checkbox toevoegen aan contactformulier - Softwareontwikkeling

woensdag 2 mei 2007 16:19

Acties:

Verwijderd

Topicstarter

heej allemaal ik heb een goed contactformulier alleen wil ik er nog een checkbox aan toevoegen voor het akkord gaan met de voorwaarden als dat checkboxje niet is aangevingt kan de email ook niet worden verstuurd wie weet hoe dat moet hier het script :

code:

 <? 
$mailto = "testemeail@showline.nl"; //jouw email adres 

if($_POST){ 
if(empty($_POST['naam'])){ 
    echo "Je hebt geen <b>naam</b> ingevuld!!!";      
} 
elseif(empty($_POST['bericht'])){ 
    echo "Je hebt geen <b>bericht</b> ingevuld!!!"; 
} 

elseif(empty($_POST['mail'])){ 
    echo "Je hebt geen <b>emailadres</b> ingevuld!!!"; 
} 
else{ 
if($_POST['onderwerp'] == 'bug'){ 
    $onderwerp = "Bug"; 
} 
elseif($_POST['onderwerp'] == 'idee'){ 
    $onderwerp = "Idee"; 
} 
elseif($_POST['onderwerp'] == 'vraag'){ 
    $onderwerp = "Vraag"; 
} 
else{ 
    $onderwerp = "Overige"; 
} 
$header = "From: ".$_POST['naam']." < ".$_POST['mail']." >"; 
mail($mailto, $onderwerp, $_POST['bericht'], $header); 
echo "Je mail is <b>succesvol</b> verzonden, ".$_POST['naam']; 
} 
} 
else{ 
echo "<form method='POST' action='".$_SERVER['REQUEST_URI']."'> 
  Naam:<br> 
  <input type='text' name='naam' size='20'><br> 
  <br> 
  Email-adres:<br> 
  <input type='text' name='mail' size='20'><br> 
  <br> 
  Onderwerp:<br> 
  <select size='1' name='onderwerp'> 
    <option value='optie 1' selected>optie 1</option> 
    <option value='optie 2'>optie 2</option> 
    <option value='optie 3'>optie 3</option> 
    <option value='optie 4'>optie 4</option> 
  </select><br> 
  <br> 
  Bericht:<br> 
  <textarea rows='11' name='bericht' cols='45'></textarea><br> 
  <br> 
  <input type='submit' value='Mail!' name='B1'> 
</form> "; 
} 
?>

woensdag 2 mei 2007 16:22

Acties:

RedHat

<input type="checkbox" name="box">

if $_POST["box"] != 1 die("foutiefje");

meer is het niet...

Ik moet ook héél eerlijk toegeven dat ik nog nooit zo'n dubbelzinnig script heb gezien

[ Voor 31% gewijzigd door RedHat op 02-05-2007 16:23 ]

woensdag 2 mei 2007 16:24

Acties:

Verwijderd

Topicstarter

hoezo dubbelzinnig script ?
kan je aub de checkbox in de script zetten zodat ik het script over kan nemen
ben niet zo goed in scripten

woensdag 2 mei 2007 16:26

Acties:

P.O. Box

PHP:

if($_POST['onderwerp'] == 'bug'){ 
    $onderwerp = "Bug"; 
} 
elseif($_POST['onderwerp'] == 'idee'){ 
    $onderwerp = "Idee"; 
} 
elseif($_POST['onderwerp'] == 'vraag'){ 
    $onderwerp = "Vraag"; 
} 
else{ 
    $onderwerp = "Overige"; 
}

vind je dit nu heel nuttig als de waarden in $onderwerp uit een dropdownlist komen?

woensdag 2 mei 2007 16:27

Acties:

P.O. Box

Verwijderd schreef op woensdag 02 mei 2007 @ 16:24:
hoezo dubbelzinnig script ?
kan je aub de checkbox in de script zetten zodat ik het script over kan nemen
ben niet zo goed in scripten

zowww....

daar hoef je niet goed voor te zijn... ga maar even lekker logisch denken

