johnny-silverhand/yoshows

A firefox addon for myshows.me adding link yohoho.cc by altering html elements.

YoShows

Расширение для сайта myshows.me, которое добавляет ссылку на yohoho.cc для онлайн-просмотра любимого сериала.

Установка с магазина расширений

FireFox: YoShows

Chrome: скоро...

yoshows's Issues

L64-L75

yoshows/main.js

Lines 64 to 75 in c79e558

	const renderProfile = () => {
	return Promise.all(Array.from(document.getElementsByClassName("Unwatched-showTitle"), item => {
	const showId = item.href.match(/\d+/g).shift()
	const element = Array.from(item.closest("div.Row-container").getElementsByClassName("Unwatched-remain")).pop()
	return fetchQuery(showId).then(result => {
	return {
	id: result,
	element: element,
	}
	})
	})).then(result => result.map(value => createYoNode(value)))
	}

1. Предлагаю варить массив промисов отдельно от Promise.all для лучшей читаемости кода

2. Опять же в строке 66 можно использовать const showId = item.href.match(/\d+/g)[0]

3.  return fetchQuery(showId).then(id=> ({
            id,
            element,
    })
   

yoshows/main.js

Line 74 in c79e558

})).then(result => result.map(value => createYoNode(value)))

})).then(result => result.map(createYoNode))

Количество параметров лямба функции равно 1, входящий параметр тоже 1. Значит необязательно ее описывать

L39-L41

yoshows/main.js

Lines 33 to 42 in c79e558

	const render = () => {
	const path = window.location.pathname
	if (path === "/profile/") {
	renderProfile().catch(error => console.log(error.message))
	} else if (path === "/search/all/") {
	renderSearch().catch(error => console.log(error.message))
	} else if (/view\/\d+\/+$/ig.exec(path)) {
	renderView(path.match(/\d+/g).shift()).catch(error => console.log(error.message))
	}
	}

yoshows/main.js

Line 40 in c79e558

renderView(path.match(/\d+/g).shift()).catch(error => console.log(error.message))

Проблема pop заключается в том, что она мутирует переменную path.
Да и создавался он для удаления элемента из массива, а не взятия из него элемента
Предлагаю использовать path.match(/\d+/g)[0] или еще для лучшей читаемости кода использовать диструктуризацию в данном месте

`
const [yourVariableName,...orher] = path.match(/\d+/g)

renderView(path.match(yourVariableName).catch(error => console.log(error.message))
`

Паттерн /\d+/g думаю можно и вынести, тк используется в нескольких местах

Обработку error => console.log(error.message) думаю можно и в отдельную функцию. Тк повторяется