вынос стандартных методов в пользовательскую функцию (теория)

Новичок в php, но не в разработке в целом. На работе с наставником возник "спор".

По коду нужно проверить существование элемента массива. Он говорит, что правильнее было бы в коде ниже вывести проверку в отдельную функцию check. И, в целом, привыкать к такому стилю. Мне же непонятно, зачем выводить в отдельную функцию, которая возвращает true/false стандартный метод isset, который сам по себе возвращает true/false. Моя позиция:

  1. в функцию надо выносить код, который используется в скрипте минимум 2 раза. Если проверка хоть в 100 строк проводится единожды - нет смысла выносить ее в отдельную функцию.
  2. превращать 1 строку кода в 3 (как в примере ниже) нецелесообразно.

<?php

//first variant
if(isset($array['key'])){
    #code here;
};


//second variant;
if(check($array['key'])){
    #code here;
}

function check($el){
    return isset($el);
}

P.S. Своими утверждениями не претендую на истину, могу ошибаться. Хочется услышать ваши мнения.


Ответы (0 шт):