[PHP/(My)SQL] voor dummies - deel 54 - MVC FTW!! | Digital Corner (DIG)

donderdag 24 januari 2008 @ 16:36:48 #226

CraZaay

prettig gestoord

Geen idee, maar die eerste 4 regels kunnen prima op 1 regel, moet je alleen even in PHP checken of sommige variabelen leeg zijn

donderdag 24 januari 2008 @ 17:15:39 #227

MuRKie

..warm as butters..

quote:
Op donderdag 24 januari 2008 16:36 schreef CraZaay het volgende:
Geen idee, maar die eerste 4 regels kunnen prima op 1 regel, moet je alleen even in PHP checken of sommige variabelen leeg zijn

Hoe bedoel je dat?

donderdag 24 januari 2008 @ 17:19:25 #228

CraZaay

prettig gestoord

RewriteRule ^!(.*)?/?(.*)?/?(.*)?/?$ index.php?component=$1&type=$2&id=$3 [L]

Misschien niet helemaal correcte syntax, maar zo kan het op 1 regel. Afhankelijk van het aantal componenten zijn 1 of meer variabelen (component, type en/of id) dan leeg.

donderdag 24 januari 2008 @ 17:31:13 #229

MuRKie

..warm as butters..

Ik ben nog al vers met PHP hoor maar ik zit met een probleem waar ik niet uitkom..
Ik ben bezig met een formulier waarin ik ook foutmeldingen wil verwerken als er iets niet goed gaat.
Ik wil dat die foutmelding in het formulier zelf wordt weergegeven met de data die de gebruiker voor het versturen had ingevuld. Ik krijg dit niet mooi voor elkaar.. Ik heb het formulier in zijn geheel in functies geschreven.

Ik heb er nu een beetje een zooi van gemaakt om tot een oplossing te komen wat dus nog steeds niet gelukt is.

Heeft iemand een voorbeeld script voor me hoe ik mooi een formulier kan opbouwen met een invoer validatie en foutafhandeling?
Ik kom er echt niet meer uit vandaag...

donderdag 24 januari 2008 @ 18:12:18 #230

no1uknow

Voor school (PHP vak) moet ik een webapplicatie maken waar mensen zich op kunnen registreren, vervolgens een adresboek moeten kunnen maken en bijhouden/wijzigen, en in de applicatie zelf een nieuwsbrief moeten kunnen opstellen die verzonden kan worden naar iedereen uit het adresboek.

Nu heb ik het registratiegedeelte bijna af, het adresboek zo goed als af, maar nu nog het nieuwsbrief gedeelte. (Vanavond moeten het adresboek+nieuwsbrief gedeelte worden ingeleverd!)

Ik heb het volgende script meegekregen van m'n leraar, die ik (denk ik..) moet gebruiken:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51

<?php

    $IsPostback   = false;
    $onderwerp    = "";
    $bericht      = "";
    $success      = false;

    if(isset($_POST['Submit']))
    {
        $IsPostback     = true;
        $adressenlijst     = "persoon1@adres1.nl, persoon2@adres2.nl"; // vul deze met adressen uit de database!

        $onderwerp         = $_POST['onderwerp'];
        $bericht           = $_POST['bericht'];

        // werkt alleen met anonymous smtp, HRO: alleen als je op school netwerk zit!, zie anders evt PEAR::Mail
            $success = @mail($adressenlijst, $onderwerp, $bericht, "From: mailinglist@adres.nl");
        // let op dat sommige SMTP servers geen niet bestaande adressen toestaan als afzender
    }

?>

<!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>mail versturen</title>
</head>

<body>

<?php if($IsPostback)
{
    if ($success) echo '<p class="success">mail succesvol verstuurd. </p>';
    else echo '<p class="error">Er is een fout opgetreden bij het versturen van de mail</p>';
}
else
{
?>
<h2>Mail:</h2>
    <form action="simple_mail.php" Method="Post">
        <label>Onderwerp:</label><input type="text" name="onderwerp"  /> <br />
        <label>Bericht:</label><textarea name="bericht" > </textarea> <br />
        <input type="Submit" name="Submit" value="Versturen" />
    </form>

<?php
}
?>
</body>
</html>

