php - Problemer med point system - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php mysqli

Bruger #17136 @ 16.07.13 18:55

hej

overskiften siger en del, det er sådan at jeg arbejder på et point system, og det er sådan hvis man gå ind i value="40", og så retter den til 80 så gå den ind og skriver 80 i stede for 40.

Sådan her har jeg lavet mit lille point system:

PHP kode

 if($point >= 10)
 {
     if($point >= 60)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"60\"> 60 point </div>";
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"40\"> 40 point </div>";
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"20\"> 20 point </div>";
     }
     elseif($point >= 40)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"40\"> 40 point </div>";
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"20\"> 20 point </div>";
     }
     elseif($point >= 20)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"20\"> 20 point </div>";
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"10\"> 10 point </div>";
     }
     elseif($point >= 10)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"10\"> 10 point </div>";
     }
 }
 else
 {
     echo "Du kan ikke oprette spørgsmål.";
 }

En af min venner siger til mig at jeg kunne gør det med noget array, men jeg kan ikke sådan helt se hvordan jeg skulle gør det

har du et forslag eller har du tip til hvordan jeg kan lave om på kode så er du velkommen?.

Spørger hvis der er noget du vil vide eller ligne

Eller er det bedre gør det med list (dropbox) eller hvad man skal kald den.
eller input text til at gør det?

Indlæg senest redigeret d. 16.07.2013 20:47 af Bruger #17136

8 svar postet i denne tråd vises herunder
3 indlæg har modtaget i alt 7 karma

Sorter efter stemmer Sorter efter dato

Bruger #10216 @ 17.07.13 11:05

4.283

Du kan gøre det avanceret, og du kan gøre det kryptisk. Og så kan du gøre det fornuftigt

PHP kode

 function calc_points($points, $limit = false, $steps = false) {
  
      $limit = ($limit !== false) ? $limit : 10;
      $steps = ($steps !== false) ? $steps : array(60, 40, 20);
      $r     = array();
  
      if($points < $limit) return array();
   
      foreach ($steps as $value) {
          if($points >= $value) {
              $r[] = $value;
          }
      }
      
      if($limit) {
          $r[] = $limit;
      }
   
      return $r;
 }
 ?>
 <html>
     <ul>
         <?php foreach(calc_points($_POST["points"]) as $p): ?>
             <?php echo sprintf("<li class='points box'><label for='points-%1$d'>%1$d point</label><input type='radio' name='points[]' id='points-%1$d'></li>", $p); ?>
         <?php endforeach; ?>
     </ul>
 </html>

Nedestående som implementere point_boxes() er for mig, bare ikke så fornuftig som ovenstående:

PHP kode

 function calc_points($points, $limit = false, $steps = false) {
  
      $limit = ($limit !== false) ? $limit : 10;
      $steps = ($steps !== false) ? $steps : array(60, 40, 20);
      $r     = array();
  
      if($points < $limit) return array();
   
      foreach ($steps as $value) {
          if($points >= $value) {
              $r[] = $value;
          }
      }
      
      if($limit) {
          $r[] = $limit;
      }
   
      return $r;
 }
 
 function point_boxes($points, $limit = false, $steps = false) {
     foreach(calc_points($points) as $p) {
         echo sprintf("<li class='points box'><label for='points-%1$d'>%1$d point</label><input type='radio' name='points[]' id='points-%1$d'></li>", $p);
     }
 }
 
 ?>
 <html>
     <ul>
         <?php point_boxes($_POST["points"]); ?>
     </ul>
 </html>

Indlæg senest redigeret d. 17.07.2013 11:21 af Bruger #10216

Bruger #955 @ 17.07.13 00:06

1.624

Der er mange måder at gribe den an. Lad os starte med at gøre din kode, lidt mere overskuelig. Dine elseif, har jeg ændret til if, så gengange linier kunne fjernes:

PHP kode

 if($point >= 10)
 {
     if($point >= 60)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"60\"> 60 point </div>";
     }
     if($point >= 40)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"40\"> 40 point </div>";
     }
     if($point >= 20)
     {
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"20\"> 20 point </div>";
     }
 
     echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"10\"> 10 point </div>";
 
 }
 else
 {
     echo "Du kan ikke oprette spørgsmål.";
 }

