Неверный алгоритм работы кода

**punkochel** · 06.02.2020, 19:10

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

Скинь весь код, связанный с проблемой, целиком: начиная с команды и заканчивая функциями.

Callback

Открыть/закрыть

vehicle_OnPlayerKeyStateChange:

Открыть/закрыть

garage_OnPlayerKeyStateChange:

Открыть/закрыть

cmd:

Открыть/закрыть

**DeimoS** · 06.02.2020, 19:51

if(GetPlayerVehicleID(playerid) == INVALID_VEHICLE_ID) return 1;

Эта проверка неправильная. GetPlayerVehicleID не возвращает INVALID_VEHICLE_ID.

if(GetPlayerVehicleID(playerid) == PlayerVehicleID[playerid])

Если в PlayerVehicleID хранится ID авто, в котором сидит игрок, то зачем? GetPlayerVehicleID хранит ту же самую информацию.

 if(newkeys == KEY_SUBMISSION

Если у тебя не стоит задачи сделать вызов функции только когда игрок жмёт только кнопку "2", не нажимая других кнопок, то проверка должна выглядеть так:

 if(newkeys & KEY_SUBMISSION

Хотя вообще код крайне странно организован. Зачем в каждой функции прописывать проверку нажатия клавиши, если проверку можно вынести в OnPlayerKeyStateChange и уже, в зависимости от нажатой кнопки, вызывать ту или иную функцию?

Сделай так и скинь логи после нажатия:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
    printf("\n\nOnPlayerKeyStateChange(%d, %d, %d)", playerid, newkeys, oldkeys);
    new result;
 
    result = vehicle_OnPlayerKeyStateChange(playerid, newkeys);
    printf("vehicle_OnPlayerKeyStateChange", result);
 
    if (result) {
        return 1;
    }
 
    result = garage_OnPlayerKeyStateChange(playerid, newkeys);
    printf("garage_OnPlayerKeyStateChange", result);
 
    if (result) {
        return 1;
     }
    return 1;
}

**punkochel** · 06.02.2020, 20:15

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

Если в PlayerVehicleID хранится ID авто, в котором сидит игрок, то зачем? GetPlayerVehicleID хранит ту же самую информацию.

PlayerVehicleID не хранит ID авто в котором сидит игрок, она хранит ID авто которое выбрал игрок.

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

Хотя вообще код крайне странно организован. Зачем в каждой функции прописывать проверку нажатия клавиши, если проверку можно вынести в OnPlayerKeyStateChange и уже, в зависимости от нажатой кнопки, вызывать ту или иную функцию?

Не подойдет, потому-что допустим в garage_KeyStateChange есть 2 проверки на кнопки, к примеру:

garage_OnPlayerKeyStateChange(playerid, newkeys)
{
    if(newkeys & KEY_SUBMISSION)
    {    
        return 1;
    }
    if(newkeys & KEY_CROUCH)
    {
        return 1;
    }
    return 0;
}

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

Сделай так и скинь логи после нажатия:

Открыть/закрыть

**Pa4enka** · 06.02.2020, 20:37

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
    printf("\n\nOnPlayerKeyStateChange(%d, %d, %d)", playerid, newkeys, oldkeys);
    new result;
 
    result = vehicle_OnPlayerKeyStateChange(playerid, newkeys);
    printf("vehicle_OnPlayerKeyStateChange: %d", result);
 
    if (result) {
        return 1;
    }
 
    result = garage_OnPlayerKeyStateChange(playerid, newkeys);
    printf("garage_OnPlayerKeyStateChange: %d", result);
 
    if (result) {
        return 1;
    }
    return 1;
}

Подправил код DeimoS'a.

**DeimoS** · 06.02.2020, 20:56

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

Не подойдет, потому-что допустим в garage_KeyStateChange есть 2 проверки на кнопки, к примеру:

garage_OnPlayerKeyStateChange(playerid, newkeys)
{
    if(newkeys & KEY_SUBMISSION)
    {    
        return 1;
    }
    if(newkeys & KEY_CROUCH)
    {
        return 1;
    }
    return 0;
}

Создай 2 функции. В чём проблема?
Такой реализацией у тебя куча лишних действий будет впустую каждое срабатывание OnPlayerKeyStateChange.

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

Подправил код DeimoS'a.

Да, спасибо. Тупанул :D

**punkochel** · 06.02.2020, 22:45

