Question

我有两个功能

function ListFiles($dir) {
    if($dh = opendir($dir)) {
        $files = array();
        $topics = array();
        $inner_files = array();
        while($file = readdir($dh)) {
            if($file != "." && $file != ".." && $file[0] != '.') {
                array_push($topics, $file);
                if(is_dir($dir . "/" . $file)) {
                    $inner_files = ListFiles($dir . "/" . $file);
                    if(is_array($inner_files)) $files = array_merge($files, $inner_files);
                } else {
                    array_push($files, $dir . "/" . $file);
                }
            }
        }
        closedir($dh);
        $topics = array();
        $i = 0;

        foreach ($files as $file) {
//wrong result
            $topics[] = getTopicFromPath($file);

//correct result
//$topics[] = getTopicFromPath("/Users/Unknown/Sites/sample/training/topic/acq/19ddb673359747ee9095.txt")
        }
        return $topics;
    }
}

function getTopicFromPath($path){
//$path = /Users/Unknown/Sites/sample/training/topic/acq/19ddb673359747ee9095.txt

    $string1 = substr($path,strpos($path,"topic/"));
//$string1 = topic/acq/19ddb673359747ee9095.txt

    $string2 = str_replace("topic/", "", $string1);
//$string2  = acq/19ddb673359747ee9095.txt

    $string3 = strstr($string2, '/', true);
//$string3 = null
//expecting $string3 = 'acq'

    return $string3;;
}

问题是getTopicFromPath（$ path）无法从readdir（）方法解析字符串。但是，如果我放一个纯字符串，结果是正确的。请检查代码是否清楚。

我想要做的是获取文件路径，将其父文件夹保存为主题。

使用其他方法获取文件可能会解决问题。但我很好奇这些功能有什么问题？

Answer 1

主要问题是您的代码需要清理和简化。

1 - 在函数getTopicFromPath（）中，如果 string3 为NULL，则 string2 中未找到“/”。也许你在Windows下，你的目录分隔符是'\'而不是'/'？

要解决这些问题，请使用原生的 DIRECTORY_SEPARATOR 常量。

2 - 显然，此函数会尝试查找文件 $ path 的目录名称。然后你最好使用目录相关的功能，避免过于具体的编码。过于具体通常意味着依赖于背景和脆弱。

无论如何，我会用两行和两种口味重写你的功能：

function getTopicFromPath($path) {
    $dir = dirname($path);
    return substr($dir, strrpos($dir, DIRECTORY_SEPARATOR) + 1);
}

或

function getTopicFromPath($path) {
    $dir = dirname($path);
    return basename($dir);
}

在递归函数中调用

3 - getTopicFromPath（）。许多条目将被处理多次。那是多余的。

您应该通过两个独立的步骤划分您的流程：首先检索文件的完整路径，然后修剪它们。您将获得可重用性和稳健性。

4 - 最后，你应该清理 ListFile（）函数：

closedir($dh);
$topics = array();
$i = 0;

$topics = array()表示上述对此变量的赋值是无用的，因为它们将被重载。

$i在其范围内未使用。

readdir（）函数的字符串无法操作

1 个答案: