函數及其返回值的問題

Question

我製作了這種點系統，用戶可以在這裏使用點數。用戶點數確實被扣除。我沒有包含很多變數，但他們都沒問題。問題發生在return($success)和return($error_message)。

下面是代碼：

function died($error) { 
    header("Location: error_points_on.php?error=" . $error); 
    die(); 
} 

function success($success) { 
    header("Location: success_points_on.php?success=" . $success); 
    die(); 
} 
function quote_smart($value, $handle) { 

if (get_magic_quotes_gpc()) { 
$value = stripslashes($value); 
} 

if (!is_numeric($value)) { 
$value = "'" . mysql_real_escape_string($value, $handle) . "'"; 
} 
return $value; 
} 

function product($price,$points, $name, $uname, $error_message, $success_message) { 


    $user_name = "cencord"; 
    $pass_word = "cencord"; 
    $database = "cencord"; 
    $server = "cencord"; 

    $db_handle = mysql_connect($server, $user_name, $pass_word); 
    $db_found = mysql_select_db($database, $db_handle); 

if ($db_found) { // connect to DB 

      $uname = quote_smart($uname, $db_handle); 
      $SQL = mysql_query("SELECT points FROM members WHERE username=$uname"); 
      $points = mysql_fetch_row($SQL); 

      $points = $points[0]; // make it a variable rather then an array 


    if ($points >= $price) { 

     $points = $points-$price; // fjern points 

     $points = quote_smart($points, $db_handle); 

     mysql_query("UPDATE members SET points=$points WHERE username = $uname"); 

     $success_message .= "The " . $name . " has been mailed to your E-mail, please allow 5 minutes for it to arrive.<br />"; 
     return($success_message); 
    } 


     else if ($points < $price) { 

     $error_message .= "You have " . $points . " you need " . $price . " points to purchase a " . $name; 
     return($error_message); 
     } 

else if (!$db_found) { 
$error_message .= "Could not connect to the database, please contact support"; 
return($error_message); 
    } 
} 
} 



    if($Checked1 == true) { 
     product(400, $points, "Some string", $uname,  $error_message, $success_message); //price and name 
} 
    if($Checked2 == true) { 

     product(400, $points, "Some string", $uname,  $error_message, $success_message); 
} 



if(strlen($error_message) > 0) { 
    died($error_message); 
    } 
if(strlen($success_message) > 0) { 
    success($success_message); 
    } 

echo "error didnt pass at all"; 

我可以添加

header("Location: success_points_on.php?success=" . $success); 

，而不是一回的，但我希望用戶能夠購買多件商品，（加入它，而不是回報確實有效）。

Answer 1

你的邏輯嵌套是錯誤的。剔除內臟和只留下if語句，你有你的product()功能：

function product($price,$points, $name, $uname, $error_message, $success_message) { 
    if ($db_found) { // connect to DB 
     if ($points >= $price) { 
      $success_message = "blah"; 
      return($success_message); 
     } 
     else if ($points < $price) { // This if() part is redundant, btw 
      $error_message = "blah"; 
      return($error_message); 
     } 
     else if (!$db_found) { 
      $error_message .= "blah"; // This should be = instead of .= 
      return($error_message); 
     } 
    } 
} 

你要代替的是：

function product($price,$points, $name, $uname, &$error_message, &$success_message) { 
    if ($db_found) { 
     if ($points >= $price) { 
      $success_message = "blah"; 
      return($success_message); // this is redundant actually 
     } else { 
      $error_message = "blah"; 
      return($error_message); // this is redundant actually 
     } 
    } else { 
     $error_message = "blah"; 
     return($error_message); // this is redundant actually 
    } 
} 

我會強烈建議使用一些工具，如IDE可以保持您的代碼格式正確，這將使這些類型的問題更容易看到。

編輯

我也只注意到你沒有被引用，這將導致更多的問題傳遞$ ERROR_MESSAGE和$ success_message。上面所做的更改（在第二個示例中），但此代碼仍然不是我所稱的最佳實踐。

Answer 2

我同意上面所說的一切@leftclickben，但是想再補充一點觀察。我似乎在代碼中使用$error_message和$success_message的值，但在要調用product()之後的代碼中，如果要使函數更改其值，則必須使用&符號通過引用來引用它們。

你的函數原型更改爲

function product($price,$points, $name, $uname, &$error_message, &$success_message) { 

，並在函數返回後的價值變動將可用。