getelementbyid

[Mawr]

Доброго времени суток!
прошу простить, если не в тот раздел пишу(на форуме впервые, пока плохо ориентируюсь)
Javascript только только начал изучать, поэтому мой вопрос может оказаться глупым)
сам вопрос: есть много текстовых полей(каждая со своим id), в которые пользователь вводит текст. Необходимо после нажатия кнопки в цикле заполнить массив значениями текстовых полей (которые ищутся по id(который меняется в цикле) ).

function pro() {
var massiv = new Array();
var idd = 1;
  for (i=0; i<=28; i=i++) {
	massiv[i]=getElementById(idd).value;
	idd++;
     }
    }
}

Что не так?

[рони]

Mawr,
getElementById = ищем, а где будем искать?

[Mawr]

аа, т.е. надо document.getElementById ?

[рони]

Mawr,
да плюс фугурная скобка в конце лишняя

[Mawr]

да, точно, скобка лишняя закралась. Но все равно не работает

[рони]

Mawr,
сделайте пример что не работает здесь и ещё как вы смотрите работает она или нет ?

[Mawr]

function pro() {
var massiv = new Array();
var idd = 1;
  for (i=0; i<=28; i=i++) {
    massiv[i]=getElementById(idd).value;
    idd++;
     }
     alert('flaag');
    }

до места alert('flaag'); видимо не доходит

[Rootpassword]

вместо i=i++ надо i++ и var нужен и надо document.getElementById(idd).value;
for (var i=0; i<=28; i++) {

[Mawr]

вместо i=i++ надо i++ и var нужен и надо document.getElementById(idd).value;
for (var i=0; i<=28; i++) {

аа спасибо большое

[He3HaUKa]

а можно сократить кусок кода дог вот такого вида
если ты знаешь точное количество idd то можно использовать следующий код:

function pro() {
var massiv = new Array();

    for (var idd = 1; idd<=29; idd++) {
         massiv[idd-1]=getElementById(idd).value;

    }
    alert('flaag');
}

Так как idd начинается с 1 элемента а не нулевого то переберем до 29