mag dit?
Zoniet hoe ik het dan had moeten doen?
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
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
<?php
include("config.php");
include("safe.php");
include("htmltop.php");
if(isset($_SESSION['user_id'])) {
$sql = "UPDATE gebruikers SET lastactive=NOW() WHERE id='".$_SESSION['user_id']."'";
mysql_query($sql);
if($_SESSION['user_status'] == 1) {
print <<<ENDHTML
<a href="admin.php">admin</a><br>
<a href="useropties.php">instelingen</a><br>
<a href="lidonline.php">leden online</a><br>
<a href="uitloggen.php">uitloggen</a><br>
ENDHTML;
}else{
print <<<ENDHTML
<a href="useropties.php">instelingen</a><br>
<a href="lidonline.php">leden online</a><br>
<a href="uitloggen.php">uitloggen</a><br>
ENDHTML;
}
}
include("htmlbottom.php");
?>
include("config.php");
include("safe.php");
include("htmltop.php");
if(isset($_SESSION['user_id'])) {
$sql = "UPDATE gebruikers SET lastactive=NOW() WHERE id='".$_SESSION['user_id']."'";
mysql_query($sql);
if($_SESSION['user_status'] == 1) {
print <<<ENDHTML
<a href="admin.php">admin</a><br>
<a href="useropties.php">instelingen</a><br>
<a href="lidonline.php">leden online</a><br>
<a href="uitloggen.php">uitloggen</a><br>
ENDHTML;
}else{
print <<<ENDHTML
<a href="useropties.php">instelingen</a><br>
<a href="lidonline.php">leden online</a><br>
<a href="uitloggen.php">uitloggen</a><br>
ENDHTML;
}
}
include("htmlbottom.php");
?>
Verder, waarom gebruik je endhtml? Ik ken dit niet echt, maar ik zou gewoon php afsluiten en weer openen door middel van de php tags.
Als laatste zou ik wel gaan inspringen met je code (pear stijl)
Verbeter "instelingen" ook maar naar "instellingen" ;)
Roy schreef op 22.06.2008 16:20:
Verbeter "instelingen" ook maar naar "instellingen" ;)
Ik ben dislecties sorry daar voor maar ik doe mijn best.
Stefan van Iwaarden schreef op 22.06.2008 16:18:
je query is gevoelig voor mysql injection, pas hiermee op! je moet altijd de waarde van je variabele controleren.
Verder, waarom gebruik je endhtml? Ik ken dit niet echt, maar ik zou gewoon php afsluiten en weer openen door middel van de php tags.
Als laatste zou ik wel gaan inspringen met je code (pear stijl)
Verder, waarom gebruik je endhtml? Ik ken dit niet echt, maar ik zou gewoon php afsluiten en weer openen door middel van de php tags.
Als laatste zou ik wel gaan inspringen met je code (pear stijl)
Zou je een voorbeeld kunnen geven?
Ik ben nog niet zoheel goed met php & MySql.
Moet nog een beetje leren de boel vijlig te maken.
In jouw geval zou je dus zoiets krijgen:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
if(isset($_SESSION['user_id']))
{
if (is_numeric($_SESSION['user_id']))
{
$sql = "UPDATE gebruikers SET lastactive=NOW() WHERE id= '".$_SESSION['user_id']."'";
mysql_query($sql);
}
else
{
echo 'Er is een fout opgetreden bij het raadplegen van de database';
}
}
?>
if(isset($_SESSION['user_id']))
{
if (is_numeric($_SESSION['user_id']))
{
$sql = "UPDATE gebruikers SET lastactive=NOW() WHERE id= '".$_SESSION['user_id']."'";
mysql_query($sql);
}
else
{
echo 'Er is een fout opgetreden bij het raadplegen van de database';
}
}
?>
Hierboven zie je dus ook gelijk hoe je moet inspringen.
Gewijzigd op 01/01/1970 01:00:00 door Stefan van Iwaarden
Ja, maar dat doe je dus niet met is_numeric maar met ctype_digit, het lijkt mij dat het userId een integer is.......
Stefan van Iwaarden schreef op 22.06.2008 16:18:
(...)
Verder, waarom gebruik je endhtml? Ik ken dit niet echt, maar ik zou gewoon php afsluiten en weer openen door middel van de php tags.
(...)
Verder, waarom gebruik je endhtml? Ik ken dit niet echt, maar ik zou gewoon php afsluiten en weer openen door middel van de php tags.
(...)
Of gewoon echo'en, dat blijft het mooist.
Quote:
Nee! Je mag niet zo'n topictitel maken, dat je dat nog durft te vragen! …mag dit
DAT MAG NIET
Marcel schreef op 22.06.2008 17:34:
DAT MAG NIET
Quote:
Nee! Je mag niet zo'n topictitel maken, dat je dat nog durft te vragen! …mag dit
DAT MAG NIET
Ha..ha..ha..tsjoe?
Verder heredoc, ja eeuh. Doe maar niet nee. Is onhandig, en moet heel strict newline, ending delimiter, punt-komma, newline zijn. Is niet handig en anders krijg je een syntax error op de laatste regel. pff
Echt, totaal niks mis mee *kuch*
Maar goed,
Voor de rest zou ik geen ENDHTML gebruiken, maar gewoon even php afsluiten, en daarna weer openen.
En voor de rest is het inspringen niet netjes.
Wil je weten hoe je code der uit moet komen te zien?
Voer je code dan even hier in, en dan zie je hoe hij eruit hoort te zien
*gezondheid*
Marcel, bespaar ons je bijdehante opmerkingen alsjeblieft. Daar heb je net iets te weinig posts voor ;)
Waarom mag iemand met "veel posts" wel bijdehante opmerkingen maken? Doe eens even normaal ofzo, en ga niet doen alsof je beter bent omdat er een groter nummertje bij je staat. Dat is kinderachtig, irritant en kinderachtig.
Marcel is mijn mattie.
En er is er hier maar een die echt
Quote:
kinderachtig, irritant en kinderachtig.
is, En dat ben jij momenteel.
Als je gewoon lekker iedereen z'n gang beetje laat gaan, en niet bij iedere post beetje stoer gaan lopen doen, van ahh dat mag niet en dit niet (en blablabla.
Je mag er best een opmerking over maken, maar voor de rest hebben wij hier een paar hele beste mods rondlopen die dit heel erg goed doen (vind ik zelf dan hea).
Voor de rest nog een pretige avond, en tot morgen ;)
Waarom allemaal zo serieus?! Dat doe ik ook nooit!
edit: daarom heb ik ook je reactie hieronder verwijderd, boris
Gewijzigd op 01/01/1970 01:00:00 door Terence Hersbach
Nu ben ik zat, slotje erop!