Posts by NielsB

    voor de rest even opmerking:
    $mysqli = new mysqli("","","","");


    in je 2de bestand kan je beter ook de database class aanroepen?
    anders maak je meerdere connecties ipv 1!

    laat je gehele code is zien?
    ik wil even zien, want ik denkd at het beste is dat je toch niet xmlhttp gebruikt want doe maar is een console.log(xmlhttp); en kijk in firebug wat er in dat element staat.


    hoogst waarschijnlijk moet je dan werken met parentnode en children. ;)


    ik kan je wel helpen als je even je code neer zet.


    Sorry ben niet zo actief hier meer waardoor ik nu pas weer antwoord geef!

    kan het zijn dat je multi_query doet en daaronder een gewone query of iets? die gaan namelijk niet werken.


    Dit vond ik op php.net:


    WATCH OUT: if you mix $mysqli->multi_query and $mysqli->query, the latter(s) won't be executed!


    PHP
    <?php
    // BAD CODE:
    $mysqli->multi_query(" Many SQL queries ; "); // OK
    $mysqli->query(" SQL statement #1 ; ") // not executed!
    $mysqli->query(" SQL statement #2 ; ") // not executed!
    $mysqli->query(" SQL statement #3 ; ") // not executed!
    $mysqli->query(" SQL statement #4 ; ") // not executed!
    ?>


    The only way to do this correctly is:

    PHP
    <?php
    // WORKING CODE:
    $mysqli->multi_query(" Many SQL queries ; "); // OK
    while ($mysqli->next_result()) {;} // flush multi_queries
    $mysqli->query(" SQL statement #1 ; ") // now executed!
    $mysqli->query(" SQL statement #2 ; ") // now executed!
    $mysqli->query(" SQL statement #3 ; ") // now executed!
    $mysqli->query(" SQL statement #4 ; ") // now executed!
    ?>


    voor de rest kan je ook alle querys in een variabel $query zetten met = en .= waardoor je 1 variabel heb en die zet je in de multi_query moet werken. ;)


    http://php.net/manual/en/mysqli.multi-query.php

    wat vind je van het idee de column een id mee geven en dan getElementById ? dat werkt sowieso dan wel.

    regels moet je gewoon algemeen dan aanstroeven niet op 1 vlak, dus als je ergens 2 voor zet moet je dat algemeen ook neerzetten. Het is misschien te streng maar dan moet je je maar houden aan de regels!

    Lars ik vind het leuk en aardig maar probeer niet gelijk tek rijgen met die punten van je:
    Als het goed is moet die werken.
    Denk voortaan aan het volgende:
    - Netjes intabben GEEN spaties!
    - Accolades op dezelfde regel om verwarring te voorkomen
    - Gebruik één type quotes voor JavaScript / jQuery (ik raad de dubbele aan)
    - Gebruik spaties na een nieuwe parameter (dus niet 2,3,3,4 maar 2, 3, 3, 4)
    - Gebruik op het begin $(function(){ i.p.v.
    $(document).ready(function(){
    - Geef bij een Ajax request ALTIJD alle parameters op, hoe ik de request er nu in heb gezet is de juiste manier.


    Het is de manier hoe je het zelf wilt doen en wat jij makkelijker vind, ga niet beweren dat je MOET intabben MOET accolades opzelfde regel neerzetten.


    PLEASE STOP!

    Ziet er wel netjes uit, maar probeer de content beter te benutten. Probeer dus de portfolio pagina gewoon wat beter te verdelen het staat namelijk lelijk onder de titel.


    Voor de rest ziet er netjes uit, ik geef je een 7!

    Zit er geen htacces meer in ? Vroeger toen ik het pwyl nog in bezit had toen hadden we hier een htaccess bij gemaakt en deze werd dan ook gebruikt door andere sites.

    Haal die regel weg :

    PHP
    $select = mysql_query("SELECT * FROM `speldata` WHERE leven>'0'"); 
                    while($listm5 = mysql_fetch_object($select)) {


    volgens mij gebruik je hem toch nergens voor! ;) Anders maak je een if wanneer level > 0 moet zijn, maar volgens mij is dat niet eens nodig als ik even zo snel kijk.

    In dit geval heb je geen while lus nodig om die totaal aantal waarde op te halen he ! ;)


    Wanneer je gebruikt maak van een loop gebruik je += , maar snelste manier is je query zoo optimaliseren dat je in 1 keer alle data ophaal. In dit geval moet dit al voldoende zijn:

    PHP
    $query = mysql_query("SELECT SUM(waarde) AS waarde FROM `tc_garage` WHERE `soort`='1' AND `eigenaar`='".$speler['login']."'");
    $aantal = mysql_fetch_array($query);
    echo $aantal['waarde'];

    ?

    Het ligt aan htaccess wanneer je seo urls krijg. wanneer je gewoon php in je url hebt ligt het gewoon aan de link!

    PHP
    if($ontvangertje != $_POST['ontvanger']){


    moet zijn:

    PHP
    if($ontvangertje['login'] != $_POST['ontvanger']){


    overigens zou ik het volgende doen :

    PHP
    if(empty($ontvangertje['id']) && !isset($ontvangertje['id'])){


    omdat je een query maakt waarin je checkt of hij bestaat kan je net zo goed kijken of er resultaten zijn, zo niet geeft hij error.

    en anders gooi je er een strtolower over heen in je functie om te kijken of het een scheldwoord is.


    dus:

    PHP
    str_replace(strtolower($scheld[word]),"<font color=red>*Censuur*</font>",$string);

    nou de manier hoe jij het doet kan ook gewoon wanneer je dit in een loop gooit waar je alle gebruikers op haalt. Alleen zal het niet leuk zijn voor je load van je site dus wat je kan doen is het even offline op een local webserver te doen en daarna weer opnieuw op je site te gooien?


    Verder zorg er voor dat je login etc wel allemaal zo ingesteld zijn dat je die md5 md5 checks erbij heb?


    Of je laat alle gebruikers zelf een keer inloggen waardoor ze wachtwoord moeten veranderen en deze wordt dan weer opgeslagen met md5 md5.

    Ik zelf ben een professionele scripter ( paar weken geleden afgestuurd als webdeveloper ), maar ik ben eigenlijk beetje tegen de banditi source, dit komt door de opbouw van de code en het overzicht!


    Maar wat zijn de aanpassingen, ik zou graag al van te voren willen weten wat je wilt. Dit is denk voor andere mensen ook handig zodat je kan weten of je het wel aan kan. Hierdoor kan ik zien of het niet te gek veel werk is om heel de code weer te gaan snappen. ( Waardoor je eigenlijk meer tijd aan voldoet dan wanneer je een totaal unieke script schrijft los van een source )