Newcomers Player Reward 1.1

[CrazyHackGUT]

CrazyHackGUT добавил(а) новый ресурс:

Newcomers Player Reward - Одноразовые награды для игроков

Предоставляет возможность игрокам получить одноразовую награду.
Выдача награды производится через команду, которая настраивается в конфиге сервера.

Узнать больше об этом ресурсе...

 

[fl0wer]

#define CompareHandle(%0,%1) (view_as<Handle>(%0) == view_as<Handle>(%1))


Handle hDriver = SQL_ReadDriver(g_hDB, szDriverType, sizeof(szDriverType));

if (CompareHandle(hDriver, SQL_GetDriver("sqlite"))) return 0;
else if (CompareHandle(hDriver, SQL_GetDriver("mysql"))) return 1;

Почему hDriver не DBDriver? Опять костыли из ничего.

 

[CrazyHackGUT]

Почему hDriver не DBDriver? Опять костыли из ничего.

Минимальную версию SM посмотрите, пжлст.
Только в SM 1.10 вышеуказанные функции наконец-то возвращают DBDriver. Даже в 1.9 они обе ещё Handle возвращали.
Если бы я в исходнике вставил DBDriver вместо Handle - оно бы попросту на 1.9 и ниже ошибками плевало.

 

[fl0wer]

Ну Database для бд хотя бы мог заюзать, оно на 1.9 уже возвращает.

stock const char g_szLoadClientQuery[] = "SELECT 1 FROM player WHERE account_id = %d";

Что за 1? Ты вообще проверял его? Где LIMIT 1?

#define DBCallback(%0) public void %0(Handle hDB, Handle hResults, const char[] szError, int iClient)

Это новый уровень лени?

g_hRewardCommand.GetString(g_szRewardCommand, sizeof(g_szRewardCommand));
SQL_TQuery(g_hDB

Так ты используешь новый синтаксис или нет? Что за каша?

#define CompareHandle(%0,%1) (view_as<Handle>(%0) == view_as<Handle>(%1))

Вопрос зачем хэндл преобразовывать в хэндл, если они и так хэндлы остался открытым.

5 Дек 2019

Зачем грузить игрока два раза тоже не понятно.

 

[CrazyHackGUT]

Ну Database для бд хотя бы мог заюзать, оно на 1.9 уже возвращает.

Не вижу никакого смысла.

Что за 1? Ты вообще проверял его?

Оптимизация. Да, проверял.
Если записей в БД, удовлетворяющих условию нет - ничего не придёт; если есть - придёт единица. Этот запрос проверяет наличие игрока в базе данных.

Где LIMIT 1?

PRIMARY KEY гарантирует мне, что у меня в базе не будет такого, что один и тот же игрок внесётся два раза. Зачем LIMIT?

Это новый уровень лени?

https://en.wikipedia.org/wiki/Don't_repeat_yourself

Так ты используешь новый синтаксис или нет?

Использую только там, где с 1.7.3 будет работать.

Вопрос зачем хэндл преобразовывать в хэндл, если они и так хэндлы остался открытым.

Ничего не преобразуется, а кастуется лишь для внутренних проверок компилятора. В бинарник оно не попадает.

Зачем грузить игрока два раза тоже не понятно.

Где два раза?)

Изучите приличия ради всю логику плагина, и то, что именно он делает.

 

[fl0wer]

https://en.wikipedia.org/wiki/Don't_repeat_yourself

Типо "Я два раза повторять не буду!", что больше заняло написать макрос или сделать копипасте?

Нормас тебя забайтил, скриптеры изучайте.

 

[CrazyHackGUT]

Типо "Я два раза повторять не буду!", что больше заняло написать макрос или сделать копипасте?

Макрос у меня давно написан, я его просто копирую из исходника в исходник, когда он мне нужен.

Нормас тебя забайтил, скриптеры изучайте.

Забайтил на что? На ответы по вопросам?

 

[fl0wer]

CrazyHackGUT, да, люди любят доказывать другим когда те что-то ошибочно говорят (изи завязать диалог).

 

[CrazyHackGUT]

Ресурс Newcomers Player Reward был обновлен:

Обновление до 1.1

Нажмите на эту ссылку, чтобы узнать подробности обновления