兩大功能需要成爲一個我要怎麼做

function friday16mei(){ 
    $ret= ""; 
    if(time() <= strtotime('17-05-2014')){ 
    $ret = 'friday 16 mei'; 
    } 
    return $ret; 
} 
function getoptions(){ 
    $ret = ""; 
    $saturday22 = saturday22march(); 
    if(strlen($saturday22) > 0){ 
     $ret .= "<option>" . $saturday22 . "</option>"; 
    } 
    $friday16mei = friday16mei(); 
    return $ret; 
} 
$Content= ' 
<div class="content"> 
    I am signing up for the following date:<br /> 
    <select name="date[0]"> 
     '. getoptions() .' 
    </select> 
</div> 
'; 
echo $Content;

我有2個功能，現在，他們需要成爲一個我要怎麼做呢？我需要這個參數嗎？兩大功能需要成爲一個我要怎麼做

這是我可以在將來添加更多或編輯它。

來源

2014-02-11 Harryaars

你有什麼實際的目標是什麼？你的函數聽起來像是可以被泛化的（例如，將日期作爲參數傳遞給函數，該函數將日期返回爲字符串或空字符串）。 – Dehalion

同意@Dehalion，一般來說，1個函數只做1件事（最好只有1件事），最好是有2個可以獨立調用的小函數...那麼合併它們的目標是什麼？ – versvs

在旁註：您可能想要確保您堅持使用單一語言，而不是將兩種語言組合在一起。「梅」是荷蘭語，英語相當於「五月」。 – Tularis

店你的約會檢查的價值觀和你的文字一個陣列中的一個在一個循環，而不是一個方法進行比較：

function getoptions(){ 
    $check = array(
     '17-05-2014' => 'friday 16 mei', 
     '23-03-2014' => 'saturday 22 march' 
    ); 

    $now = time(); 
    $result = array(); 
    foreach($check as $date => $text) { 
     if($now <= strtotime($date)) { 
      $result[] = $text; 
     } 
    } 

    $html = ''; 
    foreach($result as $v) { 
     $html .= '<option>'.$v.'</option>'; 
    } 

    return $html; 
}

來源

2014-02-11 09:52:39 silly

是的，這是我如何，但它需要thnxs！ – Harryaars

兩大功能需要成爲一個我要怎麼做

回答

相關問題