[закрыто] makeadmin

25 июля, 2024

здравствуйте, подскажите как сделать так, чтобы команда /makeadmin после назначения игрока на админа, или повышение админа, добавляла в базу данных админку игроку, чтобы в таблице users меняла уровень админки, а в таблице admins добавляла игрока и также изменяла уровень.

Code:

CMD:makeadmin(playerid, params[])
{
    new string[128];
    if(GetAdmin(playerid, adminLogin) == false) return SCM(playerid, 0xA3A3A3FF, "{C20000}[Ошибка]"C_GREY" Вы не являетесь администратором");
    if(GetAdmin(playerid, adminLevel) < 6) return SCM(playerid, 0xA3A3A3FF, "{C20000}[Ошибка]"C_GREY" Вы не можете использовать данную команду.");
    if(sscanf(params, "ii", params[0], params[1])) return SCM(playerid, 0xA3A3A3AA, "{C20000}[Ошибка]"C_GREY" Введите: /makeadmin [ID] [Level].");
    if(!IsPlayerConnected(params[0])) return SCM(playerid, 0xA3A3A3FF, "{C20000}[Ошибка]"C_GREY" Данный игрок не в сети!");
    if(params[1] < 0 || params[1] > 8) return SCM(playerid, 0xA3A3A3FF, "{C20000}[Ошибка]"C_GREY" Уровень администратора [0-8]");
    InfoPlayer[params[0]][pAdmin] = params[1];
    format(string, sizeof(string), "Администратор %s изменил ваш уровень администратора на: %d", InfoPlayer[playerid][pName], params[1]);
    SCM(params[0], 0xFFFFFFAA, string);
    return true;
}

Screens - click

Максимальный · 25 июля, 2024

@Surfer_Selectrum, это называется избыточность: вы в двух таблицах храните и уровень прав администратора, и имя пользователя. В вашем случае правильнее хранить в столбце admin таблицы users идентификатор дескриптора (описателя) прав администратора в таблице admins. Тогда таблица admins не будет содержать столбец name, а таблица users в столбце admin не будет хранить уровень прав администратора текущего пользователя.

Тем не менее, дабы все-таки ответить на ваш вопрос, сообщаю, что в приведенном вами фрагменте кода отсутствуют какие-либо функции для записи данных в базу данных. Возможно, InfoPlayer какой-то навороченный макрос, а может быть сохранение текущих данных в базу данных осуществляется по флагу от какого-нибудь таймера, либо по событию.

Хороший · 25 июля, 2024

Если я правильно понял, то:

Создать в таблицу admins:

mysql_format(mysql, query, sizeof query, "INSERT INTO admins (name, password, level) VALUES (%s, %d, '%d')", name, password, level);

mysql_query(mysql, query, false);

Изменить в Users:

mysql_format(mysql, query, sizeof query, "UPDATE users SET admin=%d", admin);

mysql_query(mysql, query, false);

только все замени под себя

Если я все же правильно тебя понял, то нужно сделать проверку на то, есть ли уже в admins строка

Отредактировано 25 июля, 2024 пользователем moffes

Максимальный · 25 июля, 2024

@moffes, не совсем. Идея в том, чтобы можно было сопоставить данные из таблиц users и admins на основе общего поля (идентификатора аккаунта) через операторы соединения (JOIN).

Таблица users должна иметь следующие столбцы:

id (уникальный идентификатор аккаунта).
name.
password.
sex.
skin.
money.
regip.
lastip.

Обратите внимание на отсутствие столбца admin!

Таблица admins должна иметь следующие столбцы:

