Input velden verdwijnen na PHP echo
Sebastian V op 28/08/2012 13:27:22:
Hierzo lag de fout: 'Te lang! 120 karakters max.'); }
Klopt ik heb het veranderd, stond vanuit jou code nog een ) achter die weg moet :D, nu is hij eruit gehaald.
Je had Display:none; er toch uitgehaald he?
Ik krijg nu foutmeldingen op:
Regel 183 "Parse error: syntax error, unexpected T_PUBLIC":
public function sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden){
Regel 220:
public function setError($field, $errmsg){
Regel 225:
public function error_value($field){
Regel 232:
public function countErrors(){
Display none is er overal uitgehaald.
QUIZ.PHP
Code (php)
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
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
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
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
<div id="formulierruimte">
<?php
require_once("php/db.php"); /* Database Class */
require_once('php/utils/is_email.php'); /* Email Validation Script */
if(isset($_POST['newcontact'])){
$contact = new Contact();
//We'll refer to that later on, to display errors.
//unset($contact);
}
else{
//header('Location: result.php');
}
/* Class Contact */
class Contact{
private $db; /* the database obj */
//private $errors = array(); /* holds error messages */
//we have to init $errors array, as otherwise form will produce errors on missing array entry
private $errors = array( /* holds error messages */
'aanhef' => '',
'bedrijfsnaam' => '',
'email' => '',
'telefoon' => '',
'vraag1_antwoorden' => '',
'vraag2_antwoorden' => ''
);
private $num_errors; /* number of errors in submitted form */
public function __construct(){
$this->db = new DB();
if(isset($_POST['newcontact']))
$this->processNewMessage();
/* We don't need this anymore
*
else
header('Location: result.php');*/
}
public function processNewMessage(){
$aanhef = $_POST['aanhef'];
$contactpersoon = $_POST['contactpersoon'];
$bedrijfsnaam = $_POST['bedrijfsnaam'];
$telefoon = $_POST['telefoon'];
$email = $_POST['email'];
$vraag1_antwoorden = $_POST['vraag1_antwoorden'];
$vraag2_antwoorden = $_POST['vraag2_antwoorden'];
/* Server Side Data Validation */
if(!isset($aanhef)) {
$this->errors['aanhef'] = 'Vul uw aanhef in';
}
if(!isset($contactpersoon)) {
$this->errors['contactpersoon'] = 'Vul uw contactpersoon in';
} else if (isset($contactpersoon) || mb_strlen(trim($contactpersoon)) > 120) {
$this->errors['contactpersoon'] = 'Te lang! 120 karakters max.'; }
if(!isset($bedrijfsnaam)) {
$this->errors['bedrijfsnaam'] = 'Vul uw bedrijfsnaam in';
} else if (isset($bedrijfsnaam) || mb_strlen(trim($bedrijfsnaam)) > 120) {
$this->errors['bedrijfsnaam'] = 'Te lang! 120 karakters max.'; }
if(!isset($telefoon)) {
$this->errors['telefoon'] = 'Vul uw telefoon in';
} else if (isset($telefoon) || mb_strlen(trim($telefoon)) > 120) {
$this->errors['telefoon'] = 'Te lang! 120 karakters max.'; }
if(!isset($vraag1_antwoorden)) {
$this->errors['vraag1_antwoorden'] = 'Selecteer een antwoord a.u.b.';
if(!isset($vraag2_antwoorden)) {
$this->errors['vraag2_antwoorden'] = 'Selecteer een antwoord a.u.b.';
if(!isset($email)) {
$this->errors['email'] = 'Vul uw e-mail in';
} else if (isset($email) || mb_strlen(trim($email)) > 120) {
$this->errors['email'] = 'Te lang! 120 karakters max.'; }
/* No errors, insert in db
else*/
if($this->countErrors() == 0) {
if(($ret = $this->db->dbNewMessage($aanhef, $contactpersoon, $bedrijfsnaam, $email, $telefoon, $vraag1_antwoorden, $vraag2_antwoorden)) > 0){
//$json = array('result' => 1);
if(SEND_EMAIL)
$this->sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden);
//This is for relocating to successful result page
header('Location: result.php');
} else {
// else
// $json = array('result' => -2); /* something went wrong in database insertion */
//This will need special treatment. You have to prepare an errorpage
//for database-related issues.
header("Location: database-error.html");
}
/*$encoded = json_encode($json);
echo $encoded;
unset($encoded);*/
}
}
public function sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden){
/* Just format the email text the way you want ... */
$message_body = "<div style=\"font-size:12px; font-weight:normal;\">Hallo,<br><br>"
."Het volgende bedrijf heeft zich zojuist aangemeld:</div><br>"
."<table cellpadding=\"1\" cellspacing=\"1\" width=\"550px\"><tr><td style=\"font-size:12px; color:#000000\">Bedrijfsnaam:</td><td style=\"font-size:12px; color:#000000\">".$bedrijfsnaam."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Aanhef:</td><td style=\"font-size:12px; color:#000000\">".$aanhef."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Contactpersoon:</td><td style=\"font-size:12px; color:#000000\">".$contactpersoon."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Telefoonnummer:</td><td style=\"font-size:12px; color:#000000\">".$telefoon."</td></tr><tr><td style=\"font-size:12px; color:#000000\">E-mail:</td><td style=\"font-size:12px; color:#000000\">".$email."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Antwoord vraag 1:</td><td style=\"font-size:12px; color:#000000\">".$vraag1_antwoorden."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Antwoord vraag 2:</td><td style=\"font-size:12px; color:#000000\">".$vraag2_antwoorden."</td></tr></table><br>";
// Geef GELDIGE adressen op
// Een korte benaming voor jouw website
$website_naam = 'Aanmelding';
// Jouw eigen geldige emailadres
$eigen_emailadres = 'MAIL';
// Een geldig emailadres voor errors
$error_emailadres = 'MAIL';
// De naam van de verzender
$naam_verzender = ''.$bedrijfsnaam.'';
// Het geldige emailadres van de afzender
$email_verzender = ''.$email.'';
// Een geldig emailadres of helemaal leeg laten
$bcc_emailadres = '';
// HTML mail? True/False
$html = true;
// De headers samenstellen
$headers = 'From: ' . $website_naam . ' <' . $eigen_emailadres . '>' . PHP_EOL;
$headers .= 'Reply-To: ' . $naam_verzender . ' <' . $email_verzender . '>' . PHP_EOL;
$headers .= 'Return-Path: Mail-Error <' . $error_emailadres . '>' . PHP_EOL;
$headers .= ($bcc_emailadres != '') ? 'Bcc: ' . $bcc_emailadres . PHP_EOL : '';
$headers .= 'X-Mailer: PHP/' . phpversion() . PHP_EOL;
$headers .= 'X-Priority: Normal' . PHP_EOL;
$headers .= ($html) ? 'MIME-Version: 1.0' . PHP_EOL : '';
$headers .= ($html) ? 'Content-type: text/html; charset=iso-8859-1' . PHP_EOL : '';
mail(EMAIL_TO,MESSAGE_SUBJECT,$message_body,$headers);
}
public function setError($field, $errmsg){
$this->errors[$field] = $errmsg;
$this->num_errors = count($this->errors);
}
public function error_value($field){
if(array_key_exists($field,$this->errors))
return $this->errors[$field];
else
return '';
}
public function countErrors(){
return $this->num_errors;
}
};
?>
<table width="675px" cellpadding="0" cellspacing="0">
<form id="ContactForm" action="quiz.php" method="post">
<tr>
<td class="left_td"> </td><td><font size="-2">* Gelieve alle velden in te vullen.</font></td>
</tr>
<tr class="rij_aanhef">
<td class="left_td">Aanhef<font size="-2" style="vertical-align:top;">*</font></td>
<td>
<label class="label_aanhef" for="aanhef_1"><input name="aanhef" id="aanhef_1" value="Dhr." type="radio" /> Dhr.</label><label class="label_aanhef" for="aanhef_2"><input name="aanhef" id="aanhef_2" value="Mevr." type="radio" /> Mevr.</label>
<span class="error">
<?php echo $aanhef->error_value['aanhef']; ?>
</span>
</td>
</tr>
<tr class="rij_contactpersoon">
<td class="left_td">Contactpersoon</td>
<td>
<input id="contactpersoon" name="contactpersoon" class="inplaceError" maxlength="120" type="text" class="error" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $contactpersoon->error_value['contactpersoon']; ?>
</span>
</td>
</tr>
<tr class="rij_bedrijf">
<td class="left_td">Bedrijfsnaam</td>
<td>
<input id="bedrijfsnaam" name="bedrijfsnaam" class="inplaceError" maxlength="120" type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $bedrijfsnaam->error_value['bedrijfsnaam']; ?>
</span>
</td>
</tr>
<tr>
<td class="left_td">E-mail</td>
<td>
<input id="email" name="email" class="inplaceError" maxlength="120" type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $email->error_value['email']; ?>
</span>
</td>
</tr>
<tr>
<td class="left_td">Telefoonnummer</td>
<td><input id="telefoon" name="telefoon" class="inplaceError" maxlength="120" type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $telefoon->error_value['telefoon']; ?>
</span>
</td>
</tr>
</table>
</div>
<?php
require_once("php/db.php"); /* Database Class */
require_once('php/utils/is_email.php'); /* Email Validation Script */
if(isset($_POST['newcontact'])){
$contact = new Contact();
//We'll refer to that later on, to display errors.
//unset($contact);
}
else{
//header('Location: result.php');
}
/* Class Contact */
class Contact{
private $db; /* the database obj */
//private $errors = array(); /* holds error messages */
//we have to init $errors array, as otherwise form will produce errors on missing array entry
private $errors = array( /* holds error messages */
'aanhef' => '',
'bedrijfsnaam' => '',
'email' => '',
'telefoon' => '',
'vraag1_antwoorden' => '',
'vraag2_antwoorden' => ''
);
private $num_errors; /* number of errors in submitted form */
public function __construct(){
$this->db = new DB();
if(isset($_POST['newcontact']))
$this->processNewMessage();
/* We don't need this anymore
*
else
header('Location: result.php');*/
}
public function processNewMessage(){
$aanhef = $_POST['aanhef'];
$contactpersoon = $_POST['contactpersoon'];
$bedrijfsnaam = $_POST['bedrijfsnaam'];
$telefoon = $_POST['telefoon'];
$email = $_POST['email'];
$vraag1_antwoorden = $_POST['vraag1_antwoorden'];
$vraag2_antwoorden = $_POST['vraag2_antwoorden'];
/* Server Side Data Validation */
if(!isset($aanhef)) {
$this->errors['aanhef'] = 'Vul uw aanhef in';
}
if(!isset($contactpersoon)) {
$this->errors['contactpersoon'] = 'Vul uw contactpersoon in';
} else if (isset($contactpersoon) || mb_strlen(trim($contactpersoon)) > 120) {
$this->errors['contactpersoon'] = 'Te lang! 120 karakters max.'; }
if(!isset($bedrijfsnaam)) {
$this->errors['bedrijfsnaam'] = 'Vul uw bedrijfsnaam in';
} else if (isset($bedrijfsnaam) || mb_strlen(trim($bedrijfsnaam)) > 120) {
$this->errors['bedrijfsnaam'] = 'Te lang! 120 karakters max.'; }
if(!isset($telefoon)) {
$this->errors['telefoon'] = 'Vul uw telefoon in';
} else if (isset($telefoon) || mb_strlen(trim($telefoon)) > 120) {
$this->errors['telefoon'] = 'Te lang! 120 karakters max.'; }
if(!isset($vraag1_antwoorden)) {
$this->errors['vraag1_antwoorden'] = 'Selecteer een antwoord a.u.b.';
if(!isset($vraag2_antwoorden)) {
$this->errors['vraag2_antwoorden'] = 'Selecteer een antwoord a.u.b.';
if(!isset($email)) {
$this->errors['email'] = 'Vul uw e-mail in';
} else if (isset($email) || mb_strlen(trim($email)) > 120) {
$this->errors['email'] = 'Te lang! 120 karakters max.'; }
/* No errors, insert in db
else*/
if($this->countErrors() == 0) {
if(($ret = $this->db->dbNewMessage($aanhef, $contactpersoon, $bedrijfsnaam, $email, $telefoon, $vraag1_antwoorden, $vraag2_antwoorden)) > 0){
//$json = array('result' => 1);
if(SEND_EMAIL)
$this->sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden);
//This is for relocating to successful result page
header('Location: result.php');
} else {
// else
// $json = array('result' => -2); /* something went wrong in database insertion */
//This will need special treatment. You have to prepare an errorpage
//for database-related issues.
header("Location: database-error.html");
}
/*$encoded = json_encode($json);
echo $encoded;
unset($encoded);*/
}
}
public function sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden){
/* Just format the email text the way you want ... */
$message_body = "<div style=\"font-size:12px; font-weight:normal;\">Hallo,<br><br>"
."Het volgende bedrijf heeft zich zojuist aangemeld:</div><br>"
."<table cellpadding=\"1\" cellspacing=\"1\" width=\"550px\"><tr><td style=\"font-size:12px; color:#000000\">Bedrijfsnaam:</td><td style=\"font-size:12px; color:#000000\">".$bedrijfsnaam."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Aanhef:</td><td style=\"font-size:12px; color:#000000\">".$aanhef."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Contactpersoon:</td><td style=\"font-size:12px; color:#000000\">".$contactpersoon."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Telefoonnummer:</td><td style=\"font-size:12px; color:#000000\">".$telefoon."</td></tr><tr><td style=\"font-size:12px; color:#000000\">E-mail:</td><td style=\"font-size:12px; color:#000000\">".$email."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Antwoord vraag 1:</td><td style=\"font-size:12px; color:#000000\">".$vraag1_antwoorden."</td></tr><tr><td style=\"font-size:12px; color:#000000\">Antwoord vraag 2:</td><td style=\"font-size:12px; color:#000000\">".$vraag2_antwoorden."</td></tr></table><br>";
// Geef GELDIGE adressen op
// Een korte benaming voor jouw website
$website_naam = 'Aanmelding';
// Jouw eigen geldige emailadres
$eigen_emailadres = 'MAIL';
// Een geldig emailadres voor errors
$error_emailadres = 'MAIL';
// De naam van de verzender
$naam_verzender = ''.$bedrijfsnaam.'';
// Het geldige emailadres van de afzender
$email_verzender = ''.$email.'';
// Een geldig emailadres of helemaal leeg laten
$bcc_emailadres = '';
// HTML mail? True/False
$html = true;
// De headers samenstellen
$headers = 'From: ' . $website_naam . ' <' . $eigen_emailadres . '>' . PHP_EOL;
$headers .= 'Reply-To: ' . $naam_verzender . ' <' . $email_verzender . '>' . PHP_EOL;
$headers .= 'Return-Path: Mail-Error <' . $error_emailadres . '>' . PHP_EOL;
$headers .= ($bcc_emailadres != '') ? 'Bcc: ' . $bcc_emailadres . PHP_EOL : '';
$headers .= 'X-Mailer: PHP/' . phpversion() . PHP_EOL;
$headers .= 'X-Priority: Normal' . PHP_EOL;
$headers .= ($html) ? 'MIME-Version: 1.0' . PHP_EOL : '';
$headers .= ($html) ? 'Content-type: text/html; charset=iso-8859-1' . PHP_EOL : '';
mail(EMAIL_TO,MESSAGE_SUBJECT,$message_body,$headers);
}
public function setError($field, $errmsg){
$this->errors[$field] = $errmsg;
$this->num_errors = count($this->errors);
}
public function error_value($field){
if(array_key_exists($field,$this->errors))
return $this->errors[$field];
else
return '';
}
public function countErrors(){
return $this->num_errors;
}
};
?>
<table width="675px" cellpadding="0" cellspacing="0">
<form id="ContactForm" action="quiz.php" method="post">
<tr>
<td class="left_td"> </td><td><font size="-2">* Gelieve alle velden in te vullen.</font></td>
</tr>
<tr class="rij_aanhef">
<td class="left_td">Aanhef<font size="-2" style="vertical-align:top;">*</font></td>
<td>
<label class="label_aanhef" for="aanhef_1"><input name="aanhef" id="aanhef_1" value="Dhr." type="radio" /> Dhr.</label><label class="label_aanhef" for="aanhef_2"><input name="aanhef" id="aanhef_2" value="Mevr." type="radio" /> Mevr.</label>
<span class="error">
<?php echo $aanhef->error_value['aanhef']; ?>
</span>
</td>
</tr>
<tr class="rij_contactpersoon">
<td class="left_td">Contactpersoon</td>
<td>
<input id="contactpersoon" name="contactpersoon" class="inplaceError" maxlength="120" type="text" class="error" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $contactpersoon->error_value['contactpersoon']; ?>
</span>
</td>
</tr>
<tr class="rij_bedrijf">
<td class="left_td">Bedrijfsnaam</td>
<td>
<input id="bedrijfsnaam" name="bedrijfsnaam" class="inplaceError" maxlength="120" type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $bedrijfsnaam->error_value['bedrijfsnaam']; ?>
</span>
</td>
</tr>
<tr>
<td class="left_td">E-mail</td>
<td>
<input id="email" name="email" class="inplaceError" maxlength="120" type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $email->error_value['email']; ?>
</span>
</td>
</tr>
<tr>
<td class="left_td">Telefoonnummer</td>
<td><input id="telefoon" name="telefoon" class="inplaceError" maxlength="120" type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $telefoon->error_value['telefoon']; ?>
</span>
</td>
</tr>
</table>
</div>
Gewijzigd op 28/08/2012 13:53:50 door Sebas V
Ik zie een hoop in je script, waar ik niet weer waar het vandaan komt. Dat is dus heel moeilijk uitzoeken.
Dingen zoals SEND_EMAIL, MESSAGE_SUBJECT en nog wat andere dingen. Is dit door jezelf geschreven, of komt dit script van het net met wat aanpassingen?
De werking van het formulier is in principe simpel: Men vult het formulier in, zodra er op submit geklikt is, wordt er data verzonden naar een database en een e-mail gestuurd met de gegevens.
Uiteindelijke belandt de user op RESULT.PHP waar de ingevulde gegevens getoond worden.
Toevoeging op 28/08/2012 14:09:36:
SEND_EMAIL en MESSAGE_SUBJECT komen uit deze bestanden
Code (php)
1
2
3
4
2
3
4
<?php
require_once("php/db.php"); /* Database Class */
require_once('php/utils/is_email.php');
?>
require_once("php/db.php"); /* Database Class */
require_once('php/utils/is_email.php');
?>
Toevoeging op 28/08/2012 14:16:42:
db.php is gelinkt aan config.php waarin dit staat:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
2
3
4
5
6
7
8
9
10
11
12
13
<?php
define('DB_SERVER' , 'localhost');
define('DB_USER' , 'USER');
define('DB_PASS' , 'PASS');
define('DB_NAME' , 'NAME');
/*
* Email Settings
*/
define('EMAIL_TO' , 'EMAIL');
define('MESSAGE_SUBJECT' , 'Gegevens nieuwe deelnemer');
define('SEND_EMAIL' , true); /* if set to true, an email is sent to admin after a message is inserted */
?>
define('DB_SERVER' , 'localhost');
define('DB_USER' , 'USER');
define('DB_PASS' , 'PASS');
define('DB_NAME' , 'NAME');
/*
* Email Settings
*/
define('EMAIL_TO' , 'EMAIL');
define('MESSAGE_SUBJECT' , 'Gegevens nieuwe deelnemer');
define('SEND_EMAIL' , true); /* if set to true, an email is sent to admin after a message is inserted */
?>
Code (php)
Daar missen dus al bijvoorbeeld twee } om de if statements af te sluiten. Dit zijn gewoon fouten die je zelf ook kunt zien.
De validatie werkt echter nog niet. Als ik op submit klikt gaat het formulier gelijk naar RESULT.PHP
Ik heb geprobeerd:
if($this->countErrors() == 0) { te veranderen in if($this->countErrors() == '')
maar dat heeft geen effect.
Volgens mij gaat er iets fout in de class contact?
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
2
3
4
5
6
7
8
9
10
11
12
13
14
15
/* Class Contact */
class Contact{
private $db; /* the database obj */
//private $errors = array(); /* holds error messages */
//we have to init $errors array, as otherwise form will produce errors on missing array entry
private $errors = array( /* holds error messages */
'aanhef' => '',
'bedrijfsnaam' => '',
'email' => '',
'telefoon' => '',
'vraag1_antwoorden' => '',
'vraag2_antwoorden' => ''
);
class Contact{
private $db; /* the database obj */
//private $errors = array(); /* holds error messages */
//we have to init $errors array, as otherwise form will produce errors on missing array entry
private $errors = array( /* holds error messages */
'aanhef' => '',
'bedrijfsnaam' => '',
'email' => '',
'telefoon' => '',
'vraag1_antwoorden' => '',
'vraag2_antwoorden' => ''
);
Code (php)
Waarbij ik ervanuit ga dat $this->errors in eerste instantie als lege array werd gedefinieerd. Wat hier dan gebeurt is dat er elke keer dat er een error message bij komt ook num_errors wordt opgehoogd. Dat ben je nu allemaal kwijt. Volgens mij houd je nergens meer bij hoeveel errors je hebt en dus klopt er helemaal niets meer van die controles.
Gewijzigd op 28/08/2012 15:01:03 door John Berg
@John: excuses voor de hoeveelheid, ik probeer zo duidelijk mogelijk mijn probleem over te brengen. Zal ik een nieuwe post starten anders?
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public function setError($field, $errmsg){
$this->errors[$field] = $errmsg;
$this->num_errors = count($this->errors);
}
public function error_value($field){
if(array_key_exists($field,$this->errors))
return $this->errors[$field];
else
return '';
}
public function countErrors(){
return $this->num_errors;
}
$this->errors[$field] = $errmsg;
$this->num_errors = count($this->errors);
}
public function error_value($field){
if(array_key_exists($field,$this->errors))
return $this->errors[$field];
else
return '';
}
public function countErrors(){
return $this->num_errors;
}
Gewijzigd op 28/08/2012 15:02:25 door Sebas V
Dit is nu de kracht van OOP: een class die uit zichzelf kan doen wat het moet doen en je hoeft het alleen maar op de goede manier aan te roepen in een ander script om het te kunnen gebruiken.
Echter, ga je er in zitten klooien dan krijg je het nooit meer goed.
Mijn advies: begin opnieuw en laat de oorspronkelijke class onveranderd, of gebruik het in het geheel niet.
Gewijzigd op 28/08/2012 15:07:13 door Erwin H
Quote:
Regel 183 "Parse error: syntax error, unexpected T_PUBLIC":
public function sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden){
public function sendEmail($aanhef,$contactpersoon,$bedrijfsnaam,$email,$telefoon,$vraag1_antwoorden,$vraag2_antwoorden){
Het lijkt erop dat hij de T_PULBIC niet herkend, aangezien deze unexpected is.
Quote:
Regel 220:
public function setError($field, $errmsg){
public function setError($field, $errmsg){
Welke foutmelding hoort hierbij?
Quote:
Regel 225:
public function error_value($field){
public function error_value($field){
Welke foutmelding hoort hierbij?
Quote:
Regel 232:
public function countErrors(){
public function countErrors(){
Welke foutmelding hoort hierbij?
Verder ben ik het met Erwin eens. De kracht van OOP zijn de op zichzelf staande classes. Elke class in een aparte file, de class afzonderlijk testen en daarna voor de rest van je leven gebruiken.
Als je goed en modulair programmeert krijg je geen lappen code van 140 regels onder elkaar. Als ik een class functie schrijf is dat zelden meer dan 10 regels.
En voor de HTML geldt eigenlijk hetzelfde: de deelstukken in een div zetten. Bij het ontwerp eventueel een border om de div, zodat je goed kunt zien waar iets speelt.
Is er ergens een probleem, dan alleen het stuk binnen de div bekijken.
Code (php)
@John: ik ga op me gemak vnvnd het wat duidelijk proberen te schrijven. Ik ben echter bang dat als ik opnieuw begin dat ik geen raad weet waar te beginnen.
Ik snap jullie punt en ik wil me daar graag aan houden, ook voor toekomstige projecten heb je daar alleen maar profijt van.
Sorry ik was te laat met reageren.
Je schrijft 150 regels op die niets anders doen dan een tabel vullen met invoervelden.
Dat kan veel simpeler:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
$invoervelden = array( 'email', 'telefoonnummer', 'matresse1', 'maitresse2');
foreach( $invoervelden as $$invoerveld) {
?>
<tr>
<td class="left_td"><?= $invoerveld ?></td>
<td><input id="email" name="<?= $invoerveld ?>" class="inplaceError" maxlength="120 type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $email->error_value[$invoerveld]; ?>
</span>
</td>
</tr>
<?php
}
?>
$invoervelden = array( 'email', 'telefoonnummer', 'matresse1', 'maitresse2');
foreach( $invoervelden as $$invoerveld) {
?>
<tr>
<td class="left_td"><?= $invoerveld ?></td>
<td><input id="email" name="<?= $invoerveld ?>" class="inplaceError" maxlength="120 type="text" autocomplete="off" onFocus="window.scrollTo(0, 0);"/>
<span class="error">
<?php echo $email->error_value[$invoerveld]; ?>
</span>
</td>
</tr>
<?php
}
?>
Dat zou al denk ik meer dan 100 regels schelen, is gemakkelijker te onderhouden,
leesbaarder en wellicht ook eenvoudiger.
Zomaar een idee ..
Gewijzigd op 28/08/2012 15:37:44 door John Berg