Sprint 3 #67

Pillowbee · 2024-04-16T16:39:41Z

No description provided.

Sprint 1

Sprint 2

Pillowbee · 2024-04-16T20:03:37Z

Я создавал форк по инструкции, автоматические тесты проходят и 2 предыдущих спринта всё было нормально
И у меня в коде ни одного any

Данные в форме изменения профиля добавлю

gennady-bars

Здравствуйте. (Нужно развернуть общий комментарий ↓)

Посмотрите на гитхабе все комментарии к коду (нужно прокрутить вниз страницу там)

Работа проделана огромная:

Readme хорошо оформлен
Проект задеплоен и отлично работает
Отлично, что нет EOF ошибок в гите
Хорошая структура папок и файлов
Отлично, что указываете понятный текст ошибки под инпутом
Хорошая логика удаления пользователя из чата
Хорошая логика сокетов

но есть некоторые недочеты:

Клик по стрелке возле инпута сообщения не отправляет сообщение в чат https://disk.yandex.ru/i/dyaP7mku9z9O8g
Скрин https://disk.yandex.ru/i/8LX-AWdsISRUig Я могу добавить пустое сообщение в чат, а валидация не должна мне этого давать
По заданию требуется: При обновлении страницы с определённым URL должна отображаться та же самая страница. Если я перезагружаю страницу в профиле, то должен попадать в профиль. На Нетлифай это у Вас не работает
https://ya-praktikum.tech/api/v2 везде повторяется в запросах. Нужно вынести в константу BASE_URL и импортировать везде в коде проекта, добавляя в конце изменяющуюся часть (эндпоит). Это обычная практика, чтобы одним движением можно было сменить адрес, но не менять эндпоинты.
Для любых запросов на сервер в самом конце цепочки запроса нужно осуществлять проверку на возможные ошибки с помощью catch.
JSON.parse может выкинуть исключение, если данные невалидные. Стоит добавить блок try/catch для отлова возможной ошибки

Можно лучше (Это совет. Необязательно к исправлению)

Если типизация повторяется в каждом методе (функции), то нужно типизировать сам метод (функцию), а не дублировать типизацию аргументов. Код станет чище
Нужна возможность менять картинку чата
Должна быть возможность удалить сам чат
При редактировании профиля нужно вставлять в инпуты все актуальные данные пользователя, чтобы он мог изменить одну букву без необходимости опять впечатывать все его данные. А то это только расстроит пользователя, и он не будет использовать этот сервис больше.

gennady-bars · 2024-04-18T10:27:50Z

src/server.js

+})
+
+
+app.listen(3000, () => {


Можно лучше

Лучше дать возможность поменять порт из консоли, если понадобится: const PORT = process.env.PORT || 3000; . Если не задан порт, тогда будет 3000 по умолчанию.

gennady-bars · 2024-04-18T10:34:19Z

src/services/AuthService.ts

+}
+
+export class AuthService {
+  baseURL: string = 'https://ya-praktikum.tech/api/v2/auth'


https://ya-praktikum.tech/api/v2 везде повторяется в запросах. Нужно вынести в константу BASE_URL и импортировать везде в коде проекта, добавляя в конце изменяющуюся часть (эндпоит). Это обычная практика, чтобы одним движением можно было сменить адрес, но не менять эндпоинты.

А еще лучше поместить этот базовый урл 1 раз внутрь класса HTTPTransport, а в конструктор экземпляров передавать только эндпоинт

new HTTPTransport('/auth/');

gennady-bars · 2024-04-18T10:34:24Z

src/services/ChatService.ts

+}
+
+export class ChatService {
+  baseURL: string = 'https://ya-praktikum.tech/api/v2/chats'


https://ya-praktikum.tech/api/v2 везде повторяется в запросах. Нужно вынести в константу BASE_URL и импортировать везде в коде проекта, добавляя в конце изменяющуюся часть (эндпоит). Это обычная практика, чтобы одним движением можно было сменить адрес, но не менять эндпоинты.

А еще лучше поместить этот базовый урл 1 раз внутрь класса HTTPTransport, а в конструктор экземпляров передавать только эндпоинт

new HTTPTransport('/auth/');

gennady-bars · 2024-04-18T10:34:29Z

src/services/UserService.ts

+import HTTPTransport from '@/core/HTTPTransport.ts'
+
+export class UserService {
+  baseURL: string = 'https://ya-praktikum.tech/api/v2/user'


https://ya-praktikum.tech/api/v2 везде повторяется в запросах. Нужно вынести в константу BASE_URL и импортировать везде в коде проекта, добавляя в конце изменяющуюся часть (эндпоит). Это обычная практика, чтобы одним движением можно было сменить адрес, но не менять эндпоинты.

А еще лучше поместить этот базовый урл 1 раз внутрь класса HTTPTransport, а в конструктор экземпляров передавать только эндпоинт

new HTTPTransport('/auth/');

gennady-bars · 2024-04-18T10:34:46Z

src/utils/getResourceURL.ts

@@ -0,0 +1,4 @@
+export default (resourceURL: string) => {
+  const url = resourceURL.replaceAll('/', '%2F')
+  return `https://ya-praktikum.tech/api/v2/resources/${url}`


https://ya-praktikum.tech/api/v2 везде повторяется в запросах. Нужно вынести в константу BASE_URL и импортировать везде в коде проекта, добавляя в конце изменяющуюся часть (эндпоит). Это обычная практика, чтобы одним движением можно было сменить адрес, но не менять эндпоинты.

Для картинок такой же урл базовый

gennady-bars · 2024-04-18T10:36:10Z

src/pages/profilePage/profilePage.ts

+    if (resp.status === 200) {
+      router.go(routes.login)
+    }
+  })


Для любых запросов на сервер в самом конце цепочки запроса нужно осуществлять проверку на возможные ошибки с помощью catch.

Нужно исправить это везде по коду в конце запросов.

gennady-bars · 2024-04-18T10:36:53Z

src/utils/connectToMessageSocket.ts

+  }, 10000)
+
+  socket.addEventListener('message', (event) => {
+    const data = JSON.parse(event.data)


JSON.parse может выкинуть исключение, если данные невалидные. Стоит добавить блок try/catch для отлова возможной ошибки

gennady-bars · 2024-04-18T10:38:23Z

src/utils/connectToMessageSocket.ts

+    const data = JSON.parse(event.data)
+
+    if (Array.isArray(data)) {
+      store.set('messages', JSON.parse(event.data))


JSON.parse(event.data) 4 раза используете в коде тут. Нужно было 1 раз распасить и в переменную записать

И у Вас уже есть эта переменная в виде data на 36й строчке

gennady-bars · 2024-04-18T10:38:27Z

src/utils/connectToMessageSocket.ts

+    }
+    if (data.type === 'message') {
+      const messages = store.getState().messages
+      const message: MessageItemProps = JSON.parse(event.data)


JSON.parse(event.data) 4 раза используете в коде тут. Нужно было 1 раз распасить и в переменную записать

И у Вас уже есть эта переменная в виде data на 36й строчке

gennady-bars · 2024-04-18T10:38:34Z

src/utils/connectToMessageSocket.ts

+    }
+    if (data.type === 'user connected') {
+      const messages = store.getState().messages
+      const message: MessageItemProps = JSON.parse(event.data)


JSON.parse(event.data) 4 раза используете в коде тут. Нужно было 1 раз распасить и в переменную записать

И у Вас уже есть эта переменная в виде data на 36й строчке #

gennady-bars

Поздравляю! Ваша работа принята.

Вы отлично потрудились.

Удачного дальнейшего обучения.

ybortnikov and others added 27 commits March 6, 2024 03:21

Sprint 1 - 1 attempt reviewing

6f7da78

Исправления для прохождения автотестов

d08b378

Исправления после первой проверки ревьюером

e45e2aa

Merge pull request #1 from Pillowbee/sprint_1

26736c8

Sprint 1

WIP #1

45adf89

WIP #2

febda37

WIP #3

36a6bc5

WIP #4

608f32f

WIP #5

294396f

WIP #6

07dceb9

WIP #7

b72393b

WIP #8

dfcafd4

WIP #9

4788a58

WIP #10

62096a9

Попытка №1

0449a19

Попытка №1.1

e301476

Попытка №1.2

e012522

Попытка №1.3

be56b28

Попытка №1.4

2fc909b

Попытка №2

f65f5fe

Merge pull request #2 from Pillowbee/sprint_2

3ef1d89

Sprint 2

WIP #1

351ec2d

WIP #2

6e6d327

WIP #3

98904a5

First attempt in sprint_3

cea1423

Fixes for first attempt

c8750d6

Fixes for express routing

110aede

Fixes for first review

71ddb5b

gennady-bars reviewed Apr 18, 2024

View reviewed changes

Pillowbee added 3 commits April 18, 2024 15:55

Fixes for first review #3

7f5e41e

Fixes for first review #4

96496be

Fixes for review #5

756920f

gennady-bars approved these changes Apr 19, 2024

View reviewed changes

Pillowbee closed this by deleting the head repository Apr 24, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sprint 3 #67

Sprint 3 #67

Pillowbee commented Apr 16, 2024

Pillowbee commented Apr 16, 2024 •

edited

Loading

gennady-bars left a comment

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars Apr 18, 2024

gennady-bars left a comment

		})


		app.listen(3000, () => {

Sprint 3 #67

Sprint 3 #67

Conversation

Pillowbee commented Apr 16, 2024

Pillowbee commented Apr 16, 2024 • edited Loading

gennady-bars left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

gennady-bars left a comment

Choose a reason for hiding this comment

Pillowbee commented Apr 16, 2024 •

edited

Loading