Kan iemand mij dit scriptje uitleggen, en vooral de volgende dingen:

Dit stukje commentaar:

1	// werkt alleen met anonymous smtp, HRO: alleen als je op school netwerk zit!, zie anders evt PEAR::Mail

Wat bedoelt de desbetreffende leraar hiermee? Ik heb een beetje lopen zoeken naar anonymous smtp maar ik kan niks interessants of bruikbaars vinden. (Zit op de Hogeschool Rotterdam, maar daar heb ik niks aan als ik thuis zit)

1	$adressenlijst = "persoon1@adres1.nl, persoon2@adres2.nl"; // vul deze met adressen uit de database!

Ik heb een database van het adresboek (voor elke geregistreerde gebruiker een eigen tabel met daarin de fields Naam en Email)
Hoe roep ik die correct op zodat ze daar naar verzonden worden?
Een vriend van mij typte het volgende:

1
2
3
4
5
6

mysql_connect("localhost","root","speelgoed");
$resultaat = mysql_query("SELECT email FROM gebruikers002");
while($rij = mysql_fetch_array($resultaat)){
$array[] = $rij['email'].',';
}
$adressenlijst = implode(",",$array);

Klopt dat?

Wat ik tot nu toe heb begrepen is dat ik smtp erbij nodig heb. Hoe ga ik dat doen? Waar haal ik zo'n server vandaan (dus niet die schoolserver) en is dit te testen op localhost?

donderdag 24 januari 2008 @ 18:36:04 #231

MuRKie

..warm as butters..

quote:
Op donderdag 24 januari 2008 17:19 schreef CraZaay het volgende:
RewriteRule ^!(.*)?/?(.*)?/?(.*)?/?$ index.php?component=$1&type=$2&id=$3 [L]

Misschien niet helemaal correcte syntax, maar zo kan het op 1 regel. Afhankelijk van het aantal componenten zijn 1 of meer variabelen (component, type en/of id) dan leeg.

Moet die uitroepteken dan na de ^ daar staan? Want dat is toch NOT?

donderdag 24 januari 2008 @ 18:48:37 #232

CraZaay

prettig gestoord

quote:
Op donderdag 24 januari 2008 18:36 schreef MuRKie het volgende:

[..]

Moet die uitroepteken dan na de ^ daar staan? Want dat is toch NOT?

Die hoort er niet nee. Ik heb gewoon jouw code gecopy-paste

donderdag 24 januari 2008 @ 18:50:37 #233

MuRKie

..warm as butters..

quote:
Op donderdag 24 januari 2008 18:48 schreef CraZaay het volgende:

[..]

Die hoort er niet nee. Ik heb gewoon jouw code gecopy-paste

OK

[edit]Die regel werkt niet. Hij returned dan de bestandsnaam. In dit geval index.php[/edit]

[ Bericht 9% gewijzigd door MuRKie op 24-01-2008 19:06:43 ]

donderdag 24 januari 2008 @ 18:52:51 #234

MuRKie

..warm as butters..

ik heb mijn formulier nu alsvolgt gemaakt maar ik heb het idee dat ik dit niet echt efficient doe zo..

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192

<?php
// no direct access
defined('_ALLOW_ACCESS')
    or die('Restricted access');