Nu har du 3 set kodeblokke, hvor den eneste forskel nu kun er værdien!

Så lad os smide værdierne i et array:

PHP kode

 if($point >= 10)
 {
     $arr = array(60, 40, 20);
     foreach ($arr as $value) 
     {
         if($point >= $value)
         {
             echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"".$value."\"> ".$value." point </div>";
         }
     }
 
     echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"10\"> 10 point </div>";
 
 }
 else
 {
     echo "Du kan ikke oprette spørgsmål.";
 }

Jeg har ikke testet koden, så måske er der en enkelt fejl.

Indlæg senest redigeret d. 17.07.2013 00:07 af Bruger #955

Bruger #17081 @ 17.07.13 09:05

1.258

Man kunne evt. skrive det om til en funktion

PHP kode

 function point_boxes($points) {
     if($points < 10) return "<p>Du kan ikke oprette spørgsmål.<p>";
 
     $r = "";
     $tmpl = "<label class='point_box'><input type='radio' name='point[]' value='%1$d'>%1$d point</label>";
 
     foreach (array(60, 40, 20) as $value) {
         if($points >= $value) {
             $r .= sprintf($tmpl, $value);
         }
         $r .= sprintf($tmpl, 10);
     }
 
     return $r;
 }
 
 echo point_boxes($_POST["points"]);

Indlæg senest redigeret d. 17.07.2013 09:08 af Bruger #17081

Bruger #17136 @ 17.07.13 09:41

Ja igen, nogle gang skal der flere øjne til at gøre tingene på og jeg valgt at bruge din, men jeg kun også have brugt Dan's, jeg synes begge var god og de virker fint hos mig!.

#Stefan Svensson Der var ingen fejl hos mig ikke ud over at jeg skal små tilpasse den efter min kode altså til at hente ned fra database hvor mege point bruger have osv..

#dan Jeg synes virkelig også din var god, og ja jeg har ikke så meget andet at sige tak for hjælpen både til dig og stefan !

i begge få et like fra min side! da begge er utrolig gode!

Sidste spørgsmål fra min omkring point system, hvordan er den bedste måde at validere når jeg skal ligge det ind i databasen?

Sådan her ser det ud når point skal komme frem

PHP kode

 <?php
 if ($stmt = $this->mysqli->prepare('SELECT `point` FROM `bruger` WHERE `id` = ?')) { 
 $stmt->bind_param('i', $id);
 $id = $_SESSION["id"];
 $stmt->execute();
 $stmt->bind_result($point);
 while ($stmt->fetch()) {
     if($point >= 10)
     {
         $arr = array(60, 40, 20);
         foreach ($arr as $value) 
         {
             if($point >= $value)
             {
                 echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"".$value."\"> ".$value." point </div>";
             }
         }
         echo "<div class=\"point_box\"><input type=\"radio\" name=\"point[]\" value=\"10\"> 10 point </div>";
     }
     else
     {
         echo "Du kan ikke oprette spørgsmål.";
     }
 }
 $stmt->close();
 } else {
     echo 'Der opstod en fejl i erklæringen: ' . $mysqli->error;
 }
 ?>

Sådan her ser det ud når jeg ligger noget ind i databasen:

PHP kode

 if ($stmt = $this->mysqli->prepare('UPDATE `bruger` SET `point`= `point` - ? WHERE `id`=?')) { 
     $stmt->bind_param('ii', $point, $id);
     $point = implode($_POST["point"]);
     $id = $_SESSION["id"];
     $stmt->execute();
     $stmt->close();
 }

Indlæg senest redigeret d. 17.07.2013 09:51 af Bruger #17136

Bruger #17081 @ 17.07.13 10:07

1.258

hmn sad lige og legede med, at man faktisk godt kunne lave den lidt mere avanceret:

PHP kode

 function point_boxes($points, $tmpl = false, $limit = false, $steps = false) {
 
     $limit = ($limit !== false) ? $limit : 10;
     $tmpl  = ($tmpl !== false) ? $tmpl : "<label class='point_box'><input type='radio' name='point[]' value='%1$d'>%1$d point</label>";
     $steps = ($steps !== false) ? $steps : array(60, 40, 20);
     $r     = "";
 
     if($points < $limit) return "<p>Du kan ikke oprette spørgsmål.<p>";
  
     foreach ($steps as $value) {
         if($points >= $value) {
             $r .= sprintf($tmpl, $value);
         }
         $r .= sprintf($tmpl, $limit);
     }
  
     return $r;
  }
  
 echo point_boxes($_POST["points"]);
 
 
 $format = "<li class='points box'><label for='points-%1$d'>%1$d point</label><input type='radio' name='points[]' id='points-%1$d'></li>";
 
 echo "<ul>". point_boxes($_POST['points'], $format) ."</ul>";
 
 
 echo point_boxes($_POST["points"], false, 5, array(30,15,10);

Indlæg senest redigeret d. 17.07.2013 10:10 af Bruger #17081

Bruger #17081 @ 17.07.13 11:14

1.258

Hej @Michael der er nogle små slåfejl / tastefejl i dit eksempel og du tager ikke helt højde for points < limit problemet ... men ellers meget fikst

Indlæg senest redigeret d. 17.07.2013 11:14 af Bruger #17081

Bruger #17136 @ 17.07.13 11:17

Du kan gøre det avanceret, og du kan gøre det kryptisk. Og så kan du gøre det fornuftigt

PHP kode

 function calc_points($points, $limit = false, $steps = false) {
  
      $limit = ($limit !== false) ? $limit : 10;
      $steps = ($steps !== false) ? $steps : array(60, 40, 20);
      $r     = array();
  
      if($points < $limit) return array();
   
      foreach ($steps as $value) {
          if($points >= $value) {
              $r[] = $value;
          }
      }
      
      if($limit) {
          $r[] = $limit;
      }
   
      return $r;
 }
 ?>
 <html>
     <ul>
         <?php foreach(calc_points($_POST["points"]) as $p): ?>
             <?php echo sprintf("<li class='points box'><label for='points-%1$d'>%1$d point</label><input type='radio' name='points[]' id='points-%1$d'></li>", $value); ?>
         <?php endforeach; ?>
     </ul>
 </html>

Nedestående som implementere point_boxes() er for mig, bare ikke så fornuftig som ovenstående:

PHP kode

 function calc_points($points, $limit = false, $steps = false) {
  
      $limit = ($limit !== false) ? $limit : 10;
      $steps = ($steps !== false) ? $steps : array(60, 40, 20);
      $r     = array();
  
      if($points < $limit) return array();
   
      foreach ($steps as $value) {
          if($points >= $value) {
              $r[] = $value;
          }
      }
      
      if($limit) {
          $r[] = $limit;
      }
   
      return $r;
 }
 
 function point_boxes($points, $limit = false, $steps = false) {
     foreach(calc_points($points) as $p) {
         echo sprintf("<li class='points box'><label for='points-%1$d'>%1$d point</label><input type='radio' name='points[]' id='points-%1$d'></li>", $value);
     }
 }
 
 ?>
 <html>
     <ul>
         <?php point_boxes($_POST["points"]); ?>
     </ul>
 </html>

Jeg kan da også gør det sådan her:

PHP kode

 $arr = array(60, 40, 20);
         $point = implode($_POST["point"]);
         if(isset($_POST["opret"]))
         {
             if(in_array($point, $arr))
             {
                 if ($stmt = $this->mysqli->prepare('UPDATE `bruger` SET `point`= `point` - ? WHERE `id`=?')) { 
                     $stmt->bind_param('ii', $point, $id);
                     $point = implode($_POST["point"]);
                     $id = $_SESSION["id"];
                     $stmt->execute();
                     $stmt->close();
                 }
 .......
 }
 }

Jeg har vel ikke bygge en function ind i en function? hmm

Indlæg senest redigeret d. 17.07.2013 11:18 af Bruger #17136

Bruger #10216 @ 17.07.13 11:22

4.283

@Dan

Hej @Michael der er nogle små slåfejl / tastefejl i dit eksempel og du tager ikke helt højde for points < limit problemet ... men ellers meget fikst

Sikkert - gider ikke teste 30 linjer kode der ingen nytte gør i mine nærværende udfordringer.

Problemer med point system

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler