Hoi iedereen,

Ik ben bezig met een uitslagen formulier waar leden wedstrijd uitslagen in kunnen vullen.

De wedstrijden worden uit de database gehaald en in een while loop weergegeven, om bij elke wedstrijd een uitslag te kunnen invullen maak ik bij de sql insert query gebruik van een foreach.

In mijn formulier is de input text veld dus zo opgebouwd:

Code:

<input type="text" size="3" name="uitslag[]" />

Werkt prima met deze foreach:

Code:

foreach ($_POST['uitslag'] as $key => $value) {

                                        if(!empty($value)) {
                                            
                                            $uitslag = secure_string($value);
                                            
                                            $insert_uitslagen = mysql_query("
                                            INSERT INTO wedstrijd_uitslagen
                                            (id,lid_id,wedstrijd_id,uitslag)
                                            VALUES
                                            (
                                            '',
                                            '".secure_string($_SESSION["lid_id"])."',
                                            '".secure_string($_POST['wedstrijd_id'])."',
                                            '".$uitslag."'
                                            )")or die (mysql_error());
                                            
                                            
                                            
                                            
                                    }
                                            }

Zoals je ziet worden er 3 dingen opgeslagen, lid_id, wedstrijd_id en uitslag.
Er wordt netjes voor elke nieuwe uitslag en nieuwe rij aangemaakt en de uitslag daarin weergegeven, alleen het probleem zit in "wedstrijd_id". Die wordt in het formulier nu als een hidden field meegegeven, maar bij het maken van de query merk ik dat alleen de laatst gegeven wedstrijd_id in het form bij alle nieuwe uitslagen in komt te staan.

Dit wordt in de database gezet (fictief):

id,lid_id,wedstrijd_id,uitslag
'1','4','20','4-5'
'2','4','20','2-1'
'3','4','20','3-1'

Zoals je ziet blijft wedstrijd_id hetzelfde in het formulier bij elke uitslag terwijl dit eigenlijk het wedstrijd_id moet zijn die bij die uitslag hoort.

Dus ik zat te denken om wedstrijd_id ook in een foreach loop te achterhalen en of moet ik dit anders verwerken in mijn form?

Mag ik je hele form zien? Lijkt me stug dat je hele formulier werkt op basis van 1 input field.
Misschien kun je meerdere hidden inputs meegeven met elk een unieke id (wat dus de wedstrijdid is).

Als in je Database de velden een INT zijn waar je een getal invoerd dan hoef je sowieso geen '' om getallen te zetten. Dit vergroot je leesbaarheid.

Code:

<form action="" method="post">
<input type="hidden" name="wedstrijd_id" value="'.stripslashes($wedstrijd['id']).'" />
<input type="text" size="3" name="uitslag[]" />
</form>

Dit is mijn formulier, waarbij wedstrijd_id de id van de wedstrijd ophaalt uit de database. Alleen er gaat iets fout met het toevoegen aan de database, waarbij de foreach wel werkt maar wedstrijd_id niet dynamisch mee gaat vanuit het formulier maar er wordt 1 wedstrijd_id (de laatste uit het formulier) toegekend aan de nieuwe uitslagen.

Hoi,

Gaat het dan al niet fout bij het ophalen van de wedstrijden?

Martijn

@Martijn

Nee het hidden field vertoond de juiste ID's. Ik zat te denken om misschien de hidden field ook als foreach weer te geven.:

Code:

<input type="hidden" name="wedstrijd_id[]" value="'.stripslashes($wedstrijd['id']).'" />

En dan die 2 foreach combineren zodat de uitslag wel bij de juiste wedstrijd_id gezet wordt?

Begrijp ik het goed dat je meerdere van deze formulieren op 1 pagina hebt? Of meerdere van deze onder elkaar plaatst? Laat eens wat meer zien.

Als er PER wedstrijd maar 1 uitslag veld is, en je plaatst meerdere uitslag velden op je pagina moet je natuurlijk ook zorgen dat je meerdere wedstrijd IDs meestuurt, dus:

<form>
<input type="hidden" name="wedstrijd_id[]" value="1">
<input type="text" name="uitslagen[1]">
<input type="hidden" name="wedstrijd_id[]" value="2">
<input type="text" name="uitslagen[2]">
<input type="hidden" name="wedstrijd_id[]" value="3">
<input type="text" name="uitslagen[3]">
</form>

Nu kun je in je foreach kiezen om de wedstrijden op te halen en de array keys te gebruiken om je uitslag te vinden:

PHP Code:

foreach($_POST['wedstrijd'] as $key => $wedstrijd_id) {
  if(!empty($_POST['uitslagen'][ $key ])) {
    $uitslag = secure_value($_POST['uitslagen'][$key]);
    $wedstrijd_id = secure_value($wedstrijd_id);
    ...
  }
} 


De reden waarom in het formulier WEL waardes tussen uitslagen[] staat (de 1,2 of 3) is omdat deze waarde leeg zou kunnen blijven en dan zou je een foute key krijgen; bijv:

uitslagen[] = '1-1';
uitslagen[] = '';
uitslagen[] = '2-1';

levert een array op:

array(
0 => '1-1',
1 => '2-1'
);

en niet een een 3e key met daarin een lege waarde.

In jouw code krijg je dus:

<input type="hidden" name="wedstrijd_id[]" value="'.stripslashes($wedstrijd['id']).'" />
<input type="text" name="uitslag['.stripslashes($wedstrijd['id']).']" size="3" />

Laat eens wat meer van je script zien?
Dus vanaf het begin waar je die formulier gaat maken.

@Joshua was mij net voor.

@Joshua / Martijn

Het klopt inderdaad dat er meerdere velden in het form zijn en hoe jij het nu weergeeft Joshua werkt het. Heb nu dit:

Foreach:

Code:

foreach($_POST['wedstrijd_id'] as $key => $wedstrijd_id) {
                                              if(!empty($_POST['uitslag'][ $key ])) {
                                                $uitslag = secure_string($_POST['uitslag'][$key]);
                                                $wedstrijd_id = secure_string($wedstrijd_id);
                                                $insert_uitslagen = mysql_query("
                                            INSERT INTO wedstrijd_uitslagen
                                            (id,lid_id,wedstrijd_id,uitslag)
                                            VALUES
                                            (
                                            '',
                                            '".secure_string($_SESSION["lid_id"])."',
                                            '".$wedstrijd_id."',
                                            '".$uitslag."'
                                            )")or die (mysql_error());
                                                
                                              }
                                            }

Form:

Code:

<form action="" method="post">
<input type="hidden" name="wedstrijd_id[]" value="'.stripslashes($wedstrijd['id']).'" />
<input type="text" size="3" name="uitslag['.stripslashes($wedstrijd['id']).']" />
<form>

In de broncode gecheckt en er wordt de juiste wedstrijd_id en id bij uitslag weergegeven. Alleen nu ga ik een uitslag invullen en submit deze. Dan komt de foreach aan te pas, die voegt nu toe met de uitslag die ik hebingegeven alleen staat er een andere wedstrijd_id opgeslagen.

Bijvoorbeeld mijn eerste uitslag die ik toevoeg is ID 8 dus in het form staat bij "wedstrijd_id[]" value="8" en "uitslag[8]".

De uitslag staat in de database, maar bij wedstrijd_id staat "15" om een of andere vage reden. De tabel heb ik van de te voren geleegd en gecheckt met de wedstrijden maar zie geen overeenkomst met de 15 die is toegevoegd. Dit geldt trouwens voor alle wedstrijden die ik ga invullen dat er een andere ID komt te staan. Mijn uitslag staat dus nu bij wedstrijd_id 15 wat uiteraard een andere wedstrijd moet voorstellen.

Probeer je resultaten eens te tonen met <pre> print_r($_POST); </pre> dan kun je zien welke array terug gestuurd wordt en kun je kijken of je koppelingen kunt maken met de resultaten en wedstrijd IDs. Even de query commenten zodat je niet de hele tijd de boel hoeft leeg te maken.

Je kunt ook in je foreach een regel echoën (echo "Wedstrijd id: $wedstrijd_id met uitslagen $uitslag";) en kijken wat je daar in kunt vinden./

Het is gelukt, blijkbaar geeft $wedstrijd_id een foutieve waarde mee, heb nu $key geprobeerd wat juist werkt en nu worden de juiste wedstrijd ID's bij de juiste uitslag weergegeven.

Snap niet echt waarom $wedstrijd_id een andere waarde heeft dan $key.

Bedankt Joshua!

$wedstrijd_id = de waarde van het veld en $key is de key. Wat jij doet is:

$wedstrijd_id[0] = 15; (0 = key, 15=id) matchen met $uitslagen[0];

in plaats van

$wedstrijd_id[0] = 15; (0 = key, 15=id) matched met $uitslagen[15];

Je krijgt dan

$uitslag = secure_input($_POST['uitslagen'][ $wedstrijd_id ]);

Is overigens mijn fout in mijn voorbeeld zie ik :-))

Bedankt voor de uitleg Joshua, snap nu ook waarom ;)

Thanx!

Alles werkt goed en mijn form met wedstrijden worden netjes toegevoegd aan de database met de juiste uitslag.

Nu merk ik alleen dat als de wedstrijd_id hoger dan 88 niet toegevoegd worden. De rest werkt perfect zoals het moet. Zo worden de wedstrijden 88 t/m/ 98 niet toegevoegd aan de database, maar krijg ook geen foutmeldingen gewoon lege echo's in de foreach.

Ik heb in de foreach even een echo gedaan en gekeken als ik het formulier in ga vullen wat er precies fout gaat. Overal krijg ik netjes de uitslag en wedstrijd_id te zien vanuit de echo maar vanaf ID's 88 zie ik niks geen echo.

Foreach:

Code:

if(isset($_POST['submit']))
                                    
                                        {
                                            foreach($_POST['wedstrijd_id'] as $key => $wedstrijd_id) {
                                              if(!empty($_POST['uitslag'][ $key ])) {
                                                $uitslag = secure_string($_POST['uitslag'][$key]);
                                                $wedstrijd_id1 = secure_int($key);
                                                echo $uitslag.'<br />';
                                                echo $wedstrijd_id1.'<br />';
                                                $insert_uitslagen = mysql_query("
                                            INSERT INTO wedstrijd_uitslagen
                                            (id,lid_id,wedstrijd_id,uitslag)
                                            VALUES
                                            (
                                            '',
                                            '".secure_string($_SESSION["lid_id"])."',
                                            '".$wedstrijd_id1."',
                                            '".$uitslag."'
                                            )")or die (mysql_error());
                                                
                                              }
                                            }

Formulier

Code:

while($wedstrijd = mysql_fetch_assoc($select_wedstrijden))
                                        
   {
echo'<form action="" method="post">    
<input type="hidden" name="wedstrijd_id[]" value="'.$wedstrijd['id'].'" />
<input type="text" size="3" name="uitslag['.$wedstrijd['id'].']" />
</form>
';
   }

Iemand enig idee waarom het wel alles tot ID 87 gewoon netjes toegevoegd wordt maar vanaf 88 niet.

Database structuur `wedstrijd_uitslagen`:
id (int11)
lid_id (int11)
wedstrijd_id (int11)
uitslag(varchar10)