search

Patriot0911 / MW-discord-bot

Source for Magic Words [Discord Bot]
0 stars 0 forks source link

Review 1.0 #1

Open Dmutre opened 1 year ago

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/end.js#LL1

В тебе тут неймінг різний, інколи в тебе з великої. Хоча можливо це назва класу і вони мають бути з великої, тоді ок.

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/end.js#L15

Правда тут вже інший стиль неймінгу. Якщо так треба, то ок, потім скажеш.

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/end.js#L5

Функція дуже велика і трохи складно читати. +ти її одразу запихуєш в модуль на екпорт, можна було б описати її зверху, щоб легше рефакторити було.

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/end.js#L33

Такі фрази можна винести в конфіг, щоб не доводилося рефакторити код з великим шматком тексту.

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/start.js#L11

Незрозуміло звідки 10. Я не розумію чого десять

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/start.js#L65

Знову запихнув функцію в модуль. Їх складнувати читати так

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/commands/start.js#L66

Функція дуже-дуже велика. Було б круто декомпонувати її. + в ній знову є текст, який ти до цього писав у іншому файлі, здається. Це все можна перенести до твого файлу з фразами

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/events/interaction.js#L3

Знову в модуль запихнув.

Dmutre commented 1 year ago

https://github.com/Patriot0911/MW-discord-bot/blob/0ee1e88d453b1fdf1d2c694ea7fa7c3d1181ee42/events/ready.js#L38

Magic value

Dmutre commented 1 year ago

Хоча я в тебе повсюду бачу довгі функції, але розбивати їх якось і не хочеться. Вони роблять одне, але для цього потрібно багато написати, тому залишаю їх на твій розсуд