• Inloggen
  • Registreer
  • Zoeken
Dit topic
  • Overal
  • Dit topic
  • Dit forum
  • Artikelen
  • Pagina's
  • Forum
  • Downloads
  • Meer opties

ICTscripters

Dé plek voor IT

Dé plek voor IT

Aanmelden

Geavanceerde opties
  1. Home
  2. Forum
    1. Alle berichten
    2. Recente activiteiten
    3. Onopgeloste topics
  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
  1. Home
  2. Forum
    1. Alle berichten
    2. Recente activiteiten
    3. Onopgeloste topics
  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
  1. Home
  2. Forum
    1. Alle berichten
    2. Recente activiteiten
    3. Onopgeloste topics
  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
  1. Dé plek voor IT - ICTscripters
  2. Forum
  3. Scripting & programmeren
  4. PHP + SQL

Forum

  • Externe testers gezocht — Saphaxis AI Chat Organizer 0.1.59

    FreelanceAP 4 november 2026 om 07:42
  • nieuwe game

    tigermaffia 2 november 2026 om 15:21
  • Criminolz

    qevanto 31 oktober 2026 om 20:52
  • [Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

    frio 27 oktober 2026 om 12:22
  • Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

    Jeroen.G 26 oktober 2026 om 10:33
  • Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

    eeyk 22 oktober 2026 om 18:43
  • Crime Club volledig herschreven vanaf nul

    frio 19 oktober 2026 om 23:57
  • Malavita (nieuwe maffia game, van 0)

    jos 19 oktober 2026 om 23:18

Marktplaats

  • [Aanbod] Stadskaart voor je browsergame: van menu naar klikbare stad, vanaf €200

    frio 29 oktober 2026 om 18:17
  • [Verkoop] G4G (Gangsters 4 Gangsters): volledige broncode, nieuw gebouwd in PHP 8.3 | € 30 per domein

    frio 27 oktober 2026 om 18:54
  • [TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

    Dpm_x 26 oktober 2026 om 00:22

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; 
    					}
    			}
    Meer tonen

    Kritiek welkom :)!

    Bij voorbaat dank!
    Lancer.

  • Gast, 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
    }
    Meer tonen


    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?

Registratie

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

Nu registreren Aanmelden

ICT Nieuws

  • PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

    Aaron 22 oktober 2026 om 22:19

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

Gebruikers die dit topic bekijken

  • 1 Gasten
  1. Marktplaats
  2. Design
  3. Voorwaarden
  4. Ons team
  5. Leden
  6. Geschiedenis
  7. Regels
  8. Links
  9. Privacyverklaring
  10. Contact
ICTscripters ©2005 - 2026 , goedkope hosting door DiMoWeb.com, BE0558.915.582
Sponsors: Beste kattenhotel provincie Antwerpen
Style: Nexus by cls-design
Stylename
Nexus
Manufacturer
cls-design
Licence
Commercial styles
Help
Supportforum
Visit cls-design