if (filterConfig !== undefined) лучше заменить на if (filterConfig). Лучше читается
не ленитесь ставить фигурные скобки вокруг тел даже с одним выражением.
photoPostsToRemake = photoPostsToRemake.filter(Date =>: нельзя называть свои переменные/объекты/массивы заведомо существующими названиями. К примеру: Date - объект из стандартной библиотеки JavaScript. Если вы создаете переменную Date, вы очень сбиваете читающего и, возможно, самого себя в будущем.
photoPosts.findIndex(index => index.id === photoPost.id) уделяйте больше внимания пониманию смысл переменных. Тут, например, index это не индекс, а фотопост.
editPhotoPost:
по ТЗ в этот метод может приходить неполный объект фотопоста. А при вашей реализации он не пройдет валидацию.
CleanCode:
Исправил недочеты, сделал код более читабельным.
editPhotoPost:
Я знал что может приходить неполный объект поста, на валидацию проверял пост, который изменяется, просто ошибся в написании. Исправил.
CleanCode:
if (filterConfig !== undefined)
лучше заменить наif (filterConfig)
. Лучше читаетсяphotoPostsToRemake = photoPostsToRemake.filter(Date =>
: нельзя называть свои переменные/объекты/массивы заведомо существующими названиями. К примеру:Date
- объект из стандартной библиотеки JavaScript. Если вы создаете переменную Date, вы очень сбиваете читающего и, возможно, самого себя в будущем.photoPosts.findIndex(index => index.id === photoPost.id)
уделяйте больше внимания пониманию смысл переменных. Тут, например,index
это не индекс, а фотопост.editPhotoPost: