2012-11-23 50 views
-5

當我嘗試驗證我的數據即時得到這個錯誤,我不明白爲什麼它是「意外」驗證意外{

//Requests fields from form on previous page 
$CDID = $_REQUEST['CDID']; 
$CDTitle = isset($_REQUEST['CDTitle']) ? $_REQUEST['CDTitle'] :null; 
$CDYear = isset($_REQUEST['CDYear']) ? $_REQUEST['CDYear'] :null; 
$catID = isset($_REQUEST['catID']) ? $_REQUEST['catID'] :null; 
$pubID = isset($_REQUEST['pubID']) ? $_REQUEST['pubID'] :null; 


if(empty($CDTitle) || (empty($CDYear) || (empty($catID) || (empty($pubID)){ 
die("<p> You must enter all fields</p>\n");} 
+0

Downvoting問題地獄...你正在做的很好,所以。 – eisberg

+2

@eisberg因此,你認爲計算某人的括號來找到一個微不足道的錯誤應該是衆包? – lanzz

+0

@lanzz我相信幫助他會沒事的。我們每個人都在某個地方開始。 – eisberg

回答

0

多餘的括號..

變化:

if(empty($CDTitle) || (empty($CDYear) || (empty($catID) || (empty($pubID)){ 

到:

if(empty($CDTitle) || empty($CDYear) || empty($catID) || empty($pubID)){ 
+2

這仍然不行,你有一個額外的括號前的最後一個'空' – Mansfield

+0

soz我的錯誤,更正 – vlcekmi3

+0

括號過多將永遠不會導致腳本失敗,只要語法是正確的,他們成對。 – George

2

你if語句應該是:

if(empty($CDTitle) || empty($CDYear) || empty($catID) || empty($pubID)){

你有太多的支架,所以PHP期望看到更多的右括號關閉您的條件。我刪除了不必要的括號。

+0

ahhhh yes !!!解釋它 –

+0

@BenArmstrong如果它對你有幫助,一定要點擊答案左邊的綠色複選標記將其標記爲已接受的答案。 – Mansfield

0

)丟失:

if(empty($CDTitle) || (empty($CDYear) || (empty($catID) || (empty($pubID)){ 

應該是:

if(empty($CDTitle) || (empty($CDYear)) || (empty($catID)) || (empty($pubID))){ 

if(empty($CDTitle) || empty($CDYear) || empty($catID) || empty($pubID)){ 
+2

這也不起作用,第二個和第三個'emptys'需要移除開口支架或額外的閉合支架。 – Mansfield

+0

@Mansfield,編輯,謝謝 – nicolast