• Login
  • Register
  • Zoek
This Thread
  • Everywhere
  • This Thread
  • This Forum
  • Articles
  • Pages
  • Forum
  • Filebase Entry
  • More Options

ICTscripters

Dé plek voor IT

Dé plek voor IT

Login

Geavanceerde opties
  1. Home
  2. Forum
    1. Alle berichten
    2. Recente activiteiten
  3. ICT Nieuws
  4. Blog
  5. Marktplaats
    1. Werk
    2. Advertenties
    3. Domeinnamen
    4. Websites
    5. Design & lay-outs
    6. Scripts
    7. Overige
  6. Design
  7. Leden
    1. Actieve bezoekers
    2. Team
    3. Leden zoeken
  8. Downloads
  9. Goedkope domeinnamen
  1. Home
  2. Forum
    1. Alle berichten
    2. Recente activiteiten
  3. ICT Nieuws
  4. Blog
  5. Marktplaats
    1. Werk
    2. Advertenties
    3. Domeinnamen
    4. Websites
    5. Design & lay-outs
    6. Scripts
    7. Overige
  6. Design
  7. Leden
    1. Actieve bezoekers
    2. Team
    3. Leden zoeken
  8. Downloads
  9. Goedkope domeinnamen
  1. Home
  2. Forum
    1. Alle berichten
    2. Recente activiteiten
  3. ICT Nieuws
  4. Blog
  5. Marktplaats
    1. Werk
    2. Advertenties
    3. Domeinnamen
    4. Websites
    5. Design & lay-outs
    6. Scripts
    7. Overige
  6. Design
  7. Leden
    1. Actieve bezoekers
    2. Team
    3. Leden zoeken
  8. Downloads
  9. Goedkope domeinnamen
  1. Dé plek voor IT - ICTscripters
  2. Forum
  3. Scripting & programmeren
  4. PHP + SQL

Forum

  • Beta-testers gezocht voor Crypto-oefenplatform

    Syntax 29 januari 2026 om 16:11
  • Na 15 jaar terug van weggeweest: iCriminals.nl is terug (BETA)!

    Syntax 19 januari 2026 om 09:34
  • Developer Gezocht

    Mikevdk 10 januari 2026 om 18:57
  • Op zoek naar de legends

    Syntax 5 januari 2026 om 13:50
  • [FREE] WeFact Hosting module

    Jeroen.G 13 oktober 2025 om 14:09
  • Help testers nodig voor android app Urgent

    urgentotservices 26 september 2025 om 10:21
  • Versio vervanger

    Jeroen.G 25 augustus 2025 om 15:56
  • Afspraken systeem met planbeperking

    Lijno 1 augustus 2025 om 23:04

Marktplaats

  • 350 Nieuwe Domeinnamen Januari 2026

    shiga 1 februari 2026 om 14:21
  • 321 Nieuwe Domeinnamen December 2025

    shiga 1 januari 2026 om 10:26
  • Meerdere mafia game template te koop

    Syntax 26 december 2025 om 00:07

