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;
}

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

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


Ссылка на сообщение
Поделиться на другие сайты
  В 1/17/2022 в 20:13, 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/17/2022 в 20:35, 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 позволит изменить состояние двигателя на противоположное.

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

  открыть спойлер

 

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

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

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


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

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

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

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

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

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

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


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

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

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


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

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

  открыть спойлер

 

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


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

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

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

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

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


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

Войти

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


Войти

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

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

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

    • Dev_Persona
      От Dev_Persona
      Приветствую всех, у меня возник такой вопрос:
       
      - хочу изменить базовую регистрацию в админке в крмп, на Болле сложную то-есть, сейчас изначально пароль от админки может быть только из цифр, хотелось бы узнать как можно было бы сделать что бы можно было использовать и цифры и буквы, вот код:
       
      case DIALOG_ADMIN_REG:
         {
          if(!response) return FixKick(playerid);
       
          if(strval(inputtext) < 100000 || strval(inputtext) > 999999) return ShowPlayerDialog(playerid, DIALOG_ADMIN_REG, DIALOG_STYLE_INPUT, "{FF6347}"PROJECT_NAME" {ffffff}| Придумайте пароль", "{FFFFFF}Для того, чтобы продолжить дальше - придумайте и введите\n{FFFFFF}пароль от панели администратора в диалоговое поле ниже.\n\n{FFFFFF}Пароль должен быть из 6 цифр.\n", "Готово", "");
                   SetPlayerData(playerid, P_APASS, strval(inputtext));
       
                   UpdatePlayerDatabaseInt(playerid, "AdminPassword", GetPlayerData(playerid, P_APASS));
       
                   AdminAuthorization(playerid);
         }
    • MetaLife Online
      От MetaLife Online
      Приветствую! Код мода не отпровляет данные на БД все правильно подключенно и написанно(копировал с phpmyadmin) но данные не отпровляет. но с БД на сервер приходит сообщение 
       
      new MySQL:dbHandle; new MySQL:dbHandle; #define Host "127.0.0.1" #define User "root" #define DataBase "metalife" #define Password_SQL "" dbHandle = mysql_connect(Host,User,Password_SQL, DataBase);     if(mysql_errno())     {         printf("Подключение к базе данных %s не удалось.",DataBase); // %s означает строку. То есть будет выводить название базы данных которое указано в #define DataBase     }     else     {         printf("Подключение к базе данных %s успешно",DataBase);     }     format(query, sizeof(query), "INSERT INTO `Users`(`Nick`, `Password`, `Gender`, `Skin`) VALUES ('%s','%s','%d','%d')",player_name, player_info[playerid][P_PASS], player_info[playerid][P_GENDER], player_info[playerid][P_SKIN]);     mysql_tquery(dbHandle, query); SA-MP Dedicated Server ---------------------- v0.3.7-R2, (C)2005-2015 SA-MP Team [15:34:54] weburl = "www.sa-mp.com"  (string) [15:34:54]  [15:34:54] Server Plugins [15:34:54] -------------- [15:34:54]  Loading plugin: mysql [15:34:55]  >> plugin.mysql: R41-4 successfully loaded. [15:34:55]   Loaded. [15:34:55]  Loaded 1 plugins. [15:34:55]  [15:34:55] Filterscripts [15:34:55] --------------- [15:34:55]   Loaded 0 filterscripts. [15:34:55] Подключение к базе данных metalife успешно [15:34:55]  ---------------------------------- [15:34:55]  MetaLife Online [15:34:55] ---------------------------------- [15:34:55] Number of vehicle models: 1 При Регестрации не опровляються данные (фото)
      Но если записать рукой данные (фото 2)  
      выполняеться эта эта функция(фото 3(чат) и код )
      forward CheckAccountFromMySql(playerid); public CheckAccountFromMySql(playerid) {     new str[256];     if(cache_num_rows() == 0)     {         new player_name[MAX_PLAYER_NAME + 1];         GetPlayerName(playerid, player_name, MAX_PLAYER_NAME);         format(str, sizeof(str), " {ffffff}Добро пожаловать!\n Аккаунта с таким ником %s нет\nДля Регестрации придумайте и введите пароль в поле ниже!", player_name);         SPD(playerid, DLG_REG, DSP, "Регестрация", str, "Войти", "Выйти");     }     else     {         SCM(playerid, color_green, "Вы  зарегестрырованы!");     }     return 1; } // выполняеться else  ошибу и варнинги mysql
      [15:47:39] [WARNING] mysql_connect: no password specified (C:\Project\Metalife\gamemodes\MetaLife.pwn:92) dbHandle = mysql_connect(Host,User,Password_SQL, DataBase); // 92 строка