Foutje in deze script

Status
Niet open voor verdere reacties.
met deze code lukt het niet.

<?php

if($submit)
{
/* Voorloopspaties en naloopspaties weghalen */
/* ========================================= */
$VoorEnAchternaam = ltrim ($VoorEnAchternaam);
$VoorEnAchternaam = rtrim ($VoorEnAchternaam);
$Bedrijf = ltrim ($Bedrijf);
$Bedrijf = rtrim ($Bedrijf);
$Telefoonnummer = ltrim ($Telefoonnummer);
$Telefoonnummer = rtrim ($Telefoonnummer);
$Email = ltrim ($Email);
$Email = rtrim ($Email);
$Keuze = ltrim ($Keuze);
$Keuze = rtrim ($Keuze);
$Bericht = ltrim ($Bericht);
$Bericht = rtrim ($Bericht);


if (($VoorEnAchternaam != "") and ($Bedrijf != "") and

($Telefoonnummer != "") and ($Email != "") and ($Keuze != "") and

($Bericht != ""))
{
/* Email kan nu worden verzonden */
/* ============================== */

$Pro = "Producten";
$Ser = "Service" ;
$Web = "Website";
if($Keuze == $Pro || $Keuze == $Ser)
{
$versturen_naar = "bart@telefoneren.net";
}
else
{
if($Keuze == $Web)
{
$versturen_naar = "samvansteenbrugge@pandora.be";
}
else
{
?>
<center>

<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Gelieve alle
velden in te vullen! </font></p>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Klik op de "vorige"-knop
van uw webbrowser.</font></p>
</center>
<?php
}
}

$onderwerp = "Contact via Telefoneren.net";
$ingevoerde_gegevens = "Voor- en achternaam:

\n".$_POST['VoorEnAchternaam']. "\n\nBedrijf: \n".$_POST['Bedrijf'].

"\n\nTelefoonnummer: \n". $_POST['Telefoonnummer']. "\n\nE-mail: \n".

$_POST['Email']. "\n\nKeuze: \n". $_POST['Keuze']."\n\nBericht: \n".

$_POST['Bericht'];


mail ($versturen_naar, $onderwerp, $ingevoerde_gegevens);
/* PHP even onderbreken met "?>", zie hieronder */
?>

<center>
<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Bedankt voor
de interesse, </font></p>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">We nemen zo spoedig
mogelijk contact op.</font></p>
</center>
<?php /* PHP weer met "<?php" activeren */
}

else
{
/* Niet alle velden zijn ingevuld! */
/* ================================*/
/* PHP even onderbreken met "?>", zie hieronder */
?>


<center>
<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Gelieve alle
velden in te vullen! </font></p>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Klik op de "vorige"-knop
van uw webbrowser.</font></p>
</center>

<?php } ?>


Wat doe ik verkeerd?
 
Geplaatst door MrBeanstra
Je sluit iig de laatste else tage niet af met:
PHP:
<?php } ?>
Oh jawel hoor :)

Wat je misschien kan doen is de code even in
PHP:
 tags zetten, dat is wat makkelijker lezen :)
 
PHP:
<?php 

if($submit) 
{ 
/* Voorloopspaties en naloopspaties weghalen */ 
/* ========================================= */ 
$VoorEnAchternaam = ltrim ($VoorEnAchternaam); 
$VoorEnAchternaam = rtrim ($VoorEnAchternaam); 
$Bedrijf = ltrim ($Bedrijf); 
$Bedrijf = rtrim ($Bedrijf); 
$Telefoonnummer = ltrim ($Telefoonnummer); 
$Telefoonnummer = rtrim ($Telefoonnummer); 
$Email = ltrim ($Email); 
$Email = rtrim ($Email); 
$Keuze = ltrim ($Keuze); 
$Keuze = rtrim ($Keuze); 
$Bericht = ltrim ($Bericht); 
$Bericht = rtrim ($Bericht); 


if (($VoorEnAchternaam != "") and ($Bedrijf != "") and 

($Telefoonnummer != "") and ($Email != "") and ($Keuze != "") and 

($Bericht != "")) 
{ 
/* Email kan nu worden verzonden */ 
/* ============================== */ 

$Pro = "Producten"; 
$Ser = "Service" ; 
$Web = "Website"; 
if($Keuze == $Pro || $Keuze == $Ser) 
{ 
$versturen_naar = "bart@telefoneren.net"; 
} 
else 
{ 
if($Keuze == $Web) 
{ 
$versturen_naar = "samvansteenbrugge@pandora.be"; 
} 
else 
{ 
?> 
<center> 

<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1> 
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Gelieve alle 
velden in te vullen! </font></p> 
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Klik op de "vorige"-knop 
van uw webbrowser.</font></p> 
</center> 
<?php 
} 
} 

$onderwerp = "Contact via Telefoneren.net"; 
$ingevoerde_gegevens = "Voor- en achternaam: 

\n".$_POST['VoorEnAchternaam']. "\n\nBedrijf: \n".$_POST['Bedrijf']. 

"\n\nTelefoonnummer: \n". $_POST['Telefoonnummer']. "\n\nE-mail: \n". 

$_POST['Email']. "\n\nKeuze: \n". $_POST['Keuze']."\n\nBericht: \n". 

$_POST['Bericht']; 


mail ($versturen_naar, $onderwerp, $ingevoerde_gegevens); 
/* PHP even onderbreken met "?>", zie hieronder */ 
?> 

<center> 
<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1> 
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Bedankt voor 
de interesse, </font></p> 
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">We nemen zo spoedig 
mogelijk contact op.</font></p> 
</center> 
<?php /* PHP weer met "<?php" activeren */ 
} 

else 
{ 
/* Niet alle velden zijn ingevuld! */ 
/* ================================*/ 
/* PHP even onderbreken met "?>", zie hieronder */ 
?> 


<center> 
<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1> 
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Gelieve alle 
velden in te vullen! </font></p> 
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Klik op de "vorige"-knop 
van uw webbrowser.</font></p> 
</center> 

<?php } ?>
 
Geplaatst door masterprut

Oh jawel hoor :)

