Current Events > Anybody notice what's wrong with my php code?

(+) Yeah! (+)

<?php
function myFunction(){
$firstName = $_GET["fname"];
$midName=$_GET["mname"];
$lastName = $_GET["lname"];

if (($firstName !== null) && ($midName == "") && ($lastName == ""))
echo "<p>Hello $firstName! Welcome to php</p>";

else if
(($firstName !== null) && ($lastName !== null) && ($midName == null))
echo "<p> Hello $firstName $lastName ! It is a pleasure to meet you</p>";

else if
(($firstName !== null) && ($midName !== null) && ($lastName !== null)
echo "<p>Hello $firstName $midName $lastName ! Your middle name is very unique.</p>";
else if
(($firstName == null) && ($midName == null) && ($lastName == null)
echo "<p>"You did not supply any names</p>";

}

myFunction();
?>

That's the code, and I'm using XAMPP. It doesn't work and I don't knwo where I;m going wrong.
If you want the html just ask.
---

I cant really remember php, but I think your last two conditions are missing close parens
---

gandalfl0 posted...

I cant really remember php, but I think your last two conditions are missing close parens

yeah this

gandalfl0 posted...

I cant really remember php, but I think your last two conditions are missing close parens

---

I put that in but it's saying there's an error on this line:

echo "<p>"You did not supply any names</p>";

The error is this

Parse error: syntax error, unexpected 'You' (T_STRING), expecting ',' or ';' in C:\xampp\htdocs\Week9Lab\week9labTask1.php on line 25
---

You have an extra "
---

Ok, I changed that and it outputs now but the output is wrong for when they don't enter a name, if they dont it came up as if they entered $firstName
---

Why are you mixing up null and ""

An empty string is not the same thing as null
---

I'm not too versed in php tbh but maybe try adding some lines for if they didn't enter a name at all to either repromt them or just output whatever you want
---

God dammit I didn't mean to delete my post

Anyway, as I was saying, an empty string isn't null and you shouldn't be mixing them up in your conditionals anyway. If they didn't enter a name at all, the program runs on the first if statement because

-an empty string isn't null, so the first condition is true
-the other two fields are empty strings so the other two conditions are true
---

MacadamianNut3 posted...

God dammit I didn't mean to delete my post

Anyway, as I was saying, an empty string isn't null and you shouldn't be mixing them up in your conditionals anyway. If they didn't enter a name at all, the program runs on the first if statement because

-an empty string isn't null, so the first condition is true
-the other two fields are empty strings so the other two conditions are true

Ah, so I would have to do something like $firstName == "" ?
---

Yeah, it's in PHP.
---