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

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

stock IsVehicleMoving(vehicleid)
{
    static Float:x, Float:y, Float:z;
    return (GetVehicleVelocity(vehicleid, x, y, z) != 0) &&
        ((_:x != 0) || (_:y != 0) || (_:z != 0));
}
 
stock IsVehicleBackwordsDirection(vehicleid)
{
    static Float:x, Float:y, Float:z, Float:z_angle;
    if (GetVehicleVelocity(vehicleid, x, y, z)) {
        if (_:z_angle < 0x42b40000) {
            if (_:x > 0 && _:y < 0)
                return 1;
        } else if (_:z_angle < 0x43340000) {
            if (_:x > 0 && _:y > 0)
                return 1;
        } else if (_:z_angle < 0x43870000) {
            if (_:x < 0 && _:y > 0)
                return 1;
        } else {
            if (_:x < 0 && _:y < 0)
                return 1;
        }
    }
        return 0;
}

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

По поводу читаемости кода: тебе ничто не мешает юзать дефайны.

А в чём, собственно, оптимизация? Показал бы хотя бы вариант до оптимизации, чтоб было с чем сравнить.

Цитата:

---

Сообщение от DeimoS

А в чём, собственно, оптимизация? Показал бы хотя бы вариант до оптимизации, чтоб было с чем сравнить.

---

Из кода были убраны все floatcmp вызовы.

Цитата:

---

Сообщение от x86

Из кода были убраны все floatcmp вызовы.

---

На самом деле в экономии это мизер, а вот что касается такого вырвиглазного кода взамен:

Цитата:

---

Сообщение от x86

if (_:z_angle < 0x42b40000) {

---

то скорее даже больше вреда от этого будет, чем пользы.

Не говоря уже о том, что такая "оптимизация" сама по себе неправильная.
(-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 выдаст неправильный результат.

Цитата:

---

Сообщение от x86

Из кода были убраны все floatcmp вызовы.

---

Если обе функции должны вызываться одним и тем же таймером, то ты, скорее, больше будешь тратить от двух вызовов GetVehicleVelocity, чем от всех floatcmp вместе взятых.

Тему можно закрывать, так как это был бред (хотя что еще можно было ожидать от двух часов ночи). Вообще стоило сделать тесты и бенчмарки.