Shin x blog

コンピュータエンジニア向けのLifeHack本です。

仕事術としてタイムマネジメントが紹介されているのですが

• タスクリストは全て外部記憶に追い出す（脳はより生産的な事に使う）
• 大きなタスクは実現しやすい細かいタスクに分ける
• 仕事とプライベートのタスクを同じように管理する

など、GTDに通じるところがあります。

さらにこの本独自の

• 自動化できるものは自動化してしまう
• 目的に応じたドキュメントを作る
• 日々のタスクと中長期的なタスクの折り合い
• タイムマネジメントと顧客対応の両立

といったより実践的な内容もあります。

エンジニア向けと銘打っているだけあって自動化のところではUnixコマンドが紹介されるなど、この仕事をしている人ならではのTipsが参考になりました。仕事術系の本でawkの使い方を解説しているのは見たことないです。;-)

私の場合、GTDで躓いたのが「やりたいリスト」を出した後、それらを上手く分類・実践していくことでした。（実際今もリストに埋もれているままだったりしてます。。。）

そこでこの本のタイムマネジメントの方法（日々の始めに実施するタスクを決める、優先順位を付ける等）を組み合わせると上手くいくではないかと考えています。

「Just do it」の精神で早速実践ですね。

PHPUnit3がリリースされました。このバージョンにはMockがサポートされているのですが、PHPUnit3はPHP5を対象としているので残念ながらPHP4では動作しません。ですのでPHP4対応のPHPUnitでMockクラスを使う方法を探ってみました。

使うのはPHPUnitと並ぶユニットテストツールのSimpleTestです。こちらはPHP4対応でMockクラスがサポートされています。これをPHPUnitと組み合わせて使ってみます。

SimpleTestインストール

PEARパッケージが以前はあったようなのですが、sourceforgeにはそれらしいものが見当たりませんでした。
zuzara : symfonyチュートリアル実践【第14～16日目】を参考に以下のコマンドでインストールしました。

$ pear install http://jaist.dl.sourceforge.net/sourceforge/simpletest/simpletest_1.0.0.tgz

PEARパッケージが見つからない場合でもsourceforgeからソースを取得してinclude_pathで参照できる位置に配置すればOKです。

PHPUnit用Mockクラス

SimpleTestのSimpleMockを継承してPHPUnitで使うMockクラスを記述します。

< ?php
require_once 'simpletest/mock_objects.php';

class PHPUnitMock extends SimpleMock {
  function PHPUnitMock($test, $wildcard) {
    $this->SimpleMock($test, $wildcard);
  }
}
SimpleTestOptions::setMockBaseClass('PHPUnitMock');
?>

テスト対象クラス

テスト対象のクラスはこちらです。selectCount()はテーブル名を引数に取り、テーブルのレコード数を整数で返します。擬似的に処理時間が100秒かかるようにしています。

< ?php
class Database {
  function selectCount($table) {
    sleep(100);
    return null;
  }
}
?>

テストケース

Databaseクラスのテストケースを記述します。Mockクラスを使用することに定義した処理結果[100]がすぐに返ってきます。

< ?php
require_once 'PHPUnit.php';
require_once 'PHPUnitMock.php';
require_once 'Database.php';

Mock::generate('Database');

class DatabaseTest extends PHPUnit_TestCase {
  function DatabaseTest ($name) {
    $this->PHPUnit_TestCase($name);
  }

  function setup() {
  }

  function testQuery() {
    $mock =& new MockDatabase($this);
    $mock->setReturnValue('selectCount', 100, 'users');

    $ret = $mock->selectCount('users');
    $this->assertEquals($ret, 100);
  }
}

$suite = new PHPUnit_TestSuite('DatabaseTest');
$result = PHPUnit::run($suite);
echo $result->toString();
?>

SimpleTestはMockクラスの他にHTTPテストも行えるなど高機能なのですが、すでにPHPUnitを使用していたので今更移行するのも億劫でした。あとはコードカバレッジも使えると良いのですが。;-)

マジックメソッドの__get()ですが、配列を返すとforeachでNoticeが発生する場合があります。

< ?php
error_reporting(E_ALL);