Я чет вообще курю, ладно бы мод бы переполнен разным функционалом, так ведь он чист. Осталось только на new.pawn попробовать.
Вот в этой ситуации выполняется функция что в if и в else:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
    if(newkeys & KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
    {
        if(GetPlayerVirtualWorld(playerid) == PlayerInfo[playerid][pHouseID])
        {
            for(new i; i < MAX_VEHICLE_PLAYER; i++)
            {
                if(IsPlayerInVehicle(playerid, PlayerInfo[playerid][pVehicle][i]))
                {
                    if(PlayerVehicleID[playerid] == 0)
                    {
                        PlayerVehicleID[playerid] = PlayerInfo[playerid][pVehicle][i];
                        SetVehicleVirtualWorld(PlayerVehicleID[playerid], 0);
                        LinkVehicleToInterior(PlayerVehicleID[playerid], 0);
                        SetVehiclePos(PlayerVehicleID[playerid], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosX], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosY], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosZ]);
                        SetVehicleZAngle(PlayerVehicleID[playerid], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosFacingAngle]);
                        SetPlayerVirtualWorld(playerid, 0);
                        SetPlayerInterior(playerid, 0);
                        PutPlayerInVehicle(playerid, PlayerVehicleID[playerid], 0);
                        return 1;
                    }
                    else
                    {
                        SCMError(playerid, "Для смены транспорта, все автомобили должны находится в гараже. Введите: /fixcar");
                        return 1;
                    }
                }
            }
        }
        else
        {
            new Float:carhp, engine, lights, alarm, doors, bonnet, boot, objective;
            if(!IsPlayerInAnyVehicle(playerid)) return 1;
            if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return 1;
            GetVehicleParamsEx(GetPlayerVehicleID(playerid), engine, lights, alarm, doors, bonnet, boot, objective);
            GetVehicleHealth(GetPlayerVehicleID(playerid), carhp);
            if(engine == 0)
            {
                if(VehicleInfo[GetPlayerVehicleID(playerid)][vFuel] <= 0)
                {
                    GameTextForPlayer(playerid,"~r~FUEL TANK IS EMPTY", 3000, 3);
                    return 1;
                }
                else if(carhp <= 300.0)
                {
                    GameTextForPlayer(playerid,"~r~ENGINE DAMAGED", 3000, 3);
                    return 1;
                }
                SetVehicleParamsEx(GetPlayerVehicleID(playerid), true, true, alarm, doors, bonnet, boot, objective);
            }
            else
            {
                SetVehicleParamsEx(GetPlayerVehicleID(playerid), false, false, alarm, doors, bonnet, boot, objective);
            }
            return 1;
        }
    }
    return 1;
}

**DeimoS** · 07.02.2020, 13:04

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

Я чет вообще курю, ладно бы мод бы переполнен разным функционалом, так ведь он чист. Осталось только на new.pawn попробовать.
Вот в этой ситуации выполняется функция что в if и в else:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
    if(newkeys & KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
    {
        if(GetPlayerVirtualWorld(playerid) == PlayerInfo[playerid][pHouseID])
        {
            for(new i; i < MAX_VEHICLE_PLAYER; i++)
            {
                if(IsPlayerInVehicle(playerid, PlayerInfo[playerid][pVehicle][i]))
                {
                    if(PlayerVehicleID[playerid] == 0)
                    {
                        PlayerVehicleID[playerid] = PlayerInfo[playerid][pVehicle][i];
                        SetVehicleVirtualWorld(PlayerVehicleID[playerid], 0);
                        LinkVehicleToInterior(PlayerVehicleID[playerid], 0);
                        SetVehiclePos(PlayerVehicleID[playerid], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosX], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosY], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosZ]);
                        SetVehicleZAngle(PlayerVehicleID[playerid], HouseInfo[PlayerInfo[playerid][pHouseID]][hCarPosFacingAngle]);
                        SetPlayerVirtualWorld(playerid, 0);
                        SetPlayerInterior(playerid, 0);
                        PutPlayerInVehicle(playerid, PlayerVehicleID[playerid], 0);
                        return 1;
                    }
                    else
                    {
                        SCMError(playerid, "Для смены транспорта, все автомобили должны находится в гараже. Введите: /fixcar");
                        return 1;
                    }
                }
            }
        }
        else
        {
            new Float:carhp, engine, lights, alarm, doors, bonnet, boot, objective;
            if(!IsPlayerInAnyVehicle(playerid)) return 1;
            if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return 1;
            GetVehicleParamsEx(GetPlayerVehicleID(playerid), engine, lights, alarm, doors, bonnet, boot, objective);
            GetVehicleHealth(GetPlayerVehicleID(playerid), carhp);
            if(engine == 0)
            {
                if(VehicleInfo[GetPlayerVehicleID(playerid)][vFuel] <= 0)
                {
                    GameTextForPlayer(playerid,"~r~FUEL TANK IS EMPTY", 3000, 3);
                    return 1;
                }
                else if(carhp <= 300.0)
                {
                    GameTextForPlayer(playerid,"~r~ENGINE DAMAGED", 3000, 3);
                    return 1;
                }
                SetVehicleParamsEx(GetPlayerVehicleID(playerid), true, true, alarm, doors, bonnet, boot, objective);
            }
            else
            {
                SetVehicleParamsEx(GetPlayerVehicleID(playerid), false, false, alarm, doors, bonnet, boot, objective);
            }
            return 1;
        }
    }
    return 1;
}

Если это сообщение о твоей изначальной проблеме, то Pa4enka выше скинул код, который так же нужно протестировать и скинуть логи.

