let text = data.text;
if (text !== null) {
if (mark === 1) {
searchTextbox.val(text);
searchTextbox.keyup();
} else if (mark === 0) {
feedbackTextbox.val(text);
}
} else if (text === null) {
if (mark === 1) {
searchTextbox.attr('placeholder', currentTranslateEror);
} else if (mark === 0) {
feedbackTextbox.attr('placeholder', currentTranslateEror);
}
};
})
Зачем } else if (text === null) { , и так же понятно
да
Не скажу
выглядит как говнокод
А это код говна
Ну вот так можно, сразу нагляднее (data)=>{ let text = data.text; if (mark === 1) { if (text !== null) { searchTextbox.val(text); searchTextbox.keyup(); } else { searchTextbox.attr('placeholder', currentTranslateEror); } } if (mark === 0) { if (text !== null) { feedbackTextbox.val(text); } else { feedbackTextbox.attr('placeholder', currentTranslateEror); } } }
но ты же просто поменял порядок ифов
Да, но алгоритм стал нагляднее, теперь сразу видно, что если оценка один, это про одно поле, если оценка 2 это про другое поле, а то что можно это в отдельную функцию вынести, ну вынеси, в первой версии алгоритма как раз не было видно что выносить
ладно, согласен, стало лучше
Да, по сути изменилось не много но код лучше.
Ну вынеси в отдельную функцию установку вэлью и плейсходера если хочешь
Правда эта конструкция еще и в .then это обработка fetch ответа 😁
Обсуждают сегодня