2017-09-07 1 views
0
if(!empty($act2) && empty($l2)) { 
     unset($act2); 
     unset($l2);   
} 

if($act2 == "empty" && !empty($l2)) { 
     unset($act2); 
     unset($l2);   
} 

if(!empty($act3) && empty($l3)) { 
     unset($act3); 
     unset($l3);   
} 

if($act3 == "empty" && !empty($l3)) { 
     unset($act3); 
     unset($l3);     
} 

if(!empty($act4) && empty($l4)) { 
     unset($act4); 
     unset($l4);    
} 

if($act4 == "empty" && !empty($l4)) { 
     unset($act4); 
     unset($l4);   
} 

//Wie dieses Skript kürzer

if(!empty($l1) && !empty($act1) && empty($l2) && empty($act2) && empty($l3) && empty($act3) && empty($l4) && empty($act4)) { $validOrder = "true"; $l2 = ""; $act2 = ""; $l3 = ""; $act3 = ""; $l4 = ""; $act4 = ""; } 
if(!empty($l1) && !empty($act1) && !empty($l2) && !empty($act2) && empty($l3) && empty($act3) && empty($l4) && empty($act4)) { $validOrder = "true"; $l3 = ""; $act3 = ""; $l4 = ""; $act4 = ""; } 
if(!empty($l1) && !empty($act1) && !empty($l2) && !empty($act2) && !empty($l3) && !empty($act3) && empty($l4) && empty($act4)) { $validOrder = "true"; $l4 = ""; $act4 = "";} 
if(!empty($l1) && !empty($act1) && !empty($l2) && !empty($act2) && !empty($l3) && !empty($act3) && !empty($l4) && !empty($act4)) { $validOrder = "true"; } 

Grundsätzlich zu machen, ich habe ein Formular und einen PHP-Code, im ersten Teil vor dem // es bestätigt, wenn der Link gesetzt und Wenn die Handlung leer ist und die Handlung leer ist und die Verknüpfung gesetzt ist, wird die Verknüpfung und die Handlung dadurch aufgehoben. Im zweiten Teil wird nach einer gültigen Bestellung gesucht, so dass die Person keine Verbindung nur mit act1 & 4 herstellen kann, sondern einer arthetischen Reihenfolge folgt und die Variablen leer setzt. Wie man es kürzer und effektiver macht.

+2

Ich denke, das auf gehört [Codereview] (https://codereview.stackexchange.com/) – GrumpyCrouton

+1

Statt getrennte Variablen zu verwenden, können Sie ein Array mit allen Daten erstellen und dann erstellen Sie einfach eine Funktion, wo Sie übergeben Sie das Array und welche Schlüssel zu überprüfen. –

+0

+ GrumpyCrouton, du hast die alten Fragen beantwortet, was ist falsch daran? :( –

Antwort

0

Basierend auf den Kommentaren, nicht sicher, dass dies den Code kürzer und freundlich dynamisch für Ihre mehrere Variablenpaare aussehen wird.

$act_array = array("act2","act3","act4"); 
$l_array = array("l2","l3","l4"); 

$x = 1; 

While ($x <= 3) 
{ 
    if(!empty($act_array[x+1]) && empty($l_array[x+1])) 
    { 
     unset($act_array[x+1]); 
     unset($l_array[x+1]);    
    } 

if($act_array[x+1] == "empty" && !empty($l_array[x+1])) 
    { 
     unset($act_array[x+1]); 
     unset($l_array[x+1]);   
    } 

    $x++; 
} 
+0

und der zweite Teil ist okay, so: o? –

+0

gibt es irgendwo einen Fehler –

+0

@JDaniels Ich legte ein extra ')' in der 'unset ($ act_array [x + 1])));' und im nächsten Teil entfernen Sie auch das – LONG