id (уникальный номер записи).
accountid (уникальный идентификатор аккаунта, аналогичный полю id таблицы users.
password.
level.

Теперь для выдачи прав администратора игроку нужно создать новую запись в таблице admins. Для изменения текущих прав администратора нужно, соответственно, изменить существующую запись. Удаление записи влечет лишение игрока прав администратора. Для того, чтобы при подключении игрока к серверу одним запросом получить всю информацию как из таблицы users, так и из таблицы admins, нужно соединить обе этих таблицы по столбцу admins. Например, вот так:

"SELECT * FROM users LEFT JOIN admins ON users.id = admins.accountid WHERE users.name = '%s' and users.password = '%s'", PlayerInfo[playerid][pAccountID], PlayerInfo[playerid][pPassword]

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

27 июля, 2024

@Cawfee а куда этот код в конце вставлять примерно?

Хороший · 27 июля, 2024

@Surfer_Selectrum Чтобы решить твою проблему правильно: нужно учитывать несколько моментов архитектуры базы данных и обеспечения целостности данных. В твоем же случае наблюдается что ты хранишь уровень администратора и имя пользователя в двух таблицах (`admins` и `users`), что не является оптимальным подходом.

Правильнее было бы использовать одну таблицу для хранения информации об уровне администратора и ссылаться на нее из основной таблицы пользователей. Это уменьшит избыточность и упростит управление данными. Например, в таблице `users` у тебя должен быть столбец `admin_id`, который ссылается на идентификатор в таблице `admins`.

Твой текущий код не включает операции вставки и обновления в базу данных. Вот как это можно сделать, сохраняя корректность и эффективность работы:

CMD:makeadmin(playerid, params[])
{
    new string[128];
    new pname[MAX_PLAYER_NAME];
    GetPlayerName(playerid, pname, sizeof(pname));

    if(GetAdmin(playerid, adminLogin) == false) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Вы не являетесь администратором");
    if(GetAdmin(playerid, adminLevel) < 6) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Вы не можете использовать данную команду");

    if(!IsPlayerConnected(params[1])) return SCM(playerid, 0xA3A3A3AA, "[C20000]Ошибка[]C_GREY] Данный игрок не в сети!");
    if(!strlen(params[1]) || !strlen(params[2])) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Введите: /makeadmin [ID игрока] [Уровень администратора 0-8]");

    new targetid = strval(params[1]);
    new level = strval(params[2]);

    if(level < 0 || level > 8) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Уровень администратора [0-8]!");

    format(string, sizeof(string), "SELECT `id` FROM `users` WHERE `name`='%s'", pname);
    new Cache:cache = db_query(mysqlHandle, string);
    if(cache_get_row_count(cache) == 0) {
        db_free_result(cache);
        return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Пользователь не найден в базе данных");
    }
    db_free_result(cache);

    if(level == 0) {
        format(string, sizeof(string), "DELETE FROM `admins` WHERE `name`='%s'", pname);
        db_query(mysqlHandle, string);
    } else {
        format(string, sizeof(string), "REPLACE INTO `admins` (`name`, `password`, `level`) VALUES ('%s', '%s', %d)", pname, "somepassword", level);
        db_query(mysqlHandle, string);
    }

    format(string, sizeof(string), "UPDATE `users` SET `admin`=%d WHERE `name`='%s'", level, pname);
    db_query(mysqlHandle, string);

    InfoPlayer[targetid][pName] = level;
    format(string, sizeof(string), "Администратор %s изменил ваш уровень администратора на: %d", pname, level);
    SCM(params[0], 0xFFFFFFAA, string);

    return true;
}

В этом коде выполняется сначала проверка наличия пользователя в таблице `users`. Если он найден, затем проверяется уровень администратора. Если он равен 0, то запись удаляется из таблицы `admins`. В противном случае используется `REPLACE INTO`, чтобы обновить или вставить новую запись в `admins`. Также обновляется поле `admin` в таблице `users`.

Такой подход минимизирует избыточность данных и поддерживает целостность и консистентность базы данных.

Отредактировано 27 июля, 2024 пользователем D2DChat
Причина: Орфографические ошибки.

27 июля, 2024

@D2DChat спасибо! сейчас пойду проверять.

27 июля, 2024

@D2DChat

@D2DChat ../server/admins/cmd.inc(30) : error 017: undefined symbol "mysqlHandle"
../server/admins/cmd.inc(30) : warning 213: tag mismatch: expected tag "Cache", but found "DBResult"
../server/admins/cmd.inc(31) : warning 213: tag mismatch: expected tag none ("_"), but found "Cache"
../server/admins/cmd.inc(32) : warning 213: tag mismatch: expected tag "DBResult", but found "Cache"
../server/admins/cmd.inc(35) : warning 213: tag mismatch: expected tag "DBResult", but found "Cache"
../server/admins/cmd.inc(39) : error 017: undefined symbol "mysqlHandle"
../server/admins/cmd.inc(42) : error 017: undefined symbol "mysqlHandle"
../server/admins/cmd.inc(46) : error 017: undefined symbol "mysqlHandle"

некоторые ошибки исправил

Отредактировано 27 июля, 2024 пользователем Surfer_Selectrum

Хороший · 27 июля, 2024

@Surfer_Selectrum Могу узнать какая у вас версия библиотеки MYSQL ?

27 июля, 2024

@D2DChat 5.6

Хороший · 27 июля, 2024

@Surfer_Selectrum

CMD:makeadmin(playerid, params[])
{
    new string[256];
    new pname[MAX_PLAYER_NAME];
    GetPlayerName(playerid, pname, sizeof(pname));

    if(GetAdmin(playerid, adminLogin) == false) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Вы не являетесь администратором");
    if(GetAdmin(playerid, adminLevel) < 6) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Вы не можете использовать данную команду");

    if(!IsPlayerConnected(params[1])) return SCM(playerid, 0xA3A3A3AA, "[C20000]Ошибка[]C_GREY] Данный игрок не в сети!");
    if(!strlen(params[1]) || !strlen(params[2])) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Введите: /makeadmin [ID игрока] [Уровень администратора 0-8]");

    new targetid = strval(params[1]);
    new level = strval(params[2]);

    if(level < 0 || level > 8) return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Уровень администратора [0-8]!");

    format(string, sizeof(string), "SELECT `id` FROM `users` WHERE `name`='%s'", pname);
    mysql_tquery(mysqlHandle, string, "OnAdminCheckQuery", "ii", playerid, level);

    return true;
}

forward OnAdminCheckQuery(playerid, level);
public OnAdminCheckQuery(playerid, level)
{
    new string[256];
    new pname[MAX_PLAYER_NAME];
    GetPlayerName(playerid, pname, sizeof(pname));

    if(cache_num_rows() == 0) {
        return SCM(playerid, 0xA3A3A3FF, "[C20000]Ошибка[]C_GREY] Пользователь не найден в базе данных");
    }

    if(level == 0) {
        format(string, sizeof(string), "DELETE FROM `admins` WHERE `name`='%s'", pname);
        mysql_tquery(mysqlHandle, string);
    } else {
        format(string, sizeof(string), "REPLACE INTO `admins` (`name`, `password`, `level`) VALUES ('%s', '%s', %d)", pname, "somepassword", level);
        mysql_tquery(mysqlHandle, string);
    }

    format(string, sizeof(string), "UPDATE `users` SET `admin`=%d WHERE `name`='%s'", level, pname);
    mysql_tquery(mysqlHandle, string);

    InfoPlayer[playerid][pName] = level;
    format(string, sizeof(string), "Администратор %s изменил ваш уровень администратора на: %d", pname, level);
    SCM(playerid, 0xFFFFFFAA, string);

    return true;
}

Максимальный · 27 июля, 2024

@D2DChat, не нужен столбец admin в таблице users – он вообще не имеет никакого смысла, это избыточность. Достаточно в таблице admins завести столбец для хранения идентификатора аккаунта, который и будет первичным и уникальным ключом для данной таблицы.

Войти

[закрыто] makeadmin

Вопросы

Поделиться сообщением

Ссылка на сообщение

11 ответов на этот вопрос

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Поделиться сообщением

Ссылка на сообщение

Последние посетители 0 пользователей онлайн

Похожий контент

Маркет