Vouchers verkoop systeem (deel 2)

<?php
$query = "SELECT id, ticket FROM tickets WHERE 'valid' = 1 LIMIT 1";
$result = mysql_query($query);

if($result && mysql_num_rows($result) == 1)
{
//Gelukt!

$row = mysql_fetch_array($result);

echo $row['ticket'];

$query = "UPDATE tickets SET 'valid' = 0 WHERE id = '".$row['id']."'";
$result = mysql_query($query);

if($result && mysql_affected_rows($result) == 1)
{
//Gelukt!
}
}
?>

Zoiets? Wellicht even aanpassen dat de tabel en kolomnamen kloppen. Nog mooier is om hier een transactie van te maken, maargoed :).

Wat bedoel je met transactie?
Als betaling transactie bedoeld dan gebruik ik daarvoor de voorbeeld scripts van mollie ik zet dan jouw voorbeeld in de pay.php
althans dat ga ik nu erin proberen te verwerken brrr

Even kijken hoe ik dit simpel kan uitleggen, met een transactie kan je enkele querys 'aan elkaar koppelen', zodat als er ergens iets fout gaat de hele boel word teruggedraaid.

Stel je koopt Fortis over voor een paar tientjes en je besluit de software opnieuw te gaan schrijven. Bij een transactie heb je twee querys, eentje om het geld van mijn jouw bankrekening af te halen en eentje om dat bedrag vervolgens bij iemand anders bij te schrijven. Als die tweede querys mislukt is het geld al wel van jouw bankrekening afgehaald (!). Vandaar dus transacties, dan kan je de eerste query ongedaan maken.

Ikzelf gebruik op dit moment Codeigniter, een PHP framework. Transacties zitten daar al standaard ingebakken.

$this->db->trans_start();
$this->db->query('AN SQL QUERY...');
$this->db->query('ANOTHER QUERY...');
$this->db->query('AND YET ANOTHER QUERY...');
$this->db->trans_complete();

Als alle querys gelukt zijn worden ze toegepast (commit) en als er ergens iets fout gaat worden ze teruggedraaid (rollback).

Hopelijk begrijp je nu waarom ik dit meedeel; tussen de eerste query (het ophalen van een record) en de tweede (het updaten van dat record) zit weliswaar een ongelofelijk kleine hoeveelheid tijd maar stel je voor dat er twee mensen op exact hetzelfde moment een ticket aanvragen... dan krijgen ze allebei eenzelfde ticket!

Arjan Kapteijn schreef op 09.01.2009 19:41

Ikzelf gebruik op dit moment Codeigniter, een PHP framework. Transacties zitten daar al standaard ingebakken.

Transacties zitten in je database en niet in een PHP-framework. De database zal dus transacties moeten ondersteunen, het overbekende MySQL met de zeer populaire MyISAM-engine ondersteunt bijvoorbeeld géén transacties! En je kunt daar transacties op loslaten tot je een ons weegt, de MySQL database verrekt het om je daar ook maar 1 foutmelding op te geven. Op een verkeerd moment kom je dan ook van een koude kermis thuis...

Gebruik in MySQL uitsluitend innoDB anders heb je geen transacties tot je beschikking. Ook niet wanneer je een framework gebruikt die een schil om transacties heeft gezet, het is de database die transacties regelt/zou moeten regelen.

hmm ja dat zou een probleem kunnen zijn inderdaad.
De kans dat dat gebeurd is denk ik niet zo groot.
Als ik zoveel users heb dat dit soort problemen zich vaker voordoen dan kan ik ook al wel een pro betalen.

Voor mij is het allemaal ongelooflijk ingewikkeld ik ben al blij als dit allemaal ook maar een klein beetje gaat werken zodat mensen zelf een ticket kunnen aanschaffen.

Ik zal proberen een en ander te optimaliseren en zo mijn kennis groter te maken. Als ik ooit op het nivo kom dat ik zoiets kan maken zoals jij uitlegt dan zal ik dat zeker gaan toepassen maar voorlopig eerst nog maar even wat basic dingen eigen maken.

Als ik al eens weet hoe je precies fatsoenlijk een simpele php script maakt ben ik al een heel eind.

Ik krijg het dus niet aan de praat.
Ik heb jouw script in een test bestand gezet als volgt

<?php
require('connect.php');

$query  = "SELECT id, tickets FROM vouchers WHERE 'valid' = 1 LIMIT 1";
$result = mysql_query($query);

if($result && mysql_num_rows($result) == 1)
{
    //Gelukt!
    
    $row = mysql_fetch_array($result);
    
    echo $row['ticket'];

    $query  = "UPDATE vouchers SET 'valid' = 0 WHERE id = '".$row['id']."'";
    $result = mysql_query($query);
    
    if($result && mysql_affected_rows($result) == 1)
    {
        //Gelukt!
    }
}

mysql_close($con);

?>

Maar ik kom er niet uit waar ik de database table moet invullen en waar welke records etc

Mijn database heet dus test (deze word geselcteerd in connect.php)
Dan de table in test heet vouchers waarin ik 3 velden heb te weten "id", "tickets" en "valid". Waar zet ik nu wat neer om bovenstaand te laten werken?

Niet Bumpen:

Twee of meer keer achter elkaar in een topic posten heet bumpen. Bumpen is pas na 24 uur toegestaan en kan een reden zijn voor de admins en moderators om een topic te sluiten. Gebruik indien nodig de knop om je tekst aan te passen.

SanThe.

Echo $query, gooi dat in PHPMyAdmin en kijk eens wat er gebeurd.

Ik probeer het te begrijpen
Ik log in in phpmyadmin ga de database "test" in en druk op SQL in het menu en plak daarin Echo $query dat lever het volgende op

MySQL retourneerde: Documentatie
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Echo $query' at line 1

Ik doe dus wat vekeerd

Ik klik op table vouchers dan op de SQL knop
dan verschijnt er:

SELECT * FROM `vouchers` WHERE 1

Ik zit ondertussen ook nog hier http://www.freewebmasterhelp.com/tutorials/phpmysql/4 te zoeken of ik eruit kom maar m'n ogen beginnen al aardig te tollen hahaha

Haal dat SELECT * FROM `vouchers` WHERE 1 dan weg.
En druk DAN pas op CTRL + V (om te plakken).

Fout

SQL-query:

Echo $query

MySQL retourneerde: Documentatie
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Echo $query' at line 1


:(