Conversation
| }); | ||
| }, | ||
| deletProduct(product) { | ||
| this.putJson(`/api/cart/${product.id_product}`, { quantity: -1 }) |
There was a problem hiding this comment.
С уменьшением кол-ва товара этим запросом согласен, а вот с удалением таким образом - нет. Для этого у нас должен быть отдельный запрос к серверу
|
|
||
| const add = (cart, req) => { | ||
| cart.contents.push(req.body); | ||
| logging('ADD', req.body.product_name); |
There was a problem hiding this comment.
По записи в лог при изменении корзины тоже не соглашусь. Тут проблема в том, что потенциально сервер после этого может спокойно упасть (или даже без падения) и не записать обновленную корзину. То есть по факту изменений в БД не будет, а если мы будем смотреть лог - там они есть, получается лог неактуален. Стоит писать в лог, только после перезаписи json файла корзины
| if (req.body.quantity < 0) { | ||
| logging('DELETE', find.product_name); | ||
| } else { | ||
| logging('ADD', find.product_name); | ||
| } |
There was a problem hiding this comment.
Вот тут странновато - метод меняет товар корзины, не добавляет его и не удаляет. Как в логе отделить операцию добавления нового товара и изменения кол-ва уже имеющегося товара? Стоит логировать именно изменение (change) товара, для удаления и добавления должны быть отдельные запросы
| let time = `${moment().format('L')} ${moment().format('LTS')}` | ||
| let logJson = JSON.parse(data); | ||
| let logItem = { | ||
| action, | ||
| productName, | ||
| time |
There was a problem hiding this comment.
Переменная time выглядит лишней, можно сразу у объекта присвоить свойству time соответствующее значение
2 participants
No description provided.