class design - view_htmlクラスを改善する

Question

このクラスは、ユーザが作成したコンテンツをページに提供します。

コールがこのようなHTMLで作られている。

<?php new route('tweets'); ?> 

又は

<?php new route('bookmarks'); ?> 

ルートview_htmlをインスタンス化沿っPARAMTERを通過します。

どのような改善ができますか？

/*view_html*/ 

class view_html extends database 
    { 
    function __construct($type) 
    { 
    parent::__construct();  
    switch ($type) 
     { 
     case "bookmraks": 
     $this->bookmarks(); 
     break; 
     case "tweets": 
     $this->tweets(); 
     break; 
     default: 
     echo "Invalid View Type"; 
     break; 
     } 
    } 
    private function bookmarks() 
    { 
    $email = $_SESSION['email']; 
    $query_return = database::query("SELECT * FROM bo WHERE email='$email' ORDER BY name ASC"); 
    while ($ass_array = mysqli_fetch_assoc($query_return)) 
     { 
     $fav=$this->fav($ass_array['url']); 
     echo "<img name=\"bo_im\" class=\"c\" src=\"$fav\"/ onerror=\"i_bm_err(this)\"><a target=\"_blank\" name = \"a1\" class = \"b\" href = \"$ass_array[url]\">$ass_array[name]</a>"; 
     }  
    } 
    private function tweets() 
    { 
    $query_return = database::query("SELECT * FROM tw ORDER BY time DESC LIMIT 7"); 
    $time = time(); 
    while ($a = mysqli_fetch_assoc($query_return)) 
     { 
     echo "<div class=\"Bb2b\"><img class=\"a\" src=\"p/$a[email].jpg\" alt=\"\"/><a class=\"a\" href=\"javascript:void(0)\">$a[fname] posted <script type=\"text/javascript\">document.write(v0($a[time],$time))</script></a><br/><p class=\"c\">$a[message]</p></div>"; 
     } 
    } 
    private function fav($url)  
    {  
    $pieces = parse_url($url);  
    $domain = isset($pieces['host']) ? $pieces['host'] : '';  
    if(preg_match('/(?P<domain>[a-z0-9][a-z0-9\-]{1,63}\.[a-z\.]{2,6})$/i', $domain, $regs))  
     {  
     return $pieces['scheme'] . '://www.' . $regs['domain'] . '/favicon.ico';  
     }  
     return false; 
    } 
    } 

Answer 1

これは、コードレビューサイトではありません、あなたはここでhttps://codereview.stackexchange.com/

をこの質問を服用したほうが良いと思いますしたい場合はしかし、あなたはステートメントスイッチブロックを使用して代わりの場合は検討する必要があります可能なタイプの数を後で拡張します。 http://php.net/manual/en/control-structures.switch.php

Answer 2

このタイプでは、コンストラクタでスイッチケース構成を使用できます。デフォルトの場合、InvalidArgumentExceptionがスローされる可能性があります。