// --- formulier verwerken --- \\
function processForm()
{
    if($_POST['title'] != "" && $_POST['itemID'] != "")
    {
        if($_POST['ID'] != "" && fetchType() == "edit")
        {
            $varQuery    = "UPDATE menuitem ";
            $varSubQuery = "WHERE ID = '" . mysql_real_escape_string(htmlentities($_POST['ID'])) . "'";
        }
        else if($_POST['ID'] == "" && fetchType() == "new")
        {
            $varQuery    = "INSERT INTO menuitem";
            $varSubQuery = "";
        }
        else
        {
            $varComponentResult .= displayComponentResult(retryForm());
            exit;
        }
        $varQuery .= " SET title = '" . mysql_real_escape_string(htmlentities($_POST['title'])) . "'";
        $varQuery .= ", component = '" . mysql_real_escape_string(htmlentities("content")) . "'"; //frontend components moeten nog in de database verwerkt worden. Daarna zal deze functie gemaakt kunnen worden!
        $varQuery .= ", itemID = '" . mysql_real_escape_string(htmlentities($_POST['itemID'])) . "'";
        $varQuery .= ", publish = '" . mysql_real_escape_string(htmlentities($_POST['publish'])) . "'";
        $varQuery .= $varSubQuery;
        $varRun = mysql_query($varQuery)
            or die(_DATABASE_QUERY_ERROR . " " . mysql_error());

        redirect(cmsConfig_siteRoot . 'cbs/?component=menuitem&type=list');
    }
    else
    {
        $varComponentResult .= displayComponentResult(retryForm());

        return $varComponentResult;
    }
}

// --- lijst maken van beschikbare paginas --- \\
function listPages($pageID)
{
    $varQuery  = "SELECT ID, title";
    $varQuery .= " FROM content";
    $varRun    = mysql_query($varQuery)
            or die(_DATABASE_QUERY_ERROR . " " . mysql_error());
    $intRecords   = mysql_affected_rows();
    $varHtml   = "\t\t\t\t" . _CMN_PAGES . ":<br />\n";
    $varHtml  .= "\t\t\t\t<select name=\"itemID\" size= \"" . $intRecords . "\">\n";

    while($arrResult = mysql_fetch_assoc($varRun))
    {
        $varHtml .= "\t\t\t\t\t<option value=\"" . $arrResult['ID'] . "\"";
        if($arrResult['ID'] == $pageID)
        {
            $varHtml .= " selected";
        }
        $varHtml .= ">";
        $varHtml .= $arrResult['title'];
        $varHtml .= "</option>\n";
    }

    $varHtml .= "\t\t\t\t</select><br />\n";

    return $varHtml;
}

// --- velden voor nieuw formulier aanmaken --- \\
function newForm()
{
    $arrForm['type']    = "new";
    $arrForm['ID']      = "";
    $arrForm['title']   = "";
    $arrForm['itemID']  = "";
    $arrForm['page']    = listPages($arrForm['itemID']);
    $arrForm['publish'] = "";
    $arrForm['submit']  = _CMN_CREATE;

    return $arrForm;
}

// --- velden aanmaken voor onvolledig ingevuld formulier --- \\
function retryForm()
{
    $arrForm['type']    = mysql_real_escape_string(htmlentities($_GET['type']));
    $arrForm['ID']      = mysql_real_escape_string(htmlentities($_POST['ID']));
    $arrForm['title']   = mysql_real_escape_string(htmlentities($_POST['title']));
    $arrForm['itemID']  = mysql_real_escape_string(htmlentities($_POST['itemID']));
    $arrForm['page']    = listPages($_POST['itemID']);
    $arrForm['publish'] = mysql_real_escape_string(htmlentities($_POST['publish']));
    $arrForm['submit']  = mysql_real_escape_string(htmlentities($_POST['submit']));

    return $arrForm;
}

// --- velden voor wijzigings formulier aanmaken --- \\
function editForm()
{
    $intID = fetchID();
    $varQuery  = "SELECT ID,
                  title,
                  itemID,
                  publish";
    $varQuery .= " FROM menuitem";
    $varQuery .= " WHERE ID = '" . $intID . "'";
    $varRun    = mysql_query($varQuery)
        or die(_DATABASE_QUERY_ERROR . " " . mysql_error());
    $arrResult = mysql_fetch_assoc($varRun);

    $arrForm['type']    = "edit";
    $arrForm['ID']      = $arrResult['ID'];
    $arrForm['title']   = $arrResult['title'];
    $arrForm['itemID']  = $arrResult['itemID'];
    $arrForm['page']    = listPages($arrForm['itemID']);
    $arrForm['publish'] = $arrResult['publish'];
    $arrForm['submit']  = _CMN_EDIT;

    return $arrForm;
}