woensdag 2 mei 2007 16:29

Acties:

Verwijderd

Topicstarter

ja ik dacht al dat ej eht daarover had dat moet ik nog veranderen maar waar moet ik dat :

code:

<? 
$mailto = "testemeail@showline.nl"; //jouw email adres 

if($_POST){ 
if(empty($_POST['naam'])){ 
    echo "Je hebt geen <b>naam</b> ingevuld!!!";      
} 
elseif(empty($_POST['bericht'])){ 
    echo "Je hebt geen <b>bericht</b> ingevuld!!!"; 
} 

elseif(empty($_POST['mail'])){ 
    echo "Je hebt geen <b>emailadres</b> ingevuld!!!"; 
} 
else{ 
if($_POST['onderwerp'] == 'bug'){ 
    $onderwerp = "Bug"; 
} 
elseif($_POST['onderwerp'] == 'idee'){ 
    $onderwerp = "Idee"; 
} 
elseif($_POST['onderwerp'] == 'vraag'){ 
    $onderwerp = "Vraag"; 
} 
else{ 
    $onderwerp = "Overige"; 
} 
$header = "From: ".$_POST['naam']." < ".$_POST['mail']." >"; 
mail($mailto, $onderwerp, $_POST['bericht'], $header); 
echo "Je mail is <b>succesvol</b> verzonden, ".$_POST['naam']; 
} 
} 
else{ 
echo "<form method='POST' action='".$_SERVER['REQUEST_URI']."'> 
  Naam:<br> 
  <input type='text' name='naam' size='20'><br> 
  <br> 
  Email-adres:<br> 
  <input type='text' name='mail' size='20'><br> 
  <br> 
  Onderwerp:<br> 
  <select size='1' name='onderwerp'> 
    <option value='optie 1' selected>optie 1</option> 
    <option value='optie 2'>optie 2</option> 
    <option value='optie 3'>optie 3</option> 
    <option value='optie 4'>optie 4</option> 
  </select><br> 
  <br> 
  Bericht:<br> 
  <textarea rows='11' name='bericht' cols='45'></textarea><br> 
  <br>
<input type="checkbox" name="box">

if $_POST["box"] != 1 die("foutiefje");
 
  <input type='submit' value='Mail!' name='B1'> 
</form> "; 
} 
?>

neerzetten wil je alsjeblieft het even in het script verwerken ?

woensdag 2 mei 2007 16:29

Acties:

Verwijderd

Topicstarter

ik heb het nu helemal onderaan maar nu werkt het neit meer ?

woensdag 2 mei 2007 16:30

Acties:

OkkE

CSS influencer :+

Even een aantal opmerkingen..

1. Als je de code tussen [ php] en [ /php] zet, dan is het nog duidelijker aangezien er dan kleur gebrukt wordt.

2. Een beetje eigen moeite mag wel. Dat je niet goed bent in scripten is natuurlijk niet erg; niemand weet alles. Maar met het antwoord van RedHat en de huidige code is het toch met een beetje eiden inzet wel te maken.

3. Dubbelzinnig script als in: Er worden nog al wat checks gedaan die best op een andere manier gedaan kunnen, zodat het minder code in neemt. Verder is het op deze manier niet echt een veilig script; de $_POST['naam'] wordt bijv. gewoon in de $header gezet zonder verdere checks..

“The best way to get the right answer on the Internet is not to ask a question, it's to post the wrong answer.”
QA Engineer walks into a bar. Orders a beer. Orders 0 beers. Orders 999999999 beers. Orders a lizard. Orders -1 beers.

woensdag 2 mei 2007 16:31

Acties:

P.O. Box

logisch denken!

kijk naar de rest van de code... denk logisch na... en dan kom je er wel uit....

woensdag 2 mei 2007 16:33

Acties:

Verwijderd

Topicstarter

PHP:

<? 
$mailto = "testemeail@showline.nl"; //jouw email adres 

