search

Nikita0703 / Front_Angular

0 stars 0 forks source link

Code review #1

Open EugeneHryz opened 10 months ago

EugeneHryz commented 10 months ago

  1. Чтобы копировать property с таким же названием в другой объект можно использовать spread оператор https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/adduser/adduser.component.ts#L81-L89

  2. Зачем здесь создавать объект? Почему он не может быть null? https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/adduser/adduser.component.ts#L64C10-L64C10

  3. Зачем здесь async? https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/uppdateuser/uppdateuser.component.ts#L107

  4. Лучше здесь удалить из массива users, а не обновлять страницу https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/viewuser/viewuser.component.ts#L32-L34

Nikita0703 commented 10 months ago

Добрый день Мне исправить и вам отправить или Ирине Геннадьевне?

понедельник, 30 октября 2023 г. пользователь EugeneHryz < @.***> написал:

1.

Чтобы копировать property с таким же названием в другой объект можно использовать spread оператор https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L81-L89 https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/adduser/adduser.component.ts#L81-L89 2.

Зачем здесь создавать объект? Почему он не может быть null? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L64C10-L64C10 https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/adduser/adduser.component.ts#L64C10-L64C10 3.

Зачем здесь async? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ uppdateuser/uppdateuser.component.ts#L107 https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/uppdateuser/uppdateuser.component.ts#L107 4.

Лучше здесь удалить из массива users, а не обновлять страницу https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ viewuser/viewuser.component.ts#L32-L34 https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/viewuser/viewuser.component.ts#L32-L34

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4YRMI7FMYQCOF6MROZ5RNLYB62QZAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3DQNJQHA2DEOI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

EugeneHryz commented 10 months ago

Привет! Можешь сообщить и мне, и Ирине Геннадьевне. Чтобы она тоже была в курсе.

ср, 1 нояб. 2023 г., 17:31 Nikita0703 @.***>:

Добрый день Мне исправить и вам отправить или Ирине Геннадьевне?

понедельник, 30 октября 2023 г. пользователь EugeneHryz < @.***> написал:

1.

Чтобы копировать property с таким же названием в другой объект можно использовать spread оператор https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L81-L89 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/adduser/adduser.component.ts#L81-L89>

2.

Зачем здесь создавать объект? Почему он не может быть null? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L64C10-L64C10 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/adduser/adduser.component.ts#L64C10-L64C10>

3.

Зачем здесь async? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ uppdateuser/uppdateuser.component.ts#L107 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/uppdateuser/uppdateuser.component.ts#L107>

4.

Лучше здесь удалить из массива users, а не обновлять страницу https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ viewuser/viewuser.component.ts#L32-L34 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/viewuser/viewuser.component.ts#L32-L34>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1, or unsubscribe < https://github.com/notifications/unsubscribe-auth/A4YRMI7FMYQCOF6MROZ5RNLYB62QZAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE3DQNJQHA2DEOI>

. You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1#issuecomment-1789062203, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHRSGZWIGUFQMC5E5CYO3YCJMNPAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGA3DEMRQGM . You are receiving this because you authored the thread.Message ID: @.***>

Nikita0703 commented 10 months ago

Все кроме jdbc исправил По нему есть несколько вопросов, можете, пожалуйста, подсказать как должны выглядеть пункты 2 и 4 Я не очень понимаю как это должно быть реализовано Спасибо большое)

среда, 1 ноября 2023 г. пользователь EugeneHryz @.***> написал:

Привет! Можешь сообщить и мне, и Ирине Геннадьевне. Чтобы она тоже была в курсе.

ср, 1 нояб. 2023 г., 17:31 Nikita0703 @.***>:

Добрый день Мне исправить и вам отправить или Ирине Геннадьевне?

понедельник, 30 октября 2023 г. пользователь EugeneHryz < @.***> написал:

1.

Чтобы копировать property с таким же названием в другой объект можно использовать spread оператор https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L81-L89 < https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L81-L89>

2.

Зачем здесь создавать объект? Почему он не может быть null? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L64C10-L64C10 < https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L64C10-L64C10>

3.

Зачем здесь async? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ uppdateuser/uppdateuser.component.ts#L107 < https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ uppdateuser/uppdateuser.component.ts#L107>

4.

Лучше здесь удалить из массива users, а не обновлять страницу https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ viewuser/viewuser.component.ts#L32-L34 < https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ viewuser/viewuser.component.ts#L32-L34>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1, or unsubscribe < https://github.com/notifications/unsubscribe-auth/ A4YRMI7FMYQCOF6MROZ5RNLYB62QZAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQW IX3LMV43ASLTON2WKOZRHE3DQNJQHA2DEOI>

. You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1#issuecomment- 1789062203, or unsubscribe https://github.com/notifications/unsubscribe-auth/ AOWHRSGZWIGUFQMC5E5CYO3YCJMNPAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQW IX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGA3DEMRQGM . You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1#issuecomment-1789114730, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4YRMIZG6EZJ7IRTX6ZYFTDYCJQCXAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGEYTINZTGA . You are receiving this because you commented.Message ID: @.***>

Nikita0703 commented 10 months ago

Здравствуйте, Евгений Можете, пожалуйста,эти пункты 2 и 4 уточнить Мне только они остались,а я не очень понимаю что вы имеете ввиду и как именно их надо реализовать Спасибо)

