PHP formuliertje - Softwareontwikkeling

woensdag 4 maart 2009 20:38

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

Ik ben niet zo heel erg bekend met PHP en dat blijkt ook wel: http://www.pds-galaxis.nl/contact.php

Dat formulier heb ik gevonden op internet en werkend gekregen... Maar zoals je ziet is het niet netjes uitgelijnd en zit er iets fout bovenin. Kan iemand me helpen?

Heel erg bedankt alvast!

Bob

woensdag 4 maart 2009 20:40

Acties:

0 Henk 'm!

Tommasso

je hebt iig al een session_start error, dit is omdat de headers al verzonden zet.

plaat de session_start() bovenin je script (voordat er een echo is), dan heb je dit niet meer.

verder zal ik een <dl> of een <table> oid gebruiken voor de uitlijning

woensdag 4 maart 2009 20:43

Acties:

0 Henk 'm!

MuisM4t

Pas op met gevonden scriptjes op internet! Er zitten hele goede tussen, maar ook verouderde en onveilige.
Je zou de code hier eens kunnen posten.
Voorbeeld

PHP:

1	mail($_REQUEST['ontvanger'],$_REQUEST['onderwerp'], $_REQUEST['bericht'], $headers);

[ Voor 13% gewijzigd door MuisM4t op 04-03-2009 20:45 ]

woensdag 4 maart 2009 20:50

Acties:

0 Henk 'm!

Tommasso

volgens php staat het op line 28, dat lijkt me niet boven aan. mss heb je hem 2x, dan krijg je ook val zulke errors

die code doe je met (code=php) (en dan die ( door [ vervangen

)

edit:
als er HTML boven zit, heb je hetzeflde probleem. session_start moet helemaal bovenin je pagina voordat er iets aan uitvoer is. ook als dit buiten de php blokken valt

[ Voor 32% gewijzigd door Tommasso op 04-03-2009 20:56 ]

woensdag 4 maart 2009 20:54

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

Er zit ook nog wat HTML boven voor m'n header.

woensdag 4 maart 2009 20:55

Acties:

0 Henk 'm!

jurp5

Verwijderd schreef op woensdag 04 maart 2009 @ 20:54:
Er zit ook nog wat HTML boven voor m'n header.

daar moet ie dus boven

woensdag 4 maart 2009 20:56

Acties:

0 Henk 'm!

Cadezo

Verdiep je eerst eens in de materie, voordat je aan iets dergelijks begint...

Er mag helemaal niets boven de header staan, geen HTML, geen CSS, helemaal niks.

Die uitlijning van je table, moet je trouwens regelen met HTML of CSS, dat kan niet met PHP.

offtopic:
Lees daarnaast ook eens Het algemeen beleid en dan vooral:
Forum FAQ #quickstart, Forum FAQ #groeten en Forum FAQ #verbodenanders

Daarnaast zijn scriptrequest niet toegestaan, zie: PRG Beleid

Isoleer het probleem, die bulk HTML heeft er niets mee te maken en kan je net zo goed weglaten.

[ Voor 26% gewijzigd door Cadezo op 04-03-2009 21:10 ]

woensdag 4 maart 2009 20:57

Acties:

0 Henk 'm!

Verwijderd

Overigens heeft de uitlijning van je formulier helemaal nietsss met PHP te maken

woensdag 4 maart 2009 20:58

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

jurp5 schreef op woensdag 04 maart 2009 @ 20:55:
[...]

daar moet ie dus boven

Als ik hem daar boven zet krijg ik een lege pagina..... Weet ook niet of de combi tussen HTML en PHP kwaad kan.

Volledige pagina:

PHP:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Party Disco Show GALAXIS | Contact |</title>
<link href="opmaak.css" rel="stylesheet" type="text/css" />
</head>

<body>


<div id="container">

<div id="header1">
  <object classid="clsid:D27CDB6E-AE6D-11cf-96B8-444553540000" codebase="http://download.macromedia.com/pub/shockwave/cabs/flash/swflash.cab#version=7,0,19,0" width="698" height="227">
    <param name="movie" value="images/menu pds.swf" />
    <param name="quality" value="high" />
    <embed src="images/menu pds.swf" quality="high" pluginspage="http://www.macromedia.com/go/getflashplayer" type="application/x-shockwave-flash" width="698" height="227"></embed>
  </object>
</div>
<div id="content">
  <table width="550" align="center">
    <tr bgcolor="#191919">
      <td><div id="div"><p>
        <?php
ini_set('display_errors', 1);
error_reporting(E_ALL);

ob_start();
session_start();
// Config Gedeelte
$cfg['url'] = "http://www.pds-galaxis.nl.nl";// Site waarnaar je terug gaat als je een bericht hebt achtergelaten
$cfg['naam'] = "Bob Laarmans";                // Webmaster naam
$cfg['email'] = "bob@pds-galaxis.nl";        // Webmaster E-mail
$cfg['spam'] = 60;                        // Anti Spam Tijd in Minuten ( Voer "0" om de Spam Beveiliging uit te zetten )
$cfg['text'] = TRUE;                    // Bij Fout Text Rood maken ( TRUE voor aan, FALSE voor uit )
$cfg['input'] = TRUE;                    // Bij Fout Border om Vakje Rood maken ( TRUE voor aan, FALSE voor uit )
$cfg['HTML'] = TRUE;                    // Een HTML email ( TRUE voor aan, FALSE voor uit )
$cfg['CAPTCHA'] = TRUE;                    // CAPTCHA ( TRUE voor aan, FALSE voor uit )


// Hieronder niks meer veranderen
// E-mail Checker / Validator
function checkmail($email)
{
    if(eregi("^[0-9a-z]([-_.]?[0-9a-z])*@[0-9a-z]([-.]?[0-9a-z])*\\.[a-z]{2,4}$", $email))
    {
        return TRUE;
    }
    return FALSE;
}

$formulier = TRUE;

if(!isset($_COOKIE['formulier']))
{
    if(isset($_POST['wis']) && ($_SERVER['REQUEST_METHOD'] == "POST"))
    {
        foreach($_POST as $key => $value)
        {
            unset($value);
        }
        header("Location: ".$_SERVER['PHP_SELF']."");
    }

    if(isset($_POST['verzenden']) && ($_SERVER['REQUEST_METHOD'] == "POST"))
    {
        $aFout = array();

        $naam = trim($_POST['naam']);
        $email = trim($_POST['email']);
        $onderwerp = trim($_POST['onderwerp']);
        $bericht = trim($_POST['bericht']);

        if($cfg['CAPTCHA'])
        {
            $code = $_POST['code'];
        }

        if(empty($naam) || (strlen($naam) < 3) || eregi("[<>]", $naam) )
        {
            $aFout[] = "Er is geen naam ingevuld.";
            unset($naam);
            $fout['text']['naam'] = TRUE;
            $fout['input']['naam'] = TRUE;
        }
        if(empty($email))
        {
            $aFout[] = "Er is geen e-mail adres ingevuld.";
            unset($email);
            $fout['text']['email'] = TRUE;
            $fout['input']['email'] = TRUE;
        }
        elseif(checkmail($email) == 0)
        // Wanneer je PHP 5.2 > gebruikt
        //elseif(!filter_var($email, FILTER_VALIDATE_EMAIL))
        {
            $aFout[] = "Er is geen correct e-mail adres ingevuld.";
            unset($email);
            $fout['text']['email'] = TRUE;
            $fout['input']['email'] = TRUE;
        }
        if(empty($onderwerp))
        {
            $aFout[] = "Er is geen onderwerp ingevuld.";
            unset($onderwerp);
            $fout['text']['onderwerp'] = TRUE;
            $fout['input']['onderwerp'] = TRUE;
        }
        if(empty($bericht))
        {
            $aFout[] = "Er is geen bericht ingevuld.";
            unset($bericht);
            $fout['text']['bericht'] = TRUE;
            $fout['input']['bericht'] = TRUE;
        }
        if($cfg['CAPTCHA'])
        {
            if(strtoupper($code) != $_SESSION['captcha_code'])
            {
                $aFout[] = "Er is geen correcte code ingevuld.";
                $fout['text']['code'] = TRUE;
                $fout['input']['code'] = TRUE;
            }
        }
        if(!$cfg['text'])
        {
            unset($fout['text']);
        }
        if(!$cfg['input'])
        {
            unset($fout['input']);
        }
        if(!empty( $aFout ))
        {
            $errors = '
            <div id="errors">
            <ul>';
            foreach($aFout as $sFout)
            {
                $errors .= "    <li>".$sFout."</li>\n";
            }
            $errors .= "</ul>
            </div>";
        }
        else
        {
            $formulier = FALSE;


            if($cfg['HTML'])
            {
                // Headers
                $headers = "From: \"Contact Formulier\" <".$cfg['email'].">\r\n";
                $headers .= "Reply-To: \"".$naam."\" <".$email.">\n";
                $headers .= "Return-Path: Mail-Error <".$cfg['email'].">\n";
                $headers .= "MIME-Version: 1.0\n";
                $headers .= "Content-Transfer-Encoding: 8bit\n";
                $headers .= "Content-type: text/html; charset=iso-8859-1\n";


                $bericht = '
                <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
                <html>
                <head>
                </head>

                <body>
                <br />
                <b>Naam:</b> '.$naam.'<br />
                <b>Email:</b> <a href=\"mailto:'.$email.'\">'.$email.'</a><br />
                <br />
                <b>Bericht:</b><br />
                '.$bericht.'
                <br />
                <br />
                <br />
                --------------------------------------------------------------------------<br />
                <b>Datum:</b> '.date("d-m-Y @ H:i:s").'<br />
                <b>IP:</b> <a href=\"http://sunny.nic.com/cgi-bin/whois?domain='.$_SERVER['REMOTE_ADDR'].'\">'.$_SERVER['REMOTE_ADDR'].'</a><br />
                <b>Host:</b> '.gethostbyaddr($_SERVER['REMOTE_ADDR']).'<br />
                </body>
                </html>';
            }
            else
            {
                $bericht_wrap = wordwrap ($bericht, 40, "\n", 1);
                // Headers
                $headers = "From: \"Contact Formulier\" <".$cfg['email'].">\n";
                $headers .= "MIME-Version: 1.0\n";
                $headers .= "Content-type: text/plain; charset='iso-8859-1'\n";

                // Bericht
                $message = "Naam: ".$naam."        \n";
                $message .= "E-mail: ".$email."     \n";
                $message .= "Bericht:\n".$bericht_wrap."     \n ";
                $message .= "               \n ";
                $message .= "Datum: ".date("d-m-Y H:i:s")." \n";
                $message .= "------------------------------------------------------- \n ";
                $message .= "IP: ".$_SERVER['REMOTE_ADDR']."                    \n ";
                $message .= "Host: ".gethostbyaddr($_SERVER['REMOTE_ADDR'])."                \n ";

            }

            if(mail($cfg['email'], "[Contact] ".$onderwerp, $bericht, $headers))
            {
                if(isset($_POST['stuurkopie']))
                {
                    $headers = "From: \"Contact Formulier\" <".$email.">\r\n";
                    $headers .= "Reply-To: \"".$naam."\" <".$email.">\n";
                    $headers .= "Return-Path: Mail-Error <".$email.">\n";
                    $headers .= "MIME-Version: 1.0\n";
                    $headers .= "Content-Transfer-Encoding: 8bit\n";
                    $headers .= "Content-type: text/html; charset=iso-8859-1\n";

                    mail($email, "[Contact] ".$onderwerp, $bericht, $headers);

                }

                unset($naam, $email, $onderwerp, $bericht);
                setcookie("formulier", 1, time() + ( $cfg['spam'] * 60 ) );

                echo "
                <p>
                Uw bericht is succesvol verzonden, er word zo snel mogelijk gereageerd.<br />
                <br />
                Met vriendelijke groeten,<br />
                <b>".$cfg['naam']."</b>
                </p>
                ";
            }
            else
            {
                echo "Er is een fout opgetreden bij het verzenden van de email";
            }
            header("refresh:3;url=".$cfg['url']."");
        }
    }
    if($formulier)
    {
    ?>
    <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

    <html xmlns="http://www.w3.org/1999/xhtml">
    <head>
        <link href="style.css" rel="stylesheet" type="text/css" />

        <title>Contact Formulier door Thijs</title>
        <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
        <meta http-equiv="Content-Language" content="nl" />
    </head>

    <body>
    <div id="container">
    <?php
    if(isset($errors)) {
        echo $errors;
    }
    ?>

    <h3>Contact Formulier</h3>
        <form method="post" action="<?php $_SERVER['PHP_SELF']; ?>">
        <p>
        <label <?php if(isset($fout['text']['naam'])) { echo 'class="fout"'; } ?>>Naam:</label>
        <input type="text" id="naam" name="naam" maxlength="30" <?php if(isset($fout['input']['naam'])) { echo 'class="fout"'; } ?> value="<?php if (!empty($naam)) { echo stripslashes($naam); } ?>" /><br />

        <label <?php if(isset($fout['text']['email'])) { echo 'class="fout"'; } ?>>Email:</label>
        <input type="text" id="email" name="email" maxlength="255" <?php if(isset($fout['input']['email'])) { echo 'class="fout"'; } ?> value="<?php if (!empty($email)) { echo stripslashes($email); } ?>" /><br />

        <label <?php if(isset($fout['text']['onderwerp'])) { echo 'class="fout"'; } ?>>Onderwerp:</label>
        <input type="text" id="onderwerp" name="onderwerp" maxlength="40" <?php if(isset($fout['input']['onderwerp'])) { echo 'class="fout'; } ?> value="<?php if (!empty($onderwerp)) { echo stripslashes($onderwerp); } ?>" /><br />

        <label <?php if(isset($fout['text']['bericht'])) { echo 'class="fout"'; } ?>>Bericht:</label>
        <textarea id="bericht" name="bericht" <?php if(isset($fout['input']['bericht'])) { echo 'class="fout"'; } ?> cols="35" rows="6"><?php if (!empty($bericht)) { echo stripslashes($bericht); } ?></textarea><br />

        <?php
        if($cfg['CAPTCHA'])
        {
        ?>
        <label></label>
        <img src="captcha.php" alt="" /><br />

        <label <?php if(isset($fout['text']['code'])) { echo 'class="fout"'; } ?>>Code:</label>
        <input type="text" id="code" name="code" maxlength="4" size="4" <?php if(isset($fout['input']['code'])) { echo 'class="captcha fout"'; } ?> /><br />
        <?php
        }
        ?>
        <label for="stuurkopie">Stuur mij een kopie</label><input type="checkbox" id="stuurkopie" name="stuurkopie" value="1" /><br />

        <label></label>
        <input type="submit" id="verzenden" name="verzenden" value="verzenden" />
        <input type="submit" id="wis" name="wis" value="Wis velden" />
        </p>
        </form>
    </div>

</body>
</html>
    <?php
    }
}
else
{
    echo "
    <p>
    U kunt maar eens in de ".$cfg['spam']." minuten een e-mail versturen!<br />
    U wordt nu automatisch doorgestuurd.
    </p>";
    header("refresh:3;url=".$cfg['url']."");
}
?>

      </div></td>
    </tr>
  </table>
  <p><center><script type="text/javascript"><!--
google_ad_client = "pub-2335145032074541";
/* 468x60, gemaakt 26-2-09 */
google_ad_slot = "6628201296";
google_ad_width = 468;
google_ad_height = 60;
//-->
</script>
<script type="text/javascript"
src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
</script></center>
</div>

        <div id="afteller">
        <table style="width: 540px; text-align: left; margin-left: auto; margin-right: auto;">
            <tbody>
            <tr>
            <td valign="bottom">
    <br>
            </td>
            </tr>
            </tbody>
    </table>
        </div>

<div class="style1" id="footer"><br/>
© Party Disco Show GALAXIS 2009. Alle rechten voorbehouden.
  </div>

</div>


</body>
</html>

woensdag 4 maart 2009 21:04

Acties:

0 Henk 'm!

Tommasso

Verwijderd schreef op woensdag 04 maart 2009 @ 20:58:
[...]

Als ik hem daar boven zet krijg ik een lege pagina..... Weet ook niet of de combi tussen HTML en PHP kwaad kan.

een combinatie met HTML en PHP kan geen kwaad, daar is PHP ook voor bedoelt

dus als je je session_start boven zet krijg je geen enkele output meer of zie je niks in de browser? zet je dan het hele script erboven of alleen de session_startI() (wat denk ik beter is

)?

edit:
daar moet je idd een ander forum voor gebruikt (dat heet iets van markup en webdesign? Webdesign, Markup & Clientside Scripting). als je ervaring met css hebt kan je denk ik het beste een <dl> gebruiken, anders een <table>

[ Voor 18% gewijzigd door Tommasso op 04-03-2009 21:10 ]

woensdag 4 maart 2009 21:06

Acties:

0 Henk 'm!

Cadezo

Als je dit eens bovenzet en de rest gewoon laat staan.

PHP:

ob_start(); 
session_start(); 

ini_set('display_errors', 1); 
error_reporting(E_ALL);

offtopic:
Isoleer het probleem, die bulk HTML heeft er niets mee te maken en kan je net zo goed weglaten.

woensdag 4 maart 2009 21:07

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

Dankje Cadezo

Probleem van die session is opgelost. Voor de uitlijning moet ik dus ff een ander forum proberen hier op tweakers?

[ Voor 6% gewijzigd door Verwijderd op 04-03-2009 21:07 ]

woensdag 4 maart 2009 21:10

Acties:

0 Henk 'm!

Cadezo

Dat klopt dat moet sowieso in Webdesign, Markup & Clientside Scripting.

Maar lees eens eerst het kleine, grijze stukje tekst hier Cadezo in "PHP formuliertje" en bedenkt dan rustig of en hoe je daar een topic gaat openen.

Misschien is het handig eerst een goede tutorial voor HTML/CSS (en PHP) te lezen en het dan eerst zelf te proberen. Als je er dan echt niet uitkomt kan je alsnog een topic openen.

[ Voor 36% gewijzigd door Cadezo op 04-03-2009 21:12 ]

woensdag 4 maart 2009 21:11

Acties:

0 Henk 'm!

doeternietoe

Verwijderd schreef op woensdag 04 maart 2009 @ 20:44:
Zoals ik zei heb ik er erg weinig verstand van.... Is het een optie dat ik de bestanden inpak en on-line zet zodat jullie eraan kunnen bijspijkeren? Het formulier is trouwens ook breder dan de tabel.

Het lijkt me een betere optie als je een goede cursus html/css en daarna PHP gaat volgen. Het is voor nu wel makkelijk als wij het oplossen, maar jij wilt in de toekomst denk ik liever het zelf kunnen doen en zelf kunnen begrijpen wat er gebeurt.

Voor HTML hebben we hier inderdaad Webdesign, Markup & Clientside Scripting. Maar Het lijkt me dat je er goed aan doet om eerst zelf te proberen of je eruit kunt komen, want daar leer je echt van.

@Tommasso: Ik zou zeker geen table gebruiken om uit te lijnen, dat gaat tegen alle semantische regels van html in. Tables zijn voor het overzichtelijk presenteren van data bedacht, niet voor opmaak.

woensdag 4 maart 2009 21:12

Acties:

0 Henk 'm!

Verwijderd

Topicstarter

Oh die lettertjes waren zo klein dat ik ze over het hoofd zag ;-)
Maar het is duidelijk. Dankje.

woensdag 4 maart 2009 21:14

Acties:

0 Henk 'm!

Tommasso

doeternietoe schreef op woensdag 04 maart 2009 @ 21:11:
[...]

Het lijkt me een betere optie als je een goede cursus html/css en daarna PHP gaat volgen. Het is voor nu wel makkelijk als wij het oplossen, maar jij wilt in de toekomst denk ik liever het zelf kunnen doen en zelf kunnen begrijpen wat er gebeurt.

Voor HTML hebben we hier inderdaad Webdesign, Markup & Clientside Scripting. Maar Het lijkt me dat je er goed aan doet om eerst zelf te proberen of je eruit kunt komen, want daar leer je echt van.

@Tommasso: Ik zou zeker geen table gebruiken om uit te lijnen, dat gaat tegen alle semantische regels van html in. Tables zijn voor het overzichtelijk presenteren van data bedacht, niet voor opmaak.

klopt, daarom zal ik ook voor de <dl> gaan (dat doet ik altijd), maar dara heb je wat CSS gemod voor nodig en als je dat niet kan zal ik uitwijken voor de <table> ookal is het daarvoor niet gebruikt

(is dus meer van: als het nie anders kan

)

woensdag 4 maart 2009 21:19

Acties:

0 Henk 'm!

RobIII

Admin Devschuur®

^ Romeinse Ⅲ ja!

Het is jammer dat ik het topic nu pas zie, want er schort nogal het een en ander aan;

Zo is het niet de bedoeling hier om enorme lappen code te posten; beperk je tot het relevante deel en maak desnoods een uitgeklede testcase. Ook mis ik be-hoor-lijk wat informatie in je topicstart; zie daarvoor onze Quickstart. Daarin kun je precies lezen hoe we hier graag topics zien.

Al met al: we zijn geen afhaalbalie voor kant-en-klare oplossingen; doe voortaan zelf wat meer moeite en als je dat gedaan hebt open dan gerust een topic met informatie over wat je gedaan hebt en hou dan de voorgenoemde Quickstart in je achterhoofd als je dat doet.

Oh, en we geven al helemaal geen support op code van derden; in de Devschuur hebben we het over zélf programmeren en maken. Voor support op code die je elders 'leent' mag je bij die partij aankloppen. We zijn geen servicedesk.

Verwijderd schreef op woensdag 04 maart 2009 @ 20:44:
Zoals ik zei heb ik er erg weinig verstand van.... Is het een optie dat ik de bestanden inpak en on-line zet zodat jullie eraan kunnen bijspijkeren? Het formulier is trouwens ook breder dan de tabel.

En dat is al he-le-maal ongewenst: Kan iemand even...?. Als je het zelf niet kunt, besteed het dan uit aan iemand die het wel kan, of neem de moeite om het zelf onder de knie te krijgen; maar verwacht niet dat wij hier zitten duimen te draaien tot we alsjeblieft iets voor iemand anders voor nop mogen gaan zitten oplossen als diegene daar zelf te lui voor is.

Petje af overigens voor de geduldige mensen die hier nog tijd en moeite in gestoken hebben.

[ Voor 38% gewijzigd door RobIII op 04-03-2009 21:25 ]

There are only two hard problems in distributed systems: 2. Exactly-once delivery 1. Guaranteed order of messages 2. Exactly-once delivery.

Je eigen tweaker.me redirect

Over mij