Wat je misschien kan doen is de code even in
PHP:
 tags zetten, dat is wat makkelijker lezen :) [/B][/QUOTE]

Hij sluitte 'm dus NIET af. Tel maar es de accolades openen en de accolades sluiten in het originele script. Dan zijn er twee accolades sluiten te weinig. Dus naast de eerste die ik noemde, moet er nóg eentje bij. Dan krijg je dus dit:

[PHP]
<?php

if($submit)
{
   $VoorEnAchternaam = ltrim ($VoorEnAchternaam);

   if (($VoorEnAchternaam != "") and ($Bedrijf != "") and ($Telefoonnummer != "") and ($Email != "") and ($Keuze != "") and ($Bericht != ""))
   {
      $Pro = "Producten";
      $Ser = "Service" ;
      $Web = "Website";

      if($Keuze == $Pro || $Keuze == $Ser)
      {
         $versturen_naar = "bart@telefoneren.net";
      }
      else
      {
         if($Keuze == $Web)
         {
            $versturen_naar = "samvansteenbrugge@pandora.be";
         }
         else
         {
?>
<center>

<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Gelieve alle
velden in te vullen! </font></p>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Klik op de "vorige"-knop
van uw webbrowser.</font></p>
</center>
<?php
         }
      }

      $onderwerp = "Contact via Telefoneren.net";
      $ingevoerde_gegevens = "Voor- en achternaam: \n".$_POST['VoorEnAchternaam']. "\n\nBedrijf: \n".$_POST['Bedrijf']. "\n\nTelefoonnummer: \n". $_POST['Telefoonnummer']. "\n\nE-mail: \n".$_POST['Email']. "\n\nKeuze: \n". $_POST['Keuze']."\n\nBericht: \n".$_POST['Bericht'];

      mail ($versturen_naar, $onderwerp, $ingevoerde_gegevens);

?>

<center>
<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Bedankt voor
de interesse, </font></p>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">We nemen zo spoedig
mogelijk contact op.</font></p>
</center>
<?php /* PHP weer met "<?php" activeren */
   }

   else
   {
?>

<center>
<h1><u><font size="5" face="Verdana, Arial, Helvetica, sans-serif">Contact</font></u></h1>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Gelieve alle
velden in te vullen! </font></p>
<p><font size="4" face="Verdana, Arial, Helvetica, sans-serif">Klik op de "vorige"-knop
van uw webbrowser.</font></p>
</center>

<?php
   }
}
?>

p.s. Het is ook handig om bij elke accolade openen de tekst in te laten springen en bij de accolade sluiten de inspringing weer op te heffen. Hierdoor is het makkelijker leesbaar en is in 1 oogopslag te zien of je alles wel hebt afgesloten.
 
Laatst bewerkt:
Geplaatst door Hala Madrid
Hallo,

Kan iemand eens kijken voor mij naar deze pagina en er de fout uithalen. Stuur me dan de gewijzigde php-pagina door.

Bedankt voor de hulp

Grs Sam

http://www.goedkooptelefoneren.be/contacteer_ons.html

dit is de pagina

en de php zit in de bijlage
Ik denk dat dit niet gaat werken ;)
Zeg gewoon wat het probleem is, en we moeten er uit kunnen komen hoe dat probleem opgelost kan worden.

Het heeft echt geen zin om een pagina neer te kwakken die niet werkt zonder verdere informatie ;)
 
Status
Niet open voor verdere reacties.
Terug
Bovenaan Onderaan