Scriptlibrary met Categorieeen +Admin

Door Geert-Jan , 17 jaar geleden, 2.970x 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)
PHP script in nieuw venster Selecteer het PHP script
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
--
-- 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

  1. scriptlibrary-met-categorieeen-admin

 

Er zijn 14 reacties op 'Scriptlibrary met categorieeen admin'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Robert Deiman
Robert Deiman
17 jaar geleden
 
0 +1 -0 -1
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:

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
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!
        }
?>


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.
Dennis
Dennis
17 jaar geleden
 
0 +1 -0 -1
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

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
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]
Frank -
Frank -
17 jaar geleden
 
0 +1 -0 -1
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().
Geert-Jan
Geert-Jan
17 jaar geleden
 
0 +1 -0 -1
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 ;)
Rudie dirkx
rudie dirkx
17 jaar geleden
 
0 +1 -0 -1
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.


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.
Mitchelp
Mitchelp
17 jaar geleden
 
0 +1 -0 -1
Ik heb hem ge upload alles goed gezet en een database gemaakt:
In de config.php heb ik dit zelf erbij moeten zetten:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
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 :)
Onbekend Onbekend
Onbekend Onbekend
17 jaar geleden
 
0 +1 -0 -1
Waarom zou je zelf een script lib maken? PHP Hulp 4 life!
Geert-Jan
Geert-Jan
17 jaar geleden
 
0 +1 -0 -1
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 :)

Gedaan !
Mitchelp
Mitchelp
17 jaar geleden
 
0 +1 -0 -1
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 :)

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 :)
Geert-Jan
Geert-Jan
17 jaar geleden
 
0 +1 -0 -1
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 ;)
Mitchelp
Mitchelp
17 jaar geleden
 
0 +1 -0 -1
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
Roel -
Roel -
17 jaar geleden
 
0 +1 -0 -1
Goed script, mooie programmering!
Bas
bas
16 jaar geleden
 
0 +1 -0 -1
ik vind het in de verkeerde cat staan
dit is geen beginners scripts vind ik
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Tikkes C
Tikkes C
16 jaar geleden
 
0 +1 -0 -1
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 ;)

Om te reageren heb je een account nodig en je moet ingelogd zijn.

Inhoudsopgave

  1. scriptlibrary-met-categorieeen-admin

Labels

  • Geen tags toegevoegd.

Navigatie

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.