Archief - php: mailscript error

Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.

Flipkikker

Legacy Member
yo, WarHead hier (broer van FlipKikker, te lui om acc te maken)

ik ben bezig met een online mafia rpg, (kheb nog nie veel), en bij de 'contact' sectie krijg ik een error in mn php code,
dit is de error:
Parse error: parse error, expecting `'('' in /home/www/gangworld.mybesthost.com/contact.php on line 54

dit is mn php code:
PHP:
<html>
<head>
<title>&laquo;&curren;Gang-World&curren;&raquo;</title>
<link rel="stylesheet" type="text/css"
href="stylecontact.css"/>
</head>
<body>
<img src="gangworld.jpg">
<p>Heb je <b>belangrijke</b> ideeën, opmerkingen, foutmeldingen<br>
of andere, meld het dan d.m.v het formulier hieronder!<br>
Vergeet uiteraard je emailadres niet in te vullen!</p>
<form action="contact.php" method="post">
<table>
<tr>
<td>
<select name="type">
<option value="idee">Idee
<option value="opmerking">Opmerking
<option value="foutmelding">Foutmelding
<option value="andere">Andere
</select>
</td>
</tr>
<tr>
<td>
<textarea rows="12" cols="45" name="mailtext">
</textarea>
</td>
</tr>
<tr>
<td>
<input type="text" size="35" name="sender" value="[email protected]">
<input type="submit" value="verzenden">
   </td>
</tr>
</table>
</form>
<hr><br>
<table cellspacing="0" cellpadding="0" class="menuindext1">
  <tr>
    <td class="menuindex1"><form action="index.htm"><input type="submit" name="submit" value="Login"></form></td>
<td class="menuindex1"><form action="register.htm"><input type="submit" name="submit" value="Registreren"></form></td>
    <td class="menuindex1"><form><input type="submit" name="submit" value="Wachtwoord vergeten"></form></td>
    <td class="menuindex1"><form action="contact.php"><input type="submit" name="submit" value="Contact"></form></td>
    <td class="menuindex1"><form action="faq.htm"><input type="submit" name="submit" value="FAQ"></form></td>
  </tr>
</table>
<?php
$from = ($_POST["sender"]); 
$to = "[email protected]"; 
$subject = ($_POST["type"]); 
$body = ($_POST["mailtext"]); 

  if (isset $_POST["type"]) && (isset $_POST["mailtext"]) && (isset $_POST["mailtext"]!="") && (isset $_POST["sender"]) && (isset $_POST["sender"]!="")
       {
       if(mail($to,$subject,$body,$from)) { 
            echo "Gefeliciteerd! Je mail is succesvol verzonden!";
            }
       else {
            echo "Er is iets foutgelopen terwijl je de mail aan het zenden was!";
            }
       }
  else {
            echo "Gelieve alle velden in te vullen a.u.b.!";
       }
?>
</body>
</html>

ik vind de fout niet !!!

Zero Grav

Legacy Member
if (isset($_POST["type"]) && isset($_POST["mailtext"]) && isset($_POST["mailtext"]!="") && isset($_POST["sender"]) && isset($_POST["sender"]!=""))

nuja gebruik liever

if(!empty($_POST['type']) && !empty($_POST['mailtext']) && !empty($_POST['sender']))

en controleer die variabelen liefst ook in't begin zoals bv:
$type = strip_tags($_POST['type']);

ik lees trouwens net wat je erbij zet, ge wilt dus een rpg in php bouwen terwijl ge de juiste toepassing van isset ni kent? :wtf: en die controle in't geheel is al ni echt magnifique, normaal doe ik die if() lus wa ik hier heb neergepoot ni en controleer ik elk deel individueel, which is somewhat better imho. Dan kunt ge foutmeldingen geven naargelang de fout ipv te moeten zeggen 'ja 't is ni juust he ventje'

Flipkikker

Legacy Member
kben nog maar van 3 weken geleden begonnen met (voor het eerst) html & css te leren!

nu benek volop bezig met php & mysql te leren

killgore

Legacy Member
tip: stop uw activiteiten :).
Als je een ietwat deftige rpg wilt moet je een redelijk goede engine hebben en als je spel pop. zou worden (lees: er komen 100 man op :p) ga je binnen de kortste keren last hebben van hackende script-kiddies en/of time-outs wegens je slechte scripts :).

Begin wat lager ;), iedereen hier is eenvoudig als supernoob met een slecht gescripte site begonnen hoor :).
Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.
Terug
Bovenaan