Scriptlibrary met Categorieeen +Admin
Door Geert-Jan , 20 jaar geleden, 4.335x bekeken
Dit is mijn eerste script hier op Phphulp !
Dit is een scriptlibrary die ik gebruik voor mijn site
Je moet het wel even aanpassen naar je eigen ledensysteem
Letop admin is niet beveiligd !
De admin is niet uitgebreid !
Je moet wel eerst config.php aanpassen
En de tabellen in je database uitvoeren :
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
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
--
-- Tabel structuur voor tabel `cat`
--
CREATE TABLE `cat` (
`id` int(11) NOT NULL auto_increment,
`naam` varchar(200) NOT NULL,
UNIQUE KEY `id` (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1 AUTO_INCREMENT= 0;
--
-- Gegevens worden uitgevoerd voor tabel `cat`
--
-- --------------------------------------------------------
--
-- Tabel structuur voor tabel `script`
--
CREATE TABLE `script` (
`id` int(11) NOT NULL auto_increment,
`auteur` varchar(200) NOT NULL,
`scriptnaam` varchar(200) NOT NULL,
`cat` int(2) NOT NULL,
`uitleg` longtext NOT NULL,
`source` longtext NOT NULL,
UNIQUE KEY `id` (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1 AUTO_INCREMENT=0 ;
--
-- Gegevens worden uitgevoerd voor tabel `script`
--
-- Tabel structuur voor tabel `cat`
--
CREATE TABLE `cat` (
`id` int(11) NOT NULL auto_increment,
`naam` varchar(200) NOT NULL,
UNIQUE KEY `id` (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1 AUTO_INCREMENT= 0;
--
-- Gegevens worden uitgevoerd voor tabel `cat`
--
-- --------------------------------------------------------
--
-- Tabel structuur voor tabel `script`
--
CREATE TABLE `script` (
`id` int(11) NOT NULL auto_increment,
`auteur` varchar(200) NOT NULL,
`scriptnaam` varchar(200) NOT NULL,
`cat` int(2) NOT NULL,
`uitleg` longtext NOT NULL,
`source` longtext NOT NULL,
UNIQUE KEY `id` (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1 AUTO_INCREMENT=0 ;
--
-- Gegevens worden uitgevoerd voor tabel `script`
--
Gesponsorde koppelingen
PHP script bestanden
Er zijn 14 reacties op 'Scriptlibrary met categorieeen admin'
0 
-0 
-0
Paar dingen die me zo opvallen:
- Je controleert niet bij elke query of ze gelukt zijn (de ene keer doe je het wel en de andere keer niet), doe het bijvoorbeeld zo (bijv bij de 1e insert query), en gebruik dan geen die meer:
Daarnaast ben ik er geen fan van dat je error_reporting op 0 zet. Onderdrukken van fouten is nooit goed! Probeer ze eruit te werken.
Verder kan je nog even kijken naar formulier opmaken via CSS en niet in tabellen. klik voor een tut
Verder wil ik nog wel even zeggen dat je wel overzichtelijk hebt gescript en dat het wel logisch in elkaar zit. Dat is vaak anders.
- Je controleert niet bij elke query of ze gelukt zijn (de ene keer doe je het wel en de andere keer niet), doe het bijvoorbeeld zo (bijv bij de 1e insert query), en gebruik dan geen die meer:
Code (php)
1
2
3
4
5
6
7
8
9
2
3
4
5
6
7
8
9
<?php
if($result = mysql_query($sql)){
while($row = mysql_fetch_assoc($result)){
}
}
else{
trigger_error(mysql_error()); //niet gebruiken als script online is, omdat er veel informatie uit een melding kan worden gehaald!
}
?>
if($result = mysql_query($sql)){
while($row = mysql_fetch_assoc($result)){
}
}
else{
trigger_error(mysql_error()); //niet gebruiken als script online is, omdat er veel informatie uit een melding kan worden gehaald!
}
?>
Daarnaast ben ik er geen fan van dat je error_reporting op 0 zet. Onderdrukken van fouten is nooit goed! Probeer ze eruit te werken.
Verder kan je nog even kijken naar formulier opmaken via CSS en niet in tabellen. klik voor een tut
Verder wil ik nog wel even zeggen dat je wel overzichtelijk hebt gescript en dat het wel logisch in elkaar zit. Dat is vaak anders.
0 -0
-0
wat mij opvalt is dat je telkens als je een nieuwe pagina code weergeeft reageer.php bijf hij blijft hangen in het systeem
de eerste keer dat ik het bekeek dacht ik dat het 1 geheel was
graag aanpassen
de eerste keer dat ik het bekeek dacht ik dat het 1 geheel was
graag aanpassen
Code (php)
1
2
3
4
5
6
7
8
9
10
2
3
4
5
6
7
8
9
10
reageer.php
<div align="center">
<?php
include("config.php"); // DB connectie include
$sql = "SELECT naam FROM gebruikers WHERE id='".$_SESSION['user_id']."'";
$query = mysql_query($sql);
$rij = mysql_fetch_object($query);
$naam = htmlspecialchars($rij->naam);
[code]
<div align="center">
<?php
include("config.php"); // DB connectie include
$sql = "SELECT naam FROM gebruikers WHERE id='".$_SESSION['user_id']."'";
$query = mysql_query($sql);
$rij = mysql_fetch_object($query);
$naam = htmlspecialchars($rij->naam);
[code]
0 -0
-0
Maak nog even een foreignkey aan tussen beide tabellen, dat zorgt voor het verband tussen de data. Kan je een hoop ellende schelen.
Verder gebruik je nooit meer mysql_fetch_object(), dat heeft niks met objecten te maken en is gruwelijk langzaam t.o.v. mysql_fetch_assoc().
En waarom gebruik je geen mysql_real_escape_string().
Verder gebruik je nooit meer mysql_fetch_object(), dat heeft niks met objecten te maken en is gruwelijk langzaam t.o.v. mysql_fetch_assoc().
En waarom gebruik je geen mysql_real_escape_string().
0 -0
-0
Het script werkt nog niet omdat ik ben vergeten mysql_real_escape_string().
Er in te zetten ik zal vanavond een aanpassing maken in het script ;)
Er in te zetten ik zal vanavond een aanpassing maken in het script ;)
0 -0
-0
Franks opmerking en nog iets meer:
Je kan niet (nouja, alles kan, maar toch) de FK in tabel script int(2) maken, en de PK in tabel cat int(11). Het kan dus wel, maar slaat absoluut nergens op... Denk er over na. Waarom moeten er 2 miljard (int(11) in cat) categorieen kunnen bestaan, als je maar in 100 (int(2) in script) scripts kan stoppen?
Een vriendelijke tip: Maak ELKE PK en FK int(11) UNISGNED. Dan past er altijd genoeg en overal evenveel in. Is makkelijk.
Vind het ook raar dat je nederlands en engels door elkaar gebruikt (ik kijk alleen naar je SQL, heb nog geen woord PHP gezien), maar is vooral cosmetisch en sowieso persoonlijk.
Verder zou ik de uitleg niet longtext maken (weet je hoe groot dat is??????) en misschien zelfs de source ook niet (zoek ff op google naar mysql longtext).
Ok Ok Ok ik zeg het wel. LONGTEXT ^= 2^32 bytes :) Hoeveel MB dat is? Pff te moeilijk, te vroeg. Maar daar kom je wel uit. Wel eens ooit zo'n groot script gezien?
Volgens mij is het rtouwens 4048 MB... maar kan het fout hebben.
Je kan niet (nouja, alles kan, maar toch) de FK in tabel script int(2) maken, en de PK in tabel cat int(11). Het kan dus wel, maar slaat absoluut nergens op... Denk er over na. Waarom moeten er 2 miljard (int(11) in cat) categorieen kunnen bestaan, als je maar in 100 (int(2) in script) scripts kan stoppen?
Een vriendelijke tip: Maak ELKE PK en FK int(11) UNISGNED. Dan past er altijd genoeg en overal evenveel in. Is makkelijk.
Vind het ook raar dat je nederlands en engels door elkaar gebruikt (ik kijk alleen naar je SQL, heb nog geen woord PHP gezien), maar is vooral cosmetisch en sowieso persoonlijk.
Verder zou ik de uitleg niet longtext maken (weet je hoe groot dat is??????) en misschien zelfs de source ook niet (zoek ff op google naar mysql longtext).
Ok Ok Ok ik zeg het wel. LONGTEXT ^= 2^32 bytes :) Hoeveel MB dat is? Pff te moeilijk, te vroeg. Maar daar kom je wel uit. Wel eens ooit zo'n groot script gezien?
Volgens mij is het rtouwens 4048 MB... maar kan het fout hebben.
Edit:
Voor de netheid zou ik variabelen buiten quotes halen, dus echo $var, ipv echo "$var". Scheelt de Zend Engine weer een hoop gekloot (een paar duizend keer zo'n ding in je script en je merkt het serieus).
Voor de netheid zou ik ook iets meer xhtml schrijven, dus <br /> ipv <BR>. Maar ik ben een echte xml fan, dus trek je daar vooral niets van aan.
Voor de netheid zou ik ook iets meer xhtml schrijven, dus <br /> ipv <BR>. Maar ik ben een echte xml fan, dus trek je daar vooral niets van aan.
0 -0
-0
Ik heb hem ge upload alles goed gezet en een database gemaakt:
In de config.php heb ik dit zelf erbij moeten zetten:
als dit hierboven er niet instaat gebeurt er ook vrij weinig met connecten maargoed.
En ik mis cat.php aangezien die word aageroepen wanneer je een Catogorie wilt bekijken!
Zet die er AUB nog even bij :)
In de config.php heb ik dit zelf erbij moeten zetten:
Code (php)
1
2
2
$connect = mysql_connect($db_host, $db_user, $db_pass) or die("Could not connect to server, error: ".mysql_error());
$db = mysql_select_db($db_database, $connect) or die("Could not connect to database, error: ".mysql_error());
$db = mysql_select_db($db_database, $connect) or die("Could not connect to database, error: ".mysql_error());
als dit hierboven er niet instaat gebeurt er ook vrij weinig met connecten maargoed.
En ik mis cat.php aangezien die word aageroepen wanneer je een Catogorie wilt bekijken!
Zet die er AUB nog even bij :)
0 -0
-0
Quote:
ik heb hem ge upload alles goed gezet en een database gemaakt:
In de config.php heb ik dit zelf erbij moeten zetten:
1
2
$connect = mysql_connect($db_host, $db_user, $db_pass) or die("Could not connect to server, error: ".mysql_error());
$db = mysql_select_db($db_database, $connect) or die("Could not connect to database, error: ".mysql_error());
als dit hierboven er niet instaat gebeurt er ook vrij weinig met connecten maargoed.
En ik mis cat.php aangezien die word aageroepen wanneer je een Catogorie wilt bekijken!
Zet die er AUB nog even bij :)
In de config.php heb ik dit zelf erbij moeten zetten:
1
2
$connect = mysql_connect($db_host, $db_user, $db_pass) or die("Could not connect to server, error: ".mysql_error());
$db = mysql_select_db($db_database, $connect) or die("Could not connect to database, error: ".mysql_error());
als dit hierboven er niet instaat gebeurt er ook vrij weinig met connecten maargoed.
En ik mis cat.php aangezien die word aageroepen wanneer je een Catogorie wilt bekijken!
Zet die er AUB nog even bij :)
Gedaan !
0 -0
-0
Volgens mij heb je hem rechtstreeks uit je leden systeem gehaald zonder de dingen die je leden systeem moet aansturen eruit gehaald te hebben BV:
Wanneer je een reactie wilt plaatsen krijg je eerst de volgende error:
Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in E:\xampp\htdocs\scripts\reageer.php on line 7
Geen Script ID meegegeven
Dat is zo opgelost dus ik heb het ID erin gezet:
en dan krijg je de volgende error:
Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in E:\xampp\htdocs\scripts\reageer.php on line 7
Fout bij ophalen van gebruikersnaam
Hij kan de gebruikersnaam niet ophallen..
Goed ik ga proberen de error's er zo veel moggelijk uit te hallen omdat het me wel een handig script lijkt!
Kan je misschien ook even een voorbeeld online gooien??
Thx alvast :)
Wanneer je een reactie wilt plaatsen krijg je eerst de volgende error:
Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in E:\xampp\htdocs\scripts\reageer.php on line 7
Geen Script ID meegegeven
Dat is zo opgelost dus ik heb het ID erin gezet:
en dan krijg je de volgende error:
Warning: mysql_fetch_object(): supplied argument is not a valid MySQL result resource in E:\xampp\htdocs\scripts\reageer.php on line 7
Fout bij ophalen van gebruikersnaam
Hij kan de gebruikersnaam niet ophallen..
Goed ik ga proberen de error's er zo veel moggelijk uit te hallen omdat het me wel een handig script lijkt!
Kan je misschien ook even een voorbeeld online gooien??
Thx alvast :)
Edit:
Ik heb het opgegeven aangezien er zo veel fouten in zitten en het gewoon rechtreeks uit een leden systeem is gehaald geef ik het op!!
Het zou erg leuk zijn als het script wel werkt :)
Ik heb het opgegeven aangezien er zo veel fouten in zitten en het gewoon rechtreeks uit een leden systeem is gehaald geef ik het op!!
Het zou erg leuk zijn als het script wel werkt :)
0 -0
-0
Maar het script werkt ook wel
Zoals ik heb aangegeven gebruik ik het voor mijn eigen ledensysteem
Ik had het inderdaad beter moeten aanpassen waar ik nu ook mee bezig ben ^^
Ik hoop dat je er dan wel wat aan hebt ;)
Zoals ik heb aangegeven gebruik ik het voor mijn eigen ledensysteem
Ik had het inderdaad beter moeten aanpassen waar ik nu ook mee bezig ben ^^
Ik hoop dat je er dan wel wat aan hebt ;)
0 -0
-0
Ik ben het nu zelf al aan het aanpassen als het klaar is zet ik hem zelf wel even opnieuw op PHPhulp en dan verwijs ik wel naar jou :)
ik ga de admin nammelijk uitbrijden en verbeteren :)
Greatz Mitchel
ik ga de admin nammelijk uitbrijden en verbeteren :)
Greatz Mitchel
0 -0
-0
ik vind het in de verkeerde cat staan
dit is geen beginners scripts vind ik
dit is geen beginners scripts vind ik
0 -0
-0
waarom niet dan?
@Geert-jan: Wel wat raar, in reageer.php script je heel "proper" maar in de andere scripts laat je dat wat afweten. Dit maakt het voor beginners moeilijk om goed te doorgronden en te weten wat jij nou juist doet...
verder, prima werk ;)
@Geert-jan: Wel wat raar, in reageer.php script je heel "proper" maar in de andere scripts laat je dat wat afweten. Dit maakt het voor beginners moeilijk om goed te doorgronden en te weten wat jij nou juist doet...
verder, prima werk ;)
Om te reageren heb je een account nodig en je moet ingelogd zijn.
PHP hulp
0 seconden vanaf nu