Javascript-форум - Советы по коду

Javascript-форум (https://javascript.ru/forum/)

- Ваши сайты и скрипты (https://javascript.ru/forum/project/)

- - Советы по коду (https://javascript.ru/forum/project/36192-sovety-po-kodu.html)

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

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

Цитата:

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

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

По мне так это

Цитата:

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

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

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

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

Цитата:

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

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

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

Цитата:

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

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
if (bottomTotalSumElem > clientWindow + scrollWindow && bottomTotalSumElem > clientWindow) {
...
}
Уже говнокод...
Это чем коряво и как исправить?

Цитата:

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

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

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

Цитата:

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

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

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

Цитата:

Сообщение от ksa (Сообщение 239209)

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

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

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

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

Цитата:

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

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

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

Цитата:

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

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

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

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

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