• R/O
  • SSH
  • HTTPS

jinrousiki: コミット


コミットメタ情報

リビジョン2104 (tree)
日時2018-01-03 20:26:26
作者umethyl

ログメッセージ

Loader::LoadFile(): func_get_args() 廃止

変更サマリ

差分

--- trunk/include/loader_class.php (revision 2103)
+++ trunk/include/loader_class.php (revision 2104)
@@ -18,7 +18,7 @@
1818
1919 //ファイル登録
2020 public static function RegisterFile(array &$list, $name, $file) {
21- if (require_once($file)) {
21+ if (true == require_once($file)) {
2222 $list[] = $name;
2323 //printf('◆Register: %s: %s<br>', $name, $file);
2424 return true;
@@ -103,23 +103,20 @@
103103 }
104104
105105 //ファイルロード
106- public static function LoadFile($name) {
107- $name_list = func_get_args();
108- if (is_array($name_list[0])) {
106+ public static function LoadFile(...$name_list) {
107+ if (true === is_array($name_list[0])) {
109108 $name_list = $name_list[0];
110109 }
111- if (count($name_list) > 1) {
112- foreach ($name_list as $name) self::LoadFile($name);
113- return true;
114- }
115110
116- if (is_null($name)) {
117- return false;
118- } elseif (self::IsLoaded($name)) {
119- return true;
120- } else {
121- self::LoadDependence($name);
122- return LoadManager::RegisterFile(self::$file, $name, self::GetPath($name));
111+ foreach ($name_list as $name) {
112+ if (true === is_null($name)) {
113+ throw new InvalidArgumentException("Argument is NULL: {$name}");
114+ } elseif (false === self::IsLoaded($name)) {
115+ self::LoadDependence($name);
116+ if (false === LoadManager::RegisterFile(self::$file, $name, self::GetPath($name))) {
117+ throw new RuntimeException("Load failed: {$name}");
118+ }
119+ }
123120 }
124121 }
125122
@@ -158,7 +155,7 @@
158155
159156 //ファイルパス取得
160157 private static function GetPath($name) {
161- if (isset(LoaderData::$path[$name])) {
158+ if (true === isset(LoaderData::$path[$name])) {
162159 $type = LoaderData::$path[$name];
163160 } else {
164161 $type = $name;
@@ -214,7 +211,9 @@
214211
215212 //依存解決処理
216213 private static function LoadDependence($name) {
217- if (isset(LoaderData::$depend[$name])) self::LoadFile(LoaderData::$depend[$name]);
214+ if (true === isset(LoaderData::$depend[$name])) {
215+ self::LoadFile(LoaderData::$depend[$name]);
216+ }
218217 }
219218 }
220219
旧リポジトリブラウザで表示