Another question

[Schlo_50]

Hello again,

 

I have an upload form with 3 file fields. I want to check if any of them have been used, and if they have create an array with their details. I have written some code but I wanted to see if it made sense, or whether it would need some adjustment.

 

   $aname = $_FILES['ufile']['name'][0];
   $bname = $_FILES['ufile']['name'][1];
   $cname = $_FILES['ufile']['name'][2];
   
   if (isset($aname) && isset($bname)) {
   
   $fn = array('$aname', '$bname');
   
   elseif (isset($aname) && isset($bname) && isset($cname)) {
   
   $fn = array('$aname', '$bname', '$cname');
   
   else {
   
   print "Files have been selected incorrectly.";
   
   }
             foreach ($fn as $fns) {	
$fext = substr($fns, strrpos($fns, '.'));
$jpg = ".jpg";
$jpeg = ".jpeg";

if ($fext == $jpg || $fext == $jpeg) { //     || = or

    print "foo";
  } 

 

Thanks in advance guys! (again)

[Schlo_50]

(I have just realised I've forgotten some curly braces!) Please ignore that..lol

[conker87]

Repeat after me:

 

I will check my code for curly brackets

I will check my code for curly brackets

I will check my code for curly brackets

 

[Schlo_50]

lol, Have done now!

 

Have also tested the code and have found out variables in an array need not have singles quotes surrounding them!

 

I now have:

 

   $aname = $_FILES['ufile']['name'][0];
   $bname = $_FILES['ufile']['name'][1];
   $cname = $_FILES['ufile']['name'][2];
   
   if (isset($aname)) {
   
   $fn = array($aname);
   
   }
   
   elseif (isset($aname) && isset($bname)) {
   
   $fn = array($aname, $bname);
   
   }
   
   elseif (isset($aname) && isset($bname) && isset($cname)) {
   
   $fn = array($aname, $bname, $cname);
   
   }
   
   else {
   
   print "Files have been selected incorrectly.";
   
   }
   
foreach ($fn as $fns) {	
print $fns;
}

 

However, my loop will not print more than one file's details if I upload more than one file. Whats wrong with my if else statements?