// --- formulier opbouwen --- \\
function displayComponentResult($arrForm)
{
    $varHtml  = "<form action=\"" . cmsConfig_siteRoot . "cbs/?component=menuitem&type=" . $arrForm['type'];
    if($arrForm['ID'] != "")
    {
        $varHtml .= "&id=" . $arrForm['ID'];
    }
    $varHtml .= "\" method=\"post\" name=\"menuitem\">\n";
    $varHtml .= "\t\t\t\t<input name=\"ID\" type=\"hidden\" value=\"" . $arrForm['ID'] . "\" />\n";
    $varHtml .= "\t\t\t\t" . _CMN_TITLE . ":<br />\n";
    $varHtml .= "\t\t\t\t<input name=\"title\" type=\"text\" size=\"25\" maxlength=\"50\" value=\"";
    $varHtml .= $arrForm['title'];
    $varHtml .= "\" /><br />\n";
    $varHtml .= $arrForm['page'];
    $varHtml .= "\t\t\t\t" . _CMN_PUBLISH . ":<br />\n";
    $varHtml .= "\t\t\t\t<select name=\"publish\">\n";
    $varHtml .= "\t\t\t\t\t<option value=\"yes\"";
    if($arrForm['publish'] == "yes")
    {
        $varHtml .= " selected";
    }
    $varHtml .= ">" . _CMN_YES . "</option>\n";
    $varHtml .= "\t\t\t\t\t<option value=\"no\"";
    if($arrForm['publish'] == "no")
    {
        $varHtml .= " selected";
    }
    $varHtml .= ">" . _CMN_NO . "</option>\n";
    $varHtml .= "\t\t\t\t</select><br />\n";
    $varHtml .= "\t\t\t\t<input name=\"submit\" type=\"submit\" value=\"" . $arrForm['submit'] . "\" />\n";
    $varHtml .= "\t\t\t</form>\n";

    return $varHtml;
}

// --- functie bepalen --- \\
if(isset($_POST['submit']) && $_POST['submit'] == _CMN_CREATE || $_POST['submit'] == _CMN_EDIT)
{
    $varComponentResult = processForm();
}
else if(fetchType() == "edit" && !fetchID())
{
    redirect(cmsConfig_siteRoot . 'cbs/?component=menuitem&type=list&msg=' . _MSG_NO_ID);
}
else if(fetchType() == "new")
{
    $varComponentResult = displayComponentResult(newForm());
}
else if(fetchType() == "edit")
{
    $varComponentResult = displayComponentResult(editForm());
}
?>

kan iemand mij wegwijs maken in een betere methode?

[ Bericht 16% gewijzigd door MuRKie op 24-01-2008 19:52:44 ]

donderdag 24 januari 2008 @ 23:46:14 #235

MuRKie

..warm as butters..

Misbruik ik met de volgende toepassing van OOP het OOP principe of "mag" deze methode gewoon?

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48

<?php
class MenuItem
{
    protected $ID;
    protected $title;

    function __construct()
    {

    }

    function setID($ID)
    {
        $this->ID = $ID;
    }

    function getID()
    {
        return $this->ID;
    }

    function setTitle($title)
    {
        $this->title = $title;
    }

    function getTitle()
    {
        return $this->title;
    }



    function Display()
    {
        $this->setID('7');
        $this->setTitle('Test met OOP?');
        $html  = "ID    = " . $this->getID() . "\n";
        $html .= "Title = " . $this->getTitle() . "\n";

        return $html;
    }

}

