Разложить массив

[Rost93]

Добрый вечер, вопрос у меня достаточно странный. Дело в том что я на 200% уверен что код должен работать, но он не работает. Вот код:

PHP код:

for (var i = 1; i < countOfFields; i++) {
    var ves_pol = new Array();
    ves_pol[i] = document.getElementById("ves"+i).value;
    var ves_pol1 = 0;
    ves_pol[i] = parseInt(ves_pol[i]);
    ves_pol1 = ves_pol[i] + ves_pol1;
} 


Значит у меня неизвестное количество поле типа текст, мне нужно сложить все значения что будут туда вводить, я написал вроде бы простой цикл, но он почему-то не работает. В переменную ves_pol1 заноситься только последние значения ves_pol[i], т.е. почему-то не складывает. Подскажите пожалуйста в чем дело.

[Кольша]

for (var i = 1; i < countOfFields; i++) { 
    var ves_pol = new Array(); 
    ves_pol[i] = document.getElementById("ves"+i).value; 
    var ves_pol1 = 0; 
    ves_pol1[i] = parseInt(ves_pol[i]); 
    ves_pol1 = ves_pol[i] + ves_pol1; 
}

может так

[Rost93]

Тоже самое, заноситься только последние значение.

[Naive]

Сообщение от Rost93

Добрый вечер, вопрос у меня достаточно странный. Дело в том что я на 200% уверен что код должен работать, но он не работает. Вот код:

PHP код:

for (var i = 1; i < countOfFields; i++) {
    var ves_pol = new Array();
    ves_pol[i] = document.getElementById("ves"+i).value;
    var ves_pol1 = 0;
    ves_pol[i] = parseInt(ves_pol[i]);
    ves_pol1 = ves_pol[i] + ves_pol1;
} 


Значит у меня неизвестное количество поле типа текст, мне нужно сложить все значения что будут туда вводить, я написал вроде бы простой цикл, но он почему-то не работает. В переменную ves_pol1 заноситься только последние значения ves_pol[i], т.е. почему-то не складывает. Подскажите пожалуйста в чем дело.

Вы в каждой итерации цикла заново пересоздаете массив.

var ves_pol = [];
for (var i=countOfFields-1;i++;) {
  ves_pol.push(document.getElementById("ves"+(i-1)).value-0)
}

Нужно было что-то в этом роде?
п.с. я бы не советовал использовать конструкцию parseInt:

>>> parseInt('07');
7
>>> parseInt('08');
0
>>> parseInt('09');
0
>>> parseInt('10');
10

консоль ff10
п.п.с. в том коде для приведения к целому числу я использовал вычитание нуля из строки.
Если в тексте после цифр будут другие символы, то воспользуйтесь parseFloat:

'9'-0
9
'9a'-0
NaN
parseInt('9a')
9
parseFloat('9a')
9
parseFloat('09a')
9
parseInt('09a')
0

[Rost93]

Я в теме ошибся, мне нужно как бы сложить в значения. Т.е. в первой строке ввели 25 во второй 20, мне нужно получить 45 и занести это в переменную.
Я почитал функция push просто как бы прилепливает все значения в одну строку.

[Naive]

var ves_pol = 0;
for (var i=countOfFields-1;i++;) {
  ves_pol+=document.getElementById("ves"+(i+1)).value-0;
}

В одну строку сливает элементы массива функция join.
Push добавляет к массиву элемент.

[Rost93]

Вставил в точности ваш код, появилась какая-то ошибка скрипт весь не работал. Переделал не много

PHP код:

var ves_pol = 0;
for (var i = 1; i < countOfFields; i++) {
  ves_pol += parseFloat(document.getElementById("ves"+i).value);
} 


Теперь все ок, складывает. Большое спасибо!
А теперь если не трудно объясните пожалуйста чем отличается
Ваш вариант от моего.
Чтобы больше подобных ошибок не было!

[Naive]

Сообщение от Rost93

Вставил в точности ваш код, появилась какая-то ошибка скрипт весь не работал. Переделал не много

PHP код:

var ves_pol = 0;
for (var i = 1; i < countOfFields; i++) {
  ves_pol += parseFloat(document.getElementById("ves"+i).value);
} 


Теперь все ок, складывает. Большое спасибо!
А теперь если не трудно объясните пожалуйста чем отличается
Ваш вариант от моего.
Чтобы больше подобных ошибок не было!

Знать бы еще какая именно ошибка. Я не представляю какая у вас структура документа и какие значения в полях, и вполне возможно, что я где-то опечатался, так как код не тестировал. На вскидку по поводу данных я писал выше:

... в том коде для приведения к целому числу я использовал вычитание нуля из строки. Если в тексте после цифр будут другие символы, то воспользуйтесь parseFloat:

'9'-0//9
'9a'-0//NaN
parseInt('9a')//9
parseFloat('9a')//9
parseFloat('09a')//9
parseInt('09a')//0

[Rost93]

От вас я узнал про функцию parseFloat();