2017-08-30 15 views
0

文字列値を保持しているforeachループを使用してデータベースからフェッチして表示するラジオボタンがあります。私は、各データのidを保持しているラジオボタンと共に、別の隠れた入力を設定します。ラジオボタンの非表示入力を取得

$sql = "SELECT DISTINCT(class) FROM program WHERE level = '".$level."' AND title = '".$title."'"; 
$result = $conn->query($sql); 

if ($result->num_rows > 0) { 
    $i = 1; 
    while($row = $result->fetch_assoc()) { 
     echo '<ul>'; 
     echo '<li>'; 
     echo $row['class']; 
     echo '<input type="hidden" value="'.$row['class'].'"'; 
     echo '</li>'; 
     echo '<li>'; 
     echo '<ul><li><input name="timeslot['.$i.']" type="radio" value="" checked>N/A</li></ul>'; 

     $sql2 = 'SELECT * FROM program WHERE class = "'.$row['class'].'"'; 
     $result2 = $conn->query($sql2); 

     if ($result->num_rows > 0) { 
      while($row2 = $result2->fetch_assoc()) { 
       echo '<ul>'; 
       echo '<li>'; 
       echo '<input name="timeslot['.$i.']" type="radio" value="'.$row['class'].', '.$row2['timeslot'].'" />'.$row2['timeslot']; 
       echo '<input type="hidden" name="slot_id['.$i.']" value="'.$row2['id'].'"'; 
       echo '</li>'; 
       echo '</ul>'; 
      } 
     } 

     echo '</li>'; 
     echo '</ul>'; 
     $i++; 
    } 
} 

フォームを送信するとき、私の隠し値は常に各グループの最後の行を取得します。チェックされたラジオボタンでid値を取得するにはどうすればよいですか?

+0

SQLクエリを作成するときは、変数をバインドするだけでなく、文字列を連結する必要があります。あなたがしていることは潜在的に攻撃に対して脆弱です。 https://stackoverflow.com/questions/1860130/how-to-bind-sql-variables-in-phpをご覧ください。 – ivo

答えて

0

あなたは非表示のフィールドを入力していません。

また、ラジオの代わりにこの方法を変更し、サーバーのすべてのIDを必要としない場合は非表示にすることをお勧めします。

echo '<input name="timeslot['.$row2['id'].']" type="radio" value="'.$row['class'].', '.$row2['timeslot'].'" />'.$row2['timeslot']; 
関連する問題