Функция или отдельный файл?

Question

[Посторонним В.]

На данный момент имеется скрипт session.php, который проверяет наличие у юзера идентификационного куки (сохраняется при логине, когда отмечено "запомнить меня") и, если таковой существует (и данные верны), автоматом логинит юзера, загружая из БД все его данные в сессию.

Вопрос: правильно ли этот скрипт хранить отдельно и подключать его в нужных файлах с помощью require_once, или же лучше сделать из него функцию и перенести в библиотеку функций? А затем вызывать функцию везде, где это нужно.

Вообще, какие преимущества/недостатки включения кода в библиотеку функций и подключения кода отдельным скриптом? Как правильно выбирать нужный вариант?

Спасибо.

Comments

[Алексей Немиро]

Я бы сделал функцией. Порядка больше. Удобней.

Отдельно вообще не стал бы размещать такую мелочь. Сделал бы класс для работы с пользователями :-)

[Сергей Протько]

Алексей Немиро: если делать класс для работы с пользователем - придется делать классы для работы с базой и сессиями и т.д. И все в отдельных файлах. А вообще стоит брать готовые решения для этого, любой нормальный фреймворк имеет для этого готовые средства. Выйдет явно лучше чем писать с нуля.

[Посторонним В.]

Сергей Протько: дело в том, что у меня задача состоит в том, чтоб писать с нуля. И пока что исключительно в процедурном стиле.

[Алексей Немиро]

Сергей Протько: не представляю, как можно жить без класса работы с БД :-)

[Сергей Протько]

Посторонним В.: сочувствую... Это пустая трата времени, 2015-ый на дворе. В 2005-ом это было модно да.

Алексей Немиро: можно, хотя не знаю зачем. Я вот всюду пихаю Doctrine ORM и не могу без нее жить (во всяком случае пока она не становится узким местом приложения).

Answer 1

[FanatPHP]

ну то есть, у тебя в каждом скрипте рядышком стоят два инклюда:

include 'session.php';
include 'auth.php';

Первый заполняет сессию из БД, а второй проверяет сессию и получает из нее данные.

Мне кажется, что глядя на эти две строчки, ответ становится очевиден - одна из них лишняя

Answer 2

[Денис Сафронов]

function check_session() {
    include '/full/path/to/session.php';
}

Comments

[FanatPHP]

это будет самый дурацкий вариант

[Денис Сафронов]

FanatPHP: да ну!

class Session
{
  private $_path;
  public function __construct($path) {
      $this->setPath($path);
  }
  public funcion setPath($path) {
      if (is_dir($path)) {
          $this->_path = $path;
      } else {
          echo 'Path is not dir.';
      }
  }
  public function check_session() {
     if (!$this->_path) {
         echo 'Call setPath() before checking session.';
     } else {
         if (file_exists($this->_path.'session.php')) {
             include $this->_path.'session.php';
         } else {
             echo 'Session.php not found.';
         }
     }
  }
}

$session = new Session('/full/path/to/');
$session->check_session();

[FanatPHP]

ААААААААААААААА, мои глаза! что это за велосипед?!!!!111одинодин

[Денис Сафронов]

FanatPHP: это ООП! Ещё чуть-чуть изменить и там может быть и MVC! Вот тогда это будет ультракрутой код!