Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Первый скрипт. Все плохо ? или работать будет? (https://javascript.ru/forum/project/66156-pervyjj-skript-vse-plokho-ili-rabotat-budet.html)

krasavchik 28.11.2016 18:13

Первый скрипт. Все плохо ? или работать будет?
 
Оцените первые мои «3 строчки» кода. Цель авто заполнение поля.
Если есть за что прошу попинать. Только сильно не пинайте это мой первый код на JS. Выложил, я сам не понимаю зачем, с одной стороны, что тут может быть не так. Но с другой вдруг допущена серьезная ошибка?


<script language="JavaScript" type="text/javascript">
function func(field)
{
	var checked_y= document.getElementById("pickup").checked; // Проыеряем наличие чекбокса
	//alert(checked_y); // Тестовая строка
	var selind = document.getElementById("zav_x").options.selectedIndex; // Узнаем порядковый номер select
	//alert(selind); // Тестовая строка
	var txt= document.getElementById("zav_x").options[selind].text; // Узнаем значения описания select

	// alert(txt); // Тестовая строка
	//var val= document.getElementById("zav_x").options[selind].value; // Узнаем значения value select

	var text=document.getElementById("text"); // Считываем значение тектового поля

	reg= new RegExp(/^Самовыоз из +.[а-яА-Я- ]+. /, 'gi'); // Удаляем предыдущее значение из строки
	str= text.value.replace(reg,''); // // Удаляем предыдущее значение из строки
		
	if (checked_y == true) // Чекбокс есть
	{ 
		text.value="Самовыоз из "+txt+". " +str;  // Зписываем новое значение + все что осталось в адресе.
	}   else    {
		text.value=""+str+" "; // Зписываем новое значение без завода + все что осталось в адресе.
	}
}
</script>
<form name="formName">


<SELECT NAME="zav_x"   id="zav_x" onclick="func(this)" >
<option value=""  > Все заводы </option> 
<option value="1" > Базел </option>  
<option value="4" > Волок-к </option>

 </SELECT>	

 <input type="checkbox" name="pickup" id="pickup" value="true" onclick="func(this)">Автозаполнение
<textarea id="text"  rows="5" cols="100" >Что то написано.</textarea>
</form>

krasavchik 28.11.2016 18:17

А почему нет кнопки запустить у кода ? Что можно править ? или это могут делать только модераторы в своих кодах ?

Coriolan161 28.11.2016 18:31

krasavchik,
Цитата:

А почему нет кнопки запустить у кода ?
Разработчики сделали так что как только ты вставляешь код, автоматически создается фидл и пен в codepen + доступна опция автоматического создания репозитория на гитхаб.
Цитата:

могут делать только модераторы в своих кодах
Они и только они.

krasavchik 29.11.2016 17:06

Цитата:

Сообщение от Rise (Сообщение 436442)

Спасибо, поправил. а по коду есть нарекания ?


Часовой пояс GMT +3, время: 23:59.