1

Недавно решал тестовое задание на стажировку в одну контору. Не смог тогда в положенное время решить одну задачу, собственно пролетел. Потом решил её для себя, но у меня возник вопрос - достаточно ли моё решение оптимально, или, исходя из условий задачи, его можно как-то улучшить? Вот сама задача: Описание задачи

Вот моё окончательное решение (после строки комментария):

<?php
$cars = [
    ['name' => 'Такси 1', 'position' => rand(0, 1000), 'isFree' => (bool) rand(0, 1)],
    ['name' => 'Такси 2', 'position' => rand(0, 1000), 'isFree' => (bool) rand(0, 1)],
    ['name' => 'Такси 3', 'position' => rand(0, 1000), 'isFree' => (bool) rand(0, 1)],
    ['name' => 'Такси 4', 'position' => rand(0, 1000), 'isFree' => (bool) rand(0, 1)],
    ['name' => 'Такси 5', 'position' => rand(0, 1000), 'isFree' => (bool) rand(0, 1)],
];

$passenger = rand(0, 1000);

/* ===== Ваш код ниже ===== */

$min = null;
foreach($cars as $car){
    if($car['isFree']==true){
        if(abs($car['position'] - $passenger) < $min OR $min === null){
            $min = abs($car['position'] - $passenger);
     } 
 }
}

foreach($cars as $car){
    echo $car['name'].", стоит на ".$car['position']." км, до пассажира ".abs($car['position'] - $passenger)." км";
    if($car['isFree']==true){
        echo " (свободен)";
        if(abs($car['position'] - $passenger)==$min){
        echo " - едет эта машина";
        }
        echo "<br/>";
    } else{
        echo " (занят)"."<br/>";
    }   
}
?>

Я знаю, что эта задача здесь уже фигурировала, но тогда отвечающий при её решении использовал создание функции, а это запрещено условием. Заранее спасибо)

3
  • Говоря об оптимальности надо указать критерий оптимальности.
    – u_mulder
    9 дек 2020 в 14:53
  • а можно узнать парочку хотя бы, какие навскидку тут могли бы быть критерии? просто автор задачи не указал их явно 9 дек 2020 в 15:05
  • зачем вы пишите $var == true ? у вас слева от равно - булевое значение, формально вы сравниваете true == true или false == true.данные равенства это логические выражения, которое будет вычислено и вернет булевый результат. А результат этот будет равен true только в случае, когда ваш аргумент истинный. Так так что не надо ничего сравнивать там, а просто использовать if($var)
    – teran
    9 дек 2020 в 16:43

1 ответ 1

0

Безотносительно PHP, общечеловеческие замечания:

Зачем два (в итоге даже три) раза вычислять для каждой машины abs($car['position'] - $passenger)? Функция min поможет оформить сравнение более кратко. Если она вдруг запрещена - можно записать в переменную.

Что будет, если до двух машин одинаковое расстояние? Обе поедут?

При обновлении минимума нужно запоминать индекс, тогда вывод будет однозначен

2
  • Спасибо!) Но ведь в любом случае придётся сначала найти минимум в этом массиве, а потом уже выводить строку? Как-то совместить это не получится? 9 дек 2020 в 15:17
  • Нет смысла, т.к. другие машины тоже нужно выводить, а минимум будет известен только после проверки всех
    – MBo
    9 дек 2020 в 15:27

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.