Краш сервера

[SteveStage]

Эмм, ну так ты данные грузишь в ячейки от 0 и дальше, а обращаться к ним пытаешься по ID авто.

Так что-ли?

stock ShowPTS(playerid, carid){    static           fmt_str[] =               "\                   {00C0FF}Владелец:\t\t\t\t{F81414}%s\                   \n{00C0FF}ID транспорта:\t\t\t\t{F81414}%d\                   \n{00C0FF}Модель:\t\t\t\t\t{F81414}%s\                   \n{00C0FF}Класс:\t\t\t\t\t{F81414}%s\               ";    new        string[sizeof(fmt_str)-8+MAX_PLAYER_NAME+11+25+1+1];    format(string, sizeof(string), fmt_str, car[car[carid][cID]][cName], car[carid][cID], ModelName[car[car[carid][cID]][cModel]-400],    car_class[car[car[carid][cID]][cClass]-1]);     SPD(playerid, 9866, DSM, !"{00C0FF}ПТС", string, !"{00C0FF}Понятно", !"");}

UPD: Да! Это сработало! Наконец-то, DeimoS большое спасибо за наводку)

[DeimoS]

Эмм, нет, не так. Стоит тебе нарушить порядок автомобилей (например, если игрок купит авто во время работы сервера), то твоя система перестанет работать. Тебе нужно загрузку данных переписывать так, чтоб ячейка, в которую записывается информация, совпадала с ID авто. Тогда твой текущий код будет работать надёжно. А сейчас тебе просто повезло, что всё сработало (и то, скорее всего, ты получил данные от не того автомобиля, от которого нужно было).

[SteveStage]

(и то, скорее всего, ты получил данные от не того автомобиля, от которого нужно было).

Я получил данные именно от того авто, которое у меня (владелец, класс, модель совпадают)

Эмм, нет, не так. Стоит тебе нарушить порядок автомобилей (например, если игрок купит авто во время работы сервера), то твоя система перестанет работать. Тебе нужно загрузку данных переписывать так, чтоб ячейка, в которую записывается информация, совпадала с ID авто.

Тоесть, по твоему, паблик LoadCars должен выглядить как-то так?

