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 ответа 😁
Обсуждают сегодня