Tutorial Cms System V1

Kun je in je toelichting nog even vermelden wat het script doet?

Uhm het script is eigenlijk voor webmasters van een tutorial site je kan dus tutorials op je site toevoegen j kan ze dan naaderhand ook verwijderren en bewerken uhm die tutorials houd ook de views bij hoeveel keer het gezien is enz maar de ubb kan ik nog nie dus die zal voor versie 2 zijn

Dit klopt niet denk ik (veronderstel dat het op zijn minst php4 is):
Dit wordt tegenwoordig ook anders gedaan: (request_method == POST)
Je doet geen input controle (mysql_real_escape_string bv)
Waarom neem je hier geen DATETIME of iets dergerlijks?

Kortom, er is nog veel aan te verbeteren.

ja maar he ik ben ook nog een noob in php dus ik weet ook nie zo goe hoe alles werkt dus als je aub de lines + codes wil geven die ik moet veranderren pleas typ ze dan verander ik ze wel

ik heb je duidelijk dingen aangegeven wat er fout is en hoe je het beter kan doen.
Een beetje google en daarna een topic op phphulp doet wonderen.

Een paar opmerkingen:
1. gebruik $_GET ipv $_REQUEST
2. Als je die of exit gebruik stopt ook alles, dus ook je layout en zit je soms met een halve layout. Beter is gewoon een if/else
3. Je gebruikt in je tabel tinyint, deze kan niet meer dan 127 records opslaan. Gebruik int(11) daar heb je zat aan.
4. Vars buiten quotes (echo "Ik heet $naam"; => echo "Ik heet ". $naam;) en om een int (getal) geen quotes. (Dit doe je wel in add.php maar niet bij de rest??)
5. Echo wat minder html, minder parse tijd.
6. In dbconnect.php staat op regel 13 voor mysql_select_db() een punt. Gaat waarschijnlijk fouten opleveren?
7. In je tabel gebruik je heel veel text als veldtype. Author, Email kan je ook gewoon varchar(80) ofzo van maken, Email mss iets meer varchar(250). En views is altijd een getal dus int(11) is beter dan varchar.
8. Gebruik ook tabs, dat maakt het allemaal iets duidelijker dan wat je nu hebt.

Dit is wat ik zo even zag. En doe wat Hipska zei, want de dingen die hij zei heb ik hier niet vermeld omdat het dan dubbel op was :]

amai se gene ene kan hier iets positiefs zeggen leuk op te weten nogmaal ik ben een beginner ik ken amper sh*t van php dus ipv van te zegge wat ik verkeerd doe zeg dan in wat ik het moet veranderren je bent hier niet tegen een pro aan het praten :s

Plaats dan ook geen script als het aan alle kanten lekt. Ga gewoon php leren en kom dan over een jaar terug ofzo. En van fouten leer je het meest en ik heb toch gezet wat fout is en hoe je het kan oplossen?

Klinkt hard, maar als je nog niet genoeg kennis hebt moet je geen halve scripts posten en ook niet waar fouten inzitten want beginners kunnen het dan ok fout gaan leren.

Vind ik..

@TreX
ik vindt persoonlijk dat het nog wel meevalt met het aantal fouten. De meeste foutjes zijn foutjes die de meeste beginners maken (doe ik zelf ook wel eens... mijn eigen ingebakken foutjes :P ). Er zijn wel scripts gepost (ook op sites naast PHPhulp) waar ze zeggen dat ze gevorderde PHP'ers zijn, terwijl ze dingen doen waar een beginner niet eens aan zou durven denken (qua fouten e.d.). Zie het niet als negatieve berichten, maar zie het eens als allemaal serieuze berichten waarmee je op een makkelijke manier je eigen foutjes leert zien en zo kunt verbeteren.

P.s. je vraagt zelf trouwens of als mensen foutjes zien of ze die willen corrigeren... dus mensen doen dat voor je omdat jij dat graag wou (of wil) ;)

Ik snap het echt het nut van dit script. Als ik jouw voorbeeld alleen al bekijk raak ik in de war :S

Ik kan dit niet echt een CMS noemen.

@TreX, leuke poging, maar ga eerst naar de bibliotheek om een boek te halen die up-to-date(); is, vervolgens ga je eerst kleine scripts maken en als je dan dus wat van php afweet, herschrijf je dit alles ;)

Toch vind ik de kritiek wel beetje té: het is niet voor niets een 'beginnersscript'

Het staat er toch duidelijk bij. Verwacht dan ook geen pro-script.