@_LoadCars();@_LoadCars(){    new         rows,         fields;    cache_get_data(rows, fields);     for(new i; i != rows; i++)    {        car[i][cModel] = cache_get_field_content_int(i, "Model");        car[i][cX] = cache_get_field_content_float(i, "X");        car[i][cY] = cache_get_field_content_float(i, "Y");        car[i][cZ] = cache_get_field_content_float(i, "Z");        car[i][cA] = cache_get_field_content_float(i, "A");        car[i][cHP] = cache_get_field_content_float(i, "HP");        car[i][cColor1] = cache_get_field_content_int(i, "Color1");        car[i][cColor2] = cache_get_field_content_int(i, "Color2");        car[i][cDmg1] = cache_get_field_content_int(i, "Dmg1");        car[i][cDmg2] = cache_get_field_content_int(i, "Dmg2");        car[i][cDmg3] = cache_get_field_content_int(i, "Dmg3");        car[i][cDmg4] = cache_get_field_content_int(i, "Dmg4");        car[i][cPrice] = cache_get_field_content_int(i, "Price");        //car[i][cID] = cache_get_field_content_int(i, "ID");        car[i][cID] = i;        car[i][cPaintjob] = cache_get_field_content_int(i, "Paintjob");        car[i][cClass] = cache_get_field_content_int(i, "Class");        cache_get_field_content(i, "Name", car[i][cName], MAX_PLAYER_NAME+1);        allcars++;        if(!strcmp(car[i][cName], "None"))            allbuycars++;        LoadInfoCar(i);    }    return true;}

[Kovshevoy]

Предложил бы такой вариант.

@_LoadCars();@_LoadCars(){    new        rows,        fields;     cache_get_data(rows, fields);     for(new i = 0, idx = 1; i < rows; i++, idx++)    {         car[idx][cModel] = cache_get_field_content_int(i, "Model");        car[idx][cX] = cache_get_field_content_float(i, "X");        car[idx][cY] = cache_get_field_content_float(i, "Y");        car[idx][cZ] = cache_get_field_content_float(i, "Z");        car[idx][cA] = cache_get_field_content_float(i, "A");        car[idx][cHP] = cache_get_field_content_float(i, "HP");        car[idx][cColor1] = cache_get_field_content_int(i, "Color1");        car[idx][cColor2] = cache_get_field_content_int(i, "Color2");        car[idx][cDmg1] = cache_get_field_content_int(i, "Dmg1");        car[idx][cDmg2] = cache_get_field_content_int(i, "Dmg2");        car[idx][cDmg3] = cache_get_field_content_int(i, "Dmg3");        car[idx][cDmg4] = cache_get_field_content_int(i, "Dmg4");        car[idx][cPrice] = cache_get_field_content_int(i, "Price");        //car[idx][cID] = cache_get_field_content_int(i, "ID");        car[idx][cID] = i;        car[idx][cPaintjob] = cache_get_field_content_int(i, "Paintjob");        car[idx][cClass] = cache_get_field_content_int(i, "Class");        cache_get_field_content(i, "Name", car[idx][cName], MAX_PLAYER_NAME+1);         allcars++;         if(!strcmp(car[idx][cName], "None"))            allbuycars++;         LoadInfoCar(i);    }    return true;}

- - - Добавлено - - -

Или же

@_LoadCars();@_LoadCars(){    new        rows,        fields;     cache_get_data(rows, fields);     for(new i = 0, idx = 1; i < rows; i++, idx++)    {        idx = cache_get_field_content_int(i, "ID");         ....

Тогда сразу будет грузиться ID тачки нужный. Хорошо это или плохо, думаю более подробней обьяснит DeimoS, ибо я не имею возможности нормально обьяснить :D

[DeimoS]

Эмм, а что-то существенно изменилось?

Нет. Создаёшь локальные переменные и сначала грузишь данные в них. После создаёшь автомобиль и уже копируешь данные из локальных переменных в массив.
Получится как-то так:

new vehicleid, /*остальные переменные*/;for(new i; i < rows; i++){    /*локальная переменная для cModel*/ = cache_get_field_content_int(i, "Model");    /*локальная переменная для cX*/ = cache_get_field_content_float(i, "X");    // И так далее...     vehicleid = CreateVehicle(/*локальная переменная для cModel*/, /*локальная переменная для cX*/, /*...*/);     car[vehicleid][cModel] = /*локальная переменная для cModel*/;    car[vehicleid][cX] = /*локальная переменная для cX*/;}

Тут где-то уже задавали схожий вопрос и я даже чуть ли не готовый код давал, вроде.

- - - Добавлено - - -

Вот, собственно, одна из таких тем (вроде, были ещё): *ссылка*

[SteveStage]

Вы упускаете, что LoadInfoCar в конце паблика - и есть код спавна транспорта

stock LoadInfoCar(i){    car[i][cVeh] = CreateVehicle(car[i][cModel], car[i][cX], car[i][cY], car[i][cZ], car[i][cA], car[i][cColor2], car[i][cColor2], 0);    UpdateVehicleDamageStatus(car[i][cVeh], car[i][cDmg1], car[i][cDmg2], car[i][cDmg3], car[i][cDmg4]);    SetVehicleHealth(car[i][cVeh], car[i][cHP]);    if(car[i][cPaintjob] > 0)        ChangeVehiclePaintjob(car[i][cVeh], car[i][cPaintjob]-1);    else if(!strcmp(car[i][cName], "None"))    {        car[i][cPrice] = ModelPrice[car[i][cModel]-400];        static const            fmt_str[] = "Автомобиль продается за %d$";        new            str[sizeof(fmt_str)-2+11+1];        format(str, sizeof(str), fmt_str, car[i][cPrice]);        car[i][cTDPrice] = CreateDynamic3DTextLabel(str, COLOR_YELLOW, 0, 0, 0.6, 20.0, INVALID_PLAYER_ID, i, 0, -1, -1, -1, 100.0);    }    return true;}

Или вы имеете ввиду, что можно обойтись без него одним пабликом?

[SteveStage]

@_LoadCars();@_LoadCars(){    new         rows,         fields;    cache_get_data(rows, fields);     new veh_id, veh_model, Float:veh_x, Float:veh_y, Float:veh_z, Float:veh_a, Float:veh_hp, veh_color1, veh_color2, veh_dmg1, veh_dmg2, veh_dmg3, veh_dmg4,        veh_price, veh_paintjob, veh_class, veh_name[MAX_PLAYER_NAME+1], Text3D:veh_tdprice;    for(new i; i < rows; i++)    {        veh_model = cache_get_field_content_int(i, "Model");        veh_x = cache_get_field_content_float(i, "X");        veh_y = cache_get_field_content_float(i, "Y");        veh_z = cache_get_field_content_float(i, "Z");        veh_a = cache_get_field_content_float(i, "A");        veh_hp = cache_get_field_content_float(i, "HP");        veh_color1 = cache_get_field_content_int(i, "Color1");        veh_color2 = cache_get_field_content_int(i, "Color2");        veh_dmg1 = cache_get_field_content_int(i, "Dmg1");        veh_dmg2 = cache_get_field_content_int(i, "Dmg2");        veh_dmg3 = cache_get_field_content_int(i, "Dmg3");        veh_dmg4 = cache_get_field_content_int(i, "Dmg4");        veh_price = cache_get_field_content_int(i, "Price");        veh_paintjob = cache_get_field_content_int(i, "Paintjob");        veh_class = cache_get_field_content_int(i, "Class");        veh_id = cache_get_field_content_int(i, "ID");        cache_get_field_content(i, "Name", veh_name, MAX_PLAYER_NAME+1);         veh_id = CreateVehicle(veh_model, veh_x, veh_y, veh_z, veh_a, veh_color1, veh_color2, 0);        UpdateVehicleDamageStatus(veh_id, veh_dmg1, veh_dmg2, veh_dmg3, veh_dmg4);        SetVehicleHealth(veh_id, veh_hp);        if(veh_paintjob > 0)            ChangeVehiclePaintjob(veh_id, veh_paintjob-1);        if(!strcmp(veh_name, "None"))        {            veh_price = ModelPrice[veh_model-400];            static                fmt_str[] = "Автомобиль продается за %d$";            new                str[sizeof(fmt_str)-2+11+1];            format(str, sizeof(str), fmt_str, veh_price);            veh_tdprice = CreateDynamic3DTextLabel(str, COLOR_YELLOW, 0, 0, 0.6, 20.0, INVALID_PLAYER_ID, i, 0, -1, -1, -1, 100.0);        }         allcars++;        if(!strcmp(veh_name, "None"))            allbuycars++;         car[veh_id][cModel] = veh_model;        car[veh_id][cX] = veh_x;        car[veh_id][cY] = veh_y;        car[veh_id][cZ] = veh_z;        car[veh_id][cA] = veh_a;        car[veh_id][cHP] = veh_hp;        car[veh_id][cColor1] = veh_color1;        car[veh_id][cColor2] = veh_color2;        car[veh_id][cDmg1] = veh_dmg1;        car[veh_id][cDmg2] = veh_dmg2;        car[veh_id][cDmg3] = veh_dmg3;        car[veh_id][cDmg4] = veh_dmg4;        car[veh_id][cPrice] = veh_price;        car[veh_id][cPaintjob] = veh_paintjob;        car[veh_id][cClass] = veh_class;        car[veh_id][cID] = veh_id;        cache_get_field_content(i, "Name", car[veh_id][cName], MAX_PLAYER_NAME+1);    }    return true;}

Так? Но почему нельзя записывать значения сразу в энум, не записывая в переменные, а потом перезаписывать в энум, если энум все равно используется и в него все равно запишутся значения из бд? Даже в той теме ты писал, что можно лучше - избавиться от перезаписи

Догадка: чтобы обойтись без переменной цикла i, т.к. она используется только при старте сервера и вызывается один раз, а при покупке авто его нужно загружать повторно

[DeimoS]

Во-первых, потому что ты не сможешь так просто определить то, в какую ячейку можно записать данные, а какая уже заполнена.
Во-вторых, обращение к переменной будет происходить быстрее, нежели к массиву (хоть это и не столь существенно, но всё же лишним не будет).
В-третьих, код более опрятный за счёт коротких имён (то же не существенно, но и не лишнее).


Какое логическое предназначение вот этого?

 car[veh_id][cID] = veh_id;

На всякий случай помечу более явно:

car[veh_id][cID] = veh_id;

Ну а в целом, вроде, верно. Я бы ещё добавил проверку на значение в veh_id после создания авто, проверяя невалидные значения, которые возвращает CreateVehicle в случае, если по каким-то причинам не смогла создать автомобиль. Такого, по хорошему, происходить не должно, но лучше писать систему, которая будет устойчива к подобным ошибкам.

[SteveStage]

Какое логическое предназначение вот этого?

 car[veh_id][cID] = veh_id;

Записать в vID айди транспорта, иначе как я буду его использовать?

UPD: я сказал про vID потому, что подумал, что лучше будет назвать энум не car, а veh (самолет не автомобиль же)

veh[veh_id][vModel] = veh_model;veh[veh_id][vX] = veh_x;veh[veh_id][vY] = veh_y;veh[veh_id][vZ] = veh_z;veh[veh_id][vA] = veh_a;veh[veh_id][vHP] = veh_hp;veh[veh_id][vColor1] = veh_color1;veh[veh_id][vColor2] = veh_color2;veh[veh_id][vDmg1] = veh_dmg1;veh[veh_id][vDmg2] = veh_dmg2;veh[veh_id][vDmg3] = veh_dmg3;veh[veh_id][vDmg4] = veh_dmg4;veh[veh_id][vPrice] = veh_price;veh[veh_id][vPaintjob] = veh_paintjob;veh[veh_id][vClass] = veh_class;veh[veh_id][vID] = veh_id;cache_get_field_content(i, "Name", veh[veh_id][vName], MAX_PLAYER_NAME+1);

UPD2: можно строчку с ID просто закомментировать и использовать только vehicleid, подобно тому, что в энуме игрока хранится ID, но используется playerid

UPD3: может ты имел ввиду исправить на это?

veh[veh_id][vID] = i;

Я бы ещё добавил проверку на значение в veh_id после создания авто, проверяя невалидные значения, которые возвращает CreateVehicle в случае, если по каким-то причинам не смогла создать автомобиль. Такого, по хорошему, происходить не должно, но лучше писать систему, которая будет устойчива к подобным ошибкам.

Так?

if(veh_id == INVALID_VEHICLE_ID) return i--;

UPD: на всякий случай

if(veh_id == INVALID_VEHICLE_ID) return i--;else if(veh_id != INVALID_VEHICLE_ID) continue;

[SteveStage]

Мда... переписал я, как ты просил, и опять проблемы AMX Backt*** и т.д. в ShowPTS и dc_cmd_pts

@_LoadCars();@_LoadCars(){    new         rows,         fields;    cache_get_data(rows, fields);     new veh_id, veh_model, Float:veh_x, Float:veh_y, Float:veh_z, Float:veh_a, Float:veh_hp, veh_color1, veh_color2, veh_dmg1, veh_dmg2, veh_dmg3, veh_dmg4,        veh_price, veh_paintjob, veh_class, veh_name[MAX_PLAYER_NAME+1];    for(new i; i < rows; i++)    {        veh_model = cache_get_field_content_int(i, "Model");        veh_x = cache_get_field_content_float(i, "X");        veh_y = cache_get_field_content_float(i, "Y");        veh_z = cache_get_field_content_float(i, "Z");        veh_a = cache_get_field_content_float(i, "A");        veh_hp = cache_get_field_content_float(i, "HP");        veh_color1 = cache_get_field_content_int(i, "Color1");        veh_color2 = cache_get_field_content_int(i, "Color2");        veh_dmg1 = cache_get_field_content_int(i, "Dmg1");        veh_dmg2 = cache_get_field_content_int(i, "Dmg2");        veh_dmg3 = cache_get_field_content_int(i, "Dmg3");        veh_dmg4 = cache_get_field_content_int(i, "Dmg4");        veh_price = cache_get_field_content_int(i, "Price");        veh_paintjob = cache_get_field_content_int(i, "Paintjob");        veh_class = cache_get_field_content_int(i, "Class");        veh_id = cache_get_field_content_int(i, "ID");        cache_get_field_content(i, "Name", veh_name, MAX_PLAYER_NAME+1);         veh_id = CreateVehicle(veh_model, veh_x, veh_y, veh_z, veh_a, veh_color1, veh_color2, 0);        if(veh_id == INVALID_VEHICLE_ID) return i--;        else if(veh_id != INVALID_VEHICLE_ID) continue;        UpdateVehicleDamageStatus(veh_id, veh_dmg1, veh_dmg2, veh_dmg3, veh_dmg4);        SetVehicleHealth(veh_id, veh_hp);        if(veh_paintjob > 0)            ChangeVehiclePaintjob(veh_id, veh_paintjob-1); // 1, 2, 3 - нумерация винилов по порядку, 4 - отсутствие винил        if(!strcmp(veh_name, "None"))        {            veh_price = ModelPrice[veh_model-400];            static                fmt_str[] = "Автомобиль продается за %d$";            new                str[sizeof(fmt_str)-2+11+1];            format(str, sizeof(str), fmt_str, veh_price);            CreateDynamic3DTextLabel(str, COLOR_YELLOW, 0, 0, 0.6, 20.0, INVALID_PLAYER_ID, i, 0, -1, -1, -1, 100.0);        }         allvehs++;        if(!strcmp(veh_name, "None"))            allbuyvehs++;         veh[veh_id][vModel] = veh_model;        veh[veh_id][vX] = veh_x;        veh[veh_id][vY] = veh_y;        veh[veh_id][vZ] = veh_z;        veh[veh_id][vA] = veh_a;        veh[veh_id][vHP] = veh_hp;        veh[veh_id][vColor1] = veh_color1;        veh[veh_id][vColor2] = veh_color2;        veh[veh_id][vDmg1] = veh_dmg1;        veh[veh_id][vDmg2] = veh_dmg2;        veh[veh_id][vDmg3] = veh_dmg3;        veh[veh_id][vDmg4] = veh_dmg4;        veh[veh_id][vPrice] = veh_price;        veh[veh_id][vPaintjob] = veh_paintjob;        veh[veh_id][vClass] = veh_class;        veh[veh_id][vID] = i;        cache_get_field_content(i, "Name", veh[veh_id][vName], MAX_PLAYER_NAME+1);    }    return true;}

CMD:pts(playerid, params[]){    if(!IsPlayerInAnyVehicle(playerid))        return SCM(playerid, COLOR_RED, !"Вы должны находиться в транспорте, чтобы показать ПТС транспорта!");    new vehicleid = GetPlayerVehicleID(playerid);    if(strcmp(veh[vehicleid][vName], GetName(playerid)))        return SCM(playerid, COLOR_RED, !"Это не ваш транспорт!");    if(spawn_car{vehicleid} == true)        return SCM(playerid, COLOR_RED, !"У созданных администраторами транспортных средств нету ПТС");    new        targetid;     if(sscanf(params, "u", targetid))        return SCM(playerid, COLOR_RED, !"Образец показа ПТС транспорта: /pts [id]");    if(!IsPlayerConnected(targetid))        return SCM(playerid, COLOR_RED, !"Игрок с введенным вами ID не в сети!");     static const        fmt_str[] = "*%s показал ПТС автомобиля %s";     new        str[sizeof(fmt_str)-4+MAX_PLAYER_NAME*2+1];     format(str, sizeof(str), fmt_str, GetName(playerid), GetName(targetid));    ProxDetector(playerid, 20.0, COLOR_ACTION, str);    ShowPTS(targetid, vehicleid);    return true;}

stock ShowPTS(playerid, vehid){    static           fmt_str[] =               "\                   {00C0FF}Владелец:\t\t\t\t{F81414}%s\                   \n{00C0FF}ID транспорта:\t\t\t\t{F81414}%d\                   \n{00C0FF}Модель:\t\t\t\t\t{F81414}%s\                   \n{00C0FF}Класс:\t\t\t\t\t{F81414}%s\               ";    new        string[sizeof(fmt_str)-8+MAX_PLAYER_NAME+11+25+1+1];    format(string, sizeof(string), fmt_str, veh[vehid][vName], veh[vehid][vID], ModelName[veh[vehid][vModel]-400],    car_class[veh[vehid][vClass]-1]);     SPD(playerid, 9866, DSM, !"{00C0FF}ПТС", string, !"{00C0FF}Понятно", !"");}