Changeset 17419

Timestamp:

2008/07/11 17:05:07 (16 years ago)

Author:

Seasoft

Message:

配列走査ブロックの統合

Location:

branches/comu-ver2

Files:

• convert.php (modified) (1 diff)
• html/install/index.php (modified) (1 diff)

branches/comu-ver2/convert.php

@@ -116 +116 @@
  */
 function listdirs($dir) {
-    static $alldirs = array();
+    static $alldirs = array(); // FIXME 1実行で複数回外部から呼ばれると累積されない? 参照渡しで持ちまわるのが妥当な気がする。html\install\index.php に良い実装が存在する気がする。ただし、このファイルはインクルード用で無いようなので、これで良いのかも。
     $dirs = glob($dir . '/*');
-    if (is_array($dirs) && count($dirs) > 0) {
-        foreach ($dirs as $d) $alldirs[] = $d;
-    }
     if (is_array($dirs)) {
-        foreach ($dirs as $dir) listdirs($dir);
+        foreach ($dirs as $d) {
+            $alldirs[] = $d;
+            listdirs($d);
+        }
     }
     return $alldirs;

branches/comu-ver2/html/install/index.php

@@ -1085 +1085 @@
     $dirs = glob($dir . '/*');
     if (is_array($dirs) && count($dirs) > 0) {
-        foreach ($dirs as $d) $alldirs[] = $d;
-    }
-    if (is_array($dirs)) {
-        foreach ($dirs as $dir) listdirs($dir);
+        foreach ($dirs as $d) {
+            $alldirs[] = $d;
+            listdirs($d);
+        }
     }
     return $alldirs;