Size: a a a

JavaScript Noobs — сообщество новичков

2021 July 29

R

Rafael 🦠 in JavaScript Noobs — сообщество новичков
по этой причине условие всегда должно быть задано явно, поэтому (length > 0) гораздо удобнее проверить
источник

BB

Bogdan Bida in JavaScript Noobs — сообщество новичков
Так если это таки переменная то согласен, но к примеру если так
if (array.length)

Типа ясно что это переменная однозначно отвечает за длинну массива а не абы откуда взялась
источник

BB

Bogdan Bida in JavaScript Noobs — сообщество новичков
Грубо говоря это проверка не на пустоту а на вообще существование массива как такого, ибо вдруг там дичь пришла а не массив и length undefined

Хотя с другой стороны... если выскочит ошибка то будет ясно что где-то прокол
источник

R

Rafael 🦠 in JavaScript Noobs — сообщество новичков
ты сразу привел плохой пример

правильный пример:
if (SomeObjectBlablaBla.length)

откуда я знаю что SomeObjectBlablaBla это массив? почему такое поле не может прийти от бекенда например? и опять придется скролить вверх и смотреть, а что это за фигня такаая

и вот я проскролил выше, а там:
const SomeObjectBlablaBla = BlablaBla.someProp
источник

D

Dmitriy in JavaScript Noobs — сообщество новичков
;))) красота. Спасибо
источник

R

Rafael 🦠 in JavaScript Noobs — сообщество новичков
если ожидаешь что может прийти не массив, значит такую проверку нужно добавить явно
источник

R

Rafael 🦠 in JavaScript Noobs — сообщество новичков
if ( Array.isArray(SomeObjectBlablaBla) && SomeObjectBlablaBla.length)
источник

BB

Bogdan Bida in JavaScript Noobs — сообщество новичков
Тут согласен, но иногда ведь очень очевидно что массив, когда его объявление происходит сразу же парой строчек выше или буквально на одном "экране"

Типа результат какого нибуть .map или .filter и последующая проверка на длинну
источник

Ш

Шепс in JavaScript Noobs — сообщество новичков
Ctrl + лкм по перемененной
источник

Ш

Шепс in JavaScript Noobs — сообщество новичков
Хз читабельность важна и можно ради читабельности пренебречь чистотой
источник

Ш

Шепс in JavaScript Noobs — сообщество новичков
Но тут все так банально в примере выше
источник

c

credo in JavaScript Noobs — сообщество новичков
Там говорили про code review на гитлаб вроде без ide
источник

Ш

Шепс in JavaScript Noobs — сообщество новичков
То есть человек решил сам себе жизнь усложнить и ревьюить в браузере… пох на него
источник

R

Rafael 🦠 in JavaScript Noobs — сообщество новичков
ты находишь в гитлабе, потому что проверяешь мерж риквест
источник

Ш

Шепс in JavaScript Noobs — сообщество новичков
А ну да в таком случае
источник

R

Rafael 🦠 in JavaScript Noobs — сообщество новичков
это не усложнение, когда ревьювих хотябы проектов 5
источник

Ш

Шепс in JavaScript Noobs — сообщество новичков
Думаю да, не подумал, сори, в таких условиях не работал никогда просто
источник

НК

Николай Коржиков... in JavaScript Noobs — сообщество новичков
Ребята, привет.

/* 
   Task4:
 
   Дан код:

   const getNumber = n => {
       return new Promise(r => {
           setTimeout(() => {
               r(n);
           },2000)
       })
   }

   const printNumber = n => {
       return new Promise(r => {
           getNumber(n).then(res => {
               r(res)
           })
       })
   }

   printNumber(100).then(data => console.log(data));


   Перепишите его используя операторы async/await
*/


Никак не могу понять что тут можно переписать на async/await, кто может подсказать новичку?
источник

BB

Bogdan Bida in JavaScript Noobs — сообщество новичков
Место then сразу внутрь консол лога засунуть вызов функции принтНамбер с использованием авейт

А еще вместо создания промисов новых тоже можно юзать авейт а сами функции сделать асинк тогда они будут возвращать промис и так
источник

B

Bohdan in JavaScript Noobs — сообщество новичков
printNumber = async (n) =>  await new Promise
источник