2009年10月19日月曜日

"define"でなく"const" - PHP

スズキです。

うれしいことに、この記事に対して、
http://blog.suz-lab.com/2009/10/php.html

このようなコメントをいただきました。
http://www.haloscan.com/comments/iretsuzuki/7658573933420455263/?a=22777
> MobileTool.php のソースを拝見させていただきました。
> 定数定義されている
> define("MOBILE_CAREER_DOCOMO" , 1);
> 等を以下のようにクラス定数にすれば、グローバル空間を汚染せず、
> より隠蔽化されたライブラリになると思います。
> class MobileTool extends Tool {
>   const MOBILE_CAREER_DOCOMO = 1;
>   :
>   :
>   public function isCareerDocomo() {
>     return ($this->career === self::MOBILE_CAREER_DOCOMO);

ということで、早速反映してコミットです。
http://code.google.com/p/suz-lab-php/source/browse/trunk/suz-lab-php/php/lib/Tool/MobileTool.php

HaloScanはやめたいなー...

--------
http://www.suz-lab.com

0 コメント: