Mag ik ook vragen waarom? Als het op je server aanstaat gaat het prima. En ik vind het er zelf erg netjes uit ziet. Wel vind ik dat je consequent moet zijn. OF je gebruikt shorttags. Of je gebruikt het andere. Niet beide.
- is dit goede php??
-
02-03-2010, 16:53 #16
- Berichten
- 173
- Lid sinds
- 17 Jaar
-
02-03-2010, 17:01 #17
- Berichten
- 2.392
- Lid sinds
- 17 Jaar
Ten tweede, omdat je er rekening mee moet houden dat het in toekomstige versies eruit gaat ;-)
-
02-03-2010, 17:34 #18
- Berichten
- 2.971
- Lid sinds
- 18 Jaar
Dit MD5en is onnodig gezien de bezoeker het toch al niet ziet.
-
02-03-2010, 20:51 #19
- Berichten
- 257
- Lid sinds
- 15 Jaar
Misschien ook niet helemaal handig, je hebt immers geen garantie dat een gebruiker elk request met hetzelfde ip-adres doet. Dus die zou, onterecht, bij elke wissel eruit geknikkerd worden.
-
02-03-2010, 20:53 #20
- Berichten
- 2.971
- Lid sinds
- 18 Jaar
Kies je dus voor veiligheid of klantvriendelijkheid.
(en hoeveel zullen er tijdens een browser sessie een ander ip krijgen, ik denk dat je dan grotere problemen hebt dan sessies ;-) )
-
02-03-2010, 21:01 #21
- Berichten
- 257
- Lid sinds
- 15 Jaar
Volgens mij kan dat best voorkomen wanneer gebruikers achter proxy-servers zitten. Dus toch beetje mee oppassen.
Je kunt je ook afvragen hoe groot de kans is dat een cookie met korte time-to-live, gehasht met salt, gekraakt wordt.
-
02-03-2010, 21:10 #22
- Berichten
- 2.971
- Lid sinds
- 18 Jaar
Maar gehasht met salt, mag jij me uitleggen wat het nut is om van een hash een hash te maken?
-
02-03-2010, 21:25 #23
- Berichten
- 257
- Lid sinds
- 15 Jaar
Kan ik niet uitleggen. Was even in de war met die checksum.
Overigens ga ik nu niet meer reageren. Ik denk dat dit de originele vraag van de poster niet meer echt beantwoordt.
-
03-03-2010, 07:39 #24
- Berichten
- 750
- Lid sinds
- 15 Jaar
@TS
Ja dat stukje PHP code is correct
Maar goed je gooit je zelf nu wel open voor cross site scripting (CSS of XSS afkorting) waarmee session riding mogelijk is, altijd htmlspecialchars gebruiken op informatie die van de client afkomt. Immers de client moet de username ingeven bij aanmelden.
Verdere opmerkingen over die gegevens correctheid geen hidden veld gebruiken is te makkelijk te
manipuleren. En ook geen cookie zoals Lars Kikkert voorstelt is namelijk ook te makkelijk te manipuleren.
Gebruik maken van een sessie die bij elk request een nieuwe session_id krijgt in combinatie met IP locking is redelijk safe
Maar het moraal van het verhaal, stuur geen gevoelige informatie naar de client die makkelijk aanpasbaar is, aangezien in jou'n geval, ik gewoon een "Admin" bericht kan plaatsen door het hidden veld aan te passen. Hou gevoelige informatie gewoon lekker in een sessie en pas op met gegevens van de client zie cross side scripting.
-
03-03-2010, 10:29 #25
Elephant Media GbR
- Berichten
- 1.253
- Lid sinds
- 18 Jaar
Ik probeer altijd in mijn codes zo weinig mogelijk html te gebruiken. Als je een apart template bestand maakt en deze laad, dan kun je werken met @naam@ of {naam}. zo kun je dus gewoon richten op het scripten, en heb je geen onoverzichtelijke vlakken met html in je sources staan.
Dit is in mijn ogen goed php omdat je echt php script, en niet half om half.
-
03-03-2010, 11:18 #26
- Berichten
- 606
- Lid sinds
- 18 Jaar
en zo dan?
<?php echo'<input type="hidden" name="name" value="'.$user['username'].'" >;?>
-
03-03-2010, 11:24 #27
- Berichten
- 2.392
- Lid sinds
- 17 Jaar
Dan gaat hij al helemaal dood Dirk, verkeerde syntax...
<?php echo '<input type="hidden" name="name" value="'.$user['username'].'" />'; ?>
-
03-03-2010, 11:32 #28
- Berichten
- 606
- Lid sinds
- 18 Jaar
hoe bedoel je Chris?
Update: Ah ja, zie m, maar wat ik bedoel is algemeen alles in een echo zetten. Vind dit persoonlijk fijner bij een if/else statement en dergelijke
-
03-03-2010, 12:54 #29
- Berichten
- 257
- Lid sinds
- 15 Jaar
Ik zou dan ook nog liever gebruik maken van de functies printf() en sprintf() van PHP. Iets overzichtelijker.
De volgende keer dat ik mijn site ga vernieuwen dan ga ik gebruik maken van een template engine (Smarty) zodat ik html en php volledig kan scheiden. (tenminste, dat ga ik proberen).
Uiteindelijk komt denk ik iedereen er een keer achter dat het scheiden van diverse onderdelen: html, css, php etc. echt heel veel voordelen oplevert.
-
03-03-2010, 13:24 #30
- Berichten
- 690
- Lid sinds
- 15 Jaar
<? word niet meer ondersteund in de nieuwe PHP versies.
Plaats een
- + Advertentie
- + Onderwerp
Marktplaats
Webmasterforum
- Websites algemeen
- Sitechecks
- Marketing
- Domeinen algemeen
- Waardebepaling
- CMS
- Wordpress
- Joomla
- Magento
- Google algemeen
- SEO
- Analytics
- Adsense
- Adwords
- HTML / XHTML
- CSS
- Programmeren
- PHP
- Javascript
- JQuery
- MySQL
- Ondernemen algemeen
- Belastingen
- Juridisch
- Grafisch ontwerp
- Hosting Algemeen
- Hardware Info
- Offtopic