可以优化此循环浏览文件夹中照片的代码吗？

Question

我已经编写了这段代码，现在已经 2 年没有使用 PHP 来循环遍历照片文件夹并按字母顺序将它们写入页面。 这是一个相当简单的请求，但我花了 15 分钟的时间来写。

if ($handle = opendir('photos')) {
  $count = 0;
  $list[] = array();
  while (false !== ($file = readdir($handle))) {
    if ($file != "." && $file != "..") {
      $list[$count] = $file;
      $count ++;
    }
  }
  closedir($handle);
  asort($list);
  $sorted_list = array();
  $sorted_list = array_values($list);
  foreach ($sorted_list as $i => $value) {
    echo "<li><img src=\"photos/$sorted_list[$i]\" alt=\"$sorted_list[$i]\" title=\"\"></li>\n";
  }
}

我是不是完全写错了？ 有什么方法可以改进代码吗？ 很高兴收到任何建设性的反馈。

Answer 1

您可以利用 scandir() 函数，该函数将处理读取目录并对结果进行排序。

$files = scandir('photos');
if ($files !== false) 
{
    foreach($files as $f) {
        if ($f == '..' || $f == '.') continue;      
        echo '<li><img src="photos/'.$f.'" alt="'.$f.'" title=""></li>'."\n";
    }
}

为了便于阅读，我对其进行了一些编辑。

Answer 2

试试这个：

$photos = glob('photos/*');
foreach($photos as $photo) {
    echo "<li><img src=\"{$photo}" alt=\"{$photo}\" title=\"\"></li>\n";

}

http://us.php.net/manual/en/function .glob.php

Answer 3

您不需要 $count。 这将为您提供相同的结果

$list[] = array();
while (false !== ($file = readdir($handle))) {
    if ($file != "." && $file != "..") {
        $list[] = $file;
    }
}

将排序和显示替换为：

sort($list);
for ($i = 0; $i < count($list); $i++) {
    echo "<li><img src=\"photos/{$list[$i]}\" alt=\"{$list[$i]}\" title=\"\"></li>\n";
}

Answer 4

您可以替换

foreach ($sorted_list as $i => $value) {
    echo "<li><img src=\"photos/$sorted_list[$i]\" alt=\"$sorted_list[$i]\" title=\"\"></li>\n";
  }

为

foreach ($sorted_list as $value) {
    echo "<li><img src=\"photos/$value\" alt=\"$value\" title=\"\"></li>\n";
}

然后您不需要调用 array_values()，因为数组键不按数字顺序并不重要。

Answer 5

更简单的方法是使用 scandir 函数：

$dir = 'photos';
$files = array_diff( scandir($dir), array(".", "..") );
foreach ($files as $i => $value) {
    echo "<li><img src=\"photos/$value\" alt=\"$value\" title=\"\"></li>\n";
}

祝你好运！