169 похожих чатов

Пипл, как вы счетаете это говнокод? .then((data) => {

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);
}
};
})

15 ответов

11 просмотров

Зачем } 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); } } }

Denis Deniskov
Ну вот так можно, сразу нагляднее (data)=>{ le...

но ты же просто поменял порядок ифов

Anton Permyakov
но ты же просто поменял порядок ифов

Да, но алгоритм стал нагляднее, теперь сразу видно, что если оценка один, это про одно поле, если оценка 2 это про другое поле, а то что можно это в отдельную функцию вынести, ну вынеси, в первой версии алгоритма как раз не было видно что выносить

First name- Автор вопроса
Denis Deniskov
Ну вот так можно, сразу нагляднее (data)=>{ le...

Да, по сути изменилось не много но код лучше.

First name
Да, по сути изменилось не много но код лучше.

Ну вынеси в отдельную функцию установку вэлью и плейсходера если хочешь

First name- Автор вопроса
Denis Deniskov
Ну вынеси в отдельную функцию установку вэлью и пл...

Правда эта конструкция еще и в .then это обработка fetch ответа 😁

Похожие вопросы

Обсуждают сегодня

Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
А вот это что за конструкция? Вернее, она тут нафига?
Serjone
10
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Мужики. привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных элементов...
Kraszx
2
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Товарищи, кто работа с iphelper? Или может я в самой логике ошибки фигачу, не пойму.... var ifTable : PMIB_IFTABLE; size, corSize: DWORD; Buffer ...
Warfarellen
4
я так понимаю, я так подозреваю, что создание такого плагина для человека, кто умеет писать плагины для делфи потребует минут 5-10 времени. но это мое подозрение. хотелось бы ...
Kraszx
7
Всем привет! Кто пользуется DevExpress, подскажите пожалуйста, реализован ли в TcxGrid в новых версиях поиск по датам как в Экселе (ну т.е. не просто список чекбоксов со значе...
A Z
4
Карта сайта