Как в константу класса PHP вставить даты?

Question

[m4son]

Осваиваю ООП. Подскажите, нужно файл логов помещать в папки по датам. Как в константу класса вставить даты?
Я и через функцию, и через конструктор пробовал. Никак не получается. Вот последний мой вариант

public static function getDate($k) {
    switch ($k) {
      case "Y":
        return date("Y");
        break;
      case "m":
        return date("m");
        break;
      case "d":
        return date("d");
        break;
      case "H-i-s":
        return date("H-i-s");
        break;
    }
  }

const LOG_FILENAME = "{$_SERVER["DOCUMENT_ROOT"]}/logs/" .  self::getDate("Y") . "/" . self::getDate("m") . "/" . self::getDate("d") . ".txt";

Comments

[Сергей delphinpro]

Эта статическая функция вообще никакого отношения к классу не имеет, а значит ее нужно вынести просто как отдельную функцию. (Оставлю за скобками ее целесообразность , т.к. она просто повторяет стандартную date())

Потом просто используйте эту функцию для получения даты, и еще одну, для получения пути к файлу. Константы не нужны.

[lil_koi]

а зачем тебе вообще эта функция? Почему нельзя сделать так?
retrun date($k);

[m4son]

Сергей delphinpro, я понимаю, что это обход через date(), просто не получалось и решил ссылаться на функцию класса, а не функцию PHP

Answer 1

[キム ファイブプラス]

Это не ООП.
Константа потому так и называется, что она константа. Смекаешь?

Answer 2

[Newto]

В вашем случае это было бы как-то так:

class Someclass
{
	public static function getDate($k) 
	{
		switch ($k) {
		  case "Y":
			return date("Y");
			break;
		  case "m":
			return date("m");
			break;
		  case "d":
			return date("d");
			break;
		  case "H-i-s":
			return date("H-i-s");
			break;
		}
	}
}

define('LOG_FILENAME',  "{$_SERVER["DOCUMENT_ROOT"]}/logs/" .  Someclass::getDate("Y") . "/" . Someclass::getDate("m") . "/" . Someclass::getDate("d") . ".txt");

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

Comments

[m4son]

Есть два файла лога: ошибок и результата.
Лог ошибок у него постоянное место хранения, я его вставил в константу и эту константу легко вставляю в разные методы класса.
А лог результата зависит от даты. Я пробовал объявлять класс указывая переменную объект дата и через конструктор вставлять в переменные класса, и потом эти переменные (будучи просто строкой) вставлять в константу, но не получилось

[Дмитрий]

m4son, запишите в свойство и вставляйте как свойство, а не как константа

[m4son]

Дмитрий, я сделал так, не получилось

public static $date;
  public static $year;
  public static $month;
  public static $day;
  public static $now;

  public function __construct() {
    $this->date = new DateTime();
    $this->year = $this->date->format("Y");
    $this->month = $this->date->format("m");
    $this->day = $this->date->format("d");
  }

// и потом в коде ссылаюсь на this->year

[Сергей delphinpro]

m4son, зачем вы везде static суёте? =)

[m4son]

Сергей delphinpro, при работе с framework-ами иногда ругается, уже привычка )

[Дмитрий]

m4son, ну как бы у статики нет
$this есть self::

[Newto]

m4son, Видимо проблема в const. Когда вы определяете константу через const -- она не может включать в всебя динамические переменные. А если определить через define, то может.

Answer 3

[Siverius]

А откуда берётся эта $k?