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