class A {
  private $array = array(1, 2, 3);

  function __get($name) {
    if ($name == 'hoge') {
      return $this->array;
    }
  }
}

$obj = new A();
foreach ($obj->hoge as $elem) {
  var_dump($elem);
}
?>

PHP Notice:  Indirect modification of overloaded property A::$hoge has
 no effect in __get.php on line 15

The reason for this is that __get() only returns variables in read mode, while foreach() wants a variable in read/write mode as it tries to modify the internal array pointer. As it can’t do this PHP 5.2 will now throw a warning on this.

Overloaded properties (__get) – Derick Rethans

理由はforeachではread/writeモードが要求されるのに__get()ではreadモードの値が返されるためのようです。__get()がreadモードで返すのは当然のような気がしますがちと不便ですね。

対策としては__get()内で(array)でキャストするか

< ?php
  function __get($name) {
    if ($name == 'hoge') {
      return (array)$this->array;
    }
  }
?>

__get()の値を別の変数に代入すれば良いようです。

< ?php
$array = $obj->hoge
foreach ($array as $elem) {
?>

この現象は5.2.0で発生し、5.1.6ではNoticeはでませんでした。__get()は便利で何だか使いたくなる機能ですが、配列を返す場合は注意しましょう。

strtotime()で様々な日付をunixタイムスタンプで取得できます。

Keep it simple and just do strtotime(‘-1 Sunday’,$currentDay);

Calculating start and end dates of a week. – Derick Rethans (2006-11-18)

コメント欄からの引用です。

エントリにある週の始め、終わりの日にちを取得するには以下で取得できます。

< ?php
$now = time();
// 週の始め（月曜日）
echo date('Y/m/d', strtotime('-1 Monday', $now)) . "\n";
// 週の終わり（日曜日）
echo date('Y/m/d', strtotime('+1 Sunday', $now)) . "\n";
?>

PHPマニュアルに他の記述も載っていますのでご参考にどうぞ。

■参考サイト
PHPマニュアル－strtotime

ユニットテストでプログラムで生成したデータを検証する方法です。

CSVファイルを検証するテストをPHPUnitで書いてみました。

< ?php
require_once "PHPUnit.php";

class DataTest extends PHPUnit_TestCase {
  function DataTest($name) {
    $this->PHPUnit_TestCase($name);
  }

  function testData() {
    // データ読み込み
    $fp = fopen('data.csv', 'r');
    if (!$fp) {
      die('data not found.');
    }

    // テストループ
    $line = 0;
    while(!feof($fp)) {
      $line++;
      $row = fgetcsv($fp, 4096);
      if (empty($row)) {
        continue;
      }

      if (count($row) != 3) {
        $this->fail('Line: ' . $line);
        continue;
      }

　　　// id は数値のみ
      $id = $row[0];
      $this->assertRegExp('/^[0-9]+$/', $id, $line);

　　　// code は英数字のみ
      $code = $row[1];
      $this->assertRegExp('/^[A-Za-z0-9]+$/', $code, $line);

　　　// name は値があれば良い
      $name = $row[2];
      $this->assertTrue(strlen($code) > 0, $line);
    }

    fclose($fp);
  }
}


$suite = new PHPUnit_TestSuite('DataTest');
$result = PHPUnit::run($suite);
echo $result->toString();
?>

ポイントは「assert失敗時のメッセージにデータを特定する項目を含めておく」くらいでしょうか。（このソースでは行数を入れていますが、DBのRecordsetを検証する時はpkeyの値を入れています。）あとはdata.csvが何万行あろうが何回テストしようがphpunitがきっちりテストしてくれます。やっている事は単純なのですが効果は高いです。

他にも複雑なSQLで生成されるデータをPHPで取得して検証したり、DBに問い合わせてデータが正しいかを検証するという方法もあります。もちろん正となるデータを用意しておいてそれと比較するというのもありです。

あと、ここではPHPUnitを使いましたが、データを相手にテストを行うので別にSimpleTestでも良いですし、PerlでもRubyでも何でもOKです。Javaで開発したシステムのデータをPHPで検証する、なんてこともできますね。（LL言語の利用範囲がこんなところにも;-)）

