If Else Nesting Question

[RobRocker]

Hello,

What is wrong with my nested IF ELSE's?

<?php 
		  		 $deletelink = "&quot;delete/sold.php&quot;";
		  		 $idtext = "&quot;id&quot;";
				 $test = "&quot;Are you sure you want to delete this image?&quot;";
				 if($row_Recordset1['img_1'] != ""){
				 Print "<input type='hidden' name='img_1' value='" . $row_Recordset1['img_1'] . "'>";
				 Print "<img src='". $row_Recordset1['img_1'] ."' width='200' height='150' style='margin-bottom: 5px;'/>";
				 	if($row_Recordset1['img_2'] != ""){
					Print "<input type='hidden' name='img_2' value='" . $row_Recordset1['img_2'] . "'>";
		 			Print "<img src='". $row_Recordset1['img_2'] ."' width='200' height='150'/>";
					}
						if($row_Recordset1['img_3'] != ""){
						Print "<input type='hidden' name='img_3' value='" . $row_Recordset1['img_2'] . "'>";
		 				Print "<img src='". $row_Recordset1['img_3'] ."' width='200' height='150'/>";
						}else{
							echo "<form action='".uploadSinglePicture()."'><table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
							echo "<tr><td>Upload Image</td><td><input type=file name='images' class='bginput'></td></tr>";
							echo "<tr><td><input name='submit' type='submit' value='Add Picture'/></td></tr>";
							echo "</table></form>";
						}
					else{
					$max_no_img=2;  // Maximum number of images value to be set here
					echo "<form action='".uploadTwoPictures()."'><table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
						for($i=1; $i<=$max_no_img; $i++){
						echo "<tr><td>Upload $i</td><td>
						<input type=file name='images[".$i."]' class='bginput'></td></tr>";
						}
					echo "<tr><td><input name='submit' type='submit' value='Add Pictures'/></td></tr>";
					echo "</table></form>";
					}
				else{
				$max_no_img=3;  // Maximum number of images value to be set here
				echo "<form action='".$editFormAction."'><table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
				echo "<table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
					for($i=1; $i<=$max_no_img; $i++){
					echo "<tr><td>Images $i</td><td>
					<input type=file name='images[".$i."]' class='bginput'></td></tr>";
					}
				echo "<tr><td><input name='submit' type='submit' value='Add Pictures'/></td></tr>";
				echo "</table></form>";
				}?>

[NogDog]

The first thing I see is that you have two consectuive else's here:

                        }else{
                            echo "<form action='".uploadSinglePicture()."'><table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
                            echo "<tr><td>Upload Image</td><td><input type=file name='images' class='bginput'></td></tr>";
                            echo "<tr><td><input name='submit' type='submit' value='Add Picture'/></td></tr>";
                            echo "</table></form>";
                        }
                    else{
                    $max_no_img=2;  // Maximum number of images value to be set here
                    echo "<form action='".uploadTwoPictures()."'><table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
                        for($i=1; $i<=$max_no_img; $i++){
                        echo "<tr><td>Upload $i</td><td>
                        <input type=file name='images[".$i."]' class='bginput'></td></tr>";
                        } 

Not knowing (or having the time to figure out) your logic, you'll need to determine if you just need a closing } before that 2nd else, or if the logic needs reworking.

[RobRocker]

Hmm, Can you not have two else statements?

[englishtom]

no you cant have two else but I think you could use..

switch($numberofpics){

case '0':

echo 'no picture to upload';

break;

case '1':

echo 'one picture to upload';

break;

case '2':

echo 'two pictures to upload';

break;
}

Dont know if that will help but good luck getting it to work!

[RobRocker]

Hmm...

Well I used
if
elseif
elseif
else

And that works!

Thanks for the help guys, You always come through.

[Plasma]

You can only have one ELSE, you can have multiple ELSE IF's

Else is the last condition, ie, if none of the above succeeded - do this code block.

You may want to reorganise what you have done if it becomes difficult to read.

[Weedpacket]

As Plasma and englishtom said, you can't have two else's for a single if statement.
Think about it:

if(test)
{
    // This code runs if the test succeeds
}
else
{
    // This code runs if the test fails
}
else
{
    // So when is this code supposed to run?
}