局部变量保持未初始化

Question

私有阵列$list_of_files保持未初始化状态。我如何从while循环更新它？

class listOfFiles { 
private $list_of_files = []; 

function __construct() { 
    if ($handle = opendir(WEB_STORAGE_DIR)) { 

    while (false !== ($entry = readdir($handle))) { 
     $this->list_of_files[$entry] = filesize(WEB_STORAGE_DIR.DIRECTORY_SEPARATOR.$entry); 
    } 

    closedir($handle); 

    // Remove . and .. from the list 
    unset($list_of_files['.']); 
    unset($list_of_files['..']); 
    } 
} 

function is_empty() { 
    return empty($list_of_files); 
} 
} 

Answer 1

$list_of_files是指的是一个与$this->list_of_files属性不同的变量。

变量声明/在一个函数引用是仅在该功能有效（除非你使用全球 - 不过这被普遍认为是“邪恶”的，应该避免）

属性都可以从类中的所有方法（除非它们是静态的）并坚持物体的生命。

<?php 
//lets show all error so we can see if anything else is going on.. 
error_reporting(E_ALL & ~E_NOTICE); 

class listOfFiles { 
    private $list_of_files = []; 

    function __construct() { 
     if ($handle = opendir(WEB_STORAGE_DIR)) { 

     while (false !== ($entry = readdir($handle))) { 
      $this->list_of_files[$entry] = filesize(WEB_STORAGE_DIR.DIRECTORY_SEPARATOR.$entry); 
     } 

     closedir($handle); 

     // Remove . and .. from the list 
     unset($this->list_of_files['.']); 
     unset($this->list_of_files['..']); 
     } 
    } 

    function is_empty() { 
     return empty($this->list_of_files); 
    } 
} 

问题是目录不存在？这将是更好试图打开前检查这一点，并且还允许当它存在什么样的事，但你不能真正阅读：

<?php 
//lets show all error so we can see if anything else is going on.. 
error_reporting(E_ALL & ~E_NOTICE); 

class listOfFiles { 
    private $list_of_files = []; 

    function __construct() { 
     if(!is_dir(WEB_STORAGE_DIR)){ 
     throw new Exception("Missing Web Storage Directory"); 
     } 
     $handle = opendir(WEB_STORAGE_DIR); 
     if (!$handle) { 
     throw new Exception("Could not read Web Storage Directory"); 
     } 
     else{ 

     while (false !== ($entry = readdir($handle))) { 
      $this->list_of_files[$entry] = filesize(WEB_STORAGE_DIR.DIRECTORY_SEPARATOR.$entry); 
     } 

     closedir($handle); 

     // Remove . and .. from the list 
     unset($this->list_of_files['.']); 
     unset($this->list_of_files['..']); 
     } 
    } 

    function is_empty() { 
     return empty($this->list_of_files); 
    } 
} 

我已经加入error_reporting(E_ALL & ~E_NOTICE);的例子，因为这将确保您会看到任何错误并可能有助于调试您的问题。更多的信息在这里：http://php.net/manual/en/function.error-reporting.php

Answer 2

访问一个属性，你需要使用$this，否则你正在创建一个局部变量。你在一个地方这样做，但例如不在这里

return empty($list_of_files); 

由于该变量从未设置，这将始终返回相同的东西。

return empty($this->list_of_files); 

这同样适用于其他财产的引用，使得完整的代码（这是未经测试，当然，因为你没有提供任何可检验）是这个样子

class listOfFiles { 
    private $list_of_files = []; 

    function __construct() { 
    if ($handle = opendir(WEB_STORAGE_DIR)) { 

     while (false !== ($entry = readdir($handle))) { 
     $this->list_of_files[$entry] = filesize(WEB_STORAGE_DIR.DIRECTORY_SEPARATOR.$entry); 
     } 

     closedir($handle); 

     // Remove . and .. from the list 
     unset($this->list_of_files['.']); 
     unset($this->list_of_files['..']); 
    } 
    } 

    function is_empty() { 
    return empty($this->list_of_files); 
    } 
}