среда, 1 ноября 2023 г. пользователь никита новиченко < @.***> написал:

Все кроме jdbc исправил По нему есть несколько вопросов, можете, пожалуйста, подсказать как должны выглядеть пункты 2 и 4 Я не очень понимаю как это должно быть реализовано Спасибо большое)

среда, 1 ноября 2023 г. пользователь EugeneHryz @.***> написал:

Привет! Можешь сообщить и мне, и Ирине Геннадьевне. Чтобы она тоже была в курсе.

ср, 1 нояб. 2023 г., 17:31 Nikita0703 @.***>:

Добрый день Мне исправить и вам отправить или Ирине Геннадьевне?

понедельник, 30 октября 2023 г. пользователь EugeneHryz < @.***> написал:

1.

Чтобы копировать property с таким же названием в другой объект можно использовать spread оператор https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L81-L89 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/adduser/ adduser.component.ts#L81-L89>

2.

Зачем здесь создавать объект? Почему он не может быть null? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L64C10-L64C10 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/adduser/ adduser.component.ts#L64C10-L64C10>

3.

Зачем здесь async? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ uppdateuser/uppdateuser.component.ts#L107 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/uppdateuser /uppdateuser.component.ts#L107>

4.

Лучше здесь удалить из массива users, а не обновлять страницу https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ viewuser/viewuser.component.ts#L32-L34 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/viewuser/ viewuser.component.ts#L32-L34>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1, or unsubscribe < https://github.com/notifications/unsubscribe-auth/A4YRMI7FMY QCOF6MROZ5RNLYB62QZAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43AS LTON2WKOZRHE3DQNJQHA2DEOI>

. You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1# issuecomment-1789062203, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHRSGZW IGUFQMC5E5CYO3YCJMNPAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43O SLTON2WKQ3PNVWWK3TUHMYTOOBZGA3DEMRQGM . You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1#issuecomment-1789114730, or unsubscribe https://github.com/notifications/unsubscribe-auth/A4YRMIZG6EZJ7IRTX6ZYFTDYCJQCXAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGEYTINZTGA . You are receiving this because you commented.Message ID: @.***>

EugeneHryz commented 10 months ago

Привет, Никита

По 2-му и 4-му пунктам написал на проекте с jdbc.

чт, 2 нояб. 2023 г. в 17:48, Nikita0703 @.***>:

Здравствуйте, Евгений Можете, пожалуйста,эти пункты 2 и 4 уточнить Мне только они остались,а я не очень понимаю что вы имеете ввиду и как именно их надо реализовать Спасибо)

среда, 1 ноября 2023 г. пользователь никита новиченко < @.***> написал:

Все кроме jdbc исправил По нему есть несколько вопросов, можете, пожалуйста, подсказать как должны выглядеть пункты 2 и 4 Я не очень понимаю как это должно быть реализовано Спасибо большое)

среда, 1 ноября 2023 г. пользователь EugeneHryz @.***> написал:

Привет! Можешь сообщить и мне, и Ирине Геннадьевне. Чтобы она тоже была в курсе.

ср, 1 нояб. 2023 г., 17:31 Nikita0703 @.***>:

Добрый день Мне исправить и вам отправить или Ирине Геннадьевне?

понедельник, 30 октября 2023 г. пользователь EugeneHryz < @.***> написал:

1.

Чтобы копировать property с таким же названием в другой объект можно использовать spread оператор https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L81-L89 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/adduser/ adduser.component.ts#L81-L89>

2.

Зачем здесь создавать объект? Почему он не может быть null? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ adduser/adduser.component.ts#L64C10-L64C10 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/adduser/ adduser.component.ts#L64C10-L64C10>

3.

Зачем здесь async? https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ uppdateuser/uppdateuser.component.ts#L107 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/uppdateuser /uppdateuser.component.ts#L107>

4.

Лучше здесь удалить из массива users, а не обновлять страницу https://github.com/Nikita0703/Front_Angular/blob/ 1c29eb70660ad8813c5605cd9a25abf41728954f/src/app/components/ viewuser/viewuser.component.ts#L32-L34 < https://github.com/Nikita0703/Front_Angular/blob/1c29eb70660 ad8813c5605cd9a25abf41728954f/src/app/components/viewuser/ viewuser.component.ts#L32-L34>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1, or unsubscribe < https://github.com/notifications/unsubscribe-auth/A4YRMI7FMY QCOF6MROZ5RNLYB62QZAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43AS LTON2WKOZRHE3DQNJQHA2DEOI>

. You are receiving this because you are subscribed to this thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1# issuecomment-1789062203, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHRSGZW IGUFQMC5E5CYO3YCJMNPAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43O SLTON2WKQ3PNVWWK3TUHMYTOOBZGA3DEMRQGM . You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub < https://github.com/Nikita0703/Front_Angular/issues/1#issuecomment-1789114730>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/A4YRMIZG6EZJ7IRTX6ZYFTDYCJQCXAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBZGEYTINZTGA>

. You are receiving this because you commented.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/Nikita0703/Front_Angular/issues/1#issuecomment-1790879694, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHRSFX3RMVANUPEMI37PDYCOXFHAVCNFSM6AAAAAA6WDWPO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJQHA3TSNRZGQ . You are receiving this because you authored the thread.Message ID: @.***>