kwakkers.nl.tt
dit is de site van mijn wagenbouwgroep
Wat moet er gereviewed? De lay-out heb je namelijk gewoon gedownload:
zal ik proberen
En wat als ik bijv. zou zeggen: Wat een lelijke site?.
Verder vind ik de code nei echt netjes:
- Variabelen buiten quotes
- Missende controles op functies en acties.. Waarom is lelijk een scheldwoord? nou en. het werkt:P
ok :)
lawl .tt
Als er iets te reviewen is dan is het je spelling en grammatica. Deze zin: : Heb jij zin om in de vakantie de kwakkers te helpen?? voor de optocht dan kan!! kan echt niet. En zo staat er nog meer.
maar wat vinden jullie van de site
Wat? De layout? Dat is een template... De tekst? Die zit vol spelfouten en komt niet echt professioneel over. He gastenboek dan? Die is van een gratis site... (Je zit hier op een PHP site, dus eigenlijk zou je zo'n gastenboek zelf moeten maken ;) )
Douwe M schreef op 05.08.2008 13:55:
Die is van een gratis site... (Je zit hier op een PHP site, dus eigenlijk zou je zo'n gastenboek zelf moeten maken ;) )
zal ik proberen
Okay, dan is er in ieder geval iets te beoordelen.
http://kwakkers.x10hosting.com/gastenboek.php
het gasten boek : Gewijzigd op 01/01/1970 01:00:00 door Bas
Okay, nice. De frontend is niet verschrikkelijk, maar als je het wilt laten beoordelen: what's the source?
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
216
217
218
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
216
217
218
<!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>GASTENBOEK</title>
<script language='javascript' type='text/javascript'>
function emoticon(text) {
text = ' ' + text + ' ';
if (document.book.message.createTextRange && document.book.message.caretPos) {
var caretPos = document.book.message.caretPos;
caretPos.text = caretPos.text.charAt(caretPos.text.length - 1) == ' ' ? text + ' ' : text;
document.book.message.focus();
} else {
document.book.message.value += text;
document.book.message.focus();
}
}
</script>
</head>
<body>
<?php
include "connect.php";
$actie=$_GET['actie'] OR $actie=$_POST['actie'];
if ($actie == 'teken')
{
?>
<center>
<form action='gastenboek.php?actie=toevoegen' method='post' name="book">
<table border='1' cellspacing='0' width="600">
<tr>
<td>
Naam:
</td>
<td>
<input type='tekst' name='naam' size='20'>
</td>
</tr>
<tr>
<td>
Email:
</td>
<td>
<input type='tekst' name='email' size='20'>
</td>
</tr>
</table>
<table border='1' cellspacing='0' width="600">
<tr>
<td>
<textarea name='message' rows='10' cols='71'></textarea>
</td>
</tr>
<tr>
<table width='600'>
<tr>
<td>
<img src="images/smilies/icon4.gif" border="0" onclick="javascript:emoticon(':p')">
</td>
<td>
<img src="images/smilies/icon1.gif" border="0" onclick="javascript:emoticon(':)')">
</td>
<td>
<img src="images/smilies/icon10.gif" border="0" onclick="javascript:emoticon('(H)')">
</td>
<td>
<img src="images/smilies/biggrin.gif" border="0" onclick="javascript:emoticon(':D')">
</td>
<td>
<img src="images/smilies/finger.gif" border="0" onclick="javascript:emoticon('(m)')">
</td>
<td>
<img src="images/smilies/bored.gif" border='0' onclick="javascript:emoticon(':S')">
</td>
<td>
<img src="images/smilies/mad.gif" border='0' onclick="javascript:emoticon(':@')">
</td>
<td>
<img src="images/smilies/smiley3.gif" border='0' onclick="javascript:emoticon(':-O')">
</td>
<td>
<img src="images/smilies/LOL.gif" border='0' onclick="javascript:emoticon('(LOL)')">
</td>
<td>
<img src="images/smilies/tong.gif" border='0' onclick="javascript:emoticon('P:')">
</td>
</tr>
</table>
</tr>
<tr>
<td align='center'>
<input type='submit' name='actie' value='toevoegen'>
</td>
</tr>
</table>
</form>
</center>
<?php
}
else if ($actie =='toevoegen')
{
$naam=$_POST['naam'];
$naam = ucfirst(strtolower($_POST['naam']));
$email=$_POST['email'];
$reactie=$_POST['message'];
if ($naam == '' OR $email == '' OR $reactie =='')
{
print 'Je moet wel alles ingevullen';
}
else if($naam == 'Admin' OR $naam =='Webmaster' OR $naam =='Administrator' OR $naam =='Moderator' )
{
echo "probeer je niet voor te doen als iemand die je niet bent";
}
else
{
//naam
$naam = htmlspecialchars($naam);
//email
$email = htmlspecialchars($email);
//reactie
$reactie = htmlspecialchars($reactie);
$reactie = nl2br ($reactie);
$reactie = str_replace(':p','<img src="images/smilies/icon4.gif">',$reactie);
$reactie = str_replace(':)','<img src="images/smilies/icon1.gif">',$reactie);
$reactie = str_replace('(H)','<img src="images/smilies/icon10.gif">',$reactie);
$reactie = str_replace(':D','<img src="images/smilies/biggrin.gif">',$reactie);
$reactie = str_replace('(m)','<img src="images/smilies/finger.gif">',$reactie);
$reactie = str_replace(':S','<img src="images/smilies/bored.gif">',$reactie);
$reactie = str_replace(':@','<img src="images/smilies/mad.gif">',$reactie);
$reactie = str_replace(':-O','<img src="images/smilies/smiley3.gif">',$reactie);
$reactie = str_replace('(LOL)','<img src="images/smilies/LOL.gif">',$reactie);
$reactie = str_replace('P:','<img src="images/smilies/tong.gif">',$reactie);
$reactie = str_replace('(m)','<img src="images/smilies/finger.gif">',$reactie);
// scheldwoorden er uit filteren
$reactie = str_replace('tering','*', $reactie);
$reactie = str_replace('kanker','*', $reactie);
$reactie = str_replace('kenker','*', $reactie);
$reactie = str_replace('vagina','*', $reactie);
$reactie = str_replace('anaal','*', $reactie);
$reactie = str_replace('tyfus','*', $reactie);
$reactie = str_replace('tievus','*', $reactie);
$reactie = str_replace('godverdomme','*', $reactie);
$reactie = str_replace('kut','*', $reactie);
$reactie = str_replace('lul','*', $reactie);
$reactie = str_replace('flikker','*', $reactie);
$reactie = str_replace('homo','*', $reactie);
$reactie = str_replace('nicht','*', $reactie);
$reactie = str_replace('likker','*', $reactie);
$reactie = str_replace('suck','*', $reactie);
$reactie = str_replace('fuck','*', $reactie);
$reactie = str_replace('zuigen','*', $reactie);
$reactie = str_replace('zuige','*', $reactie);
$reactie = str_replace('sukkel','*', $reactie);
$reactie = str_replace('lelijk','*', $reactie);
$reactie = str_replace('trut','*', $reactie);
$reactie = str_replace('new holland','*', $reactie);
$reactie = str_replace('ford','*', $reactie);
$reactie = str_replace('fend','*', $reactie);
//alles toevoegen
$sql = ("INSERT INTO gastenboek (id, naam, reactie, email) VALUES ('', '$naam', '$reactie', '$email')");
$result = mysql_query ($sql) or die (mysql_error());
echo"
je bericht is toegevoegd.<br>
klik <a href='gastenboek.php'>hier</a> om je bericht te bekijken.";
}
}
else
{
?>
<div id="center">
<center>
<table border='1' cellspacing='0' width="600">
<tr>
<td>
WELKOM BIJ DIT GASTENBOEK
</td>
</tr>
<tr>
<td>
<a href='gastenboek.php?actie=teken'>klik hier om iets in het gastenboek te schrijven</a>
</td>
</tr>
</table>
<table border='1' cellspacing='0' width="600">
<?php
$result = mysql_query ("SELECT * FROM gastenboek order by id desc")or die (mysql_error());
while ($array = mysql_fetch_array($result))
{
echo "
<tr>
<td width='50'>
<a href='mailto:$array[email]'>$array[naam]</a>
</td>
<td width='550'>
$array[reactie]";
echo"
</td>
</tr>
";
}
?>
</center>
</div>
<?php
}
?>
</body>
</html>
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>GASTENBOEK</title>
<script language='javascript' type='text/javascript'>
function emoticon(text) {
text = ' ' + text + ' ';
if (document.book.message.createTextRange && document.book.message.caretPos) {
var caretPos = document.book.message.caretPos;
caretPos.text = caretPos.text.charAt(caretPos.text.length - 1) == ' ' ? text + ' ' : text;
document.book.message.focus();
} else {
document.book.message.value += text;
document.book.message.focus();
}
}
</script>
</head>
<body>
<?php
include "connect.php";
$actie=$_GET['actie'] OR $actie=$_POST['actie'];
if ($actie == 'teken')
{
?>
<center>
<form action='gastenboek.php?actie=toevoegen' method='post' name="book">
<table border='1' cellspacing='0' width="600">
<tr>
<td>
Naam:
</td>
<td>
<input type='tekst' name='naam' size='20'>
</td>
</tr>
<tr>
<td>
Email:
</td>
<td>
<input type='tekst' name='email' size='20'>
</td>
</tr>
</table>
<table border='1' cellspacing='0' width="600">
<tr>
<td>
<textarea name='message' rows='10' cols='71'></textarea>
</td>
</tr>
<tr>
<table width='600'>
<tr>
<td>
<img src="images/smilies/icon4.gif" border="0" onclick="javascript:emoticon(':p')">
</td>
<td>
<img src="images/smilies/icon1.gif" border="0" onclick="javascript:emoticon(':)')">
</td>
<td>
<img src="images/smilies/icon10.gif" border="0" onclick="javascript:emoticon('(H)')">
</td>
<td>
<img src="images/smilies/biggrin.gif" border="0" onclick="javascript:emoticon(':D')">
</td>
<td>
<img src="images/smilies/finger.gif" border="0" onclick="javascript:emoticon('(m)')">
</td>
<td>
<img src="images/smilies/bored.gif" border='0' onclick="javascript:emoticon(':S')">
</td>
<td>
<img src="images/smilies/mad.gif" border='0' onclick="javascript:emoticon(':@')">
</td>
<td>
<img src="images/smilies/smiley3.gif" border='0' onclick="javascript:emoticon(':-O')">
</td>
<td>
<img src="images/smilies/LOL.gif" border='0' onclick="javascript:emoticon('(LOL)')">
</td>
<td>
<img src="images/smilies/tong.gif" border='0' onclick="javascript:emoticon('P:')">
</td>
</tr>
</table>
</tr>
<tr>
<td align='center'>
<input type='submit' name='actie' value='toevoegen'>
</td>
</tr>
</table>
</form>
</center>
<?php
}
else if ($actie =='toevoegen')
{
$naam=$_POST['naam'];
$naam = ucfirst(strtolower($_POST['naam']));
$email=$_POST['email'];
$reactie=$_POST['message'];
if ($naam == '' OR $email == '' OR $reactie =='')
{
print 'Je moet wel alles ingevullen';
}
else if($naam == 'Admin' OR $naam =='Webmaster' OR $naam =='Administrator' OR $naam =='Moderator' )
{
echo "probeer je niet voor te doen als iemand die je niet bent";
}
else
{
//naam
$naam = htmlspecialchars($naam);
$email = htmlspecialchars($email);
//reactie
$reactie = htmlspecialchars($reactie);
$reactie = nl2br ($reactie);
$reactie = str_replace(':p','<img src="images/smilies/icon4.gif">',$reactie);
$reactie = str_replace(':)','<img src="images/smilies/icon1.gif">',$reactie);
$reactie = str_replace('(H)','<img src="images/smilies/icon10.gif">',$reactie);
$reactie = str_replace(':D','<img src="images/smilies/biggrin.gif">',$reactie);
$reactie = str_replace('(m)','<img src="images/smilies/finger.gif">',$reactie);
$reactie = str_replace(':S','<img src="images/smilies/bored.gif">',$reactie);
$reactie = str_replace(':@','<img src="images/smilies/mad.gif">',$reactie);
$reactie = str_replace(':-O','<img src="images/smilies/smiley3.gif">',$reactie);
$reactie = str_replace('(LOL)','<img src="images/smilies/LOL.gif">',$reactie);
$reactie = str_replace('P:','<img src="images/smilies/tong.gif">',$reactie);
$reactie = str_replace('(m)','<img src="images/smilies/finger.gif">',$reactie);
// scheldwoorden er uit filteren
$reactie = str_replace('tering','*', $reactie);
$reactie = str_replace('kanker','*', $reactie);
$reactie = str_replace('kenker','*', $reactie);
$reactie = str_replace('vagina','*', $reactie);
$reactie = str_replace('anaal','*', $reactie);
$reactie = str_replace('tyfus','*', $reactie);
$reactie = str_replace('tievus','*', $reactie);
$reactie = str_replace('godverdomme','*', $reactie);
$reactie = str_replace('kut','*', $reactie);
$reactie = str_replace('lul','*', $reactie);
$reactie = str_replace('flikker','*', $reactie);
$reactie = str_replace('homo','*', $reactie);
$reactie = str_replace('nicht','*', $reactie);
$reactie = str_replace('likker','*', $reactie);
$reactie = str_replace('suck','*', $reactie);
$reactie = str_replace('fuck','*', $reactie);
$reactie = str_replace('zuigen','*', $reactie);
$reactie = str_replace('zuige','*', $reactie);
$reactie = str_replace('sukkel','*', $reactie);
$reactie = str_replace('lelijk','*', $reactie);
$reactie = str_replace('trut','*', $reactie);
$reactie = str_replace('new holland','*', $reactie);
$reactie = str_replace('ford','*', $reactie);
$reactie = str_replace('fend','*', $reactie);
//alles toevoegen
$sql = ("INSERT INTO gastenboek (id, naam, reactie, email) VALUES ('', '$naam', '$reactie', '$email')");
$result = mysql_query ($sql) or die (mysql_error());
echo"
je bericht is toegevoegd.<br>
klik <a href='gastenboek.php'>hier</a> om je bericht te bekijken.";
}
}
else
{
?>
<div id="center">
<center>
<table border='1' cellspacing='0' width="600">
<tr>
<td>
WELKOM BIJ DIT GASTENBOEK
</td>
</tr>
<tr>
<td>
<a href='gastenboek.php?actie=teken'>klik hier om iets in het gastenboek te schrijven</a>
</td>
</tr>
</table>
<table border='1' cellspacing='0' width="600">
<?php
$result = mysql_query ("SELECT * FROM gastenboek order by id desc")or die (mysql_error());
while ($array = mysql_fetch_array($result))
{
echo "
<tr>
<td width='50'>
<a href='mailto:$array[email]'>$array[naam]</a>
</td>
<td width='550'>
$array[reactie]";
echo"
</td>
</tr>
";
}
?>
</center>
</div>
<?php
}
?>
</body>
</html>
En is het helemaal zelfgemaakt?
Dat vraag ik me ook af ja...
ja
En wat als ik bijv. zou zeggen: Wat een lelijke site?.
Verder vind ik de code nei echt netjes:
- Variabelen buiten quotes
- Missende controles op functies en acties..