PHP类是上市内容的有效解决方案吗？

Question

我想在我们的网站上列出我的投资组合，我写了一个课程，客户端设置和获取有关每个客户的基本知识的参数，即：客户名称，网站，它的特色，是否发布，为他们做，等等。

这是相当原始的，但我正确地认为最好为每个客户端创建一个对象（并保持每个数据在一起），而不是为每个参数创建数组，并从这些数组中调用相应的元素每次？

这是我的课，我知道它仍然是原始的，但我也可能创建一些其他功能，所以现在我只是使用基本的getter和setter来获取或设置每个客户端的参数。

<?php 

class Client { 
    private $name = ""; 
    private $items = array(); 
    private $website = ""; 
    private $featured = false; 
    private $published = false; 
    private $type = array(); 
    private $filename = null; 
    private $extension = null; 

    public function __set($name, $value) { 
     //echo "Setting ".$name." to value: ".$value."\n"; 
     $this->{$name} = $value; 
    } 

    public function __get($name) { 
     //echo "Getting ".$name.": ".$this->{$name}; 
     return $this->{$name}; 
    } 

    public function __isset($name) { 
     return isset($this->{$name}); 
    } 

    public function __unset($name) { 
     unset($this->{$name}); 
    } 
} 

?> 

我仍在计划我可能想要添加到课程中的功能，但现在就是这样。

这里是我的其他代码我使用创建每个客户对象：

<?php 

// create Client object for every client 

$files = array(); 
// files to take out of file listing, I'm developing on Mac, i.e. ._DS_Store file 
$bad_files = array(".","..","._DS_Store"); 
$portfolio = "portfolio"; 
$images = "images"; 
$details = "details"; 
$thumbs = "thumbs"; 

// get all *.txt files listed in portfolio, so a client will not be added to portfolio without the necessary details. 
if (is_dir("$images/$portfolio")) { 
    if (is_dir("$images/$portfolio/$details")) { 
     $files = scandir("$images/$portfolio/$details"); 

     sort($files); 
    } 
} 
$files = array_diff($files, $bad_files); 
sort($files); 

// keeps a list of all clients 
$clients = array(); 

foreach ($files as $file) { 
    $value = file_get_contents("$images/$portfolio/$details/$file"); 

    $newClient = new Client(); 

    $filename = explode(".",$file); 
    $newClient->filename = $filename[0]; 
    $client_image = glob("$images/$portfolio/$images/".$newClient->filename.".*"); 
    $newClient->image = $client_image[0]; 
    $client_thumb = glob("$images/$portfolio/$thumbs/".$newClient->filename.".*"); 
    $newClient->thumb = $client_thumb[0]; 

    $client_items = array(); 
    $client_type = array(); 

    // gets variables from text file contents and explode string to array [key=value] values 
    $content = explode("&",$value); 
    for ($j=0; $j<count($content); $j++) { 
     $client = explode("=", $content[$j]); 
     if (strpos($client[0],"name") !== false) $newClient->name = $client[1]; 
     if (strpos($client[0],"items") !== false) $client_items = $client[1]; 
     if (strpos($client[0],"website") !== false) $newClient->website = $client[1]; 
     if (strpos($client[0],"featured") !== false) $newClient->featured = $client[1]; // show on frontpage 
     if (strpos($client[0],"published") !== false) $newClient->published = $client[1]; // show at all 
     if (strpos($client[0],"type1") !== false) $client_type[] = $client[1]; // show for specific type, eg. business card, website 
     if (strpos($client[0],"type2") !== false) $client_type[] = $client[1]; // show for specific type, eg. business card, website 
    } 

    // these parameters need array type values 
    $newClient->type = $client_type; 
    $newClient->items = explode(", ",$client_items); 

    // adds client to list of clients 
    $clients[] = $newClient; 
} 

?> 

这里是我用来输出每个客户的旗帜和细节代码：

<div id="banner_content"> 
    <? 
     foreach ($clients as $client) { 
     // client must be published to show at all 
      if ((($page == "home" && $client->featured) || $page != "home") && $client->published) { 
    ?> 
    <div class="banner_container"><img src="<? echo $client->image; ?>" width="809" height="324" alt="<? echo $client_name; ?>" title="<? echo $client_name; ?>" /> 
     <div class="banner_details"> 
      <div class="client_name">Client: <b><? echo (!empty($client->name) ? $client->name : "Unknown"); ?></b></div> 
      <div class="client_items"><? echo (!empty($client->items) ? "Items: <b>".join(", ",$client->items)."</b>" : ""); ?></div> 
      <div class="client_website"><? echo (!empty($client->website) ? "Website: <b><a href=\"http://".strtolower($client->website)."\">".$client->website."</a></b>" : ""); ?></div> 
     </div> 
    </div> 
    <? 
    } 
} 
?> 
</div> 

任何帮助或建议将不胜感激。提前致谢。

//编辑

我忘了提，我居然写了类，因为会有一个投资组合页面，其中将包含有关客户的不仅仅是上述信息的详细信息。我知道一个班级只是在横幅中列出图片是有点矫枉过正的。

Answer 1

我对web开发没有任何经验，但在这里使用一门课是绝对好的编程习惯。
管理多个平行阵列很麻烦。例如，无论何时插入新项目或需要移动项目，都需要记住每个数组。此外，在这里使用类可以使您的代码更有组织，因此更易于维护。
我认为这当然值得实施这个类的开销，尤其是一旦你已经掌握了在PHP中使用类的过程！

Answer 2

如果一个类是一个有效的解决方案取决于你如何实现它。这不像你的代码会奇迹般地被增强，因为你对某个东西进行了类声明。

使用类的关键思想之一是封装状态和责任。因此，封装与客户端相关的状态的类Client是一个好主意。给它的方法呈现为List或Table不是。我知道，你没有这样做，但你也没有使用一个有责任将客户端呈现为列表或表格的类。您也没有课程来查找相关的客户端文件。您的第二个发布代码块是一个程序脚本。它不使用OOP。确定问题并将其分解为小（可测试）单元。将它们合并到适当的类中。

有一个不错的series of articles by Lorna Jane Mitchell about PHP5 and OOP有a number of good Q&A on StackOverflow as well。你也想看看what Design Patterns are。和here is a shameless self-plug。