Удаление личной машины игрока

[verteich]

Я сделал удаление машины игрока, но после удаления созданная следующая машина занимает ID удаленной и считается личной
Код в команде:

    DestroyDynamic3DTextLabel(CarText[carid]);    DestroyVehicleEx(OwnableCar[carid]);    OWNABLECARS --;

stock IsAOwnableCar(carid){    return (OwnableCar[0] <= carid <= OwnableCar[OWNABLECARS-1]);}

[DeimoS]

Ну так а что ты хочешь с такой проверкой? Либо цикл запускай, либо переделывай код создания авто, чтоб ID авто был равен ID ячейки в массиве OwnableCar.
То бишь:

OwnableCar[vehicleid] = CreateVehicle(...);  

И тогда проверка будет выглядеть так:

if(OwnableCar[vehicleid] != 0) // Транспорт личный  

Только, естественно, нужно обнулять ячейку после удаления авто.

[verteich]

Ну так а что ты хочешь с такой проверкой? Либо цикл запускай, либо переделывай код создания авто, чтоб ID авто был равен ID ячейки в массиве OwnableCar.
То бишь:

OwnableCar[vehicleid] = CreateVehicle(...);  

И тогда проверка будет выглядеть так:

if(OwnableCar[vehicleid] != 0) // Транспорт личный  

Только, естественно, нужно обнулять ячейку после удаления авто.

А какой цикл нужен?

[DeimoS]

А какой цикл нужен?

С перебором всего массива OwnableCar и проверкой того, равно ли значение какой-либо из ячеек ID автомобиля?

[verteich]

С перебором всего массива OwnableCar и проверкой того, равно ли значение какой-либо из ячеек ID автомобиля?

Не очень понял

CMD:dellcar(playerid, params[]) {    if(!PI[playerid][pFullDostup])        return 1;     new vehid;    if(sscanf(params, "d", vehid))        return SendClientMessage(playerid, COLOR_LIGHTRED, !"[Îøèáêà] {FFFFFF}Èñïîëüçóéòå: /dellcar [carid]");     new        carid = GetVehicleID(vehid);    CarInfo[carid][cID] = -1;    CarInfo[carid][cVirtualWorld] =    CarInfo[carid][cInterior] =    CarInfo[carid][cColor_1] =    CarInfo[carid][cColor_2] =    CarInfo[carid][cShtrafer] =    CarInfo[carid][cLock] =    CarInfo[carid][cShtraf] =    CarInfo[carid][cKeyIN] =    CarInfo[carid][cAlarmON] = 0;    CarInfo[carid][cMilage] = 0;    CarInfo[carid][cSell_X] = 0;    CarInfo[carid][cSell_Y] = 0;    CarInfo[carid][cSell_Z] = 0;    CarInfo[carid][cSell_A] = 0;    CarInfo[carid][cPos_X] = 0;    CarInfo[carid][cPos_Y] = 0;    CarInfo[carid][cPos_Z] = 0;    CarInfo[carid][cPos_A] = 0;    SetString(CarInfo[carid][cOwner], "The State");    SetString(CarInfo[carid][cKeyer], "The State");    SetString(CarInfo[carid][cNumber], "Arizona ST");    new string[128];    format(string, sizeof(string), "DELETE FROM `ownable` WHERE ID = %d", carid);    mysql_tquery(dbHandle, string);    DestroyDynamic3DTextLabel(CarText[carid]);    DestroyVehicleEx(OwnableCar[carid]);    OWNABLECARS --;    SendClientMessage(playerid, COLOR_LIGHTRED, !"Авто удалено");    return 1;}

Если делаю CreateVehicle сыпится run time error 4 на респавн авто после его созадния
Если после создания авто задебажить его ид в массиве, то он совпадает на сервере

[DeimoS]

После удаления автомобиля обнуляешь соответствующую ячейку в OwnableCar. А в IsAOwnableCar делаешь цикл, в котором каждая ячейка OwnableCar будет сверяться с переданным ID авто

[verteich]

После удаления автомобиля обнуляешь соответствующую ячейку в OwnableCar. А в IsAOwnableCar делаешь цикл, в котором каждая ячейка OwnableCar будет сверяться с переданным ID авто

так?

