Помогите сократить код

[nikto93i7]

Привет всем!) Прошу помочь с кодом (сократить его).
Думаю, что проще все будет выглядеть с помощью счетчика...

Код рабочий, но выглядет страшно и совсем безграмотно

function positive() {
    if ($("#term1").val() == 1){
	positiveUp = [ 0, tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 2) {
    positiveUp = [ 0, tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 3) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 4) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 5) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(),
 tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 6) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(),
 tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 7) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(),
 tcoSavings(), tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 8) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(),
 tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 9) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(),
 tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings()];
	return positiveUp
    } else if ($("#term1").val() == 10) {
    positiveUp = [ 0, tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(),
 tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings(), tcoSavings()];
	return positiveUp
    }
};

Буду признателен за помощь

[ild]

function positive() {
	return [ 0, tcoSavings($("#term1").val())];
};

[nikto93i7]

Что-то странный совет... такое не работает и не будет работать

[ild]

Я не знаю что там в функции tcoSavings(), но вместо 10 вызова просто ему даем цифру 10, чтобы вернул что-то 10 раз, то есть немного там тоже нужно изменить

[KosBeg]

примерно так

tcoSavings = function ( ) { // я не знаю вашу функцию, по этому просто "заглушка"
  return 1
}

function positive() {
//  var data = $("#term1").val(), // кешируем значение, так как зачем нам лишнее обращение к DOM’у?
  var data = 10, // чисто для примера
  arr = [0];
  if(typeof data === 'number') { // проверка на цифру, наверное это ненужно, но всеже
    for (; data; data--) {
      arr.push(tcoSavings())
	}
    return arr
  }
}

alert(positive())

[KosBeg]

Блин, я в шоке от своего говнокода
ОН РАБОТАЕТ)))

[nikto93i7]

От своего или моего?

[KosBeg]

От своего или моего?

я в шоке от своего говнокода

а хоть так как нужно работает?

[nikto93i7]

пока нет, но вроде уже сам разбираться начал)))
tcoSavings() - это число, которое подставляется в массив.
допустим, при выборе ($("#term1").val() == 7), должен выводиться массив positiveUp = [ 0, 5000, 5000, 5000, 5000, 5000, 5000, 5000];

[рони]

nikto93i7,
вам же давали решение для таких случаев
Замена значения в массиве