Доработки

[f0cusX]

• fetchJS/js/main.js

  Line 111 in 2ddaeac

  fetch('https://jsonplaceholder.typicode.com/photos?albumId=' + albumId)

  Вынести в отдельную функцию. Чушь(

  fetchJS/js/main.js

  Line 63 in 2ddaeac

  fetch('https://jsonplaceholder.typicode.com/photos/?albumId=1')

  )

• fetchJS/js/main.js

  Line 116 in 2ddaeac

  if (data[0].albumId == albumId) {

  зачем? ты в запросе уже указал номер альбома, зачем тут его еще раз проверять?

• fetchJS/js/main.js

  Line 117 in 2ddaeac

  var img = document.getElementsByClassName('thumb');

  Если у нас в текущем альбоме 100 фоток, а в следующем 10?

• fetchJS/js/main.js

  Line 121 in 2ddaeac

  var link = document.getElementsByClassName('image');

  Аналогично выше.

• fetchJS/js/main.js

  Line 127 in 2ddaeac

  fetch('https://jsonplaceholder.typicode.com/albums/')

  Берешь все альбомы, а потом выбираешь нужный? Мб сразу обратиться к нужному?

• fetchJS/js/main.js

  Lines 51 to 52 in 2ddaeac

  	var img = document.createElement('img');
  	img.className = "imagePopup";

  может функцию написать? в initAlbum что-то аналогичное?

• fetchJS/js/main.js

  Line 86 in 2ddaeac

  var h1 = document.getElementsByTagName('h1')[0];

  может класс все таки для заголовка сделать?

• fetchJS/js/main.js

  Lines 106 to 110 in 2ddaeac

  	if (attr == 'prev') {
  	albumId--;
  	} else {
  	albumId++;
  	}

  нужны функции на следующий и предыдущий альбом.

• не увидел ни одного .catch