Phing и pre-processing исходного кода

Question

[SpiRi7]

Интересует вопрос препроцессинга исходного кода проекта используя phing.
Что имеется ввиду
Имеется исходных код вида:

 protected function _doImportant() {
#ifdef DEBUG
  $this->log("doing important stuff now");
#endif
  foreach ($this->import as $moreimportant) {
#ifdef DEBUG
   $this->log("as I said, important or $moreimportant");
#endif
  }
#ifdef DEBUG
  $this->log("finished doing ach so important thing");
#endif
 }


После препроцессинга, если имеется/включена директива DEBUG

 protected function _doImportant() {
  $this->log("doing important stuff now");
  foreach ($this->import as $moreimportant) {
   $this->log("as I said, important or $moreimportant");
  }
  $this->log("finished doing ach so important thing");
 }


Если нет:

 protected function _doImportant() {
    foreach ($this->import as $moreimportant) {
  
    }
 }


Answer 1

[Imenem]

Попробуйте использовать Replace Regexp для этого. Задача позволяет реализовать поиск/замену с использованием регулярок. Или переходите на Ant, для него есть класс для препроцессинга.

Comments

[SpiRi7]

Спасибо, похоже один из вариантов. Ant пользовался для сборки ява приложений, там я препроцессинг использую. Видимо придется менять build скрипт.

Answer 2

[Aecktann]

1) В чем ваш вопрос? Хабр — не гугл :)
2) Не проще ли вместо препроцессинга использовать что-нибудь наподобие:

const DEBUG = TRUE;
protected function _doImportant() {
  if (self::DEBUG) {
    $this->log("doing important stuff now");
  }
  foreach ($this->import as $moreimportant) {
    if (self::DEBUG) {
      $this->log("as I said, important or $moreimportant");
    }
  }
  if (self::DEBUG) {
    $this->log("finished doing ach so important thing");
  }
}

Const DEBUG находится в том же классе, ибо вы используете $this->log().

Хотя у меня для этого используется Log::debug($message), который в случае, если Log::DEBUG == 0, ничего не выводит. Пытаться на этом экономить — зло, как и любая преждевременная оптимизация.

Comments

[SpiRi7]

Я понимаю что Хабр не гугл, спрашиваю потому что гугл не дает ответа на мой вопрос.
Пример который вы привели правильный, но в исходном коде будет находиться константа DEBUG, а мне надо что бы она убралась.
Как вы понимаете, пример приведенный мной абстрактный. Это не единственное применение нужного мне функционала.

[Aecktann]

>Я понимаю что Хабр не гугл, спрашиваю потому что гугл не дает ответа на мой вопрос.
Лингвистическая шутка :). Дело в том, что вы не задали сам вопрос. Вы только сказали, что он вас интересует. Я иногда ищу в гугле запросы в стиле «php 5.3 замыкания», в которых тоже не содержится вопроса, если я не знаю, что конкретно я хочу узнать.

>Пример который вы привели правильный, но в исходном коде будет находиться константа DEBUG, а мне надо что бы она убралась.
Даже если подобная функциональность нужна не только в примере с debug'ом продукта, общую логику это не отменит. Препроцессинг ради препроцессинга — не PHP-way. Да, это удобно. Когда я пишу на C, я тоже применяю препроцессинг, но если язык предполагает другую идеологию — не стоит ее нарушать. Действия, которые должны выполняться/не выполняться в зависимости от режима работы программы (то есть это статическое условие относительно контекста выполнения) с 99% вероятностью можно инкапсулировать в отдельный класс. На примере с debug'ом:

class Log {
  const DEBUGLEVEL=0;
  const WARNINGLEVEL = 1;
  /*
  ...
  */
  const DEBUGMODE = TRUE;

  public function plainWrite($message) {
    //smth
  }
  
  public function write($message, $level) {
    //smth
  }

  public function debug($message) {
    if (self::DEBUGMODE) {
      $this->write($message, self::DEBUGLEVEL);
    }
  }
}

Не стоит ходить со своим уставом в чужой монастырь другого языка.

[Aecktann]

Не дописал. Тогда брюки превращаются в элегантные шорты:

protected function _doImportant() {
  $log->debug("doing important stuff now");
  foreach ($this->import as $moreimportant) {
    $log->debug("as I said, important or $moreimportant");
  }
  $log->debug("finished doing ach so important thing");
}

Никакого засорения константами или чем-либо еще. Да и выглядит лучше, чем множество #ifdef и т. п.

[evilbloodydemon]

да топикстартер может разные версии исходников хочет поставлять, чего голову морочить-то своими советами по совсем другой теме. меня тоже в свое время интересовал вопрос препроцессинга пхп, но ничего внятного не нашлось ибо «пхп — это препроцессор».

[Aecktann]

Топикстартер хочет иметь одну версию исходников, и компилировать из этой версии само приложение с помощью препроцессинга (возможно в нескольких вариантах). В таком случае надо не исходники мучить и заниматься метапрограммированием (в каком-то смысле), а логику разделять.

[SpiRi7]

Aecktann спасибо за разъяснение вашего ответа. я соглашусь с Вами — это немного не php way, но вот была такая необходимость. Evilbloodydemon прав нужно не только для логов, но и для разных версий исходников. Самый явный пример — триальная версия скрипта/модуля. В этом случае к сожалению простыми константами и классами не обойдешься. Крайне желательно вырезать куски исходников.