Помогите исправить алгоритм удаления дома (всегда удаляется последний дом)

[PawnoNoob]

Приветствую. Столкнулся с проблемой при создании одной из систем. В общем, есть система - удаление домов через диалоговое окно с использованием цикла. Для наглядности приложу код:

есть enum с переменными:

#define MAX_HOUSES (25) // для начала сделал именно 25, чтобы потом увеличивать;enum hinfo{    house_ID,    Float:house_X,    Float:house_Y,    Float:house_Z}new hInfo[MAX_HOUSES][hinfo], TOTALHOUSES;

естественно, есть и загрузка:

public LoadHouses(){    new rows;    cache_get_row_count(rows);    if(rows)    {        for(new i = 1; i <= rows; i++)        {            TOTALHOUSES++;            cache_get_value_name_int(i-1, !"id", hInfo[i][house_ID]);            cache_get_value_name_float(i-1, !"x", hInfo[i][house_X]);            cache_get_value_name_float(i-1, !"y", hInfo[i][house_Y]);            cache_get_value_name_float(i-1, !"z", hInfo[i][house_Z]);             // здесь планировал создание пикапов и тому подобных вещей, но не об этом речь;        }    }    return true;}

есть команда для удаления:

cmd:ahouse(playerid){    return ShowPlayerDialog(playerid, 0, DIALOG_STYLE_LIST, !"ahouse", !"1. Delete house", !"Ok", !"No");}

и, соответственно, код в самом диалоге:

...case 0:{    for(new i = 1; i <= TOTALHOUSES; i++)    {        // здесь показывает список домов от 1 до последнего (из TOTALHOUSES);    }}

Проблема в том, что при удалении, например, первого дома, его id остаётся в списке, но пропадает последний. В базе, соответственно, тоже удаляется (пикапы, тексты и тому подобные вещи удаляются тоже).

Сомневаюсь в решении, но нужно ли использовать в этой ситуации итераторы? Если да, то каким образом это реализовать можно?
Заранее благодарен за ответ!

[pawnoholic]

Сомневаюсь в решении, но нужно ли использовать в этой ситуации итераторы? Если да, то каким образом это реализовать можно?

Примерно, как то так:

new Iterator:House<MAX_HOUSES>; public LoadHouses(){    new         idx,        rows = cache_num_rows();     for (new i; i < rows; i++)    {        if ((idx = Iter_Free(House)) == INVALID_ITERATOR_SLOT) {            break;        }         cache_get_value_name_int(i, "id", hInfo[idx][house_ID]);        // ...         Iter_Add(House, idx);    }}

[PawnoNoob]

Примерно, как то так:

new Iterator:House<MAX_HOUSES>; public LoadHouses(){    new         idx,        rows = cache_num_rows();     for (new i; i < rows; i++)    {        if ((idx = Iter_Free(House)) == INVALID_ITERATOR_SLOT) {            break;        }         cache_get_value_name_int(i, "id", hInfo[idx][house_ID]);        // ...         Iter_Add(House, idx);    }}

А можете дать ссылку на тему, где есть урок с использованием итераторов?

И ещё. Можете сказать, как удалить этот итератор? Ну, при удалении дома, например

[pawnoholic]

А можете дать ссылку на тему, где есть урок с использованием итераторов?

http://forum.sa-mp.com/showthread.php?t=571159

И ещё. Можете сказать, как удалить этот итератор? Ну, при удалении дома, например

Iter_Remove(House, id);

[PawnoNoob]

http://forum.sa-mp.com/showthread.php?t=571159

Iter_Remove(House, id);

А как их выводить? Ну, список домов в диалоговое окно

[StevenH]

А как их выводить? Ну, список домов в диалоговое окно

С помощью foreach, например:

foreach(new h: House) 
{ 
    // какой то код 
}  

На самом деле итераторы довольно полезная и удобная штука)

[DeimoS]

Примерно, как то так:

new Iterator:House<MAX_HOUSES>; public LoadHouses(){    new         idx,        rows = cache_num_rows();     for (new i; i < rows; i++)    {        if ((idx = Iter_Free(House)) == ITER_NONE) {            break;        }         cache_get_value_name_int(i, "id", hInfo[idx][house_ID]);        // ...         Iter_Add(House, idx);    }}

Не совсем понятно зачем тут Iter_Free. У тебя ведь итератор имеет тот же размер, что и массив с домами, а значит логично просто проверить количество возвращённых строк, дабы получить тот же эффект, но без постоянного насилования Iter_Free (который, к слову, содержит в себе цикл). Да и не совсем понятен смысл поля "id", если у тебя есть итератор, который отлично сортирует данные.

И не стоит использовать cache_num_rows, если ты записываешь значение в массив :) Этим ты вдвойне насилуешь стэк, ибо cache_num_rows является обычным stock, в котором создаётся переменная и вызывается cache_get_row_count.


Я бы сделал так:
Поле id оставил бы в виде "AUTO_INCREMENT" и не трогал бы его. Пусть этот столбец будет нужен исключительно MySQL. А для ID домов на сервере я создал бы ещё один столбец (например, "house_id") и контролировал бы его уже сам.
Зачем это нужно? Во-первых, это позволит быть уверенным в том, что твой запрос на выгрузку данных вернёт ровно столько строк, сколько есть ячеек (если ты, конечно, не уменьшишь число ячеек в один момент, указав меньшее число, чем будет домов на сервере). Во-вторых, это позволит использовать этот ID в качестве индекса массива, тем самым ты сможешь обращаться к базе с домами, не обращаясь, при этом, к массиву, дабы получить ID дома, так как он будет равен индексу. Возможно, не очень понятно, так что покажу пример загрузки.

new Iterator:House<MAX_HOUSES>; public LoadHouses(){    new row_count;    cache_get_row_count(row_count);     if(row_count > MAX_HOUSES)// Проверка нужна как раз на случай, если ты решишь изменить значение MAX_HOUSES и укажешь его меньше, чем есть домов в базе    {        row_count = MAX_HOUSES;        print("Warning: Тут сообщение с предупреждением о том, что часть домов не загрузилось");    }     for (new i, idx; i < rows; i++)    {        cache_get_value_name_int(i, "house_id", idx);// Выгружаем ID дома         cache_get_value_name_float(i, "x", hInfo[idx][house_Y]);// И тут уже используем значение "idx" в качестве индекса        cache_get_value_name_float(i, "y", hInfo[idx][house_Y]);        cache_get_value_name_float(i, "z", hInfo[idx][house_Z]);         // ...         Iter_Add(House, idx);// + заносим индекс в итератор    }     printf("Загружено %d домов", row_count);    return 1;}

А при создании дома уже используем Iter_Free, тем самым сразу проверяя лимит домов и определяя свободный ID для базы

new id = Iter_Free(House);if(id == ITER_NONE) {    SendClientMessage(playerid, -1, "Лимит домов исчерпан")    return 1;} format(...., "INSERT INTO house(house_id,...)VALUE(%d,....)", id);// Указываем значение Iter_Free в качестве значения "house_id"// ...Iter_Add(House, id);

[PawnoNoob]

Не совсем понятно зачем тут Iter_Free. У тебя ведь итератор имеет тот же размер, что и массив с домами, а значит логично просто проверить количество возвращённых строк, дабы получить тот же эффект, но без постоянного насилования Iter_Free (который, к слову, содержит в себе цикл). Да и не совсем понятен смысл поля "id", если у тебя есть итератор, который отлично сортирует данные.

И не стоит использовать cache_num_rows, если ты записываешь значение в массив :) Этим ты вдвойне насилуешь стэк, ибо cache_num_rows является обычным stock, в котором создаётся переменная и вызывается cache_get_row_count.


Я бы сделал так:
Поле id оставил бы в виде "AUTO_INCREMENT" и не трогал бы его. Пусть этот столбец будет нужен исключительно MySQL. А для ID домов на сервере я создал бы ещё один столбец (например, "house_id") и контролировал бы его уже сам.
Зачем это нужно? Во-первых, это позволит быть уверенным в том, что твой запрос на выгрузку данных вернёт ровно столько строк, сколько есть ячеек (если ты, конечно, не уменьшишь число ячеек в один момент, указав меньшее число, чем будет домов на сервере). Во-вторых, это позволит использовать этот ID в качестве индекса массива, тем самым ты сможешь обращаться к базе с домами, не обращаясь, при этом, к массиву, дабы получить ID дома, так как он будет равен индексу. Возможно, не очень понятно, так что покажу пример загрузки.

new Iterator:House<MAX_HOUSES>; public LoadHouses(){    new row_count;    cache_get_row_count(row_count);     if(row_count > MAX_HOUSES)// Проверка нужна как раз на случай, если ты решишь изменить значение MAX_HOUSES и укажешь его меньше, чем есть домов в базе    {        row_count = MAX_HOUSES;        print("Warning: Тут сообщение с предупреждением о том, что часть домов не загрузилось");    }     for (new i, idx; i < rows; i++)    {        cache_get_value_name_int(i, "house_id", idx);// Выгружаем ID дома         cache_get_value_name_float(i, "x", hInfo[idx][house_Y]);// И тут уже используем значение "idx" в качестве индекса        cache_get_value_name_float(i, "y", hInfo[idx][house_Y]);        cache_get_value_name_float(i, "z", hInfo[idx][house_Z]);         // ...         Iter_Add(House, idx);// + заносим индекс в итератор    }     printf("Загружено %d домов", row_count);    return 1;}

А при создании дома уже используем Iter_Free, тем самым сразу проверяя лимит домов и определяя свободный ID для базы

new id = Iter_Free(House);if(id == INVALID_ITERATOR_SLOT) {    SendClientMessage(playerid, -1, "Лимит домов исчерпан")    return 1;} format(...., "INSERT INTO house(house_id,...)VALUE(%d,....)", id);// Указываем значение Iter_Free в качестве значения "house_id"// ...Iter_Add(House, id);

И как потом, допустим, по номеру (id) дома посмотреть его информацию? Или присвоить значение pHouse игроку (которое равно id дома)

[DeimoS]

И как потом, допустим, по номеру (id) дома посмотреть его информацию?

Эмм, так во всех случаях, когда ты работаешь с домами на сервере, у тебя есть индекс этого дома в массиве hInfo. Вот точно так же, как ты получаешь индекс для массива, получай ID дома - теперь это одно и то же.

Только теперь не забывай добавлять "LIMIT 1" ко всем запросам, где нужно изменить/получить лишь какую-то одну строку, дабы MySQL не продолжал обрабатывать все строки после нахождения нужной. В случае с выборкой по "AUTO_INCREMENT" MySQL делает это автоматически, ибо "A_I" поля не могут повторять значения и логично, что если одна строка найдена, то искать дальше нет смысла. А вот в случае с выборкой для полей без "AUTO_INCREMENT" лучше указывать "LIMIT" явно.

Или присвоить значение pHouse игроку (которое равно id дома)

При авторизации делай запрос в базу с домами и ищи ID аккаунта игрока среди ID владельцев (надеюсь, ты не хранишь в базе с домами ники, а хранишь ID аккаунтов).

[StevenH]

Дополню.. довольно правильное решение при авторизации аккаунта делать запрос в базу с домами и таким образом проверять владельца и записать в переменную, ту же pHouse (но без сохранения, и соответственно с обнулением при коннекте)..

Почему же?

Может быть довольно много проблем, например не раз замечал, что дом у игрока якобы есть, а в таблице с домами его нет (или наоборот). Конечно, тут косяк самой реализации, но лучше сделать удобнее и потом не мучаться!)