Как определить свободный слот транспорта?

Доброго времени суток, у меня назрел такой вопрос, что будет лучше использовать в плане оптимизации из этих двух функциях?
На мой взгляд, 1 будет более лучше, нежели 2 так как 2, ему еще понадобится синхронизировать с сервером.

И еще, функция IsValidVehicle только на стороне сервера или клиента тоже?

1. вариант

PHP код:

---

stock GetVehicleLastID()
{
    for (new i = 1; i < MAX_VEHICLES; i++) {
        if (!IsValidVehicle(i)) {
            return i;
        }
    }
    return 0;
} 


---

2. вариант

PHP код:

---

stock GetVehicleLastID()
{
    new
        vehicleid = CreateVehicle(400, 0.0, 0.0, 0.0, 0.0, 0, 0, -1);

    if (vehicleid != INVALID_VEHICLE_ID) {
        DestroyVehicle(vehicleid);
    }
    return vehicleid;
} 


---

Использовать GetVehiclePoolSize?

Твои оба варианта, кстати, не рабочие

Если тебе необходимо узнать максимальный ID валидного транспорта, то уже ответили выше - GetVehiclePoolSize();
Если проблема в том, что создать ли автомобиль игроку или добавить в очередь на создание (copy AZ-RP), то необходимо искать свободное место, должно быть, что-то вроде этого:

stock GetFreeVehicleId() {
 
    if(GetVehiclePoolSize()+1 < MAX_VEHICLES)
        return GetVehiclePoolSize()+1;
 
 
    for(new i; i < MAX_VEHICLES; i++) {
 
        if(IsValidVehicle(i))
            continue;
 
        return i;
    }
 
    return INVALID_VEHICLE_ID;
}

P.S: Набросал за пару минут, поэтому подумай, может что можно сделать лучше.

oukibt Кто вам такое сказал? Я сам писал, я знаю как они должны работать

punkochel Я конечно все понимаю,твой вариант равносильный моему первому. Но суть вопроса был не в этом

Сам вопрос поставлен некорректно. Что значит "синхронизировать с сервером" во втором варианте? И что значит "IsValidVehicle на стороне клиента тоже"? Если мы говорим про сам натив, то очевидно это часть серверного функционала.

Касаемо вариантов в первом посту, оба имеют недоработки:
Первый вариант остановится и вернёт не сущесвующий ид машины, который не создан на сервере, что на практике будет означать всегда "последний валидный ид тачки + 1", либо 1, если тачек не создано вообще. По этой же причине "return 0" там это почти невыполнимый код, потому что даже если на сервере машин не будет вообще, сработает условие из цикла в первой же его итерации и вернет i. (0 вернет только в случае, если сервер забит машинами до их лимита)
Второй вариант вернет уже 65535 как последний созданный ид тачки, если машин будет создано до лимита. Плюс та же проблема, что и в первом варианте с возвратом "последнего валидного ида тачки + 1" из-за возвращения ида уже удаленной машины.

Кароче, как тебе сказали выше, оба варианта если и не неработающие полностью, то уж точно не годные для использования в таком виде, ну и также про GetVehiclePoolSize - незачем изобретать велосипед, если нужно узнавать именно наивысший созданный ид тс.

Цитата:

---

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

Сам вопрос поставлен некорректно. Что значит "синхронизировать с сервером" во втором варианте? И что значит "IsValidVehicle на стороне клиента тоже"? Если мы говорим про сам натив, то очевидно это часть серверного функционала.

Касаемо вариантов в первом посту, оба имеют недоработки:
Первый вариант остановится и вернёт не сущесвующий ид машины, который не создан на сервере, что на практике будет означать всегда "последний валидный ид тачки + 1", либо 1, если тачек не создано вообще. По этой же причине "return 0" там это почти невыполнимый код, потому что даже если на сервере машин не будет вообще, сработает условие из цикла в первой же его итерации и вернет i. (0 вернет только в случае, если сервер забит машинами до их лимита)
Второй вариант вернет 65535 как последний созданный ид тачки, если машин будет создано до лимита. Плюс та же проблема, что и в первом варианте с возвратом "последнего валидного ида тачки + 1" из-за возвращения ида уже удаленной машины.

