Antoxa39

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

В теме 10 сообщений

И здрасте! Я начинающий скриптер,  и мне самому не помешали бы уроки) xD Но все же,я хочу вам дать готовый скрипт на заводку авто.
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. Ну вот и все!) Наш скрипт уже готов.

Отредактировано пользователем keyl
Причина: добавил форматирование

Заметка от _bogdan_ , создано

3.2. Код больше 10 строк необходимо брать в теги "Код" и "Спойлер". Как это сделать рассказано в разделе "FAQ".

Устное предупреждение.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
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.");
    }
это разве не проверка?

 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
1 минуту назад, Cawfee сказал:

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

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

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

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

@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 верно подметил про проверку на "игрок в автомобиле" и "игрок водитель". Я даже не обратил внимание на это 9_9

Отредактировано пользователем Cawfee

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

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

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

Отредактировано пользователем Cawfee

Заметка от Cawfee , создано

Для ответа пользователям используйте кнопку со значком собачки (@), а не кнопку "Цитата".

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

@Antoxa39 А можно сделать заводка двигателя через команду? Просто у меня мобайл проект.

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

@Vlad0s1k, вставить логику из обработчика нажатия клавиш в обработчик команды:

Спойлер

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

 

 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты

Для публикации сообщений создайте учётную запись или авторизуйтесь

Вы должны быть пользователем, чтобы оставить комментарий

Создать учетную запись

Зарегистрируйте новую учётную запись в нашем сообществе. Это очень просто!


Регистрация нового пользователя

Войти

Уже есть аккаунт? Войти в систему.


Войти

  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу

  • Похожий контент

    • Mister Odin
      От Mister Odin
       
      new
       g_teleport_list[39][E_TELEPORT_LIST] = {  {"Спавн пгт.Батырево", 1802.3612, 2508.4824, 15.8887},  {"Спавн г.Арзамас", -113.5887, 977.7222, 12.0346},     {"Спавн г.Южный", 2744.2770, -2446.7246, 21.8988},     {"Центральный банк", 1852.020385,2040.791381,15.892713},  {"Рублёвка", -837.7783, 888.6161, 12.7232},  {"Казино", 1332.4252, 2358.7014, 17.6641},  {"Битва за контейнеры", 614.5167, 1722.3566, 12.0709},  {"Автосалон низкого класса", 2493.9616, -722.9544, 12.3315},  {"Автосалон среднего класса", 1407.7449, 455.4801, 13.1630},  {"Автосалон высокого класса", -14.0637, 2614.5664, 10.9892},   {"Мотосалон Harley Davidson", 785.720153, 750.465087, 12.000024},  {"Автошкола", 1909.174438,2227.679687,15.708162},  {"Военкомат", 1916.778564,2302.411376,15.574637},  {"Правительство области", -139.140975,593.789611,12.145712},  {"Отдел полиции №1 (ГИБДД)", 1906.805786,-2234.382812,11.257631},  {"Отдел полиции №2 (УМВД)", 2581.309326,-2416.139892,21.960090},  {"Отдел ФСБ", 1823.904663,2095.636718,15.848405},  {"Городская больница г.Арзамас", -285.797210,581.851562,12.120290},   {"СМИ", -317.708038,821.221679,13.051450},  {"Воинская часть", 1703.197265,1678.025878,15.279437},  {"Арзамасская ОПГ", 438.705322,1046.505126,12.002637},  {"Батыревская ОПГ", 1941.203857,2161.006591,15.705187},  {"Лыткаринская ОПГ", -2346.002441,75.257041,21.002962},  {"Шахта", 2381.5727, 1726.4451, -2.1506},  {"Завод", -1062.6697, 2204.1894, 38.0964},  {"Инкосация", 1864.1374, 2011.7136, 15.8546},  {"Транспортная Компания", 2362.1115, 1972.5627, 15.5530},  {"База механиков", 1840.918090,-118.025146,15.695312},  {"Курьерская служба доставки", 2764.509765,-2396.882568,21.890625},  {"Аренда автобуса(Южный)", 2771.041992,-2454.780761,21.845964},  {"Аренда автобуса(Арзамас)", -127.487670,943.652648,12.142824},   {"Аренда автобуса(Батырево)", 1798.986328,2529.591552,15.664262},  {"Гоночная трасса", -1568.7784, 1611.8305, 36.3971},  {"Дом на горе", -789.5397, -456.7479, 741.1422},  {"СТО и тюнинг-ателье", 1856.1375, -122.3266, 15.6888},  {"Аренда транспорта пгт.Батырево", 1763.641723,2255.673339,15.865348},  {"Аренда транспорта г.Арзамас", -36.923530,1360.673583,12.002090},  {"Аренда транспорта г.Южный", 1966.599975,-2603.943603,10.820312},  {"Тайный интерьер в Лыткарино", -2424.5818, 2846.3291, 40.7908} };