login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Wat kan beter?

Offline ikki007 - 16/12/2006 12:28
Avatar van ikki007Gouden medailleGouden medaille

PHP ver gevorderde		 Ik gebruik op het moment deze code voor het registreren:

http://www.plaatscode.be/3471/

Mijn vraag is, is deze veilig? En wat kan beter/sneller/veiliger?

Ikki

15 antwoorden

Gesponsorde links
Offline Wim - 16/12/2006 12:32
Avatar van Wim Crew algemeen
php code - Bekijk de code zonder highlighting - Klap code in 
  1. #
  2. mkdir ("/home/wwwmponl/public_html/images/foto/" . $gebruikersnaam . "", 0755);
  3. #
  4. chmod ("/home/wwwmponl/public_html/images/foto/" . $gebruikersnaam . "", 0755);


waarom die chmod nog? Je geeft bij je mkdir al mee dat het een chmod 755 moet worden...!
Offline Stijn - 16/12/2006 12:40 (laatste wijziging 16/12/2006 12:45)
Avatar van Stijn PHP expert regel 6: if( $_SERVER['REQUEST_METHOD'] == 'POST' ) {
regel 8: $gebruikersnaam = addslashes($_POST['gebruikersnaam']);
dit geld voor alle array's die je gebruikt! altijd single quotes gebruiken!!!
$array[key] => verkeerd
$array['key'] => goed , omdat key een string is , bij integers kan je die quotes weglaten ( $array[0]; )
regel 38: is niet slecht maar ik zie liever dit:
(op regel 1: ob_start();)
header('Refresh: 2;URL=index.php');
regel 48-54: wtf je doet hetzelfde , wat is daar nu het nut van???? die if..else kan weg en je hoeft maar één insert query te doen
regel 68-74: natuurlijk html en php gescheiden houden voor de overzichtelijkheid
Offline vinTage - 16/12/2006 12:43
Avatar van vinTage Nieuw lid Je beveiligd geen enkele input..
Offline ikki007 - 16/12/2006 13:02
Avatar van ikki007 Gouden medailleGouden medaille

PHP ver gevorderde		 Hier de nieuwe: http://www.plaatscode.be/3472/

@jvriet, ik ben nog een functie aan het maken die ik dan voor elke input kan gebruiken ;)

@stijn: over dat html en php gescheiden, als ik php afsluit en dan een nieuwe blok begin lopen de variabelen uit het eerste blok door met het 2de? Dus

php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. $test = werkt
  3. ?>

<htmlmode>lalalla</htmlmode>
php code - Bekijk de code zonder highlighting - Klap code in 
  1. <?php
  2. echo "Kan ik hem dus hier gebruiken? $test";
  3. ?>
Offline Webtijn - 16/12/2006 13:47
Avatar van Webtijn PHP interesse Ja, dat werkt. Maar het is minder net.. ;)
Offline Stijn - 16/12/2006 13:49 (laatste wijziging 16/12/2006 13:51)
Avatar van Stijn PHP expert idd , daarom werk ik met templates 

lijn 48: $_SERVER['REMOTE_ADDR'] !! idem voor lijn 21 ['email']
Offline Webtijn - 16/12/2006 13:52 (laatste wijziging 16/12/2006 13:52)
Avatar van Webtijn PHP interesse [Verwijder]
Offline ikki007 - 16/12/2006 14:00 (laatste wijziging 16/12/2006 14:09)
Avatar van ikki007 Gouden medailleGouden medaille

PHP ver gevorderde		 http://www.plaatscode.be/3478/

Hoe is die nu ?
Offline dreamworld - 16/12/2006 14:19 (laatste wijziging 16/12/2006 14:20)
Avatar van dreamworld HTML beginner Je hebt een fout op regel 51 (kijk naar de kleurtjes..)  

EDIT: je had het al veranderd zie ik..
Offline ikki007 - 16/12/2006 15:27
Avatar van ikki007 Gouden medailleGouden medaille

PHP ver gevorderde		 Hij is nu super ge-update

http://www.plaatscode.be/3480/

en die cc_input functie is:

http://www.plaatscode.be/3481/
Offline stijnhau - 16/12/2006 18:04
Avatar van stijnhau Onbekend or die(mysql_error) kan je wel gebruiken.
en htmlspecialchars kan nog bij in die functie
Offline Gerard - 17/12/2006 04:10
Avatar van Gerard Ouwe rakker Waarom al die checks doorlopen met if - elseif.
Nu krijg je altijd maar 1 error terug terwijl ik kan zien aan de .= dat je juist wilt dat er meerdere errors worden teruggegeven.
Offline ikki007 - 17/12/2006 11:31
Avatar van ikki007 Gouden medailleGouden medaille

PHP ver gevorderde		 Ja klopt dat wilde ik ook ja, alleen ik snap niet hoe dit moet  
Offline Dark_Paul - 17/12/2006 11:57
Avatar van Dark_Paul PHP ver gevorderde Ipv elseifs zet je dan alleen if-statements neer.
Dus
if(dit) {}
if (dat) {}
if (zoiets) {}
zal 3x iets teruggeven, mits aan de voorwaarde voor de desbetreffende statement wordt voldaan.
Dit
if(dit) {}
elseif(dat) {}
elseif (zoiets) {}
zal maar één ding teruggeven. Als niet aan de voorwaarde van de eerste statement wordt voldaan, wordt pas naar de tweede statement gekeken. Als er dan ook niet aan de voorwaarde van de tweede statement wordt voldaan, kijkt 'ie naar de derde statement en zo door..
Offline stijnhau - 24/12/2006 11:33
Avatar van stijnhau Onbekend $_POST[email] dat is ook niet goed he.
$_POST['email'] zo moet het.
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.205s