Paar vragen!

  • Lancer
  • 20 februari 2010 om 22:08
  • Lancer
    Regelnasi
    Berichten
    1.043
    • 20 februari 2010 om 22:08
    • #1

    Beste Cp'ers,

    Ik heb hier en daar al aardig wat code gezien, en ben het een beetje verloren wat nu echt moet, en wat totaal niet meer kan.

    Zo heb ik een vendetta script bekeken, en daar staat alles dus zo:

    PHP
    if($data->status == dood OR $data->status == Dood)  {
    	$dood = mysql_query("SELECT * FROM `vermoord` WHERE `login`='{$data->login}'");

    Allebei de dingen die ik graag wil weten zitten toevallig bij elkaar in hetzelfde script.

    Is het nou het bovenstaande, of is het juist dit:

    PHP
    if($data->status == "dood" OR $data->status == "Dood")  {
    	$dood = mysql_query("SELECT * FROM `vermoord` WHERE `login`='" .$data->login. "'");

    En hoe kan ik de mysql_query verbeteren?
    Want ik zie velen scripters niet met `vermoord` `login` e.d. werken.

    Dit is gewoon een vraag om te weten te komen wat beter is voor mijn scripting, om wat professioneler te worden.

    PS: Ikzelf heb dit stukje opnieuw geschreven, wat vinden jullie hiervan?

    PHP
    if ($jisin == "1")
    			{
    				if ($tim == "1") 
    					{ 
    						$time = gmdate('s', $tim); $word = seconde; $boete = $jail->boete;
    					}
    				elseif ($tim < "61")
    					{ 
    						$time = gmdate('s', $tim); $word = seconden; $boete = $jail->boete;
    					}
    				else 
    					{ 
    						$time = gmdate('i:s', $tim); $word = minuten; $boete = $jail->boete; 
    					}
    			}
    Toon Meer

    Kritiek welkom :)!

    Bij voorbaat dank!
    Lancer.

  • Guest, wil je besparen op je domeinnamen? (ad)
  • Darsstar
    Vreemd
    Berichten
    1.173
    • 20 februari 2010 om 22:24
    • #2

    De tweede snippet is beter.
    Die geeft geen notices en door de variablen buiten quotes te halen kun je gemakkelijker variable in strings herkennen.

    Nog beter:

    PHP
    if(strtolower($data->status) == "dood")  { 
        $dood = mysql_query("SELECT * FROM `vermoord` WHERE `login`='" .mysql_real_escape_sring($data->login). "'");

    PS. Ik zou de accolades niet hun eigen niveau geven, en ook na elke punt-komma een enter plaatsen:

    PHP
    if ($jisin == 1)
    {
        if ($tim < 61)
        {
            $time = gmdate('s', $tim);
            $word = 'seconden'; // quotes toegevoegd
        }
        else
        {
            $time = gmdate('i:s', $tim);
            $word = 'minuten'; // weer quotes
        }
    
    
        $boete = $jail->boete; // DRY
    }
    Toon Meer


    Ok, ik heb nog wat meer dingen verbeterd, maar zo vind ik het behoorlijk wat netter.

    miauw!

  • Lancer
    Regelnasi
    Berichten
    1.043
    • 20 februari 2010 om 22:34
    • #3

    Dus ik moet een if zoals deze:

    PHP
    if ($jisin == 1)


    nooit tussen "" zetten?

    Enter na een ; kan ik begrijpen!

    strtolower maakt dus de gehele naam naar kleine letters?
    Dus Lancer naar lancer, klopt dit?

    Waarom zou ik de accolades geen eigen niveau geven?
    Opzich is jou script een stuk overzichtelijker omdat alles onder elkaar staat, en de mijne is een weerwar...

    Bedankt voor de uitleg, maar ik kamp nog met 1 ding!
    Is er geen andere manier om alle $data->login e.d. te beveiligen met een mysql_real_escape_string?

    Of kan ik dit beter handmatig blijven doen, omdat ik dan weet dat het functioneert?

    PS: Beveiling moet ik inderdaad nog doen!
    PSS: Ik dacht dat beveiling zoals mysql_real_escape_string alleen bij POST's en GET's moest?

    Bedankt Darsstar!

  • Darsstar
    Vreemd
    Berichten
    1.173
    • 20 februari 2010 om 23:08
    • #4
    Citaat van Lancer

    Dus ik moet een if zoals deze:

    PHP
    if ($jisin == 1)


    nooit tussen "" zetten?


    Cijfers zijn cijfers en geen strings en hoeven daardoor niet tussen quotes.

    Citaat van Lancer

    strtolower maakt dus de gehele naam naar kleine letters?
    Dus Lancer naar lancer, klopt dit?


    Dat klopt.

    Citaat van Lancer

    Waarom zou ik de accolades geen eigen niveau geven?
    Opzich is jou script een stuk overzichtelijker omdat alles onder elkaar staat, en de mijne is een weerwar...


    Daarom dus, ik zie de accolades trouwens liever op de zelfde regel als de if, elseif en else dan op hun eigen niveau (maar het liefst zoals in mijn voorbeeld, maar smaak valt niet te twusten)

    Citaat van Lancer

    Ik dacht dat beveiling zoals mysql_real_escape_string alleen bij POST's en GET's moest?


    Ik weet niet aan welke voorwaarden de login van jouw spel hangen.
    Als (bijvoorbeeld) quots toegestaan zijn, en je gebruikt alleen bij het registreren mysql_real_escape_string() dan is je spel niet bepaald veilig...
    Je moet user input beveiligen, en de login was ooit user input...
    Ik ben vanavond een beetje paranoia volgens mij...

    miauw!

  • Lancer
    Regelnasi
    Berichten
    1.043
    • 20 februari 2010 om 23:13
    • #5

    Bedankt voor de informatie, en de naam is inderdaad user input.
    Maar als je dat door middel van een preg_replace " tegenhoudt, hoeft dit dus niet?

Participate now!

Heb je nog geen account? Registreer je nu en word deel van onze community!

Maak een account aan Login

ICT Nieuws

  • Fijne feestdagen

    tcbhome 28 december 2025 om 13:55
  • Kritieke update voor Really Simple Security-plug-in

    K.Rens 16 november 2024 om 16:12
  • ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

    K.Rens 2 november 2024 om 16:09

Blogs

  • Functioneel ontwerp

    Dees 28 december 2014 om 12:38
  • Access Control List implementatie in PHP/MySQL - deel 1/2

    FangorN 28 december 2018 om 12:35
  • Access Control List implementatie in PHP/MySQL - deel 2/2

    FangorN 29 december 2018 om 12:37
  1. Marktplaats
  2. Design
  3. Voorwaarden
  4. Ons team
  5. Leden
  6. Geschiedenis
  7. Regels
  8. Links
  9. Privacy Policy
ICTscripters ©2005 - 2026 , goedkope hosting door DiMoWeb.com, BE0558.915.582
Sponsors: Beste kattenhotel provincie Antwerpen | Beste Zetes eid kaartlezer webshop
Style: Nexus by cls-design
Stylename
Nexus
Manufacturer
cls-design
Licence
Commercial styles
Help
Supportforum
Visit cls-design