[Решение] Заводка двигателя

[Antoxa39 Хороший 18]

И здрасте! Я начинающий скриптер,  и мне самому не помешали бы уроки)  Но все же,я хочу вам дать готовый скрипт на заводку авто.
1. new engine, lights, alarm, doors, bonnet, boot, objective; // Ко всем переменным (массивам) (если сделать ее локальной, то будут ерроры)

2. Далее переходим в паблик OnPlayerStateChange. Вставляем туда это:

if(IsPlayerInAnyVehicle(playerid))
    {
        SendClientMessage(playerid,-1,"[СИСТЕМА]  Для того чтобы завести двигатель нажмите клавишу N.");
    }

3. Все,с этим пабликом закончили,идем далее! Переходим сюды OnPlayerKeyStateChange.
и вот это  все, туда!
  

  new t,b;
    t = true;
    b = false;
    new carid = GetPlayerVehicleID(playerid);
    if(newkeys & KEY_NO)
    {
        GetVehicleParamsEx(carid, engine, lights, alarm, doors, bonnet, boot, objective);
        if(engine == b)
        {
              SetVehicleParamsEx(carid, t, lights, alarm, doors, bonnet, boot, objective);
        }
        else
        {
            SetVehicleParamsEx(carid, b, lights, alarm, doors, bonnet, boot, objective);
        }
    }

4. Ну вот и все!) Наш скрипт уже готов.

Edited January 17 by keyl
Причина: добавил форматирование

[Sleash`en Хороший 73]

Добрый день, небольшая ошибка:

public OnPlayerStateChange(playerid, newstate, oldstate)
{
    if(newstate == PLAYER_STATE_DRIVER)
    {
        SendClientMessage(playerid,-1,"[СИСТЕМА]  Для того чтобы завести двигатель нажмите клавишу N.");
    }
    return true;
}

Так как у вас нет проверки на то, что игрок только что сел в машину, по сути вот она и есть.

[Antoxa39 Хороший 18]

15 минут назад, Sleash`en сказал:

Добрый день, небольшая ошибка:

public OnPlayerStateChange(playerid, newstate, oldstate)
{
    if(newstate == PLAYER_STATE_DRIVER)
    {
        SendClientMessage(playerid,-1,"[СИСТЕМА]  Для того чтобы завести двигатель нажмите клавишу N.");
    }
    return true;
}

Так как у вас нет проверки на то, что игрок только что сел в машину, по сути вот она и есть.

if(IsPlayerInAnyVehicle(playerid))
    {
        SendClientMessage(playerid,-1,"[СИСТЕМА]  Для того чтобы завести двигатель нажмите клавишу N.");
    }
это разве не проверка?

 

[Cawfee Максимальный 443]

@Antoxa39, просто нет смысла вызывать нативную функцию в callback, который предоставляет возможность проверить текущее местонахождение игрока без использования нативных функций 

[Antoxa39 Хороший 18]

1 минуту назад, Cawfee сказал:

@Antoxa39, просто нет смысла вызывать нативную функцию в callback, который предоставляет возможность проверить текущее местонахождение игрока без использования нативных функций 

Извиняюсь,ведь я сам только изучаю павно) А так впринципе, как оцените? правильно или вообще плохо?)

[Sleash`en Хороший 73]

У вас стоит проверка на то, сидит ли игрок в любом виде авто, то есть:

1. Не учитывается то. за рулём он или нет, функция выдаёт "положительно" если игрок сидит в авто

2. Нет уточнений по времени и ситуации, но так как это паблик OnPlayerStateChange, то будет проверка при смене "статуса" на любой известный

А вообще код неплох, но ты видимо просто запутался в определённых функциях

P. S. Вместо true и false можно использовать 1 и 0 соответственно.

[Cawfee Максимальный 443]

@Antoxa39, ну, конкретно я бы учел следующие пункты:

1. Использование глобальных переменных не есть хорошо в тех случаях, когда их можно попытаться избежать. Многие для упрощения выносят создание переменных-аргументов engine, lights, alarm, doors, bonnet, boot, objective в глобальную область памяти, хотя необходимости в этом нет (разве что лень автора исходного текста программы). То, что у вас возникают ошибки при попытках сделать эти переменными локальными, - не есть хорошо: проблему нужно разрешать, а не избегать ее, вынося создание указанных переменных в глобальную область видимости.
2. Представим, что игрок нажал другую клавишу. В памяти будет выделено место для хранения переменных t, b и carid. Но зачем? По сути впустую выделяем место (и инициализируем переменные). Логичнее разместить их в более локальную область видимости, то есть внутри условной конструкции, предусматривающей нажатие клавиши KEY_NO.
3. Переменные t и b имеют очень интересные и непонятные названия. Хорошо бы их переименовать.
4. В принципе, вы можете обойтись какой-нибудь одной переменной: t или b. Соответственно, будет достаточно просто инвертировать значение этой переменной (из 0 делать 1 или из 1 делать 0 через оператор "НЕ", обозначающийся восклицательным знаком: '!').
5. Если игрок не в автомобиле, тогда в carid будет INVALID_VEHICLE_ID, для которого будет осуществляться попытка получить состояние всех компонентов.
6. Если игрок в автомобиле на пассажирском месте, нажатие клавиши N позволит изменить состояние двигателя на противоположное.

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

Спойлер

public OnPlayerStateChange(playerid, newstate, oldstate) {
    if(newstate == PLAYER_STATE_DRIVER) {
        SendClientMessage(playerid,-1,"[СИСТЕМА]  Для того чтобы завести двигатель нажмите клавишу N.");
    }
    return true;
}

 

if (newkeys & KEY_NO) {
    new vehicleid = GetPlayerVehicleID(playerid);
    if (vehicleid != INVALID_VEHICLE_ID) {
        if (GetPlayerState(playerid) == PLAYER_STATE_DRIVER) {
            new engine, lights, alarm, doors, bonnet, boot, objective;
            GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
            SetVehicleParamsEx(vehicleid, !engine, lights, alarm, doors, bonnet, boot, objective);  
        }
    }
}

 

 

Upd. Также пользователь @Sleash`en верно подметил про проверку на "игрок в автомобиле" и "игрок водитель". Я даже не обратил внимание на это 

Edited January 17 by Cawfee

[Antoxa39 Хороший 18]

@Cawfee, учту,я старался максимально коротко сделать)

@Sleash`en, ну спасибо за обьяснение) я пытался краткий скрипт двигателя. И да, я половину еще даже функций не знаю)

Edited January 17 by Cawfee