System med radio buttons og send til database

Har så et nyt problem. Alle radiokanpperne ender jo med at have samme navn. Så jeg kan kun vælge en af dem. :S

Det lyder mere som en opgave, end som et problem.

Kan du ikke starte med at slå de mange DB-udtræk sammen til et enkelt - så bliver det noget nemmere at overskue:

SELECT navn,klub,plads FROM player WHERE plads IN('GK','F','M','A','T') AND spiller='$user' ORDER BY plads,navn;

Dine komplicerede regler er for meget for HTMLs radio-knapper. Enten skal du lave en masse javascript til at tjekke det, ellers også skal du tjekke gyldigheden når valgene er sendt til serveren. Det sidste er det nemmeste (synes jeg).

Jeg ved ikke om det kun er tilladt at vælge en knap i hver søjle eller i hver række?

Se det var jo lidt nemmere hvis man kunne trække spillerne ud på den måde.

Jeg kan bare ikke få det til at virke da der ikke bliver udskrevet nogle spiller.

Det der skal vælges er en i hver række.
Og det er også den sidste med at tjekke når valget skal sendes til server, som jeg går efter.

Nu fik jeg den til at skrive spillerne ud.
Men de kommer bare i forkerte række følge.

Jeg skulle gerne have dem til at komme som du skriver:
GK, F, M, A, T
Men de kommer i:
T, A, GK, F, M

Tja, hvis de absolut skal komme i den rækkefølge så kan du ikke brug SQL sortering ikke.
Jeg vil stadig mene at det kan betale sige at generere alle knapper i en enkelt løkke (så du kun skal rette et sted hvis du vil lave knapperne om), men så skal du selv sortere dem:

// hent det hele
$unsorted = array();
$result = mysql_query(bla bla bla);
while ($row = mysql_fetch_array($result))
  $unsorted[] = $row;
// sorter
$sorted = array();
$pladser = array("GK","F","M","A","T");
foreach ($pladser as $plads)
  foreach ($unsorted as $row)
    if ($row['plads'] == $plads)
      $sorted[] = $row;
// vis det hele
foreach ($pladser as $key => $row) {
  $navn = htmlspecialchars($row['navn']); // hvis danske tegn
  $klub = htmlspecialchars($row['klub']);
  echo "<tr> .... bla bla bla";
}

Mon ikke databasen osse indeholder et "id" felt? F.eks.
  SELECT id,plads,navn,klub FROM ....
Så kan du bruge "id" feltet som en del af navnet på hver knap, på den måde kan man kune vælge en knap i hver række.

hov - der er vist en fejl i min PHP:

