Gå til innhold

Martin A.

Medlemmer
  • Innlegg

    2 113
  • Ble med

  • Besøkte siden sist

Innlegg skrevet av Martin A.

  1. $settinn2 = quote_smart($_POST['settinna']);
    
    if(!is_numeric($settinn2)) {

    Den kunne med fordel vært:

    $settinn2 = intval( $_POST['settinna'] );
    
    if( $settinn2 < 1 ) {

     

    POST har vel heller ingen grenser på lengen på nøklene, så beskriv de istede for slik krypisk skrot som "settinna", "setinnpen", etc. Samme gjelder SQL. penger og bankbeholdning fremfor peng og bank.

  2. Skjær meg laus! Er ikke rart det blir innviklet når du holder på på den måten der.

     

    Jeg går ut i fra at dette er en del av en funksjon som gjør at brukerne kan sette penger inn i banken? Hvorfor i alle dager minker du bankens beholdning når du da setter inn penger? Du kan gjør alt der skrotet der på seks linjer, tre om man dropper definering av mer forklarende variabler. Noe du kanskje bør lære deg.

     

    $bank = $hennt['bank'];
    $bruker = $hent['peng'];
    $settinn = is_numeric( $_POST['settinna'] ) ? intval( $_POST['settinna'] ) : die( "Feil" );
    
    $new_bruker = $bruker - $settinn;
    $new_bank = $bank + $settinn;
    
    mysql_query( "UPDATE brukere SET bank = '$new_bank', peng = '$new_bruker' WHERE id = {$_SESSION['id']}" );

     

    Slik vil din kode være i praksis, om jeg forsto den riktig.

    $innbank = 300000;
    
    $inn1 = 10000;
    $inn2 = 5000;
    $inn3 = 10000 - 5000; // = 5000;
    $inn4 = 5000;
    $inn5 = 300000;
    $inn6 = 5000 - 300000; // = -295000
    $inn7 = -295000;
    $inn8 = mysql_query("UPDATE brukere SET bank = '-295000', peng = '5000' WHERE id = '$_SESSION[id]'");

    Ikke rart det der ikke stemmer, da ingen av tallene du kommer frem til henger på grep.

  3. Se i manualen. for() krever tre parametre. En som "kjører" når for() blir initialisert, en som kjører i starten av hver "iteration" (vet ikke det norske ordet for det), og en på slutten av hver iteration.

    for( $i = 0; $i <= 10; $i++ )
    {
     echo $i . '<br />';
    }

    Noe lignende i while()

    $i = 0;
    while( $i <= 10 )
    {
     echo $i . '<br />';
    
     $i++;
    }

    foreach() gjør det samme som de to overstående, bortsettfra at den kun tar i mot array, og looper gjennom det.

    $array = array( 1, 2, 3, 4, 5, );
    foreach( $array as $i )
    {
     echo $i . '<br />';
    }

  4. <form action=""><input type="radio" name="sdk" value="yes">Yes</form>
    <form action=""><input type="radio" name="sdk" value="no">No</form>

    Der har du nok feilen. Det er ikke slik forms fungerer.

     

    <form action="send.php" method="post">
    <!-- All <input> her -->
    <input type="submit" name="submit" value="Send inn" />
    </form>

    Dessuten er det lite vits å kjøre koden i scrive.php gjennom PHP motoren om det kun er HTML du har i filen. Selv om filen har slutter .php, betyr ikke det at all kode skal være PHP. Du kan gjerne skrive rein HTML i en slik fil utenfor <?php.

  5. Sørg for at session_start(); er HELT øverst! Før alt!

    Langt fra sant. session_start() modifiserer header, og header sendes før output. session_start() må derfor stå før output. Plassere den under <?php er helt klart det enkleste, men veldig unødvendig om ikke alle funksjonene du har i fila bruker session.

     

    Sendes output før en header/session_start vil det skrives ut en feilmelding.

  6. Jeg har en modul i Notepad++ som heter "Lag spill" :)

    Der plotter jeg bare inn navnet på spillet, hvilken type spill, og skriver en røflig beskrivelse på spillet, så genererer den all PHP kode for meg :)

     

     

     

    Nei. Lær deg PHP som alle oss andre har gjort. Starte på scratch. Ingen her kommer til å skrive det for deg. Vi kan gi deg hjelp om du står fast, men å få noe servert på sølvfat kan du se rimelig langt etter.

×
×
  • Opprett ny...