PHPで定数を返す関数について

Question

PHPで定数を返す関数を記述する際、static変数を返すか変数をそのまま返すのでは
どちらがいいでしょうか？

自分ならこう記述するなどあれば教えていただけると助かります。

PHPのバージョンは「5.5.3」を想定しています。

<?php
class Hoge {
	private static $array1 = ["111", "222", "333"];
	private static $array2 = ["444", "555", "666"];
	private static $array3 = ["777", "888", "999"];

	// static変数を返す
	function get_array_static($no) {
		if ($no == 1) {
			return self::$array1;
		}
		else if ($no == 2) {
			return self::$array2;
		}
		else if ($no == 3) {
			return self::$array3;
		}
		return null;
	}

	// そのまま返す
	function get_array_direct($no) {
		if ($no == 1) {
			return ["111", "222", "333"];
		}
		else if ($no == 2) {
			return ["444", "555", "666"];
		}
		else if ($no == 3) {
			return ["777", "888", "999"];
		}
		return null;
	}
}

$hoge = new Hoge();
$ary = $hoge->get_array_static(1);
var_dump($ary);
$ary = $hoge->get_array_direct(1);
var_dump($ary);

※5.5系はarray()を[]でもいいんですね。勉強になります。

Comments

[mpyw]

本題から脱線しますが，5.5系なら array( ) 使わずに [ ] に変えちゃっていいと思います．

Answer 1

PHPの場合、配列はオブジェクトではなく、別な変数へ代入するだけで論理的には別な配列となります（実際のコピーは書き込みがあるまで遅延しますが）。つまり、外部的な動作はどちらでもまったく同じです。

PHP 5.6以上では配列定数を作れるのでそれも有効な手段ですが、5.5以前にはないので、同じ配列を1箇所しか使わないなら下手に変数化せず、配列直書きでいい気もします。

Comments

[mpyw]

ちょっとした配列の生成コストは無に等しいので，同じものが何回も登場しない限りはバンバンリテラルとして書いちゃって全然問題ないと思います．直書きを支持します．

[akabei]

なるほどどちらにしても呼び出し側の別な変数へ代入されるので動作はまったく同じなんですね。
定数なので別の箇所で利用することもないしそのまま返してよさそうですね。
ありがとうございます。スッキリしました。

Answer 2

私の場合なら前者にする事が多い気がします。
後で編集してバランス調整したりする可能性があるデータの定義は
プログラムの中に分散していたり紛れ込んだりするような配置には
なるべくしないで、極力まとめて定義しておくようにした方が保守
性があがるように思います。

Comments

[akabei]

たしかに保守することを考えたらクラスの先頭に定数がまとまっていたほうがわかりやすいですね。
ここらへんは扱う定数によって変わってくるところなんでしょうか。
ありがとうございます。参考になります。