**punkochel** · 07.02.2020, 18:07

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

Если это сообщение о твоей изначальной проблеме, то Pa4enka выше скинул код, который так же нужно протестировать и скинуть логи.

PHP код:


OnPlayerKeyStateChange(0, 16, 0)
[17:06:52] vehicle_OnPlayerKeyStateChange: 0
[17:06:52] garage_OnPlayerKeyStateChange: 0
[17:06:52] 

OnPlayerKeyStateChange(0, 0, 16)
[17:06:52] vehicle_OnPlayerKeyStateChange: 0
[17:06:52] garage_OnPlayerKeyStateChange: 0
[17:06:56] 

OnPlayerKeyStateChange(0, 16, 0)
[17:06:56] vehicle_OnPlayerKeyStateChange: 0
[17:06:56] garage_OnPlayerKeyStateChange: 0
[17:06:56] 

OnPlayerKeyStateChange(0, 0, 16)
[17:06:56] vehicle_OnPlayerKeyStateChange: 0
[17:06:56] garage_OnPlayerKeyStateChange: 0
[17:07:02] 

OnPlayerKeyStateChange(0, 512, 0)
[17:07:02] vehicle_OnPlayerKeyStateChange: 0
[17:07:02] garage_OnPlayerKeyStateChange: 1
[17:07:02] 

OnPlayerKeyStateChange(0, 0, 512)
[17:07:02] vehicle_OnPlayerKeyStateChange: 0
[17:07:02] garage_OnPlayerKeyStateChange: 0
[17:07:02] 

OnPlayerKeyStateChange(0, 512, 0)
[17:07:02] vehicle_OnPlayerKeyStateChange: 1
[17:07:02] 

OnPlayerKeyStateChange(0, 0, 512)
[17:07:02] vehicle_OnPlayerKeyStateChange: 0
[17:07:02] garage_OnPlayerKeyStateChange: 0
[17:07:08] 

OnPlayerKeyStateChange(0, 4, 0)
[17:07:08] vehicle_OnPlayerKeyStateChange: 0
[17:07:08] garage_OnPlayerKeyStateChange: 0

**Pa4enka** · 07.02.2020, 22:33

vehicle_OnPlayerKeyStateChange(playerid, newkeys)
{
    if(newkeys == KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
    {
        if(GetPlayerVehicleID(playerid) == PlayerVehicleID[playerid])
        {
            callcmd::engine(playerid);
            return 1;
        }
    }
    return 0;
}

Тыкнем пальцем в небо...

UPD: есть ещё вариант, что не проходит какое-то условие. Например массив PlayerVehicleID изначально хранит невалидное значение, ибо не обнуляется при входе игрока.

**DeimoS** · 09.02.2020, 21:07

PHP код:


OnPlayerKeyStateChange(0, 512, 0)
[17:07:02] vehicle_OnPlayerKeyStateChange: 0
[17:07:02] garage_OnPlayerKeyStateChange: 1
[17:07:02]

Судя по логам, у тебя срабатывает только garage_OnPlayerKeyStateChange в первый раз.

Попробуй

vehicle_OnPlayerKeyStateChange(playerid, newkeys)
{
    if(newkeys & KEY_SUBMISSION && GetPlayerState(playerid) == PLAYER_STATE_DRIVER)
    {
        if(GetPlayerVehicleID(playerid) == PlayerVehicleID[playerid])
        {
            callcmd::engine(playerid);
            return 1;
        }
    }
    return 0;
}
 
CMD:engine(playerid)
{
    new Float:carhp, engine, lights, alarm, doors, bonnet, boot, objective;
    if(GetPlayerVehicleID(playerid) == INVALID_VEHICLE_ID) return 1;
    if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return 1;
    GetVehicleParamsEx(GetPlayerVehicleID(playerid), engine, lights, alarm, doors, bonnet, boot, objective);
    GetVehicleHealth(GetPlayerVehicleID(playerid), carhp);
    if(engine <= 0)
    {
        if(VehicleInfo[GetPlayerVehicleID(playerid)][vFuel] <= 0)
        {
            return 1;
        }
        else if(carhp <= 300.0)
        {
            return 1;
        }
    }
    return 1;
}

И да

PHP код:


if(engine == 0)

Помимо "0" (VEHICLE_PARAMS_OFF) и "1" (VEHICLE_PARAMS_ON), у параметров автомобиля есть ещё значение "-1" (VEHICLE_PARAMS_UNSET), возвращаемое GetVehicleParamsEx когда параметр до этого не изменялся при помощи SetVehicleParamsEx. То бишь, твоя проверка внутри команды не сработает правильно Iя это уже поправил в своём коде).

+ у тебя в команде проверяется значение vehicle_OnPlayerKeyStateChange, которое задаётся только в garage_OnPlayerKeyStateChange

Тема: Неверный алгоритм работы кода

Опции темы

Поиск по теме

Информация о теме

Пользователи, просматривающие эту тему

Ваши права