Добро пожаловать на Pro Pawn - Портал о PAWN-скриптинге.
Страница 1 из 2 1 2 ПоследняяПоследняя
Показано с 1 по 10 из 16
  1. #1
    Аватар для SteveStage
    Пользователь

    Статус
    Оффлайн
    Регистрация
    05.10.2019
    Адрес
    Планета Земля
    Сообщений
    318
    Репутация:
    7 ±

    Нерабочий /lock

    Помнится, раньше я спрашивал, как создать /lock. Позже я наткнулся на такую функцию нахождения транспорта поблизости:

    1. stock GetSearchVehicle(playerid, Float:distance)
    2. {
    3. new
    4. Float:veh_x,
    5. Float:veh_y,
    6. Float:veh_z,
    7. Float:pos_veh,
    8. search_vehicle = INVALID_VEHICLE_ID;
    9.  
    10. foreach(Vehicle, vehicle)
    11. {
    12. GetVehiclePos(vehicle, veh_x, veh_y, veh_z);
    13. pos_veh = GetPlayerDistanceFromPoint(playerid, veh_x, veh_y, veh_z);
    14. if(pos_veh <= distance)
    15. {
    16. search_vehicle = vehicle,
    17. distance = pos_veh;
    18. }
    19. }
    20. return search_vehicle;
    21. }


    Но в этой комманде ты можешь открыть любое авто независимо от владельца:

    1. CMD:lock(playerid, params[])
    2. {
    3. if(login_check{playerid} == false)
    4. return true;
    5. new vehicleid = GetPlayerVehicleID(playerid);
    6. if(vehicleid == 0)
    7. vehicleid = GetSearchVehicle(playerid, 5.0);
    8. if(vehicleid == INVALID_VEHICLE_ID || strcmp(veh[vehicleid][v_owner], GetName(playerid)))
    9. return SCM(playerid, COLOR_YELLOW, !"Вы должны находиться рядом со своим личным т/с, чтобы открыть его дверь!");
    10. new
    11. string[144+1];
    12. if(lock_car{vehicleid} == false)
    13. {
    14. format(string, sizeof(string), "*%s закрыл дверь т/с", GetName(playerid));
    15. ProxDetector(playerid, 20.0, COLOR_ACTION, string);
    16. GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
    17. SetVehicleParamsEx(vehicleid, engine, lights, alarm, VEHICLE_PARAMS_ON, bonnet, boot, objective);
    18. lock_car{vehicleid} = true;
    19. return true;
    20. }
    21. if(lock_car{vehicleid} == true)
    22. {
    23. format(string, sizeof(string), "*%s открыл дверь т/с", GetName(playerid));
    24. ProxDetector(playerid, 20.0, COLOR_ACTION, string);
    25. GetVehicleParamsEx(vehicleid, engine, lights, alarm, doors, bonnet, boot, objective);
    26. SetVehicleParamsEx(vehicleid, engine, lights, alarm, VEHICLE_PARAMS_OFF, bonnet, boot, objective);
    27. lock_car{vehicleid} = false;
    28. return true;
    29. }
    30. return true;
    31. }


    Что я делаю не так?

  2. #2
    Аватар для DeimoS
    Модератор?

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Во-первых, зачем ты после
    1. if(lock_car{vehicleid} == false)

    делаешь проверку
    1. if(lock_car{vehicleid} == true)

    если и так понятно, что переменная имеет значение "true"? Заменяешь вторую проверку на "else" и избавляешься от лишнего обращения к переменной lock_car

    Во-вторых, логируй содержимое v_owner и GetName. Как вариант, либо переменная содержит пустую строку, либо функция возвращает оную, из-за чего проверка и проходит (так как в strcmp есть баг).

    Вообще зачем делать проверку на ник? Автомобиль игрока есть смысл грузить только при авторизации этого самого игрока на сервере (точнее, лучше грузить только когда игрок сам запрашивает его), ибо если ты собрался грузить все личные авто при запуске сервера, то лимит в 2000 автомобилей на сервере быстро обломает тебя, когда ты запустишь сервер и нагонишь онлайна.
    А если данные грузятся только при авторизации игрока, то и для определения владельца можно просто сохранить ID игрока (а игроку - ID автомобиля, чтоб не тратить время на перебор всех авто циклом). Сверять строки тут вообще ни к чему.
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

    Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
    Великих идей полно, на них нет спроса.
    Воплощение идеи в законченную игру требует долгой работы,
    таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
    Предложить идею просто, воплотить – вот в чём проблема

    Steve Pavlina

  3. #3
    Аватар для SteveStage
    Пользователь

    Статус
    Оффлайн
    Регистрация
    05.10.2019
    Адрес
    Планета Земля
    Сообщений
    318
    Репутация:
    7 ±
    Цитата Сообщение от DeimoS Посмотреть сообщение
    Автомобиль игрока есть смысл грузить только при авторизации этого самого игрока на сервере (точнее, лучше грузить только когда игрок сам запрашивает его), ибо если ты собрался грузить все личные авто при запуске сервера, то лимит в 2000 автомобилей на сервере быстро обломает тебя, когда ты запустишь сервер и нагонишь онлайна.
    (100*3)+(16*5) = 390 < 2000
    Последний раз редактировалось SteveStage; 08.12.2019 в 23:42.

  4. #4
    Аватар для SteveStage
    Пользователь

    Статус
    Оффлайн
    Регистрация
    05.10.2019
    Адрес
    Планета Земля
    Сообщений
    318
    Репутация:
    7 ±
    Цитата Сообщение от DeimoS Посмотреть сообщение
    А если данные грузятся только при авторизации игрока, то и для определения владельца можно просто сохранить ID игрока (а игроку - ID автомобиля, чтоб не тратить время на перебор всех авто циклом). Сверять строки тут вообще ни к чему.
    Немного не понял сути, а как тогда делать запросы на ник игрока по его айди, если его айди = айди авто? И что ты вообще подразумевал, когда писал эти строки?

  5. #5
    Аватар для DeimoS
    Модератор?

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Цитата Сообщение от Steve_Stage Посмотреть сообщение
    Немного не понял сути, а как тогда делать запросы на ник игрока по его айди, если его айди = айди авто? И что ты вообще подразумевал, когда писал эти строки?
    Какие запросы на ник игрока? В таблице с автомобилями у тебя вообще ников быть не должно. Там должен быть ID аккаунта владельца.
    При авторизации получаешь ID аккаунта вместе со всеми данными из таблицы "account" и делаешь запрос в таблицу с автомобилями, выгружая данные по этому самому ID. И уже дальнейшую работу с таблицей для игрока (обновление/чтение данных) производишь либо опять по ID аккаунта владельца, либо по номеру строки автомобиля из таблицы с автомобилями, который так же можешь выгрузить в отдельную переменную.

    ID игрока - это его ID в игре. По этому ID ты можешь напрямую обращаться к массиву игрока, получая любые данные.
    А у игрока будет хранится ID автомобиля, который сразу будет равен и индексу в ячейке массива с автомобилями, и ID автомобиля на сервере.
    Грубо говоря, так:
    1. enum e_PLAYER_INFO
    2. {
    3. /*...*/
    4. pVehID,// Тут будет хранится значение vehicleid, которое ты получишь, когда создашь автомобиль при загрузке данных
    5. /*...*/
    6. };
    7. new pInfo[MAX_PLAYERS][e_PLAYER_INFO];
    8. enum e_PLAYER_VEHICLE_INFO
    9. {
    10. /*...*/
    11. pTableRowID,// Тут будет хранится значение поля 'id'
    12. pOwnerID,// Тут будет хранится значение playerid для владельца
    13. pOwnerTableID,// Тут будет хранится значение поля 'owner_id', содержащее номер строки поля 'id' из таблицы с аккаунтами
    14. /*...*/
    15. };
    16. new pVehInfo[MAX_VEHICLES][e_PLAYER_VEHICLE_INFO];
    17.  
    18. //Соответственно, чтоб проверить, есть ли у автомобиля владелец, достаточно будет сделать так:
    19. if(pVehInfo[vehicleid][pOwnerTableID] == 0))// Владельца нет.
    20. if(pVehInfo[vehicleid][pOwnerID] != playerid))// Владелец не игрок.
    21. // И т.п.
    22.  
    23. //А для взаимодействия напрямую с транспортом игрока можно сделать так:
    24. new vehicleid = pInfo[playerid][pVeh];
    25. // И уже после проверки vehicleid на валидность (существует ли у игрока автомобиль), можно напрямую обращаться к pVehInfo
    26. pVehInfo[vehicleid][/*Нужная константа*/]
    Последний раз редактировалось DeimoS; 09.12.2019 в 02:30.
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

    Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
    Великих идей полно, на них нет спроса.
    Воплощение идеи в законченную игру требует долгой работы,
    таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
    Предложить идею просто, воплотить – вот в чём проблема

    Steve Pavlina

  6. #6
    Аватар для SteveStage
    Пользователь

    Статус
    Оффлайн
    Регистрация
    05.10.2019
    Адрес
    Планета Земля
    Сообщений
    318
    Репутация:
    7 ±
    Вот так синтаксически и логически правильно?

    1. case DLG_ID_CAR_BUY:
    2. {
    3. if(response)
    4. {
    5. new
    6. i = GetPVarInt(playerid, "BuyCarID");
    7. if(player[playerid][p_money] < veh[i][v_price])
    8. {
    9. SCM(playerid, COLOR_RED, !"[SERVER] У вас недостаточно денег, чтобы купить этот транспорт!");
    10. return RemovePlayerFromVehicle(playerid);
    11. }
    12. else
    13. {
    14. player[playerid][p_money]-=veh[i][v_price];
    15. SavePlayer(playerid, "Money", player[playerid][p_money], "d");
    16. new
    17. string[128+1];
    18. format(string, sizeof(string), "UPDATE `vehicles` SET `ownerid` = '%d' WHERE `id` = '%d'", playerid, veh[i][v_id]);
    19. mysql_function_query(connect_mysql, string, false, "", "");
    20.  
    21. format(string, sizeof(string), "UPDATE `accounts` SET `vehid` = '%d' WHERE `id` = '%d'", veh[i][v_id], player[playerid][p_id]);
    22. mysql_function_query(connect_mysql, string, false, "", "");
    23. }
    24. }
    25. else
    26. return RemovePlayerFromVehicle(playerid);
    27. }


    При загрузке аккаунта

    1. format(string, sizeof(string), "SELECT * FROM `vehicles` WHERE `id` = '%d'", player[playerid][p_vehid]);
    2. mysql_function_query(connect_mysql, string, true, "@_LoadVehs", "dd", playerid, player[playerid][p_vehid]);


    1. @_LoadVehs(playerid, vehicleid);
    2. @_LoadVehs(playerid, vehicleid)
    3. {
    4. new
    5. rows,
    6. fields;
    7. cache_get_data(rows, fields);
    8. if(rows)
    9. {
    10. new
    11. veh_id,
    12. veh_model,
    13. Float:veh_x,
    14. Float:veh_y,
    15. Float:veh_z,
    16. Float:veh_a,
    17. Float:veh_hp,
    18. veh_color1,
    19. veh_color2,
    20. veh_dmg1,
    21. veh_dmg2,
    22. veh_dmg3,
    23. veh_dmg4,
    24. veh_paintjob;
    25.  
    26. veh_model = cache_get_field_content_int(0, "model");
    27. veh_x = cache_get_field_content_float(0, "x");
    28. veh_y = cache_get_field_content_float(0, "y");
    29. veh_z = cache_get_field_content_float(0, "z");
    30. veh_a = cache_get_field_content_float(0, "a");
    31. veh_hp = cache_get_field_content_float(0, "hp");
    32. veh_color1 = cache_get_field_content_int(0, "color1");
    33. veh_color2 = cache_get_field_content_int(0, "color2");
    34. veh_dmg1 = cache_get_field_content_int(0, "dmg1");
    35. veh_dmg2 = cache_get_field_content_int(0, "dmg2");
    36. veh_dmg3 = cache_get_field_content_int(0, "dmg3");
    37. veh_dmg4 = cache_get_field_content_int(0, "dmg4");
    38. veh_paintjob = cache_get_field_content_int(0, "paintjob");
    39.  
    40. veh_id = CreateVehicle(veh_model, veh_x, veh_y, veh_z, veh_a, veh_color1, veh_color2, 0);
    41. if(veh_id != INVALID_VEHICLE_ID) return true;
    42. UpdateVehicleDamageStatus(veh_id, veh_dmg1, veh_dmg2, veh_dmg3, veh_dmg4);
    43. SetVehicleHealth(veh_id, veh_hp);
    44. if(veh_paintjob > 0)
    45. ChangeVehiclePaintjob(veh_id, veh_paintjob-1); // 1, 2, 3 - нумерация винилов по порядку, 4 - отсутствие винил
    46. allvehs++;
    47. veh[veh_id][v_price] = ModelPrice[veh_model-400];
    48. veh[veh_id][v_model] = veh_model;
    49. veh[veh_id][v_x] = veh_x;
    50. veh[veh_id][v_y] = veh_y;
    51. veh[veh_id][v_z] = veh_z;
    52. veh[veh_id][v_a] = veh_a;
    53. veh[veh_id][v_hp] = veh_hp;
    54. veh[veh_id][v_color1] = veh_color1;
    55. veh[veh_id][v_color2] = veh_color2;
    56. veh[veh_id][v_dmg1] = veh_dmg1;
    57. veh[veh_id][v_dmg2] = veh_dmg2;
    58. veh[veh_id][v_dmg3] = veh_dmg3;
    59. veh[veh_id][v_dmg4] = veh_dmg4;
    60. veh[veh_id][v_paintjob] = veh_paintjob;
    61. veh[veh_id][v_class] = cache_get_field_content_int(0, "class");
    62. veh[veh_id][v_ownerid] = playerid;
    63. veh[veh_id][v_id] = vehicleid;
    64. static
    65. fmt_str[] = "UPDATE `vehicles` SET `ownerid` = '%d' WHERE `id` = '%d'";
    66. new
    67. string[sizeof(fmt_str)-4+4*2+1];
    68. format(string, sizeof(string), fmt_str, playerid, vehicleid);
    69. mysql_function_query(connect_mysql, string, false, "", "");
    70. }
    71. return true;
    72. }


    UPD: Проверил, но машины не загружаются
    Честно уже не знаю, правильно ли логически построена эта система, голова раскалывается, мог написать полнейший бред :)
    Последний раз редактировалось SteveStage; 09.12.2019 в 22:38.

  7. #7
    Аватар для DeimoS
    Модератор?

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Сильно не вчитывался в код, но почему ты, например, вот тут
    PHP код:
    format(stringsizeof(string), "UPDATE `vehicles` SET `ownerid` = '%d' WHERE `id` = '%d'"playeridveh[i][v_id]); 
    в ownerid записываешь playerid?
    Отдохни немного и вернись к коду на свежую голову :)
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

    Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
    Великих идей полно, на них нет спроса.
    Воплощение идеи в законченную игру требует долгой работы,
    таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
    Предложить идею просто, воплотить – вот в чём проблема

    Steve Pavlina

  8. #8
    Аватар для SteveStage
    Пользователь

    Статус
    Оффлайн
    Регистрация
    05.10.2019
    Адрес
    Планета Земля
    Сообщений
    318
    Репутация:
    7 ±
    Цитата Сообщение от DeimoS Посмотреть сообщение
    Сильно не вчитывался в код, но почему ты, например, вот тут
    PHP код:
    format(stringsizeof(string), "UPDATE `vehicles` SET `ownerid` = '%d' WHERE `id` = '%d'"playeridveh[i][v_id]); 
    в ownerid записываешь playerid?
    ownerid - айди владельца в таблице `vehicles`, тоесть playerid

  9. #9
    Аватар для DeimoS
    Модератор?

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Цитата Сообщение от Steve_Stage Посмотреть сообщение
    ownerid - айди владельца в таблице `vehicles`, тоесть playerid
    Эмм, а зачем он тебе в таблице? В таблице должен хранится ID аккаунта, а не ID игрока.
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

    Широко известно, что идеи стоят 0.8333 цента каждая (исходя из рыночной цены 10 центов за дюжину).
    Великих идей полно, на них нет спроса.
    Воплощение идеи в законченную игру требует долгой работы,
    таланта, терпения и креативности, не говоря уж о затратах денег, времени и ресурсов.
    Предложить идею просто, воплотить – вот в чём проблема

    Steve Pavlina

  10. #10
    Аватар для SteveStage
    Пользователь

    Статус
    Оффлайн
    Регистрация
    05.10.2019
    Адрес
    Планета Земля
    Сообщений
    318
    Репутация:
    7 ±
    Цитата Сообщение от DeimoS Посмотреть сообщение
    Эмм, а зачем он тебе в таблице? В таблице должен хранится ID аккаунта, а не ID игрока.
    Я же говорю - полнейший бред мог написать, сори)

    Правильней будет вот так:

    1. format(string, sizeof(string), "UPDATE `vehicles` SET `ownerid` = '%d' WHERE `id` = '%d'", player[playerid][p_id], veh[i][v_id]);


    Кстати, я понял, почему машина не загрузилась - я забыл в таблице `accounts` своему акку присвоить айди авто из `vehicles`

 

 
Страница 1 из 2 1 2 ПоследняяПоследняя

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

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

Эту тему просматривают: 1 (пользователей: 0 , гостей: 1)

Ваши права

  • Вы не можете создавать новые темы
  • Вы не можете отвечать в темах
  • Вы не можете прикреплять вложения
  • Вы не можете редактировать свои сообщения
  •