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

    Статус
    Оффлайн
    Регистрация
    26.01.2019
    Сообщений
    41
    Репутация:
    6 ±

    Есть ли смысл в такой оптимизации?

    Что думаете по поводу такой оптимизации?

    1. stock IsVehicleMoving(vehicleid)
    2. {
    3. static Float:x, Float:y, Float:z;
    4. return (GetVehicleVelocity(vehicleid, x, y, z) != 0) &&
    5. ((_:x != 0) || (_:y != 0) || (_:z != 0));
    6. }
    7.  
    8. stock IsVehicleBackwordsDirection(vehicleid)
    9. {
    10. static Float:x, Float:y, Float:z, Float:z_angle;
    11. if (GetVehicleVelocity(vehicleid, x, y, z)) {
    12. if (_:z_angle < 0x42b40000) {
    13. if (_:x > 0 && _:y < 0)
    14. return 1;
    15. } else if (_:z_angle < 0x43340000) {
    16. if (_:x > 0 && _:y > 0)
    17. return 1;
    18. } else if (_:z_angle < 0x43870000) {
    19. if (_:x < 0 && _:y > 0)
    20. return 1;
    21. } else {
    22. if (_:x < 0 && _:y < 0)
    23. return 1;
    24. }
    25. }
    26. return 0;
    27. }


    P.S. оптимизация была сделана для таймера. И да, я знаю, что это снижает читаемость кода.

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

    Статус
    Оффлайн
    Регистрация
    06.03.2017
    Сообщений
    82
    Репутация:
    0 ±
    По поводу читаемости кода: тебе ничто не мешает юзать дефайны.

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

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

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

    Steve Pavlina

  4. Пользователь сказал cпасибо:
    execution (08.08.2019)
  5. #4
    Аватар для x86
    Пользователь

    Статус
    Оффлайн
    Регистрация
    26.01.2019
    Сообщений
    41
    Репутация:
    6 ±
    Цитата Сообщение от DeimoS Посмотреть сообщение
    А в чём, собственно, оптимизация? Показал бы хотя бы вариант до оптимизации, чтоб было с чем сравнить.
    Из кода были убраны все floatcmp вызовы.

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

    Статус
    Оффлайн
    Регистрация
    04.01.2015
    Адрес
    Гомель, Беларусь
    Сообщений
    547
    Репутация:
    158 ±
    Цитата Сообщение от x86 Посмотреть сообщение
    Из кода были убраны все floatcmp вызовы.
    На самом деле в экономии это мизер, а вот что касается такого вырвиглазного кода взамен:

    Цитата Сообщение от x86 Посмотреть сообщение
    1. if (_:z_angle < 0x42b40000) {
    то скорее даже больше вреда от этого будет, чем пользы.
    Не хотите постоянно проверять обновления моих скриптов?
    Подключите его последним, после всех остальных
    Nexius's Update Checker

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

    Статус
    Оффлайн
    Регистрация
    06.04.2013
    Адрес
    Novokuznetsk, Russia
    Сообщений
    2,192
    Репутация:
    2589 ±
    Не говоря уже о том, что такая "оптимизация" сама по себе неправильная.
    (-2.0 < -1.0) ==> true
    (_:-2.0 < _:-1.0) ==> false

    Пытаться оптимизировать сравнение с нулём - тоже не совсем правильно, т.к. помимо 0.00 (0x00000000) есть ещё и -0.00 (0x80000000), и по правилам вещественной арифметики эти два числа ВНЕЗАПНО равны.
    (0.0 == -0.0) ==> true
    (_:0.0 == _:-0.0) ==> false
    Т.е., к примеру, если переменная x равна -0.0, то проверка _:x > 0 выдаст неправильный результат.
    Индивидуально в ЛС по скриптингу не помогаю. Задавайте все свои вопросы здесь (click).

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

    Статус
    Оффлайн
    Регистрация
    27.01.2014
    Адрес
    Восточный Мордор
    Сообщений
    5,588
    Репутация:
    1984 ±
    Цитата Сообщение от x86 Посмотреть сообщение
    Из кода были убраны все floatcmp вызовы.
    Если обе функции должны вызываться одним и тем же таймером, то ты, скорее, больше будешь тратить от двух вызовов GetVehicleVelocity, чем от всех floatcmp вместе взятых.
    Связаться со мной в VK можно через личные сообщения этой группы
    Заказы не принимаю

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

    Steve Pavlina

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

    Статус
    Оффлайн
    Регистрация
    26.01.2019
    Сообщений
    41
    Репутация:
    6 ±
    Тему можно закрывать, так как это был бред (хотя что еще можно было ожидать от двух часов ночи). Вообще стоило сделать тесты и бенчмарки.

 

 

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

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

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

Ваши права

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