• 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

  • Het oorspronkelijke Criminals Script

    Lijno 19 juni 2026 om 15:06
  • StraatBaas is back, maar hoe?!

    Syntax 17 juni 2026 om 10:28
  • RPG game gebouwd met AI

    Frenzo.Webservice 11 juni 2026 om 19:44
  • Het Grote Vibe Code Topic

    Syntax 1 juni 2026 om 20:05
  • PWYL source gezocht

    Syntax 29 mei 2026 om 14:03
  • Ictscripters Chat

    AarClay 21 april 2026 om 11:34
  • Help testers nodig voor android app Urgent

    Servertjee 20 februari 2026 om 12:07
  • Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

    Servertjee 20 februari 2026 om 12:06

Marktplaats

  • 4-letter domein: Togi.nl

    evesi 17 juni 2026 om 17:08
  • 359 Nieuwe Domeinnamen Mei 2026

    shiga 1 juni 2026 om 12:45
  • Sicarras.com - Moderne Mafia Text-Based RPG

    Jeffrey.Hoekman 27 mei 2026 om 17:40

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.

  • 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

  • Quanscient ontvangt €10M om AI- en kwantum-native hardware engineering te bevorderen - Tech.eu

    ICTscripters 27 mei 2026 om 12:03
  • Datalek bij leverancier Canvas - Universiteit van Amsterdam

    ICTscripters 10 mei 2026 om 12:03
  • Data privacy in 2026: Hoe de naleving van GDPR verandert

    ICTscripters 8 mei 2026 om 12:16

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. 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