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

Вид для печати

01.09.2019, 21:58
verteich

Удаление личной машины игрока
Я сделал удаление машины игрока, но после удаления созданная следующая машина занимает ID удаленной и считается личной
Код в команде:
DestroyDynamic3DTextLabel(CarText[carid]);
DestroyVehicleEx(OwnableCar[carid]);
OWNABLECARS --;
stock IsAOwnableCar(carid)
{
return (OwnableCar[0] <= carid <= OwnableCar[OWNABLECARS-1]);
}
02.09.2019, 21:15
DeimoS

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

PHP код:

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

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

PHP код:

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

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

Цитата:

Сообщение от DeimoS

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

PHP код:

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

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

PHP код:

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

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

А какой цикл нужен?
02.09.2019, 21:45
DeimoS

Цитата:

Сообщение от verteich

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

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

Цитата:

Сообщение от DeimoS

С перебором всего массива 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 на респавн авто после его созадния
Если после создания авто задебажить его ид в массиве, то он совпадает на сервере

02.09.2019, 21:55
DeimoS

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

Цитата:

Сообщение от DeimoS

После удаления автомобиля обнуляешь соответствующую ячейку в 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]);
}

03.09.2019, 12:31
DeimoS

Нет =\

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

Цитата:

Сообщение от DeimoS

Нет =\

Дай код создания личных авто, в котором ты записываешь новое значение для 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

03.09.2019, 19:08
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 авто, а не абы как.