ロジックの自動テストも大事ですが、生成物の自動テスト（チェック）もあると心強いものです。

これ何気に困ってました。Firefoxのポップアップブロック数制限によりlivedoor Readerで「ピンを開く」を実行しても20画面以上開けませんでした。いつのまにかピン数が15を超えるとピンするのを躊躇したり。。。

で、livedoor Readerでブロックされるという件だけど、キー操作でウィンドウを開くってのはユーザーが起こしたイベントに紐付いたwindow.openではあるのだけれど、タイマーを使って順次開いたりしているので、ブラウザ側で判別するのは難しいだろうと思う。なんにせよサイトで許可していても開かないことがある、っていうのは、説明しづらくて非常に困る感じがする。（今のところabout:configやら拡張やらでdom.popup_maximumを増やすしか無いようだ。）

最速インターフェース研究会 :: Firefox2のポップアップブロックについて調べてみた

ありがたや。これで気にせずにピンを付けられます。;-)

Big Room » Blog Archive » PHP 5.2 – Nesting level too deep – recursive dependency?でオブジェクト同士の比較で発生するエラーについて書かれています。

class MyObj
{
    public $p;
}
class OtherObj
{
    public $q;
}

$a = new MyObj();
$b = new OtherObj();
$a->p = $b;
$b->q = $a; // the circular reference: $a->p->q === $a

$c = new MyObj();
$d = new OtherObj();
$c->p = $d;
$d->q = $c;// another circular reference: $c->p->q === $c

echo ( $a == $c ); // Fatal error:
?>
Big Room » Blog Archive » PHP 5.2 - Nesting level too deep - recursive dependency? (2006-11-16)

ポイントは循環参照と[==]での比較です。

PHPでは[==]と[===]とで処理結果が異なるのは良く言われていることですが、これはオブジェクト同士でも同様です。オブジェクトの比較にあるように[==]ならオブジェクトの持つ変数同士が比較され、[===]ならオブジェクトの参照が比較されます。（Javaの==とStrings#equals()に似ていますね。）

つまり循環参照しているオブジェクト同士を[==]で比較すると延々オブジェクトの変数が比較され続けてエラーが発生するというわけです。

ちなみにPHP4では[==]も[===]もPHP5の[==]と同じ動作をするようです。（PHP4におけるオブジェクトの比較）

呼び出し元のメソッドを使用したい場合にうっかり循環参照になっていたりするので注意が必要です。

 

# エントリではPHP5.1.6では[==]で動作していたようですが、手元の環境では5.1.6でも同じエラーが発生しました。5.2.0特有の現象というわけでは無さそうです。

cakephp.jpにあるように、1.1.10.3825のbake.phpをWindows上で使用すると上手く動作しない場合があります。

この現象はcake\とは異なるディレクトリにプロジェクトを生成しようとした場合に発生するようです。

New App Directoryが作成するプロジェクトのディレクトリなのですが、妙なパスが表示されています。[D:\tmp\d:\tmp\app]

> php -q D:\tmp\cake\scripts\bake.php -project d:\tmp\app
...
New App Directory: D:\tmp\d:\tmp\app
---------------------------------------------------------------

原因はディレクトリ区切り文字を[/]に決めうちしているためです。この問題の修正パッチを作りましたのでよろしければどうぞ。

--- cake_1.1.10.3825.org/cake/scripts/bake.php       2006-11-10 14:37:00.531250000 +0900
+++cake_1.1.10.3825/cake/scripts/bake.php       2006-11-13 18:08:16.187500000 +0900
@@ -82,12 +82,12 @@
        }
 
        $shortPath = str_replace($root, '', $app);
-       $shortPath = str_replace('../', '', $shortPath);
-       $shortPath = str_replace('//', '/', $shortPath);
+       $shortPath = str_replace('..' . DS, '', $shortPath);
+       $shortPath = str_replace('//', DS, $shortPath);
 
-       $pathArray = explode('/', $shortPath);
+       $pathArray = explode(DS, $shortPath);
        $appDir = array_pop($pathArray);
