Error

[Weeeyaa]

Hoi,

Ik heb dan toch nog eens de moed bijeen geraapt om nog eens te proberen PHP te leren.
Ik moet zeggen, dat wil goed lukken. Ik ben nu aan het proberen een mailscript te maken, waarmee je een email van je eigen mail adres naar dat van iemand anders kunt sturen.
alleen krijg ik een error, maar ik weet niet wat daar fout aan is, ik zal wel iets niet afgesloten hebben ofzo, maar toch vind ik het niet...
dit is de error die ik krijg: Parse error: syntax error, unexpected T_ELSE on line 21

en het script is:

weet een van jullie wat er fout aan is?
alvast bedankt,

[KillerSponge]

Ns even kijken. Als je alles even netjes maakt met tabjes, zie je dat er dit uitkomt:

[code="php"]<?

if (isset($verzenden) && $mailadres_van != "" && $mailadres_aan != "" && $bericht != "")
{
if (mail("$mailadres_aan", "$onderwerp", "$bericht", "From:$emailadres_van"))
{
echo "Uw E-mail is succesvol verzonden naar" .$mailadres_aan. ".";
}
else
{
echo "Uw E-mail kon niet verzonden worden. Probeer het later nog een keer!";
}
else
{
echo "Vul eerst alle velden in!";
}
?>[/code]

Zo te zien ben je een haakje vergeten:

[code="php"]<?

if (isset($verzenden) && $mailadres_van != "" && $mailadres_aan != "" && $bericht != "")
{
if (mail("$mailadres_aan", "$onderwerp", "$bericht", "From:$emailadres_van"))
{
echo "Uw E-mail is succesvol verzonden naar" .$mailadres_aan. ".";
}
else
{
echo "Uw E-mail kon niet verzonden worden. Probeer het later nog een keer!";
}
}
else
{
echo "Vul eerst alle velden in!";
}
?>[/code]

Dat is dus een handige tip: zet if-structuren op een overzichtelijke manier neer, dan zie je veel sneller waar het fout gaat. Ook is het handig om een editor te gebruiken die kan zien welke haakjes bij elkaar horen.

Veel succes verder

[Weeeyaa]

bedankt ik ga het eens proberen!
ik gebruik html-kit... heeft in iedergeval met kleurtjes

[KillerSponge]

Ik gebruik notepad2, dat is gewoon kladblok die syntaxmodules heeft voor bijna alle script/programmeer-talen, kan herkennen welke haakjes bij elkaar horen, regelnummering heeft, en automatisch kan inspringen. En meer heb je niet nodig

[Weeeyaa]

ik zal dat eens proberen

[Weeeyaa]

ook handig ja

[KillerSponge]

Ik vind het vooral handig omdat het geen overbodige meuk heeft, die veel andere editors wel hebben... Gewoon kladblok met de functies die kladblok eigenlijk standaard al zou moeten hebben ^^

[Weeeyaa]

mag ik nog een vraag stellen?

waarom zet hij de ingevulde data in de URL en komt niet de melding dat de mail verstuurt is?

[Derk]

methode="POST">
dat schrijf je niet zo geloof ik...
Is dat niet method="POST" ?

[Weeeyaa]

je hebt gelijk, nu komt t niet meer in de url, maar krijg ik niet de melding dat t is verzonden....

he bedankt he derk!

[KillerSponge]

Ik weet niet zeker of het uitmaakt, maar volgens mij hoor je variabelen niet tussen quotes te zetten, iets wat je wel doet bij de isset() functie.

[Weeeyaa]

Ik denk dat ik allemaal van die kleine schoonheidsfoutjes heb... stom

en bij de isset heb ik alleen "" dat t niet ledig mag zijn...

[code="php"](isset($verzenden) && $mailadres_van != "" && $mailadres_aan != "" && $bericht != "")[/code]

bedankt mensen,

Thijs

[KillerSponge]

Uhm, sorry, ik bedoel bij mail() ^^

[Weeeyaa]

gaan we proberen

nee, t maakt niets uit... jammer

[Derk]

[KillerSponge]