раскрыть ветку (24)
То что в каждой строчке ошибка. Не синтаксическая, конечно. Подобный код писать позволительно для того, кто только первый день чему-то учится. А заголовок "Пишем код как PRO"
раскрыть ветку (15)
Ещё позволительно такое написать если ты адепт if. Слова else, elseif или switch являются запретный колдунством.
раскрыть ветку (11)
Если можно не использовать else, то стараюсь не использовать. Но тут конечно switch просится
раскрыть ветку (6)
Я все базовое накидал. Можно же и более интересные варианты делать. На пример foreach (не знаю есть ли с js) по словарю.
раскрыть ветку (5)
А зачем foreach? Просто объявить ассоциативник.
Типа того же
getLight(color: string):
arr = {
"green": "ehay",
"red": "stoy"}
return arr[color] or 'vvedi norm cvet suka'
раскрыть ветку (3)
Как зачем? Мы же выучили. Нужно всем показать :)
А вообще удобно использовать на больших наборах данных, которые не возможно хранить в оперативной памяти. Хотя словарем в моем примере и не пахнет.
раскрыть ветку (2)
Так стоп, поясни плез.
как мы можем запустить foreach по массиву, не имея его в качестве переменной. Нихуя не понял
как мы можем запустить foreach по массиву, не имея его в качестве переменной. Нихуя не понял
раскрыть ветку (1)
Построчно считываешь из файла или иного набора данных. Часто необходимо на не структурированных данных для их упорядочивания при наличии большого объёма. То есть у тебя переменной присваивается следующая строка (если конкретный язык так позволяет) и этим же ты получаешь человеческий объект с которым можно работать (если данные предсказуем и под них можно сделать объект).
Пример. У тебя 4 набора данных по 50 гигов. Тебе нужно построить какую-то модель или как-то по другому взять нужное и объединить. Тебе нафиг не нужно все это держать в памяти (вообще её нужны на самом деле абсолютно все данные) и нет возможности закинуть все это в sql. Вот так и ебешься до получения нужного результата.
Пример. У тебя 4 набора данных по 50 гигов. Тебе нужно построить какую-то модель или как-то по другому взять нужное и объединить. Тебе нафиг не нужно все это держать в памяти (вообще её нужны на самом деле абсолютно все данные) и нет возможности закинуть все это в sql. Вот так и ебешься до получения нужного результата.
Открою тебе секрет, но что с else, что без в данном случае скомпилированный код будет одинаковым
Всё ещё не очень понимаю, что не так, разве что switch применить было бы красивее.
А так код читабельный - читабельный. А то я иногда встречал любителей тернарных операторов)
Свои функции выполняет - выполняет.
Начнём с первой строчки - название функции getLight не соответствует её реальному применению, должно быть что-то вида getActionByLight
Строчка номер два - попытка проверить переменную на цвет которая не работает для любого непустого значения, сработает только на '' NaN null undefined
Идём дальше:
- делать всё отдельными ифами не используя ифелс/свич несколько странно но в целом +/- вкусовщина
-свич для оперирования такими ситуациями подходит идеально, проверка сразу реализуется дефолтом
-ладно, без свича, больше стандартом таких ф-ций является сначала просмотреть заданные варианты, а в конце функции сделать дефолтный ретурн ошибки (а у нас дефолтный "Едь!")
Ну и на финале кода мы вызываем не ту ф-цию.
раскрыть ветку (5)
Спасибо! Возьму на заметку.
А вообще, ограничения на длину названия функции и переменных есть?
раскрыть ветку (2)
IT-юмор
5.7K постов52.6K подписчиков
Правила сообщества
Не публикуем посты:
1) с большим количеством мата
2) с просьбами о помощи
3) не относящиеся к IT-юмору