-       $rootDir = implode('/', $pathArray);
+       $rootDir = implode(DS, $pathArray);
        $rootDir = str_replace('//', '', $rootDir);
 
        if(!$rootDir) {

 

2006/11/21追記：
修正パッチを取り込んで貰えました。https://trac.cakephp.org/changeset/3872
他にも修正されていますし、いずれ1.1.11が出そうですね。

5.2.0の新機能にDateTimeクラスがあります。既存のdate関係の関数をクラスにまとめたもののようです。

以前から思っていたのですが、日付時間データを文字列や数値で持っているのに違和感がありました。特にDBからデータを取得した際、DBで日付時間型（PostgreSQLならtimestamp型）のものはPHPでもそのまま扱えないかな、と思っていました。

そこで自分が構築しているアプリケーションではDBのレコードをModelに格納する際に自作のDateクラスに変換しています。こうしておけばそのModelを使う場合は日付時間をDateクラスのメソッドで操作できます。イメージとしては以下のような感じです。

< ?php
class MyModel {
  /** 登録時間 */
  var $regiterTime = null;
}

// DB からモデル生成
// ....

// 年だけを取得
$model->registerTime->getYear();
// YYYY/MM/DD文字列で取得
$model->registerTime->getFormatString('Y/m/d');
?>

日付時間は色々なフォーマットに変換して表現する（YYYY/MM/DDにしたりYYYYだけ取ったり）のでクラス化した方が後は扱いやすいと思うのですが、あまりこういった話は聞かないですね。DBの日付時間型関数が充実しているだけにPHPでも、と考えてしまいます。

そこで冒頭のDateTimeクラスです。

いずれはPDOを使ったDBアクセスではDBの日付時間型をこのクラスに変換してくれるようになって、日付時間はDateTimeクラスで扱うというのが標準になってくれれば良いなあ、と淡い期待を抱いています。

常にDateTimeクラスにされると困る人もいるでしょうから、イメージとしてはこんな感じでしょうか。

< ?php
  // createdの内容をDateTimeオブジェクトにする
  $stmt->bindColumn('created', $created, PDO::PARAM_DATE);
?>

ひそかに注目のクラスです。

 

2006/11/10追記：PDOとの連携を考えてみました。

かなりadhocな方法ですが、いちおうDBの日付時間型をDateTimeクラスで表現できます。

< ?php
class MyModel {
  private function __set($name, $value) {
    if ($name == 'created') {
      $this->{$name} = new DateTime($value);
    } else {
      $this->{$name} = $value;
    }
  }
}

dl('php_pdo_pgsql.dll');
try {
  $dbh = new PDO('pgsql:host=localhost;dbname=xxx', 'xxx', 'pass');
  $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  $stmt = $dbh->prepare('SELECT id,created from users where id=?');

  $stmt->execute(array('1'));

  while ($ret = $stmt->fetchObject('MyModel')) {
    printf("id:%d %04d年%02d月%02d日", $ret->id
                                    , $ret->created->format('Y')
                                    , $ret->created->format('m')
                                    , $ret->created->format('d'));
  }

  $dbh = null;
} catch (PDOException $e) {
   print "error!: " . $e->getMessage() . "<br />";
   die();
}
?>

cakeError()は値を返さない、という話です。

Well, this solution would work fine as long as it is only used internally (even though it is not very elegant). But the framework allows you to write a custom error handler, and so if you want to handle a “missing helper file” error yourself, you get PHP errors when you don’t use an exit.

cakebaker » Don’t rely on side effects in your code

はじめてこの箇所のコードを追っかけた時にちょっと頭が「?」になりました。エントリでも言及されていますが、自分でエラーハンドラを作る際はこれを意識しておかないとまずい事になりますね。

どうせErrorHandlerでexitするなら、単純にcakeError()は値を返さず、メソッド内でexitすれば良いような気もします。コードで書くとこんな感じですね。

エラー呼び出し元

< ?php
...
//return $this->cakeError(...);
$this->cakeError(...); // returnしない
...
?>

Object#cakeError()

function cakeError($method, $messages) {
...
  if (class_exists('AppError')) {
    $error = new AppError($method, $messages);
  } else {
    $error = new ErrorHandler($method, $messages);
  }

  exit;
}