$test = new MenuItem;
$varComponentResult = $test->Display();
?>

Dit dient puur als voorbeeld..

Maar is OOP zo ook goed te gebruiken? Dus 1 functie (Display()) die al het nodige binnen een class uitvoerd en het eind resultaat terug geeft? Dus dat je niet van buiten de class variabelen invoert en/of wijzigt? In mijn situatie haalt hij namelijk alle waarden uit een database en komt daar geen handeling van de gebruiker aan te pas..
Dat is namelijk de reden waardoor ik twijfel of ik OOP wel gebruik zoals het hoort..

vrijdag 25 januari 2008 @ 00:27:14 #236

Geqxon

Ziet er keurig uit! Wat ik alleen persoonlijk als constructor zou doen:

1
2
3
4
5
6
7
8

<?php
    function MenuItem($ID, $title)
    {
        $this->setID($ID);
        $this->setTitle($title);
    }

?>

Zodat je bij het aanmaken van het object direct het ID en de Title meegeeft.

vrijdag 25 januari 2008 @ 00:39:27 #237

super-muffin

ik zou alleen de constructor schrijven als __constructor in plaats van de naam van de class, dat kan alleen nog maar ivm backwardcompability met PHP4.

Steam - SC2

vrijdag 25 januari 2008 @ 00:40:16 #238

Geqxon

Oeps! Dat is geen PHP4 backwardscompatibility (Scrabble?), maar een Java trekje.

vrijdag 25 januari 2008 @ 08:31:48 #240

super-muffin

En overigens, het is __construct. (zonder de 'or') op het einde.

In Java moet ik me nog een keer echt gaan verdiepen. Heb hier wel een boek liggen, maar daar staat weinig nuttigs in.

Steam - SC2

vrijdag 25 januari 2008 @ 08:59:36 #241

Xcalibur

quote:
Op vrijdag 25 januari 2008 00:27 schreef Geqxon het volgende:
Ziet er keurig uit! Wat ik alleen persoonlijk als constructor zou doen:
Zodat je bij het aanmaken van het object direct het ID en de Title meegeeft.

Dat dus.
Het nut van (de mogelijkheid tot) het zetten van een ID ontgaat me ook een beetje, aangezien je deze nu op een vaste waarde zet ín je class. Het is natuurlijk juist de bedoeling dat je dat vanuit je script doet lijkt me?

vrijdag 25 januari 2008 @ 09:15:18 #242

super-muffin

Ik zie trouwens niet in waarom je $Id en $Title niet public maakt. Die Get en Set methods zijn overbodig omdat je toch geen controle of iets uitvoert.

Steam - SC2

vrijdag 25 januari 2008 @ 09:26:51 #243

Geqxon

quote:
Op vrijdag 25 januari 2008 09:15 schreef super-muffin het volgende:
Ik zie trouwens niet in waarom je $Id en $Title niet public maakt. Die Get en Set methods zijn overbodig omdat je toch geen controle of iets uitvoert.

Wat nou als de interne werking van het instellen van de titel opeens anders wordt? Stel je voor dat het instellen van de titel naar een database weggeschreven moet worden. Dan mag je dus overal je source-code aan gaan zitten te passen, in plaats van dat je enkel de setTitle method aan hoeft te passen.

vrijdag 25 januari 2008 @ 10:22:08 #244

MuRKie

..warm as butters..

quote:
Op vrijdag 25 januari 2008 08:59 schreef Xcalibur het volgende:

[..]

Dat dus.
Het nut van (de mogelijkheid tot) het zetten van een ID ontgaat me ook een beetje, aangezien je deze nu op een vaste waarde zet ín je class. Het is natuurlijk juist de bedoeling dat je dat vanuit je script doet lijkt me?

Ja klopt, maar was ook puur ter illustratie van wat ik bedoelde..

vrijdag 25 januari 2008 @ 10:23:50 #245

MuRKie

