php jquery foto systeem, verbeteren van code
Ik probeer een galerij in jquery te maken met behulp van php.
Ik heb het als volgt:
Ik roep aan:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
2
3
4
5
6
7
8
9
10
11
12
<?php
$data1 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '0 ,12');
$data2 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '12 ,24');
$data3 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '24 ,36');
$data4 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '36 ,48');
$data5 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '48 ,50');
$data6 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '50 ,62');
$data7 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '62 ,74');
$data8 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '74 ,86');
$data9 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '86 ,98');
$data10 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '98 ,100');
?>
$data1 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '0 ,12');
$data2 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '12 ,24');
$data3 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '24 ,36');
$data4 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '36 ,48');
$data5 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '48 ,50');
$data6 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '50 ,62');
$data7 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '62 ,74');
$data8 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '74 ,86');
$data9 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '86 ,98');
$data10 = verkrijg_informatie('*', 'media', 'ORDER BY id DESC ', '98 ,100');
?>
Ik maak een lijst met pagina nummers
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
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
<ul id="flowtabs" class="navi">
<?php
if ($data1 =='') {echo " ";}
else {?> <li><a id='t1' class="bg_left" href='#'>1</a></li> <?php }
if ($data2 =='') {echo " ";}
else {?>
<li><a <?php
if ($data3 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>2</a></li> <?php }
if ($data3 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data4 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>3</a></li> <?php }
if ($data4 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data5 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>4</a></li> <?php }
if ($data5 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data6 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>5</a></li> <?php }
if ($data6 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data7 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>6</a></li> <?php }
if ($data7 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data8 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>7</a></li> <?php }
if ($data8 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data9 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>8</a></li> <?php }
if ($data9 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data10 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>9</a></li> <?php }
if ($data10 =='') {echo " ";}
else { ?>
<li><a class="bg_right" href='#'>10</a></li> <?php }
?>
<?php
if ($data1 =='') {echo " ";}
else {?> <li><a id='t1' class="bg_left" href='#'>1</a></li> <?php }
if ($data2 =='') {echo " ";}
else {?>
<li><a <?php
if ($data3 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>2</a></li> <?php }
if ($data3 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data4 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>3</a></li> <?php }
if ($data4 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data5 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>4</a></li> <?php }
if ($data5 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data6 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>5</a></li> <?php }
if ($data6 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data7 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>6</a></li> <?php }
if ($data7 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data8 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>7</a></li> <?php }
if ($data8 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data9 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>8</a></li> <?php }
if ($data9 =='') {echo " ";}
else { ?>
<li><a <?php
if ($data10 =='')
echo"class='bg_right'";
else
{ echo "class='bg_center' "; }; ?> href='#'>9</a></li> <?php }
if ($data10 =='') {echo " ";}
else { ?>
<li><a class="bg_right" href='#'>10</a></li> <?php }
?>
En vervolgens heb ik de volgende code er 10x in staan:
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
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
<?php
if ($data1 == '')
{echo ' er staat niets in data1';}
else {
?>
<div>
<ul class="thumb">
<?php
db_connect();
//And we display the results
for ($i=0; $i<count($data1); $i++)
{
?>
<li><a href="index.php?pagina=mediafull&id=<?php echo $data1[$i]['id'] ?>"><img src="<?php echo $url2.$data1[$i]['titel'] ?>" alt="" /></a></li>
<?php
}
?>
</ul>
<br clear="all" />
</div>
<?php
};
?>
if ($data1 == '')
{echo ' er staat niets in data1';}
else {
?>
<div>
<ul class="thumb">
<?php
db_connect();
//And we display the results
for ($i=0; $i<count($data1); $i++)
{
?>
<li><a href="index.php?pagina=mediafull&id=<?php echo $data1[$i]['id'] ?>"><img src="<?php echo $url2.$data1[$i]['titel'] ?>" alt="" /></a></li>
<?php
}
?>
</ul>
<br clear="all" />
</div>
<?php
};
?>
Dit is natuurlijk niet echt netjes, maar het werkt. Ik heb geprobeerd het zo te maken dat hij zelf na iedere 12 afbeeldingen een nieuw tabblad aanmaakt en een tabnummering, maar helaas dit lukt mij niet.
Is er een mogelijkheid om dit te verbeteren?
Aya
Waarom zet je een semi-colon achter je accolade? das vaag.
Als je ziet dat je herhalende code hebt, moet je daar iets aan doen. Dat is niet alleen kut qua onderhoud, maar ook kut qua performance, leesbaarheid en gewoon not-done.
Een oplossing daarvoor is een zogenoemde functie. Maar als je het helemaal goed wil doen moet je eens naar OOP kijken ;) Maar dat zal wel te lastig zijn(?) nu.
Dergelijke "herhalingen" moet je echt vermijden.
Klik
thanks
Ik heb de code wat netter gemaakt. Deze 2 variabelen moeten er bij elk deel in komen te staan (Let op dat ze niet 2x aangemaakt worden):
Code (php)
1
2
3
4
2
3
4
<?php
$limit_per_pagina = 12;
$maxPaginas = 10;
?>
$limit_per_pagina = 12;
$maxPaginas = 10;
?>
Dit is deel 1 van de code:
Code (php)
Deel 2:
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
Deel 3:
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
for($j = 0; $j < $maxPaginas; $j++){
if ($data[$j] == ''){
echo ' er staat niets in data['.$j.']';
} else {
echo "<div><ul class='thumb'>";
for ($i=0; $i<count($data[$j]); $i++) {
echo '<li><a href="index.php?pagina=mediafull&id='.$data[$j][$i]['id'].'"><img src="'.$url2.$data[$j][$i]['titel'].'" alt="" /></a></li>';
}
echo "</ul><br clear='all' /></div>";
}
}
?>
for($j = 0; $j < $maxPaginas; $j++){
if ($data[$j] == ''){
echo ' er staat niets in data['.$j.']';
} else {
echo "<div><ul class='thumb'>";
for ($i=0; $i<count($data[$j]); $i++) {
echo '<li><a href="index.php?pagina=mediafull&id='.$data[$j][$i]['id'].'"><img src="'.$url2.$data[$j][$i]['titel'].'" alt="" /></a></li>';
}
echo "</ul><br clear='all' /></div>";
}
}
?>
Gewijzigd op 22/09/2010 10:19:13 door Victor Koenders
Victor Koenders op 22/09/2010 10:00:42:
Hey niek, als je hier niet bent om antwoorden te geven ga dan lekker naar je elitest prick php siteje toe
[...]
[...]
Oh, ben ik al de hele tijd naar op zoek.
Link?
Victor Koenders op 22/09/2010 10:11:05:
Wjoow thanks man! Ik raad het je meteen aan, zijn we ook weer van jou af! :D
Niek s op 22/09/2010 10:13:12:
...
Wjoow thanks man! Ik raad het je meteen aan, zijn we ook weer van jou af! :D
Wjoow thanks man! Ik raad het je meteen aan, zijn we ook weer van jou af! :D
want ik ben hier om mensen af te zeiken en niet om mensen te helpen /sarcasm off
En, waarom gebruik je backticks in die query?
Chi Lion op 22/09/2010 10:54:06:
Victor, wtf, houdt je klep en ga gewoon even zelf naar die site.
En, waarom gebruik je backticks in die query?
En, waarom gebruik je backticks in die query?
Ik ben hier iig om mensen te helpen. Posts als "je code is slecht hier heb je een link van google ga maar php basis dingen leren" zijn niet bijzonder nuttig in mijn ogen.
en backticks in die query is macht der gewoonte. backticks voor tables en kolommen en enkele aanhalingstekens voor variabelen. Zo heb ik mysql geleerd.
Dat is de bedoeling hier, dat je er ook zelf wat tijd in steekt ipv hier heb je het kant en klaar.
Victor Koenders op 22/09/2010 10:57:27:
Ik ben hier iig om mensen te helpen. Posts als "je code is slecht hier heb je een link van google ga maar php basis dingen leren" zijn niet bijzonder nuttig in mijn ogen.
en backticks in die query is macht der gewoonte. backticks voor tables en kolommen en enkele aanhalingstekens voor variabelen. Zo heb ik mysql geleerd.
Chi Lion op 22/09/2010 10:54:06:
Victor, wtf, houdt je klep en ga gewoon even zelf naar die site.
En, waarom gebruik je backticks in die query?
En, waarom gebruik je backticks in die query?
Ik ben hier iig om mensen te helpen. Posts als "je code is slecht hier heb je een link van google ga maar php basis dingen leren" zijn niet bijzonder nuttig in mijn ogen.
en backticks in die query is macht der gewoonte. backticks voor tables en kolommen en enkele aanhalingstekens voor variabelen. Zo heb ik mysql geleerd.
Door mensen het verkeerde te leren, kom je ook niet erg ver. Uiteraard is het de bedoeling dat de mensen hier elkaar helpen, begrijp me niet verkeerd. Maar backticks voor tabellen en kolommen, is totaal niet de bedoeling. Als je die nodig hebt omdat anders je queries niet kloppen, betekend het dat je tabel- of kolom-namen niet kloppen. Lees de mysql beginnershandleiding van Blanche eens door, goed leesvoer, voor iedereen overigens ;-)