Ivor Barhansky

lopar deleted branch maksym from new-combats/game

2023-01-23 11:46:16 +00:00

lopar merged pull request new-combats/game#39

2023-01-23 11:45:16 +00:00

Обучение

lopar pushed to dev at new-combats/game

2023-01-23 11:45:14 +00:00

6095358496 Merge pull request 'Обучение' (#39) from maksym into dev

f0a9a3e8a1 Доработка #2

46c1e38ea8 Доработка

dcc6a1337c Косяк.

d1be4995e2 Убрал notise

Compare 16 commits »

lopar commented on pull request new-combats/game#39

2023-01-19 14:23:49 +00:00

Обучение

У нас есть класс User на 12 тысяч строк. Можем сначала полностью переделать его, чтобы id точно, гарантированно,…

lopar commented on pull request new-combats/game#40

2023-01-19 13:40:15 +00:00

Исправил чат.

Залил на прод. Чата нет.

lopar commented on issue new-combats/game#41

2023-01-19 13:35:55 +00:00

Сменить дизайн приёмов

Это подтверждённая задача нарезанная @evheniiacm?

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

На dcc6a1337c/main.php (L59) вызывается инстанс User. Зачем вызывать ещё один на 20 строк выше?

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Нужны ли игроку технические исключения?

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Куча замен ниже, это тоже про обучение? Костыли, увечья, хаосники, турниры..

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Убирая где-то __config.php и __db_connect.php надо вместо них вызывать Config::init() и Database::init(). Первое подтягивает настройки, второе подтягивает функции mysql_.

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Не надо такое прописывать в отдельных файлах, чтобы потом общие настройки игнорировались и перезаписывались.

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Если это константа, тогда зачем нужна переременная? Сразу в скрипт её.

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

И эта штука один раз отработала и дальше будет вхолостую колбаситься на каждое сообщение в чате?

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Это методы, возвращающие.. константу?

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Тут либо "вы указали" либо "ты указал". :)

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Прямая зависимость от PDO? Где? Как?

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Я бы от греха подальше проверил бы хотя бы на > 0, а в идеале на валидное значение. Прилетит некондит, привет "всё сломалось".

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Так ты если запрос сюда перенёс, то и переменную хорони, которую ты создаёшь в памяти и без изменений пишешь в базу. :)

lopar commented on pull request new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

А если человек поднимет не стат, а навык или особенность? Он пройдёт квест так и не подняв стат.

lopar approved new-combats/game#39

2023-01-18 14:24:01 +00:00

Обучение

Почему ChatFirstStep.php, ChatSecondQuest.php, ChatThirdQuest.php, MyUserFirstQuest.php, MyUserFirstStep.php, MyUserFourthQuest.php, MyUserFourthStep.php, MyUserSecondQuest.php, MyUserSecondStep.php, MyUserThirdQuest.php, MyUserThirdStep.php - это набор классов с константами, а не статический массив, например, или не записи в базе? 11 классов, которые вообще ничего не делают вот совсем - выглядит очень нехорошо.