Находим ближайшее транспортное средство

[geneff]

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

Аргументы:

• player - обьект нашего игрока
• distance - дистанция на которой будем искать ближайший транспорт

Возвращает:

• null - если не найдено ближайшего транспорта в нашем радиусе
• veh - обьект найденого транспорта

Сама функция:

const getNearestVehicle = (player, distance) => {
    let veh = null;
    mp.vehicles.forEachInRange(player.position, distance, vehicle => {
        const dist = player.dist(vehicle.position);
        if (dist < distance) {
            distance = dist;
            veh = vehicle;
        }
    })
    return veh;
}

Для примера сделаем команду открывания/закрывания транспортного средства:

mp.events.addCommand('lock', (player) => {
    const veh = getNearestVehicle(player, 2);

    if (veh === null)
        return player.outputChatBox('Вы должны находится возле машины.');

    veh.locked = !veh.locked;
    player.outputChatBox(veh.locked ? 'Close' : 'Open');
})

 

[seaniwe]

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

Аргументы:

• player - обьект нашего игрока
• distance - дистанция на которой будем искать ближайший транспорт

Возвращает:

• null - если не найдено ближайшего транспорта в нашем радиусе
• veh - обьект найденого транспорта

Сама функция:

const getNearestVehicle = (player, distance) => {
    let veh = null;
    mp.vehicles.forEachInRange(player.position, distance, vehicle => {
        const dist = player.dist(vehicle.position);
        if (dist < distance) {
            distance = dist;
            veh = vehicle;
        }
    })
    return veh;
}

Для примера сделаем команду открывания/закрывания транспортного средства:

mp.events.addCommand('lock', (player) => {
    const veh = getNearestVehicle(player, 2);

    if (veh === null)
        return player.outputChatBox('Вы должны находится возле машины.');

    veh.locked = !veh.locked;
    player.outputChatBox(veh.locked ? 'Close' : 'Open');
})

Чётко!

 

[Lev Angel]

Здорово! Не знал насчет Entity::dist и Pool::forEachInRange. Будет полезно

 

[skyElmax]

const getNearestVehicle = (player, distance) => {
    let veh = null;
    mp.vehicles.forEachInRange(player.position, distance, vehicle => {
        const dist = player.dist(vehicle.position);
        if (dist < distance) {
            distance = dist;
            veh = vehicle;
        }
    })
    return veh;
}

предлагаю заменить на

mp.Player.prototype.getNearestVehicle = function (distance) {
    let veh = null;
    mp.vehicles.forEachInRange(this.position, distance, vehicle => {
        const dist = this.dist(vehicle.position);
        if (dist < distance) {
            distance = dist;
            veh = vehicle;
        }
    })
    return veh;
}

и в таком случае можно обращаться красиво

player.getNearestVehicle(15);

 

[Revalto]

const getNearestVehicle = (player, distance) => {
    let veh = null;
    mp.vehicles.forEachInRange(player.position, distance, vehicle => {
        const dist = player.dist(vehicle.position);
        if (dist < distance) {
            distance = dist;
            veh = vehicle;
        }
    })
    return veh;
}

предлагаю заменить на

mp.Player.prototype.getNearestVehicle = function (distance) {
    let veh = null;
    mp.vehicles.forEachInRange(this.position, distance, vehicle => {
        const dist = this.dist(vehicle.position);
        if (dist < distance) {
            distance = dist;
            veh = vehicle;
        }
    })
    return veh;
}

и в таком случае можно обращаться красиво

player.getNearestVehicle(15);

А что если...

mp.Player.prototype.getNearestVehicle = (distance) => {
    const veh = mp.vehicles.toArray().reduce((a, b) => this.dist(a.position) < this.dist(b.position) ? a : b);
    return this.dist(veh.position) < distance ? veh : null;
}

 

[Lev Angel]

А что если...

mp.Player.prototype.getNearestVehicle = (distance) => {
    const veh = mp.vehicles.toArray().reduce((a, b) => this.dist(a.position) < this.dist(b.position) ? a : b);
    return this.dist(veh.position) < distance ? veh : null;
}

Страдает читаемость кода.
Да и вопрос что быстрее будет бегать mp.vehicles.toArray().reduce или mp.vehicles.forEachInRange? Не знаю как под капотом реализован forEachInRange. Скорее всего будет +- одинаково.

 

[Revalto]

Страдает читаемость кода.
Да и вопрос что быстрее будет бегать mp.vehicles.toArray().reduce или mp.vehicles.forEachInRange? Не знаю как под капотом реализован forEachInRange. Скорее всего будет +- одинаково.

Да, будет одинакого, ибо в любом случае идет проверка на дистанцию. Читаемость кода никуда не пропадает, это вполне обычный стандарт в JavaScript (NodeJS)

 

[Lev Angel]

Стандартный то он стандартный Просто воспринимается сложновато. Нужно потратить какое-то время чтобы понять что там происходит.

 

[Dihan48]

Да, будет одинакого, ибо в любом случае идет проверка на дистанцию. Читаемость кода никуда не пропадает, это вполне обычный стандарт в JavaScript (NodeJS)

нет, forEachInRange выполняется на много быстрее, результат на скрине.
И еще подводный камушек в reduce() - Если массив пустой и аргумент initialValue не указан, будет брошено исключение TypeError