search

AnastasiaHarost / instagram_project

0 stars 0 forks source link

Задание №4 #3

Open alexey-kozlenkov opened 5 years ago

alexey-kozlenkov commented 5 years ago
  1. Настоятельно советую не забывать форматировать код.
  2. Переменные именуются в lowerCase формате.
  3. Старайтесь не упускать return в функциях.
  4. Зачем в getPhotoPost дважды делать поиск?
  5. В removePhotoPost можно переиспользовать метод поиска поста.
  6. editPhotoPost - двойной поиск
  7. editPhotoPost - неэффективный порядок действий, сначала удаляете, потом валидируете, потом что-то добавляете назад. Проще сначала выполнить валидацию.
AnastasiaHarost commented 5 years ago

New version is ready. Check, please

ala-n commented 5 years ago

Now looks very cool for 4th lab. A couple of CleanCode problems will be part of 5th review. U've 10. thanks!