[RESOLVED] why couldnt i just be happy surfin for porn???

[mikefly]

hey guys next problem here is checkin to make sure the required fields are filled in. it works to a point if there is missing info it gives you the right error and bands2.php but if all the fields are filled in it just gives you band2.php all filled in with the info and stops the rest of the script from exicuting

    //********BEGIN ERROR HANDLing***********
if((!$bandname) || (!$password) || (!$emailaddy) || (!$bio) || (!$genre) || (!$photo) || (!$song))
  if(!$bandname) {
    echo "<center><font size='4' color='red'>Band name is needed!</font><center>";
    }
      if(!$password) {
    echo "<center><font size='4' color='red'>Password is needed!</font><center>";
    }
      if(!$bio) {
    echo "<center><font size='4' color='red'>Tell us about yourself!</font><center>";
    }
      if(!$genre) {
    echo "<center><font size='4' color='red'>What type of music do you play!</font><center>";
    }
      if(!$photo) {
    echo "<center><font size='4' color='red'>Gotta have a picture!</font><center>";
    }
      if(!$song) {
    echo "<center><font size='4' color='red'>Music we need music!</font><center>";
    }
    {
    require('bands2.php');
  exit();
  } 

as always any and all advice is welcome
as for the title to this thread i just sometimes wish i would have never started building websites

[bokeh]

The first if statement does not have curly braces encapsulating the following block. This mean it is only applied to the the first subsequent statement.

By the way every HTML tag you are using was deprecated in 1999.

[cridley]

or use elseif if you don't like curly braces

[NogDog]

While the title was pretty clever, it would be more useful to everyone here if you'd use a meanngful title that gave us some sort of clue what your post is about.

[bokeh]

or use elseif if you don't like curly braces

elseif wont work here because the require is is desired in all cases that trigger the first if statement.

[mikefly]

thanks guys the currly brackets fixed it

[cridley]

was kidding about the braces, although

if((!$bandname) || (!$password) || (!$emailaddy) || (!$bio) || (!$genre) || (!$photo) || (!$song))
if(!$bandname) {
echo "<center><font size='4' color='red'>Band name is needed!</font><center>";
}
elseif(!$password) {
echo "<center><font size='4' color='red'>Password is needed!</font><center>";
}
elseif(!$bio) {
echo "<center><font size='4' color='red'>Tell us about yourself!</font><center>";
}
elseif(!$genre) {
echo "<center><font size='4' color='red'>What type of music do you play!</font><center>";
}
elseif(!$photo) {
echo "<center><font size='4' color='red'>Gotta have a picture!</font><center>";
}
elseif(!$song) {
echo "<center><font size='4' color='red'>Music we need music!</font><center>";
}
else{
require('bands2.php');
exit();
}

is perfectly valid. ugly, maybe, but valid.
Anyway, still just kidding as all it needs is the braces.