5. sidste linie skal være:

  foreach ($sorted as $key => $row) {

Det med id var også det jeg ville til at ind på.
Da det er den eneste måde jeg kan give dem deres eget navn.

Der er et problem her:
Parse error: syntax error, unexpected T_STRING in ...test.php on line 13


  foreach ($unsorted as $row)
    if ($row['plads'] == $plads)  <--- Linje 13
      $sorted[] = $row;

Jeg kan ikke se nogen fejl i den linie.
Er du sikker på at linie 13 faktisk er den linie? Nogen gange tæller PHP linienumre på en underlig måde.

Har lige prøvet at kopiere det ind du skrev igen.
Der kom en ekstra linje på da jeg ikke havde "//hent det hele" med før.

Og nu siger den linje 14:

Du bør anstrenge dig lidt mere inde du henvender dig til eksperten.dk.

Du har en fejl her:

include('config.php); // mangler afsluttende '

Hvis du nu brugte en editor som farver teksten efter syntaks...

Weee fandt selv problemet....
Skal lige flette det ind hvor det rigtigt skal bruges.

Sådan...
Nu var det jo sådan at jeg havde prøvet det på siden hvor det skulle bruges.
Det virkede ikke og så lavede jeg lige hurtigt den test.
Her var config.php fejlen.

Men det var ikke den eneste fejl.
Og derfor virkede den ikke på siden hvor den skulle bruges.

Fejlen var i:
$result = mysql_query("SELECT plads,spiller,klub,navn from valg WHERE plads And spiller = Wagner ORDER BY navn");

Hvor jeg har "plads AND" hvilket ikke skulle være der.

Men hvis du nu kan generere en HTML side som ser ud som den skal, med knapper passende navne, og valgene bliver sendt tilbage til serveren, så kan du nu gå $_POST[] (eller $_GET[]) igennem og udtrække "id" delen af hver valg. Så kan du igen læse det hele fra databasen og begynde på det med at undersøge om valgene stemmer med dine krav.

Det er så her jeg er lidt lost om hvordan det skal laves.

Hvis man har denne linje:


Så skal man vel indsætte en masse hidden input felter til navn og plads osv?

Og kan man så bare sende det afsted sådan:

nej, hver <input> indeholder et navn og en værdi:

  <input name='foo' value='bar' />

og det er dem som bliver overført til serveren.
Alle <input> skal pakkes ind i

  <form method="POST" action="upload.php">
  <input name='foo' value='bar' />
  </form>

så vil $_REQUEST[] i upload.php kunne scannes med

  foreach ($_REQUEST as $name => $value) {
    if ($name == "foo") {
      // user entered 'bar'
    }
  }

Det vil sikkert være smart at indsætte noget foran 'id' i hvert name='', f.eks.

  echo "<tr><td><input name='foo".$row['id']."' ... ";

og så tjekke om de første 3 tegn er 'foo':

  foreach ($_REQUEST as $name => $value) {
    if (substr($name, 0, 3) == "foo") {
      $id = intval(substr($name, 3));
      // læs data for rækken $id fra database
    }
  }

Uha.. Nu er vi ude i noget jeg ikke har leget  med før eller kan gennemskue.

Det med at lave en form osv kan jeg sagtens.
Men så kommer det næste, at skulle sende det til databasen.

Skulle lave alle de If'er det tror jeg ikke lige jeg kan rode sammen.

Og de input du snakker om er det mine radio felter?

Og i trækker du id'et ud af navne feltet så du kan bruge det til at hente de andre data om spilleren i databasen?

ja, <input> er dine radio knapper.

Og når du har id'et kan du få resten af info om spilleren med:

$id = intval(substr($name, 3));
$query = "SELECT navn,plads,klub FROM player WHERE id=$id;";
...

Så skal du osse lave noget i retningen af:

$antal_GK = 0;
$antal_A = 0;

foreach ($_REQUEST ...) {
  // udtræk id
  // læs info om id fra DB
  $row = mysql_ .... ($query);
  $plads = $row['plads'];
  if ($plads == "GT")  $antal_GK++;
  if ($plads == "A")    $antal_A++;
}

og derefter kommer så dine regler:

$ok = 1;
if ($antal_GK != 1)  $ok = 0;

osv. osv.

Kan du forklare hvad der sker her:

Det med SQL sortering og udtræk kan altså løses på en meget simplere måde:

Og find_in_set() skal selvfølgelig lige tilpasses PLADS feltet:

Det var en lidt bedre.
Det var nemmere at forstå for en noobie som mig.

Jeg kender ikke mysql - bruger selv PostgreSQL som vist ikke har 'find_in_set' funktionen. Beklager.

Forklaring til seneste PHP kode: du ønsker jo at tælle op hvor mange af hver slags 'plads' som brugeren har valgt, der skal være nøjagtig 1 'GK' osv.
Så du skal gå igennem alle elementer i $_REQUEST og hvis det er et element som hører til brugerens valg, så udtrækker du 'id' for det element. 'id' var jo noget der refererede til en række i databasen, så du læser igen den række fra databasen og bruger den info til at tælle op hvormange af hver slags 'plads' der er i det valgte.

Jeg gider selvf. ikke skrive hver eneste detalje - du skal selv lave noget, ellers lærer du aldrig noget.

Nu har jeg bakset med en loesning.  Det tog mere tid end jeg havde regnet med, men jeg fandt det et interessant problem, og der sad meget personligt laerestof i.  I mellemtiden er der kommet en lang raekke indlaeg.  Det er ikke min hensigt at ignorere disse indlaeg, men lad mig nu alligevel bringe min loesning.

Foerst:  Jeg forstaar opgaven er, for en given bruger, at finde brugerens spillere i databasetabellen players, presentere disse paa en hjemmeside paa en saadan maade at brugeren kan vaelge et hold for en runde, og hvis holdet er sammensat korrekt at indfoere holdet i en databasetabel og ellers at give en fejlmelding.

Jeg vil foreslaa at du i tabellen players indfoerer en id for playerne.  Det goer oplysningerne nemmere at bruge, hvilket jeg skal demonstrere.  Yderligere vil jeg foreslaa at du paa hjemmesiden viser hver spiller med en checkbox i stedet for med radiobuttons.  Det er ogsaa nemmere at have med at goere.

For at udarbejde og teste en loesning har jeg lavet en databasetabel ddd_players og en komplet php kode.  Jeg viser begge nedenfor.

Min foreslaaede fremgangsmaade er, som ogsaa HBP2 siger, at lave en enkel forespoergsel: "SELECT id, navn, klub, plads FROM ddd_player WHERE spiller = '$spiller'"

$spiller skulle vaere $_SESSION['user'], men for test har jeg sat $spiller = 'spiller1'

For hver spiller der fremkommer fra forespoergselen gemmer jeg id og plads i en array for senere brug, og derudover laver jeg en formular der indeholder en tabel checkbox, navn, klub, og plads for spilleren, og derudover en submit knap.

Naar der er klikket i checkboxene og paa submit knappen saa dannes der en ny array med id'erne af de valgte spillere.  Denne array sammenlignens saa med den foerste array for at checke om holdet er sammesat korrekt, altsaa en GK, en A, o.s.v.  Hvis nej saa kommer der en fejlmelding.  Hvis ja saa sendes holdet til databasen.

For selve indsaetningen af det (korrekt) valgte hold har jeg ikke skrevet kode (men jeg kan i morgen hvis det stadig har interesse.)  Princippet skulle vaere "INSERT INTO runde.. SELECT ... FROM  players WHERE players.id IN([listen af valgte id'er])

Du kan proeve min kode af i http://christianjorgensen.be/ddd.php

ddd_Players tabellen jeg bruger ser saaledes ud:

        row(s) starting from record # 
in  horizontal horizontal (rotated headers) vertical mode and repeat headers after  cells     
          id  navn  klub  plads  spiller 
      1 navn1 klub1 GK spiller1
      2 navn2 klub2 GK spiller1
      3 navn3 klub3 A spiller1
      4 navn4 klub4 A spiller1
      5 navn5 klub5 T spiller1
      6 navn6 klub6 T spiller1
      7 navn7 klub7 F spiller1
      8 navn8 klub8 F spiller1
      9 navn9 klub9 F spiller1
      10 navn10 klub10 F spiller1
      11 navn11 klub11 F spiller1
      12 navn12 klub12 M spiller1
      13 navn13 klub13 M spiller1
      14 navn14 klub14 M spiller1
      15 navn15 klub15 M spiller1
      16 navn16 klub16 M spiller1
      17 navn17 klub17 A spiller2
      18 navn18 klub18 A spiller2

og php koden er her:

<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html>
<head>
<title>ddd</title>
</head>

<body>
<?
$link = mysql_connect ('christianjorgensen.be.mysql', 'christianjoygen', 'dnyBKKbe') or die(mysql_error());
mysql_select_db('christianjoygen') or die('Could not select database');

//else
{
  //$spiller = $_SESSION['user'];
  $spiller = 'spiller1';

  $result = mysql_query("SELECT id, navn, klub, plads FROM ddd_player WHERE spiller = '$spiller' ORDER BY plads")or die(mysql_error());
  echo "<table>";
  echo "<form action='' method='POST'>";
  echo "<tr><td>check</td><td>navn</td><td>club</td><td>plads</td></tr>";
  while($row = mysql_fetch_array($result))
  {
    $id = $row['id'];
    $navn = $row['navn'];
    $klub = $row['klub'];
    $plads = $row['plads'];
    $pladsarray[$id] = $plads;
    echo "<tr><td><input type='checkbox' name='spillercheck[]' value=$id></td><td>$navn</td><td>$club</td><td>$plads</td>";
  }
  echo "<tr><td colspan='4'>";
  $p = implode(",",$pladsarray);
  echo "<input type='hidden' name='p' value='$p'>";
  echo "<input type='submit' name='submitPlayers' value='Gem spillere'>";
  echo "</td></tr>";
  echo "</form>";
  echo "</table>";
}
//print_r($pladsarray);

if($_POST['submitPlayers'])
{
  $inDatabase = "FALSE";
  $spillerchecks = $_POST['spillercheck'];
  //count gk
  $gk = 0;
  foreach($spillerchecks as $value) if($pladsarray[$value] == 'GK') $gk++;
  if($gk == 1)
  {
    //count a
    $a = 0;
    foreach($spillerchecks as $value) if($pladsarray[$value] == 'A') $a++;
    if($a == 1)
    {
      //count t
      $t = 0;
      foreach($spillerchecks as $value) if($pladsarray[$value] == 'T') $t++;
      if($t == 1)
      {
        //count f
        $f = 0;
        foreach($spillerchecks as $value) if($pladsarray[$value] == 'F') $f++;
        //count m
        $m = 0;
        foreach($spillerchecks as $value) if($pladsarray[$value] == 'M') $m++;
        if($f == 2 && $m == 4 || $f == 3 && $m == 3 || $f == 4 && $m == 2) $inDatabase = "TRUE";
      }
    }   
  }
  echo "IN DATABASE = $inDatabase <br>";
}

mysql_close($link); 
?>
</body>
</html>

Jeg glemte at sige at der paa min testside for et korrekt sammensat hold, et der skulle placeres i databasen, skrives "IN DATABASE = TRUE" og ellers "IN DATABASE = FALSE."

Vi har jo alle forskellige måder at lære på.
Jeg lære bedst ved at have tingene i hånden og undersøge det.

Men det du prøver at få mig til at lave, er at der kun må være en GK?

Der må gerne være 2.
Men der må kun være en GK hvor radio input = i
Den anden skal også sendes bare med radio input = 0

Kan se at jeg havde glemt den del før. i #14
Det skulle nok have set sådan ud:


Bare du ikke misforstår min ringe måde at få forklaret tingene.

Christian --> lige med det første ser det rigtigt godt ud.
Skal nok lige lege lidt med det for at forstå det helt.

Men jeg tænkte på om det var en fordel med check boksen.
For det skal være sådan at dem man ikke tjekker af også bliver sendt til databsen som beskrevet i #25

Altså så alle bliver sendt i til databasen "valg" hvor alle oplysningerne (navn,klub,spiller,plads,io)
Hvor io skal se om de har været markeret.
Så hvis de er markeret skal der sendes et (i) og hvis de ikke er skal der sendes et (o)

Jamen hvis alle skal i databasen saa har jeg grundigt misforstaaet opgaven.  Jeg maa tilstaa at jeg ikke har studeret alle 25 indlaeg i detaljer men har koncentreret mig om dit oprindelige spoergsmaal hvor du skriver:

"Alle spillerne skal sendes til en database der hedder "runde" under følgende felter: .... Men kun hvis visse krav fra radio knapperne er godkendt. ... Hvis disse krav ikke er overholdt skal der komme en besked om fejl.
."

Det er nok mig der er tumpet, men jeg har svaert ved at fatte hvordan det kan forstaas anderledes end at spillere skal sendes til databasen men kun hvis et antal krav er opfyldt, ellers skal der komme en besked om fejl!

Du slutter dit oprindelige spoergsmaal med at oenske at "Nu håber jeg det er lidt overskueligt og nemt at forstå."  Det var det aabenbart ikke for en dummerik som mig, saa jeg beklager hvis jeg har spildt din (og min) tid ved ikke at kunne laese indenad.

Men under alle omstaendigheder, ha' fortsat en god aften.

Christian Det er skam ikke spildt.
Er meget glad for det og jeg lære også af det.
Er da ked af at jeg har svært ved at forklare mig.
:(

Det er rigtigt at kravene skal overholdes.



Er ked af hvis der kan misforståes.
Men derfor havde jeg 2 radio knapper hvor den anden var mærket "o"

Det er bare vigtigt at det er X antal der skal bruges af hver.
Som du har ramt helt rigtigt.

Jeg skal også bruge dem der ikke er valgt = "o"'erne
De skal bare sendes til databasen med et "o" istedet for et "i" i feltet "io" i databasen.

Det synes jeg f.... er svaert at forstaa ud fra det du skrev i dit oprindelige indlaeg.

Men saa giv mig (og de andre medlemmer) en beslutnings-opskrift saasom:

Hvis bruger vaelger et hold bestaaende af en GT, en A, ....o.s.v saa placer alle spillere i databasen, medlemmer af det valgte hold med i og de andre med o.

Ellers (det vil sige hvis det hold brugeren vaelger ikke er sammensat korrekt, for eksempel enten indeholder to GT'er eller ingen) .... [hvad skal der saa ske? Skal alle spillere indsaettes i databasen som i eller som o eller skal ingen spillere indsaettes i databasen eller hvad?]

Hvis du kan skaere det ud i pap saa selv jeg kan forstaa det saa skal jeg proeve i morgen at kode det.

Jamen synes faktisk jeg har forklaret det ok.
Men prøver da igen..

Jeg har fået lavet denne kode til at indtaste data med:


Jeg ville så lave send.php som den side der sendte til databasen.
Det er bare den jeg manglede at få lavet.

Hver bruger får med det overstående sine 16 spiller frem:


Ud for alle spillerne er der 2 radio input.
En med value='i' og en med value='o'
(står for "ind" og "out")

Hver bruger skal vælge 9 spiller der skal være "inde". Resten skal være "out"

De 9 spiller skal være:


Der skal komme fejl og intet skal sendes til databasen hvis følgende er valgt:


Fejl beskeden kan bare være:


Hvis kravene er godkendt skal følgende sendes til databasen "runde" i disse felter:


På den måde komme alle spillerne i databasen.
Og det er kun det antal spiller der må være inde der er inde resten er ude.


Håber at det var bedre. Du/i er meget velkommen til at spørger.

Lukker.