CMD:dellcar(playerid, params[]) {    if(!PI[playerid][pFullDostup])        return 1;     new        vehid,        carid = GetVehicleID(vehid));    if(sscanf(params, "d", vehid))        return SendClientMessage(playerid, COLOR_LIGHTRED, !"[Ошибка] {FFFFFF}Используйте: /dellcar [carid]");     if(OwnableCar[carid] != 0) {        CarInfo[carid][cID] = -1;        CarInfo[carid][cVirtualWorld] =        CarInfo[carid][cInterior] =        CarInfo[carid][cColor_1] =        CarInfo[carid][cColor_2] =        CarInfo[carid][cShtrafer] =        CarInfo[carid][cLock] =        CarInfo[carid][cShtraf] =        CarInfo[carid][cKeyIN] =        CarInfo[carid][cAlarmON] = 0;        CarInfo[carid][cMilage] = 0;        CarInfo[carid][cSell_X] = 0;        CarInfo[carid][cSell_Y] = 0;        CarInfo[carid][cSell_Z] = 0;        CarInfo[carid][cSell_A] = 0;        CarInfo[carid][cPos_X] = 0;        CarInfo[carid][cPos_Y] = 0;        CarInfo[carid][cPos_Z] = 0;        CarInfo[carid][cPos_A] = 0;        SetString(CarInfo[carid][cOwner], "The State");        SetString(CarInfo[carid][cKeyer], "The State");        SetString(CarInfo[carid][cNumber], "Arizona ST");        new string[128];        format(string, sizeof(string), "DELETE FROM `ownable` WHERE `ID` = '%i'", carid);        mysql_tquery(dbHandle, string);        DestroyDynamic3DTextLabel(CarText[carid]);        DestroyVehicleEx(OwnableCar[carid]);        OwnableCar[carid] = 0;        OWNABLECARS --;        SendClientMessage(playerid, COLOR_LIGHTRED, !"[Информация] {FFFFFF}Личная машина удалена!");    }    else {        SendClientMessage(playerid, COLOR_LIGHTRED, !"[Ошибка] {FFFFFF}Такой машины не существует!");    }    return 1;}

stock IsAOwnableCar(carid){    return (OwnableCar[0] <= carid <= OwnableCar[carid]);}

[DeimoS]

Нет =\

Дай код создания личных авто, в котором ты записываешь новое значение для OwnableCar

[verteich]

Нет =\

Дай код создания личных авто, в котором ты записываешь новое значение для OwnableCar

stock LoadedOwnable() {    for(new i; i < OWNABLECARS; i++) {        OwnableCar[i] = J_AddStaticVehicleEx(            CarInfo[i][cModel],            CarInfo[i][cPos_X],            CarInfo[i][cPos_Y],            CarInfo[i][cPos_Z],            CarInfo[i][cPos_A],            CarInfo[i][cColor_1],            CarInfo[i][cColor_2],            60000,            CarInfo[i][cInterior],            CarInfo[i][cVirtualWorld]        );        CarText[i] = CreateDynamic3DTextLabel("-",-1,0,0,0.5,5.0,INVALID_PLAYER_ID,OwnableCar[i]);        if(!GetString(CarInfo[i][cOwner],"The State"))        {            if(!CarInfo[i][cParkedInGarage] || !CarInfo[i][cParkedInParking]) SetVehicleVirtualWorld(OwnableCar[i], 1);            else SetVehicleVirtualWorld(OwnableCar[i], CarInfo[i][cVirtualWorld]);        }        if(CarInfo[i][cSupreme] == 1) SupremeInstall(i);        if(CarInfo[i][cTwinTurbo] == 1) TwinTurboInstall(i);        BikesInstall(i);    }    for(new i; i < OWNABLECARS; i++) {        if(OwnableCar[i] != 0) {            SetVehicleToRespawn(OwnableCar[i]);        }    }    return 1;}

проверял, ид авто на сервере равен массиву OwnableCar

[DeimoS]

Это он такой только после старта сервера. Если ты удалишь личное авто, создашь новое и потом создашь личное - ничего совпадать уже не будет.
У тебя вообще сейчас крайне криво и не надёжно построена система. Если ты собираешься подобным образом работать с массивом:

DestroyVehicleEx(OwnableCar[carid]);// Хотя это странный код. Зачем обращаться к массиву по ID авто, чтоб узнать этот самый ID авто?

То тебе нужно строить код загрузки как-то так:

vehicleid = J_AddStaticVehicleEx(            CarInfo[i][cModel],            CarInfo[i][cPos_X],            CarInfo[i][cPos_Y],            CarInfo[i][cPos_Z],            CarInfo[i][cPos_A],            CarInfo[i][cColor_1],            CarInfo[i][cColor_2],            60000,            CarInfo[i][cInterior],            CarInfo[i][cVirtualWorld]        );OwnableCar[vehicleid] = 1;

В этом случае ты действительно можешь работать с массивом согласно ID авто.

Собственно, как я говорил выше, можно накостылить цикл:

stock IsAOwnableCar(vehicleid){    for(new i; i < OWNABLECARS; i++)    {        if(OwnableCar[i] == vehicleid)            return 1;    }    return 0;}

Который будет работать, если переменную OwnableCar обнулять при удалении, но я бы советовал просто сесть и переписать систему в нормальный вид, сделав так, чтоб данные в массиве хранились согласно ID авто, а не абы как.