..warm as butters..

quote:
Op vrijdag 25 januari 2008 09:15 schreef super-muffin het volgende:
Ik zie trouwens niet in waarom je $Id en $Title niet public maakt. Die Get en Set methods zijn overbodig omdat je toch geen controle of iets uitvoert.

Controle komt er nog (als je bedoelt qua gebruikers invoer) en die set en get functies daarover had ik ooit gelezen dat het beter was om de variabelen te setten via een functie en niet direct?

[ Bericht 5% gewijzigd door MuRKie op 25-01-2008 11:06:00 ]

vrijdag 25 januari 2008 @ 11:58:04 #246

MuRKie

..warm as butters..

quote:
Op vrijdag 25 januari 2008 00:27 schreef Geqxon het volgende:
Ziet er keurig uit! Wat ik alleen persoonlijk als constructor zou doen:
[ code verwijderd ]

Zodat je bij het aanmaken van het object direct het ID en de Title meegeeft.

In de uiteindelijke situatie worden de ID en title uit een database gehaald.. Dit wilde ik dan binnen de class regelen. Of is dat raar? En wanneer de gegevens niet uit een database worden gehaald, dan worden ze ingevoerd via een formulier, welke ik ook in de class wilde verwerken.. vandaar dat ik de constructor leeg had gelaten..

vrijdag 25 januari 2008 @ 12:59:47 #247

Farenji

quote:
Op vrijdag 25 januari 2008 10:23 schreef MuRKie het volgende:

[..]

Controle komt er nog (als je bedoelt qua gebruikers invoer) en die set en get functies daarover had ik ooit gelezen dat het beter was om de variabelen te setten via een functie en niet direct?

Je hebt gelijk, je kan ze best direct getten/setten zonder aparte functies, maar een voordeel van getters en setters gebruiken is dat je dan ook nog andere dingen kan doen - zoals bijv een property "isModified" oid op true zetten als je een veld aanpast. Dat kun je dan weer op andere plekken gebruiken, zoals in een save functie: als er niks gewijzigd is hoef je ook niet op te slaan. Verder is het ook makkeliijk mocht je ooit nog besluiten om de structuur van je object (ingrijpend) aan te passen; met getters en setters hoef je dan alleen die functies aan te passen; als je overal objectvariabelen direct benadert moet je het overal in je code gaan aanpassen.

vrijdag 25 januari 2008 @ 13:03:00 #248

Farenji

quote:
Op vrijdag 25 januari 2008 11:58 schreef MuRKie het volgende:

[..]

In de uiteindelijke situatie worden de ID en title uit een database gehaald.. Dit wilde ik dan binnen de class regelen. Of is dat raar?

Ja, eigenlijk wel. Lijkt me dat je op basis van het ID het record in de database opzoekt. Dus die kan beter wel gezet worden in de constructor.

vrijdag 25 januari 2008 @ 13:38:21 #249

MuRKie

..warm as butters..

quote:
Op vrijdag 25 januari 2008 13:03 schreef Farenji het volgende:

[..]

Ja, eigenlijk wel. Lijkt me dat je op basis van het ID het record in de database opzoekt. Dus die kan beter wel gezet worden in de constructor.

Ja dat is waar, dom van me..
De ID wordt natuurlijk niet uit de DB gehaald want aan de hand van de ID haal je het record juist op..
ID kan dus wel in de __construct want die wordt via de $_GET variabele bepaald..

zaterdag 26 januari 2008 @ 21:38:41 #250

kamustra

Is het mogelijk om bij het includen van bestand (include_once etc..) een $_GET parameter mee te geven?
Edit: laat maar, nee dus

[ Bericht 18% gewijzigd door kamustra op 26-01-2008 21:51:28 ]

Forum Opties
Forumhop:
Hop naar:	(afkorting, bv 'KLB')

[PHP/(My)SQL] voor dummies - deel 54 - MVC FTW!!

» digital corner

» digital corner