Кароче, как тебе сказали выше, оба варианта если и не неработающие полностью, то уж точно не годные для использования в таком виде, ну и также про GetVehiclePoolSize - незачем изобретать велосипед, если нужно узнавать именно наивысший созданный ид тс.

---

мне и нужно несуществующий

Цитата:

---

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

мне и нужно несуществующий

---

Тогда функции из примера надо было называть не "GetVehicleLastID"

Цитата:

---

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

Тогда функции из примера надо было называть не "GetVehicleLastID"

---

Согласен, мой косяк, насчет синхронизации. Когда создаем CreateVehicle и если игроки в зоне стрима, машина должна синхронизировать всех игроков, чтобы они ее видели.

Так вопрос, что тогда будет оптимизированный вариантом, 1 вариант, перебор "всех" авто, или 2 создать/удалить

Цитата:

---

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

Согласен, мой косяк, насчет синхронизации. Когда создаем CreateVehicle и если игроки в зоне стрима, машина должна синхронизировать всех игроков, чтобы они ее видели.

Так вопрос, что тогда будет оптимизированный вариантом, 1 вариант, перебор "всех" авто, или 2 создать/удалить

---

Если создать и сразу же удалить, то не факт, что она вообще кому-то из игроков успеет прилететь. Да и создать можно на любых неудобных координатах, где игроков обычно нет. Тем не менее по эффективности могут быть разные результаты. В то время как создание/удаление тачки всегда будет занимать примерно одинаковое время и ресурсы, цикл иногда сможет очевидно быстрее определять первый попавшийся свободный ид, если машин на сервере немного. Лучше потестить, в общем.

Цитата:

---

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

Если создать и сразу же удалить, то не факт, что она вообще кому-то из игроков успеет прилететь. Да и создать можно на любых неудобных координатах, где игроков обычно нет. Тем не менее по эффективности могут быть разные результаты. В то время как создание/удаление тачки всегда будет занимать примерно одинаковое время и ресурсы, цикл иногда сможет очевидно быстрее определять первый попавшийся свободный ид, если машин на сервере немного. Лучше потестить, в общем.

---

Спасибо буду проверять

А в чём сложность просто итератор создать и перехватить создание/удаление транспорта? Благо применений у такого итератора может быть множество. Тогда и велосипедов изобретать не нужно будет в данном случае.

Цитата:

---

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

Если создать и сразу же удалить, то не факт, что она вообще кому-то из игроков успеет прилететь..

---

Пакет с созданием транспорта "прилетит" игрокам в любом случае. А вот то, "прилетит" ли пакет удаления транспорта сразу же за пакетом создания или сделает он это с задержкой - не факт (потери пакетов/пинг и т.п.).

Цитата:

---

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

А в чём сложность просто итератор создать и перехватить создание/удаление транспорта? Благо применений у такого итератора может быть множество. Тогда и велосипедов изобретать не нужно будет в данном случае.

---

А если иных применений в итоге не найдётся, то это будет примерно такой же костыль. Автор спрашивал про собственно предложенный вариант, так что от этого и отталкивался. Практической задачи всё равно поставлено не было.

Цитата:

---

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

А если иных применений в итоге не найдётся, то это будет примерно такой же костыль. Автор спрашивал про собственно предложенный вариант, так что от этого и отталкивался. Практической задачи всё равно поставлено не было.

---

Если это не какая-то отдельная библиотека, которую автор желает просто опубликовать, то я даже не смогу придумать сколь-либо крупный скрипт, в котором не потребуется как-либо обрабатывать весь транспорт: начиная от поиска свободного транспорта для какого-нибудь респавна и заканчивая каким-нибудь спидометром с прочими вычислениями, по типу бензина или пробега.
С учётом того, что автор изначально мог и не подумать в сторону итераторов, вполне разумно было бы хотя бы уточнить, что есть более мультизадачная реализация, без костылей, которые лишний раз будут нагружать интернет-канал (особенно если такая функция нужна будет где-то в цикле для создания сразу множества автомобилей). Но, впрочем, вам виднее.

В любом случае, ответ автором получен, так что тему закрываю.