Final project #91
Final project #91
Conversation
| @@ -0,0 +1,96 @@ | |||
| import SortableList from '../../components/sortable-list/index.js'; | |||
|
|
|||
| export default class Category { | |||
There was a problem hiding this comment.
good 👍
лайк за выделение данного функционала в отдельный компонент
|
|
||
| initComponents() { | ||
| const sortableList = new SortableList({ | ||
| items: this.category.subcategories.map(item => { |
There was a problem hiding this comment.
может вынести данную логику в отдельный метод?
чтобы не загромождать инициализацию SortableList?
| components; | ||
|
|
||
| constructor(category = {}) { | ||
| this.category = category; |
There was a problem hiding this comment.
category or categories?
категоря одна или их много?
| this.remove(); | ||
| this.element = null; | ||
|
|
||
| for (const component of Object.values(this.components)) { |
There was a problem hiding this comment.
👍 good
лайк за дестрой внутренних компонент
| data; | ||
|
|
||
| constructor({url = '', range = {from: new Date(), to: new Date()}, label = '', link = '', formatHeading = (str => `${str}`)} = {}) { | ||
| this.url = new URL(url, vars.BACKEND_URL); |
| @@ -1,89 +1,107 @@ | |||
| import fetchJson from '../../utils/fetch-json.js'; | |||
| import vars from '../../utils/vars.js'; | |||
There was a problem hiding this comment.
очень странное имя vars
может лучше назвать данный файл "config.js"?
| } | ||
|
|
||
| getTemplate() { | ||
| const linkA = this.link ? `<a href=${this.link} class="column-chart__link">View all</a>` : ''; |
There was a problem hiding this comment.
что такое linkA?
почему А?
| return date.toLocaleString('ru', {dateStyle: 'short'}) | ||
| months = [ | ||
| 'январь', | ||
| 'февраль', |
There was a problem hiding this comment.
подумайте как сгенерировать данный список динамично
|
|
||
| static formatDate(date) { | ||
| const year = date.getFullYear(); | ||
| const month = (date.getMonth() + 1) < 10 ? '0' + (date.getMonth() + 1) : (date.getMonth() + 1); |
| {url = '', isSortLocally = false, | ||
| sorted = {id: headerConfig.find(item => item.sortable).id, order: 'asc'}} = {}) { | ||
| this.headerConfig = headerConfig; | ||
| this.url = new URL(url, vars.BACKEND_URL); |
| </span>` | ||
| : ''; | ||
| getRows() { | ||
| const bodyRows = Array.from(this.subElements.body.querySelectorAll('.sortable-table__row')); |
There was a problem hiding this comment.
почему используется Array.from a не spread syntax?
| destroy() { | ||
| this.remove(); | ||
| this.subElements = {}; | ||
| document.addEventListener('scroll', async () => { |
| this.element.classList.add('sortable-table_empty'); | ||
| } | ||
| destroy() { | ||
| this.remove(); |
There was a problem hiding this comment.
а где удаление обработчика скрола?
| destroy() { | ||
| this.remove(); | ||
| this.subElements = {}; | ||
| document.addEventListener('scroll', async () => { |
There was a problem hiding this comment.
а где удаление обработчика скрола?
|
|
||
| } | ||
|
|
||
| initEventListeners() { |
| this.element = null; | ||
|
|
||
| for (const component of Object.values(this.components)) { | ||
| component.destroy(); |
|
|
||
| this.element = element.firstElementChild; | ||
| const promiseCategory = fetchJson(this.urlCategory); | ||
| const promiseProduct = id ? fetchJson(this.urlProduct) : Promise.resolve(this.defaultData); |
|
|
||
| this.dispatchEvent(result.id); | ||
| } catch (error) { | ||
| console.error('error', error); |
There was a problem hiding this comment.
можно также показать нотификейшен в случае ошибки
| fileInput.click(); | ||
| }; | ||
|
|
||
| onSubmit = event => { |
| import SortableTable from '../../../components/sortable-table/index.js'; | ||
| import DoubleSlider from "../../../components/double-slider/index.js" | ||
|
|
||
| const header = [ |
There was a problem hiding this comment.
может стоит вынести конфиг хедера в отдельный файл?
| isSortLocally: false, | ||
| sorted: {id: 'title', order: 'asc'} | ||
| }); | ||
| // const productId = '101-planset-lenovo-yt3-x90l-64-gb-3g-lte-cernyj'; |
There was a problem hiding this comment.
не оставляйте закомментированный код
| sortable: true, | ||
| sortType: 'date', | ||
| template: date => { | ||
| const data = new Date(Date.parse(date)); |
There was a problem hiding this comment.
шаблон должен содержать логику отображения, а не бизнес-логику
| @@ -0,0 +1,13 @@ | |||
| const vars = { | |||
| BACKEND_URL: 'https://course-js.javascript.ru', | |||
There was a problem hiding this comment.
часть из этих значений следует взять из переменных окружения,
они доступны через process.env
Часть функционала еще не доделана. Не прикручена форма добавления/редактирования товара.