if($_POST){ 
if(empty($_POST['naam'])){ 
    echo "Je hebt geen <b>naam</b> ingevuld!!!";      
} 
elseif(empty($_POST['bericht'])){ 
    echo "Je hebt geen <b>bericht</b> ingevuld!!!"; 
} 

elseif(empty($_POST['mail'])){ 
    echo "Je hebt geen <b>emailadres</b> ingevuld!!!"; 
} 
else{ 
if($_POST['onderwerp'] == 'bug'){ 
    $onderwerp = "Bug"; 
} 
elseif($_POST['onderwerp'] == 'idee'){ 
    $onderwerp = "Idee"; 
} 
elseif($_POST['onderwerp'] == 'vraag'){ 
    $onderwerp = "Vraag"; 
} 
else{ 
    $onderwerp = "Overige"; 
} 
$header = "From: ".$_POST['naam']." < ".$_POST['mail']." >"; 
mail($mailto, $onderwerp, $_POST['bericht'], $header); 
echo "Je mail is <b>succesvol</b> verzonden, ".$_POST['naam']; 
} 
} 
else{ 
echo "<form method='POST' action='".$_SERVER['REQUEST_URI']."'> 
  Naam:<br> 
  <input type='text' name='naam' size='20'><br> 
  <br> 
  Email-adres:<br> 
  <input type='text' name='mail' size='20'><br> 
  <br> 
  Onderwerp:<br> 
  <select size='1' name='onderwerp'> 
    <option value='optie 1' selected>optie 1</option> 
    <option value='optie 2'>optie 2</option> 
    <option value='optie 3'>optie 3</option> 
    <option value='optie 4'>optie 4</option> 
  </select><br> 
  <br> 
  Bericht:<br> 
  <textarea rows='11' name='bericht' cols='45'></textarea><br> 
  <br>
<input type="checkbox" name="box">

if $_POST["box"] != 1 die("foutiefje");
 
  <input type='submit' value='Mail!' name='B1'> 
</form> "; 
} 
?>

zo heb ik het meer kan ik er niet van maken maar het is in iederegeval zo niet goed

woensdag 2 mei 2007 16:35

Acties:

Verwijderd

Topicstarter

kan iemand me uitleggen dan waar ik het moet plaatsen dan weet ik eht voor de volgende keer ook

woensdag 2 mei 2007 16:39

Acties:

P.O. Box

kijk nou eens goed naar wat php-code is, en wat html-code is...
en kijk waar je controles van andere velden plaatsvinden en waar de controle op de checkbox nu plaatsvindt...

[ Voor 46% gewijzigd door P.O. Box op 02-05-2007 16:40 ]

woensdag 2 mei 2007 16:39

Acties:

Verwijderd

Topicstarter

leg het me gewoon uit ik kom er echt niet uit h0or sorry

woensdag 2 mei 2007 16:40

Acties:

P.O. Box

dat is uitleggen... maar je antwoord binnen een minuut, dus heb niet de indruk dat je echt bereid bent te kijken...

woensdag 2 mei 2007 16:43

Acties:

KatirZan

Wandelende orgaanzak

Dit formulier is een standaard formulier wat je vaak bij webserver leveranciers krijgt.....

En het is inderdaad een erg omslachtig formulier...

1 tip : als je geen verstand hebt van PHP, ga dan eerst PHP leren alvorens een simpele vraag te stellen die je via google ook had kunnen vinden...

my 24 roepies....

Wabbawabbawabbawabba

woensdag 2 mei 2007 16:45

Acties:

Verwijderd

Topicstarter

ik had al op google gezocht kan iemand nou gewoon het script zeggen dan kan ik verder

woensdag 2 mei 2007 16:48

Acties:

Swaptor

Java Apprentice

Ik denk dat je met zo'n instelling nergens komt op dit forum.

Je hebt de oplossing al op een gouden presenteerblaadje gekregen en het enige wat er van jou verwacht wordt is dat je het toepast. Voorkauwen is hier op GoT niet gewenst, eigen inzet wordt wél gewaardeerd.

