Советы по коду

[Lansaid]

Принимаю советы по коду(что сделано неправильно, коряво, говнокод-ли это и тд), который в архиве.

[ksa]

коряво, говнокод-ли это и тд

По мне так это

if (bottomTotalSumElem > clientWindow + scrollWindow && bottomTotalSumElem > clientWindow) {
...
}

Уже говнокод...

Тут else явно лишнее

if (target.tagName != 'TD') {
         return;
         }
        else {
         document.countForm.addRowButton.disabled = true;
         correctNumber(target.parentNode);
        }

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

var tr = tb1.tBodies[0].insertRow(0);
    var td1 = document.createElement('TD');
    var td2 = document.createElement('TD');
    showSum(); // Добавили строку - перещитываем координаты
    tr.appendChild(td1);
    tr.appendChild(td2);

    td1.innerHTML = '<input type="text" size="6" name="formNumber" >';
    td2.innerHTML = '<input type="button" value="Сохранить"  name ="saveButton" onclick="checkResult(this)"> ';

Как вариант...

var o = document.createElement('TD');
    o.innerHTML = '<input type="text" size="6" name="formNumber" >';
    tr.appendChild(o);
    o = document.createElement('TD');
    o.innerHTML = '<input type="button" value="Сохранить"  name ="saveButton" onclick="checkResult(this)"> ';
    tr.appendChild(o);

[Lansaid]

Сообщение от Lansaid
if (bottomTotalSumElem > clientWindow + scrollWindow && bottomTotalSumElem > clientWindow) {
...
}
Уже говнокод...
Это чем коряво и как исправить?

[ksa]

Это чем коряво?

Я запрещаю использовать длинные условия.

как исправить?

- упростить
- вычислить заранее
- использовать функцию

[Lansaid]

Я запрещаю использовать длинные условия.


- упростить
- вычислить заранее
- использовать функцию

упростить - не могу сообразить как
остальные 2 - приняты

[Lansaid]

а не надо ли все это на ООП было сделать,а не писать столько функций?

[ksa]

упростить - не могу сообразить как

Я просто привел т.с. общие принципы...

[ksa]

а не писать столько функций?

Там кода с гулькин нос! А функций еще меньше...

[Lansaid]

спасибо!

[danik.js]

(bottomTotalSumElem > clientWindow + scrollWindow && bottomTotalSumElem > clientWindow)

Если scrollWindow не может быть отрицательным, то вторая часть условия лишняя. Если x > a + b, то очевидно же что x > a (на положительных числах)