Больше деталей#4
Conversation
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
1 similar comment
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
| @@ -0,0 +1,94 @@ | |||
| const description = [ | |||
There was a problem hiding this comment.
константы-данные стоит записывать DESCRIPTIOM, NAME_USER и тд
There was a problem hiding this comment.
а также если это "описания" - то DESCRIPTIONS
| 'моя любовь', | ||
| ]; | ||
|
|
||
| const nameUser = [ |
| 'Олег', | ||
| ]; | ||
|
|
||
| const messegeUser = [ |
There was a problem hiding this comment.
USER_MESSAGES | USER_MESSAGE_LIST
There was a problem hiding this comment.
мы названием переменной должны явно показать что это список например
| comments: similarComments | ||
| }); | ||
|
|
||
| const similarFotos = Array.from ({length: quantityFotos} , createComments); |
There was a problem hiding this comment.
ожидаю что это будет функция, что бы можно было вызвать повторно
также название надо поменять чтоб было понятно что вернет список тумб/фоток
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
| const avatarNumber = getRandomInteger(1, 6); | ||
| const quantityMessege = getRandomInteger(1, 2); | ||
| const quantityComments = getRandomInteger(0, 30); | ||
| const quantityFotos = 25; |
There was a problem hiding this comment.
тк константа, будет запись QUANTITY_FOTOS
| const similarComments = Array.from ({length: quantityComments} , createComments); | ||
|
|
||
| const createFotos = () => ({ | ||
| id : generatePhotoId(), |
There was a problem hiding this comment.
ты можешь id передавать атриюутом функциюю, тк у тебя генерация идет через Array.from
второй аргумент это индекс
There was a problem hiding this comment.
и выйдет что в id подставишь id: index + 1
url также через индекс
и будут вероятней всего лишние функции
|
|
||
| function totalMessage (quantityMessege) { | ||
| const textMessege = new Set([USER_MESSAGE_LIST[getRandomInteger(0, USER_MESSAGE_LIST.length - 1)]]); | ||
| if (quantityMessege === 2) { |
There was a problem hiding this comment.
можно без проверки, просто в while смотреть на текущий размер сета и сразу сверять с тем какая длина нужна
и тогда не будет "магической двойки" и лишнего ифа
|
Не удалось смёржить пулреквест. Проверьте наличие конфликтов. Задание переведено в статус |
3 participants
🎓 Больше деталей
💥 https://htmlacademy-javascript.github.io/2487211-kekstagram-2/4/