Ontdek mij!
Proud NGS member
Stats-mod & forum-dude

woensdag 2 mei 2007 16:48

Acties:

P.O. Box

Verwijderd schreef op woensdag 02 mei 2007 @ 16:45:
ik had al op google gezocht kan iemand nou gewoon het script zeggen dan kan ik verder

van haast leer je weinig...

tot zover mijn bijdrage in dit topic

woensdag 2 mei 2007 16:52

Acties:

Verwijderd

Topicstarter

ja bedankt dat vroeg ik ook niet ik weet neit hoe ik het moet toevoegen dus vraag ik of iemand kan zeggen hoe het moet

woensdag 2 mei 2007 16:55

Acties:

KatirZan

Wandelende orgaanzak

Verwijderd schreef op woensdag 02 mei 2007 @ 16:52:
ja bedankt dat vroeg ik ook niet ik weet neit hoe ik het moet toevoegen dus vraag ik of iemand kan zeggen hoe het moet

Je voegt gewoon de code van hierboven toe onder de lijn waar je deze checkbox het liefste zou willen hebben

Geeft daar een omschrijving aan en voilá, je hebt een checkbox in je php formuliertje.

En zonder zelfstudie kom je inderdaad helemaal nergens, dus klinklare brokken op je bord krijgen schiet je niets mee op, je moet namelijk wel zelf leren kauwen

Wabbawabbawabbawabba

woensdag 2 mei 2007 17:02

Acties:

martijnve

Edwardvb schreef op woensdag 02 mei 2007 @ 16:26:

PHP:

if($_POST['onderwerp'] == 'bug'){ 
    $onderwerp = "Bug"; 
} 
elseif($_POST['onderwerp'] == 'idee'){ 
    $onderwerp = "Idee"; 
} 
elseif($_POST['onderwerp'] == 'vraag'){ 
    $onderwerp = "Vraag"; 
} 
else{ 
    $onderwerp = "Overige"; 
}

vind je dit nu heel nuttig als de waarden in $onderwerp uit een dropdownlist komen?

Er zijn zat truuks om voor een vraag met drop-down een antwoord te sturen wat er niet in voorkomt!
Simpelste is: formulier opslaan, dropdownbox in de .html vervangen voor een textbox, pagina openen en foute data sturen

Neemt natuurlijk niet weg dat het veel simpeler kan:

PHP:

1 2	if (in_array($_POST['onderwerp'], array('bug', 'idee', 'vraag'))) $onderwerp = uc_first($_POST['onderwerp']); else $onderwerp = "Overige";

