Добро пожаловать на Pro Pawn - Портал о PAWN-скриптинге.
Показано с 1 по 8 из 8
  1. #1
    Аватар для Paradox
    Пользователь

    Статус
    Оффлайн
    Регистрация
    08.06.2015
    Адрес
    Запорожье
    Сообщений
    117
    Репутация:
    1 ±

    Вопрос по sscanf

    Доброе время суток.
    Возник такой вот вопрос по sscanf, как исправить данное предупреждение?
    PHP код:
    sscanf warningStrings without a length are deprecatedplease add a destination size
    Сама команда:
    PHP код:
    COMMAND:test(playeridparams[])
    {
        new 
    targetidreason[16];
        if(
    sscanf(params"is[16]"targetidreason)) return SendClientMessage(playeridCOLOR_WHITE"Usage: /test [playerid] [reason]");
        new 
    str[32];
        
    format(strsizeof(str), "%s"reason);
        
    SendClientMessage(playeridCOLOR_WHITEstr);
        return 
    true;


  2. #2
    Аватар для m1n1vv
    Пользователь

    Статус
    Оффлайн
    Регистрация
    09.09.2015
    Сообщений
    541
    Репутация:
    78 ±
    PHP код:
    new targetidstring:reason[16]; 
    PHP код:
    extract params -> new targetidstring:reason[16]; else
        return 
    SendClientMessage(playeridCOLOR_WHITE"Usage: /test [playerid] [reason]"); 
    Последний раз редактировалось m1n1vv; 14.08.2019 в 07:54.

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

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Ты уверен, что ругается на эту команду? Данное предупреждение должно вылезать при подобном коде(
    1. if(sscanf(params, "is", ...)

    То бишь, когда для "s" не указан размер.
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

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

    Steve Pavlina

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

    Статус
    Оффлайн
    Регистрация
    08.06.2015
    Адрес
    Запорожье
    Сообщений
    117
    Репутация:
    1 ±
    Цитата Сообщение от DeimoS Посмотреть сообщение
    Ты уверен, что ругается на эту команду? Данное предупреждение должно вылезать при подобном коде(
    1. if(sscanf(params, "is", ...)

    То бишь, когда для "s" не указан размер.
    Да на нее, дело в том что я раньше работал без sscanf'a, сейчас вот, поставил, и это пока одна команда, так сказать создавал ее для теста.
    P.S я так понял оно вылетает когда ввожу больше 16 символов.

  5. #5
    Аватар для m1n1vv
    Пользователь

    Статус
    Оффлайн
    Регистрация
    09.09.2015
    Сообщений
    541
    Репутация:
    78 ±
    Используй extract params, раз вместо params используешь свои переменные

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

    Статус
    Оффлайн
    Регистрация
    08.06.2015
    Адрес
    Запорожье
    Сообщений
    117
    Репутация:
    1 ±
    спасибо, сейчас буду пробовать, отпишусь еще

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

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Цитата Сообщение от m1n1vv Посмотреть сообщение
    Используй extract params, раз вместо params используешь свои переменные
    А как он должен помочь? "extract params" - макрос, на выходе из которого получается тот же sscanf (точнее, unformat, но она перенимает весь функционал sscanf).

     Код из инклуда в качестве подтверждения
    1. // Это, собственно, sscanf и unformat, который является аналогом sscanf
    2. native sscanf(const data[], const format[], {Float,_}:...);
    3. native unformat(const data[], const format[], {Float,_}:...) = sscanf;
    4.  
    5. // А это уже extract, который превращается в unformat
    6. #define extract%0->%1; EXTRN%1;unformat(_:EXTRZ:EXTRV:EXTRX:%0,##,%1,,);
    7. #define unformat(_:EXTRZ:EXTRV:EXTRX:%0,##,%1);%2else if (unformat(_:EXTRV:EXTRX:%0,##,%1))
    8. // Тут ещё некоторое количество строк, "развёртывающих" макрос "extract"


    Собственно, вот что твой код генерирует
    1. new targetid, reason[16];if (unformat(_:EXTRV:EXTR8:EXTR9:EXTR0:EXTR1:EXTR2:EXTR3:EXTR8:EXTR9: EXTR8:EXTR9:EXTR0:EXTR1:EXTR2: params ,##is[16], targetid,reason))
    2. return SendClientMessage(playerid, COLOR_WHITE, "Usage: /test [playerid] [reason]");


    Убираем теги
    1. new targetid, reason[16];if (unformat(params ,##is[16], targetid,reason))
    2. return SendClientMessage(playerid, COLOR_WHITE, "Usage: /test [playerid] [reason]");


    Делаем нормальные переносы и заменяем unformat на sscanf:
    1. new targetid, reason[16];
    2. if(sscanf(params, "is[16]", targetid,reason))
    3. return SendClientMessage(playerid, COLOR_WHITE, "Usage: /test [playerid] [reason]");


    Ну и сравниваем с тем, что было у автора изначально, видя, что отличий нет:
    1. new targetid, reason[16];
    2. if(sscanf(params, "is[16]", targetid, reason))
    3. return SendClientMessage(playerid, COLOR_WHITE, "Usage: /test [playerid] [reason]");





    Тут одно единственное исправление может быть - увеличение значения в скобках до нужного. В идеале вообще не ограничивать игрока и давать вводить до 128 символов (именно столько помещается в окно ввода сообщения в чат за раз), ибо в этом нет никакого смысла: вы особо ничего не сэкономите от ограничения и, при этом, добавите неудобств игроку, которому придётся высчитывать количество символов, которые он ввёл, так как в SA-MP не предусмотрен такой счётчик в окне чата.

    Ну или забить и не делать ничего, ибо это просто предупреждение, которое не означает ничего плохого.
    Последний раз редактировалось DeimoS; 14.08.2019 в 15:36.
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

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

    Steve Pavlina

  8. #8
    Аватар для Daniel_Cortez
    "Это не хак, это фича"

    Статус
    Оффлайн
    Регистрация
    06.04.2013
    Адрес
    Novokuznetsk, Russia
    Сообщений
    2,192
    Репутация:
    2589 ±
    Буквально только что выложил новую версию dc_crashreport с выводом источников ошибок от sscanf2; советую проверить.
    Индивидуально в ЛС по скриптингу не помогаю. Задавайте все свои вопросы здесь (click).

 

 

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

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

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

Ваши права

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