局部变量保持未初始化

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);
  }
}