If...else lukt niet

Ik ben aan het proberen om mijn soort van noob-nieuwsbriefsysteempje te maken. Ik wil controleren of het e-mailadres een @ bevat. Toch krijg ik steeds weer de fout:
Parse error: syntax error, unexpected } in /home/***/public_html/newsletter/process.php on line 12

if(stristr($_GET[mail], @) === FALSE) {
mysql_close($con)
} else {
$sql="INSERT INTO Persons (Mail)
VALUES
($_GET[mail])";

Wat is er mis?

Je bent de
;
vergeten!

mysql_close($con)
Moet zijn:
mysql_close($con);

Eleky schreef:

Je bent de

;
vergeten!

mysql_close($con)
Moet zijn:
mysql_close($con);

Idd. Ik dacht al dat het zoiets was! Bedankt

Graag gedaan!

Nog een tip trouwens, Werk met POST ipv GET, GET kan namelijk vaak vervalst worden.

Jponsen:
Nog een tip trouwens, Werk met POST ipv GET, GET kan namelijk vaak vervalst worden.

Klopt natuurlijk, via forms altijd met post werken, maar soms is GET de enige oplossing, indien er bijvoorbeeld via een link iets moet gebeuren!

Dan wel ja, en bij een of andere generator is het ook nog wel handig..
Maar anders heb je mensen die gaan spammen met bijv. ?mail=bla@bla.nl
En dan 100x doen, heb je dr 100 binnen.

post kan net zo hard vervalst worden als get.. misschien ietsiepietsie "moeilijker", maar nog steeds kan iedereen t vervalsen.

Devon schreef:

Ik ben aan het proberen om mijn soort van noob-nieuwsbriefsysteempje te maken. Ik wil controleren of het e-mailadres een @ bevat. Toch krijg ik steeds weer de fout:

Parse error: syntax error, unexpected } in /home/***/public_html/newsletter/process.php on line 12

if(stristr($_GET[mail], @) === FALSE) {
mysql_close($con)
} else {
$sql="INSERT INTO Persons (Mail)
VALUES
($_GET[mail])";

Wat is er mis?

Heefd volgensmij niets met die ; te maken
onbekende } staad er in je script
mischien word het niet geopend met {

"unexpected }" zegt ie, dus hij verwacht die } niet.
Dat klopt ook, want doordat je mysql_close($con) niet hebt afgesloten met een puntkomma, denkt het script dat er nog wat anders achter aan komt, maar dan komt die } ineens al, dus dan snapt ie het niet meer.

wrm die
mysql_close

boven het script, hij hoort toch aan het eind, en de mysql connect in het begin...

of snap ik het nou niet?

@kingunit: als je leest kan je zien dat het al is opgelost...
@Paull: wss is dit een klein deel van het script, en is de verbinden er al boven gemaakt