Tutorial Cms System V1

Door TreX , 17 jaar geleden, 5.360x bekeken

Hoi iedereen dit is mijn eerste script die ik geschreven heb dus als er fouten zijn pleas corigeer ze uhm bij v2 ga ik dan nog eens proberen ubb codes enz toetevoegen dus mense die willen helpen altijd welkom
voor de luie mense :D
Download : http://83.149.84.72/~gangstawar/tut.rar

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

Voorbeeld: http://83.149.84.72/~gangstawar/index.php

Gesponsorde koppelingen

PHP script bestanden

  1. tutorial-cms-system-v1

 

Er zijn 13 reacties op 'Tutorial cms system v1'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Hipska BE
Hipska BE
17 jaar geleden
 
0 +1 -0 -1
Kun je in je toelichting nog even vermelden wat het script doet?
TreX
TreX
17 jaar geleden
 
0 +1 -0 -1
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
Hipska BE
Hipska BE
17 jaar geleden
 
0 +1 -0 -1
Dit klopt niet denk ik (veronderstel dat het op zijn minst php4 is):
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
PHPversie: V1

Dit wordt tegenwoordig ook anders gedaan: (request_method == POST)
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
if ($_POST['submit']){

Je doet geen input controle (mysql_real_escape_string bv)
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
$title = $_POST['frmtitle'];
$author = $_POST['frmauthor'];

Waarom neem je hier geen DATETIME of iets dergerlijks?
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
`date` varchar(50) NOT NULL,


Kortom, er is nog veel aan te verbeteren.
TreX
TreX
17 jaar geleden
 
0 +1 -0 -1
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
Hipska BE
Hipska BE
17 jaar geleden
 
0 +1 -0 -1
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.
Edit:
Hier heb je je lijn nummers waar de code beter kan:
add.php 11,12,13,14,15,16
dell.php 3,20
edit.php 3,46,47,48,49,50,51
tutorial.php 3
Ultimatum
ultimatum
17 jaar geleden
 
0 +1 -0 -1
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 :]
TreX
TreX
17 jaar geleden
 
0 +1 -0 -1
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
Ultimatum
ultimatum
17 jaar geleden
 
0 +1 -0 -1
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..
Geert
Geert
17 jaar geleden
 
0 +1 -0 -1
@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) ;)
Frank -
Frank -
17 jaar geleden
 
0 +1 -0 -1
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
`date` varchar(50) NOT NULL,
[/quote]
Een datum sla je op in een kolom van het type DATE. De kolomnaam 'date' is niet zo gelukkig gekozen, dan kan verwarrend zijn met het datatype DATE. Het wordt overigens geaccepteerd, deze fout wordt z vaak gemaakt dat de database er niet over struikeld.

Wanneer je dit datatype hebt aangepast, zul je de datum ook in het juiste formaat (ISO-formaat) moeten opslaan: yyyy-mm-dd, bv. 2007-08-12.

Wanneer je deze datum vervolgens met een SELECT-query weer ophaalt en op een andere manier wilt weergeven, dan kun je bv. de MySQL-functie DATE_FORMAT() gebruiken i.c.m. met een alias. Zie bv. de tutorials (Jan en Blanche hebben er fraaie tuts over geschreven) of de MySQL-handleiding.
W W
W W
17 jaar geleden
 
0 +1 -0 -1
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.
Chris
Chris
17 jaar geleden
 
0 +1 -0 -1
@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 ;)
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Toby hinloopen
toby hinloopen
15 jaar geleden
 
0 +1 -0 -1
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.

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

Inhoudsopgave

  1. tutorial-cms-system-v1

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.