Mini-ITX GamePC: Core i5 3470 | 16GB DDR3 | GTX 970 4GB | Samsung 830 128GB | Dell u2711 (27", IPS,1440p), 2343BW

woensdag 2 mei 2007 17:11

Acties:

OkkE

CSS influencer :+

martijnve schreef op woensdag 02 mei 2007 @ 17:02:
[...]

Er zijn zat truuks om voor een vraag met drop-down een antwoord te sturen wat er niet in voorkomt!
Simpelste is: formulier opslaan, dropdownbox in de .html vervangen voor een textbox, pagina openen en foute data sturen

Neemt natuurlijk niet weg dat het veel simpeler kan:
PHP:
1
2
if (in_array($_POST['onderwerp'], array('bug', 'idee', 'vraag'))) $onderwerp = uc_first($_POST['onderwerp']);
else $onderwerp = "Overige";

Dit is inderdaad een goede manier, maar zolang het formulier de waardes 'optie 1', 'optie 2', 'optie 3' of 'optie 4' verstuurd als onderwerp, zal dit mooie stukje code alsnog niets doen.

Maar goed, zolang er alleen om "de oplossing" gevraagd wordt, zonder dat er enige energie gestoken wordt in het zelf uitzoeken, heb ik echt geen zin om verder te helpen.

^{Oke, vooruit... [snip]}

[ Voor 7% gewijzigd door OkkE op 02-05-2007 17:17 . Reden: Waardes aangepast ]

woensdag 2 mei 2007 17:13

Acties:

Verwijderd

Maar goed, zolang er alleen om "de oplossing" gevraagd wordt, zonder dat er enige energie gestoken wordt in het zelf uitzoeken, heb ik echt geen zin om verder te helpen.

Helemaal mee eens. Want als we dat gaan doen, dan is het natuurlijk wachten op z'n volgende vergelijkbare php probleempje wat ie bij ons neerlegd omdat ie er niks van geleerd heeft.

woensdag 2 mei 2007 17:14

Acties:

Verwijderd

Topicstarter

DAT OVER DIE ONDERWERP EN OPTIES VRAAG IK NIET EENS

woensdag 2 mei 2007 17:17

Acties:

OkkE

CSS influencer :+

Verwijderd schreef op woensdag 02 mei 2007 @ 17:14:
DAT OVER DIE ONDERWERP EN OPTIES VRAAG IK NIET EENS

Sorry hoor, maar je smijt hier een lap code neer zonder dat je weet wat het doet. Je vraagt om een oplossing, en als je die krijgt maar niet zonder enige vorm van inspanning kunt gebruiken, wil je dat we alles voor je doen. Als we dan in plaats daar van tips geven over het verbeteren van je code, aangezien duidelijk is dat je die tips wel kunt gebruiken, wordt er in CAPS verteld dat dat niet nodig is?

Zoek 't uit! /ignore.

woensdag 2 mei 2007 17:23

Acties:

_Teq_

[Sarcasme]
Post even het adres en het wachtwoord van je webserver dan zorgen wij wel dat er een website op komt te draaien. Gratis, inclusief checkbox en inclusief onderhoud! Je hoeft alleen maar even te zeggen wat je wil, en hoe het eruit moet zien....
Als jij dan iets aan wil passen, dag of nacht, moet je maar gewoon even bellen en dan fixen wij het meteen!
[/Sarcasme]

woensdag 2 mei 2007 17:32

Acties:

AW_Bos

Liefhebber van nostalgie... 🕰️

Verwijderd schreef op woensdag 02 mei 2007 @ 16:19:
heej allemaal ik heb een goed contactformulier alleen wil ik er nog een checkbox aan toevoegen voor het akkord gaan met de voorwaarden als dat checkboxje niet is aangevingt kan de email ook niet worden verstuurd wie weet hoe dat moet hier het script :

code:

 <? 
$mailto = "testemeail@showline.nl"; //jouw email adres 

if($_POST){ 
if(empty($_POST['naam'])){ 
    echo "Je hebt geen <b>naam</b> ingevuld!!!";      
} 
elseif(empty($_POST['bericht'])){ 
    echo "Je hebt geen <b>bericht</b> ingevuld!!!"; 
} 

elseif(empty($_POST['mail'])){ 
    echo "Je hebt geen <b>emailadres</b> ingevuld!!!"; 
} 
else{ 
if($_POST['onderwerp'] == 'bug'){ 
    $onderwerp = "Bug"; 
} 
elseif($_POST['onderwerp'] == 'idee'){ 
    $onderwerp = "Idee"; 
} 
elseif($_POST['onderwerp'] == 'vraag'){ 
    $onderwerp = "Vraag"; 
} 
else{ 
    $onderwerp = "Overige"; 
} 
$header = "From: ".$_POST['naam']." < ".$_POST['mail']." >"; 
mail($mailto, $onderwerp, $_POST['bericht'], $header); 
echo "Je mail is <b>succesvol</b> verzonden, ".$_POST['naam']; 
} 
} 
else{ 
echo "<form method='POST' action='".$_SERVER['REQUEST_URI']."'> 
  Naam:<br> 
  <input type='text' name='naam' size='20'><br> 
  <br> 
  Email-adres:<br> 
  <input type='text' name='mail' size='20'><br> 
  <br> 
  Onderwerp:<br> 
  <select size='1' name='onderwerp'> 
    <option value='optie 1' selected>optie 1</option> 
    <option value='optie 2'>optie 2</option> 
    <option value='optie 3'>optie 3</option> 
    <option value='optie 4'>optie 4</option> 
  </select><br> 
  <br> 
  Bericht:<br> 
  <textarea rows='11' name='bericht' cols='45'></textarea><br> 
  <br> 
  <input type='submit' value='Mail!' name='B1'> 
</form> "; 
} 
?>

Als ik dit script zo zie, zou het ook wat makkelijker zijn om in te springen met je accolades. Zo houd je veel overzicht, en zou het makkelijker zijn om de code van die gegeven in te voegen en in te passen.
Enveder zou ik zowiezo dat if($_POST){
aanpassen in:

code:

1	if(isset($_POST['submit']) && ($_SERVER['REQUEST_METHOD'] == "POST")){

[ Voor 3% gewijzigd door AW_Bos op 02-05-2007 17:34 ]

☎ Telecommunicatie van vroeger
🚅Alles over spoor en treintjes

woensdag 2 mei 2007 18:51

Acties:

Cadezo

is het al gelukt...

leuk topic is dit overigens

[ Voor 7% gewijzigd door Cadezo op 02-05-2007 18:51 ]

woensdag 2 mei 2007 18:57

Acties:

Verwijderd

Ik heb er alle begrip voor dat je sommige dingen niet weet, maar met de manier waarop je reageert zullen mensen ook niet veel begrip voor je hebben en zin om je te helpen... Ikzelf heb ook weinig verstand van PHP maar ik kon tenminste nog een normale topic maken en hier normaal in reageren

[Formulier] Goedkope form hoster

Daarbij moet ik even melden dat cdezoete erg behulpzaam was, maar ik weet zeker dat andere mensen mij ook wel hadden kunnen helpen ook al begon ik op dat topic met 0.. Jij bent al iets verder en juist daardoor zou je een goede topic kunnen maken en een beetje 'verstand' kunnen tonen...

woensdag 2 mei 2007 19:40

Acties:

Verwijderd

Haal dit script alsjeblieft van je server af. We krijgen al genoeg spam. Ga alsjeblieft niet met scripts op een server aan de gang als je niet de moeite wilt nemen ervoor te zorgen dat het script écht goed werkt. En dat het dus niet gebruikt kan worden door louche figuren die je formulier volplempen met mailheaders.

woensdag 2 mei 2007 19:45

Acties:

Verwijderd

Verwijderd schreef op woensdag 02 mei 2007 @ 19:40:
Haal dit script alsjeblieft van je server af. We krijgen al genoeg spam. Ga alsjeblieft niet met scripts op een server aan de gang als je niet de moeite wilt nemen ervoor te zorgen dat het script écht goed werkt. En dat het dus niet gebruikt kan worden door louche figuren die je formulier volplempen met mailheaders.

Heb je het nu tegen TS of tegen mij? aangezien TS nog geen server heeft gepost?
En zou je misschien kunnen uitleggen wat je precies bedoelt?

woensdag 2 mei 2007 19:51

Acties:

Verwijderd

De code van de topicstarter:

PHP:

1	$header = "From: ".$_POST['naam']." < ".$_POST['mail']." >";

En wat als er iemand als volgt zijn naam invult?

code:

Pietje <pietje@example.com>
To: "Zomaar iemand" <zomaariemand@gmail.com>
Subject: Wil jij misschien ook een beetje spam?
Sender: Pietje

woensdag 2 mei 2007 20:01

Acties:

Voutloos

Het script achter het contactformulier is gewoon de verzender, dus from en to kan 100% statische code zijn. In de body van de mail kunnen vervolgens op een veilige manier de veldjes opgenomen worden.

De gebruiker is gewoon niet in een mail client bezig geweest en heeft zelf geen mail verstuurd, dus waarom hele volkstammen dit soort trucs moeten uithalen is me een raadsel.

{signature}

woensdag 2 mei 2007 20:14

Acties: