Webwereld

Classes

Gestart doorErik97 reacties359 weergaven
Log in om mee te praten

98 berichten · pagina 3 van 7

Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Dus met:

if(count($errors->aantal() => 1))
{
}


Ofzo?
0
Lennard

Lid

Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]aantal[COLOR= #007700]() ===[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700])
{[/COLOR]

  [COLOR= #FF8000]// alles verwerken[/COLOR]

[COLOR= #007700]}[/COLOR]
[URL="http://www.php.net/else"]else[/URL]
[COLOR= #007700]{[/COLOR]

  [COLOR= #FF8000]// fouten weergeven enzo[/COLOR]

[COLOR= #007700]}[/COLOR]
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Ok.
0
Lennard

Lid

Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
Want $errors->aantal() geeft al een getal terug, dus is een extra count() niet nodig. biggrin
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Ohja. biggrinEdit:
Dus dit zou een goede mail functie zijn?

public function mail()
    {
        // De headers
        $headers = Content Type: text/html;
        $headers .= From: info@ipet.nl;
        // De standaard opmaak in een variabel zetten
        $standaard_top = 
<html>
    <head>
        <title>iPet Mail</title>
        <link href="http://localhost/ipet/style.css" rel="stylesheet" type="text/css" />
    </head>
    
    <body>
        <div id="container">
            <a href="http://localhost/ipet/index/" class="header">
                <h1>iPet</h1>
            </a>
            <h1>
                iPet Mail
            </h1>
            <div class="vak">;
        $standaard_bottom = 
            </div>
        </div>
    </body>
</html>;
    }
0
Lennard

Lid

Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
Nee, want je mailt nog niets. yummie
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Wat moet je daar dan nog bijzetten? emo Ja ik weet het, ben noob met classes. emo
Edit:
Ik hoop dat me aanmelden zo goed is.. emo
[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/include"]include[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]"core.php"[/COLOR][COLOR= #007700]);[/COLOR]
[COLOR= #FF8000]// Er is op aanmelden geklikt[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/isset"]isset[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]submitA[/COLOR][COLOR= #007700]]) &&[/COLOR] [COLOR= #0000BB]$_SERVER[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]REQUEST_METHOD[/COLOR][COLOR= #007700]] ===[/COLOR] [COLOR= #DD0000]POST[/COLOR][COLOR= #007700])
{[/COLOR]
    [COLOR= #FF8000]// De variabelen klaar zetten[/COLOR]
    [COLOR= #0000BB]$gebruikersnaam[/COLOR] [COLOR= #007700]=[/COLOR] htmlweg[COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]escape[COLOR= #007700]([/COLOR][URL="http://www.php.net/addslashes"]addslashes[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]gebruikersnaam[/COLOR][COLOR= #007700]])));[/COLOR]
    [COLOR= #0000BB]$pass1[/COLOR] [COLOR= #007700]=[/COLOR] htmlweg[COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]escape[COLOR= #007700]([/COLOR][URL="http://www.php.net/addslashes"]addslashes[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]pass1[/COLOR][COLOR= #007700]])));[/COLOR]
    [COLOR= #0000BB]$pass2[/COLOR] [COLOR= #007700]=[/COLOR] htmlweg[COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]escape[COLOR= #007700]([/COLOR][URL="http://www.php.net/addslashes"]addslashes[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]pass2[/COLOR][COLOR= #007700]])));[/COLOR]
    [COLOR= #0000BB]$email[/COLOR] [COLOR= #007700]=[/COLOR] htmlweg[COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]escape[COLOR= #007700]([/COLOR][URL="http://www.php.net/addslashes"]addslashes[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]email[/COLOR][COLOR= #007700]])));[/COLOR]
    [COLOR= #0000BB]$email_state[/COLOR] [COLOR= #007700]=[/COLOR] htmlweg[COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]escape[COLOR= #007700]([/COLOR][URL="http://www.php.net/addslashes"]addslashes[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]email_state[/COLOR][COLOR= #007700]])));[/COLOR]
    [COLOR= #0000BB]$nieuwsbrief[/COLOR] [COLOR= #007700]=[/COLOR] htmlweg[COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]escape[COLOR= #007700]([/COLOR][URL="http://www.php.net/addslashes"]addslashes[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$_POST[/COLOR][COLOR= #007700][[/COLOR][COLOR= #DD0000]nieuwsbrief[/COLOR][COLOR= #007700]])));[/COLOR]
    [COLOR= #FF8000]// Geen gebruikersnaam ingevuld[/COLOR]
    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$gebruikersnaam[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #DD0000][/COLOR][COLOR= #007700])
    {[/COLOR]
        [COLOR= #FF8000]// Error zetten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Geen gebruikersnaam ingevuld.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Checken met de database of de gebruikersnaam al bezet is[/COLOR]
    [COLOR= #0000BB]$query[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"
        SELECT
            *
        FROM
            members
        WHERE
            gebruikersnaam = "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$gebruikersnaam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]"
    "[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]error[COLOR= #007700]());[/COLOR]
    [COLOR= #FF8000]// Deze gebruikersnaam is al bezet[/COLOR]
    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]num_rows[COLOR= #007700]([/COLOR][COLOR= #0000BB]$query[/COLOR][COLOR= #007700]) >[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700])
    {[/COLOR]
        [COLOR= #FF8000]// Error zetten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Je gebruikersnaam is al ingebruik.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Geen wachtwoord ingevuld[/COLOR]
    [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$pass1[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #DD0000][/COLOR][COLOR= #007700])
    {[/COLOR]
        [COLOR= #FF8000]// Error zetten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Je hebt geen wachtwoord ingevuld.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Geen tweede wachtwoord ingevuld[/COLOR]
    [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$pass2[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #DD0000][/COLOR][COLOR= #007700])
    {[/COLOR]
        [COLOR= #FF8000]// Error zettten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Je hebt geen tweede wachtwoord ingevuld.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Geen e-mail adres ingevuld[/COLOR]
    [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$email[/COLOR] [COLOR= #007700]==[/COLOR] [COLOR= #DD0000][/COLOR][COLOR= #007700])
    {[/COLOR]
        [COLOR= #FF8000]// Error zetten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Je hebt geen e-mail adres ingevuld.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Dit e-mail adres is niet geldig[/COLOR]
    [URL="http://www.php.net/elseif"]elseif[/URL][COLOR= #007700](![/COLOR]filter_var[COLOR= #007700]([/COLOR][COLOR= #0000BB]$email[/COLOR][COLOR= #007700],[/COLOR] FILTER_VALIDATE_EMAIL[COLOR= #007700]))
    {[/COLOR]
        [COLOR= #FF8000]// Error zetten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Je e-mail adres is niet geldig.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Checken met de database of dit e-mail adres er al in staat[/COLOR]
    [COLOR= #0000BB]$query2[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"
        SELECT
            *
        FROM
            members
        WHERE
            email = "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$email[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]"
    "[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]error[COLOR= #007700]());[/COLOR]
    [COLOR= #FF8000]// Dit e-mail adres is al bezet[/COLOR]
    [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]num_rows[COLOR= #007700]([/COLOR][COLOR= #0000BB]$query2[/COLOR][COLOR= #007700]) >[/COLOR] [COLOR= #0000BB]0[/COLOR][COLOR= #007700])
    {[/COLOR]
        [COLOR= #FF8000]// Error zetten[/COLOR]
        [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]add[COLOR= #007700]([/COLOR][COLOR= #DD0000]Je e-mail adres is al bezet.[/COLOR][COLOR= #007700]);
    }[/COLOR]
    [COLOR= #FF8000]// Alles goed[/COLOR]
    [URL="http://www.php.net/else"]else[/URL]
    [COLOR= #007700]{[/COLOR]
        [COLOR= #FF8000]// Dit lid toevoegen aan de database[/COLOR]
        [COLOR= #0000BB]$query3[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"
            INSERT INTO
                members
            (
                gebruikersnaam,
                wachtwoord,
                email,
                email_state,
                nieuwsbrief,
                active,
                rang
            )
            VALUES
            (
                "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$gebruikersnaam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
                "[/COLOR][COLOR= #007700].[/COLOR][URL="http://www.php.net/md5"]md5[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$pass1[/COLOR][COLOR= #007700]).[/COLOR][COLOR= #DD0000]",
                "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$email[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
                "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$email_state[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
                "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$nieuwsbrief[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
                0,
                Lid
            )
        "[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]error[COLOR= #007700]());[/COLOR]
        [COLOR= #FF8000]// Het eerste PBtje van dit lid maken[/COLOR]
        [COLOR= #0000BB]$pb[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]
        Hoi [/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$gebruikersnaam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000],<br />
        Wij vinden het leuk dat jij je hebt aangemeld op iPet!<br />
        Wij wensen je heel veel plezier op iPet.<br />
        Mocht je nog vragen hebben kun je die altijd mail naar info@ipet.nl of op het forum of in een ticket.<br />
        Groetjes,
        iPet Team[/COLOR][COLOR= #007700];[/COLOR]
        [COLOR= #FF8000]// Het eerste PBtje van dit lid versturen[/COLOR]
        [COLOR= #0000BB]$query4[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"
            INSERT INTO
                pb
            (
                to,
                of,
                read,
                subject,
                message
            )
            VALUES
            (
                "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$gebruikersnaam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
                iPet,
                0,
                Hoi "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$gebruikersnaam[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
                "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$pb[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]"
            )
        "[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]error[COLOR= #007700]());[/COLOR]
        [COLOR= #FF8000]// Het activatie mailtje maken[/COLOR]
        [COLOR= #0000BB]$mail[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]
        Hoi [/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$name[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000],
        Wij vinden het leuk dat jij je hebt aangemeld op iPet.
        Hierbij ontvang jij je gegevens en je unieke activatie link.
        http://localhost/ipet/user/activate/[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$name[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]
        Dat is je activatie link.
        Dan zie je nu je gegevens.
          --------------
        STRIKT PERSOONLIJK
          --------------
        Gebruikersnaam: [/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$name[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]
        Wachtwoord: [/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$pass1[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]
        Groetjes,
        iPet Team[/COLOR][COLOR= #007700];[/COLOR]
        [COLOR= #FF8000]// De mail versturen[/COLOR]
        [COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR][URL="http://www.php.net/mail"]mail[/URL][COLOR= #007700]();[/COLOR]
        [COLOR= #FF8000]// Weergeven dat het gelukt is[/COLOR]
        [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]
                <h1>
                    Aanmelden
                </h1>
                <div class="vak">
                    <div class="goed">
                        Je bent succesvol aangemeld.<br />
                        Check je mailbox voor je activatie link en je gegevens.<br />
                        Groetjes,
                        iPet Team
                    </div>
                </div>[/COLOR][COLOR= #007700];
    }
}[/COLOR]
[COLOR= #FF8000]// Er is nog niet op aanmelden geklikt[/COLOR]
[URL="http://www.php.net/else"]else[/URL]
[COLOR= #007700]{[/COLOR]
    [COLOR= #FF8000]// De errors weergeven[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]
                <h1>
                    Aanmelden
                </h1>
                <div class="vak">[/COLOR][COLOR= #007700];[/COLOR]
                [COLOR= #FF8000]// Er zijn errors[/COLOR]
                [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]aantal[COLOR= #007700]() ===[/COLOR] [COLOR= #0000BB]1[/COLOR][COLOR= #007700])
                {[/COLOR]
                    [COLOR= #FF8000]// De errors weergeven[/COLOR]
                    [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]enkel_meervoud[COLOR= #007700]();[/COLOR]
                    [COLOR= #0000BB]$errors[/COLOR][COLOR= #007700]->[/COLOR]list_errors[COLOR= #007700]();[/COLOR]
                    [COLOR= #FF8000]// Het formulier weergeven[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]
                    <form action="" method="post">
                        <table width="100%">
                            <tr>
                                <td class="een">
                                    Gebruikersnaam
                                </td>
                                <td class="een">
                                    <input type="text" name="gebruikersnaam" />
                                </td>
                            </tr>
                            <tr>
                                <td>
                                    Wachtwoord
                                </td>
                                <td>
                                    <input type="password" name="pass1" />
                                </td>
                            </tr>
                            <tr>
                                <td class="een">
                                    Wachtwoord <small>(Nogmaals)</small>
                                </td>
                                <td class="een">
                                    <input type="password" name="pass2" />
                                </td>
                            </tr>
                            <tr>
                                <td>
                                    E-mail adres
                                </td>
                                <td>
                                    <input type="text" name="email" />
                                </td>
                            </tr>
                            <tr>
                                <td class="een">
                                    E-mail adres weergeven?
                                </td>
                                <td class="een">
                                    <input type="checkbox" name="email_state" value="1" />
                                </td>
                            </tr>
                            <tr>
                                <td>
                                    Wil je gratis de maandelijkse nieuwsbrief?
                                </td>
                                <td>
                                    <input type="checkbox" name="nieuwsbrief" value="1" />
                                </td>
                            </tr>
                            <tr>
                                <td class="een">
                                    Accepteer je de <a href="http://localhost/ipet/av" target="_blank">Algemene Voorwaarden</a>?
                                </td>
                                <td class="een">
                                    <input type="checkbox" name="algemene_voorwaarden" value="1" />
                                </td>
                            </tr>
                            <tr>
                                <td>
                                </td>
                                <td>
                                    <input type="submit" name="submitA" value="Aanmelden" />
                                </td>
                            </tr>
                        </table>
                    </form>
                </div>[/COLOR][COLOR= #007700];
        }
    }
}[/COLOR]
?>
0
Ruben

Lid

Berichten
2.950
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Als je mysql_real_escape_string() doet hoef je geen addslashes te doen. Ook zou ik ipv

if($var == ) het met trim doen, dus:

if(trim($var) == ), dan kan je ook geen spatie invoeren.

Bij email check je of hij leeg is en of het valide is. Alleen valide is goed genoeg, een leeg emailadres is niet valide namelijk.
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Ok.
0
Lennard

Lid

Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
addslashes() heb je vrijwel nooit nodig bij databasedingen.
Stripslashes() ook niet..

Even een rijtje puntjes:
  • SELECT * (r. 22 en r. 60) is niet nodig; SELECT 1 is voldoende; je doet toch niets met de data
  • Bij elke fout heb je elseif; vervang die door if; anders krijg je max. één foutmelding
  • $pass1 kan ook een spatie bevatten; if(trim($pass1) == ) is dus handiger =D
  • Je logt wel heel vrolijk de errors ($errors->add()), maar je doet er niks mee; je afhandeling staat buiten de if waarin je de controles uitvoert, dus zal nooit uitgevoerd worden
  • Sla de data zo puur mogelijk op; geen idee wat htmlweg() doet, maar die zou ik eruit halen als het een soort htmlentities() of strip_tags() is. Als je data in je database doet, haal het dan alleen door trim() en mysql->escape() heen, en indien nodig door andere functies zoals strtolower() (voor emailadressen)
  • Je controleert ook niet alle waardes; je hebt geen flauw idee wat $nieuwsbrief is die je in regel 95 in je database stopt.
  • Vergeet de quotes op regel 121 t/m 125 niet emo
  • $core->mail() op regel 144 mist wat parameters
  • Regel 172 en verder worden alleen uitgevoerd als er één foutmelding is, dat lijkt me ook niet de bedoeling

Je bent op de goede weg, maar dr moet nog wel wat veranderd worden. clown
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
De nieuwsbrief is voor als je die wilt ontvangen. clownEdit:
Ik krijg deze error:

Fatal error: Call to a member function query() on a non-object in C:xampphtdocsipetfunctions.inc.php on line 6


Bij dit script:

[COLOR= #0000BB]<?php[/COLOR]
[COLOR= #FF8000]// De PHP error functie (De PHP errors wegwerken)[/COLOR]
[COLOR= #007700]function[/COLOR] php_error[COLOR= #007700]([/COLOR][COLOR= #0000BB]$error[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$file[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$number[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$line[/COLOR][COLOR= #007700],[/COLOR] [COLOR= #0000BB]$vars[/COLOR][COLOR= #007700])
{[/COLOR]
    [COLOR= #FF8000]// De errors in de error tabel zetten[/COLOR]
    [COLOR= #0000BB]$query[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]query[COLOR= #007700]([/COLOR][COLOR= #DD0000]"
        INSERT INTO
            errors
        (
            file,
            number,
            line,
            vars,
            error
        )
        VALUES
        (
            "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$file[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
            "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$number[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
            "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$line[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
            "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$vars[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]",
            "[/COLOR][COLOR= #007700].[/COLOR][COLOR= #0000BB]$error[/COLOR][COLOR= #007700].[/COLOR][COLOR= #DD0000]"
        )
    "[/COLOR][COLOR= #007700])[/COLOR] or [URL="http://www.php.net/die"]die[/URL][COLOR= #007700]([/COLOR][COLOR= #0000BB]$core[/COLOR][COLOR= #007700]->[/COLOR]mysql[COLOR= #007700]->[/COLOR]error[COLOR= #007700]());[/COLOR]
    [COLOR= #FF8000]// Weergeven dat er een fout is opgetreden[/COLOR]
                [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]
                Hoi,<br />
                Er is een fout op getreden.<br />
                De beheerders zijn op de hoogte gebracht.<br />
                Sorry voor het ongemak.[/COLOR][COLOR= #007700];
}[/COLOR]
[URL="http://www.php.net/set_error_handler"]set_error_handler[/URL][COLOR= #007700]([/COLOR][COLOR= #DD0000]php_error[/COLOR][COLOR= #007700]);[/COLOR]
?>
0
Lennard

Lid

Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
Ja, maar welke waarde sla je op in de database?
Iemand kan je formulier aanpassen en zorgen dat er "ewfief" komt te staan, dat wil je niet.
Je wilt een 1 als iemand een nieuwsbrief wil, en anders een 0, of iets wat er op lijkt. In ieder geval 2 waardes; voor wel en voor niet.
$nieuwsbrief kan nu alles zijn.

@ je edit: $core->mysql->query() bestaat niet in die scope; je zult $core (of $mysql) dus mee moeten geven als parameter.
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Ik sla het op als 1 of 0. 1 Is als je hebt wilt ontvangen en 0 voor niet ontvangen. Hoe kan je dat dan anders nog maken? biggrin@ Lennards edit: Dat snap ik niet? sadley
0
Lennard

Lid

Berichten
35.796
Reputatie
1
Punten
0,00
Lid sinds
Laatst gezien
Nee, dat doe je niet. emoOp regel 12 zet je $_POST[nieuwsbrief] in $nieuwsbrief, en in regel 95 zet je hem in de database, je controleert er verder niets aan.
0
Erik

Lid · Topicstarter

Berichten
7.109
Reputatie
0
Punten
0,00
Lid sinds
Laatst gezien
Hoe kan je dat dan controleren? emo
0

Jouw reactie

Deel je idee, stel een vraag of help een ander verder.

Editor laden…

Praat mee met de One2xs-community.

Inloggen om te reageren