質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

質問する

ただいまの
回答率

88.38%

mysqlテーブルの全レコードを区切ってブラウザ出力がなかなかできない…

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 1,550
退会済みユーザー

退会済みユーザー

○現在の各状態
・各テーブル
users_infoテーブル
ユーザーの基本情報
イメージ説明

days_infoテーブル
PHP側で指定してInsertした日付一覧
イメージ説明

attendances_infoテーブル
各ユーザーのその日の勤怠状況
イメージ説明

・PHPソース 

<!DOCTYPE html>
<html>
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
    <link rel="stylesheet" type="text/css" href="AppStyle.css">
    <title>勤怠状況</title>
</head>
<body>
<?php
//-- MySQL接続文省略  --//
      //ユーザー情報参照
      $query = mysql_query('select * from users_info');
      //PHP側で入力し指定した日付参照
      $days = mysql_query('select days from days_info');
      //各ユーザーの勤怠状況参照
      $attendances = mysql_query('select attendances from attendances_info');

      print("<div>");
      print("<form method='POST' action='./RegistTabaleData.php'>");
      print('<table border="1" align="center">');
      print("<tr><th></th>");

      //日付レコード参照
      while ($row = mysql_fetch_array($days)) {
            print("<th>".$row['days']."</th>");
      }

      print("</tr>");

       //送信ボタン
       $button = "<input name='submission' type='submit' value='送信'/>";
       //ユーザー名参照
       while ($row = mysql_fetch_assoc($query)) {

         $user_name = $row['user_name'];

         print("<tr><td>".$user_name."</td>");
        //-- ここが問題 --//
        //勤怠状況参照
        while($day = mysql_fetch_assoc($attendances)){    
              $selectBox = "<select name='selects[]'><option>".$day["attendances"]."</option><option>早退</option><option>休暇</option><option>出勤</option>";
              print("<td>".$selectBox."</td>");
         }
            print("</tr>");
        }


        print("<tr><td>".$button."</td></tr>");

        print('</table>');
        print("</form>");
        print("</div>");
?>
</body>
</html>

・ブラウザでの実行
イメージ説明
↓

最終的に以下の状態を望んでます。
イメージ説明

○質問
レコードを上記の画像のように区切ることがうまく行きません。
アルゴリズム的な問題かも知れませんが、やり方の問題もあるかと考えています。
どうすればうまく区切れるのか、、わかる方ご教授お願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2 件

link

+3

回答を書いてるうちに解決されてましたが、せっかくなので載せます(笑

//-- ここが問題 --//
の部分の勤怠状況参照のループ処理で全てのデータを処理してしまっているのが原因かと思われます。

端折って書くと以下のようになりますが 

//ユーザー名参照
while() { // ←ループ1
  //勤怠状況参照
  while() { // ←ループ2
    // 横一列にならぶ問題の箇所
  }
}


[ループ2]にて9回ループしているようです。(つまり3人×3日分のデータ)
ここで、一人分の処理が終わったら[ループ２]を抜けてしまえば上手くいきそうです。

ただし、 

$day = mysql_fetch_assoc($attendances)


DBから1行ずつ取得しているので、今のままのコードでは見づらかったり分かりづらかったりします。
ですので、多少処理に無駄があっても分かりやすいコードにしてみた方がいいと思います。 

/* DBからデータを取得する準備 */
// [ユーザー情報参照]と[PHP側で入力し指定した日付参照]は省略

// 各ユーザーの勤怠状況参照
// $attendances = mysql_query('select attendances from attendances_info');
// ↓ 全項目取得に修正
$attendances = mysql_query('select * from attendances_info');



/* データを取得し配列に格納する処理を行う */
// ユーザー名参照
$users = array();
while ($row = mysql_fetch_assoc($query)) { // ←ループ1
  $users[] = $row; // ←データを用意する
}

// 勤怠状況参照
$attendanceDatas= array();
while ($day = mysql_fetch_assoc($attendances)) { // ←ループ2
  $attendanceDatas[] = $day; // ←データを用意する
}



/* 用意された配列データを用いて表示を行う */
foreach ($users as $user) {
  foreach ($attendanceDatas as $data) { // ここでの$dataは質問者さんの$dayに相当
    if ($user['user_name'] == $data['user_name']) {
      // 表示を行う、または表示するものを準備する
    }
  }
}


大体このようになるかと思います(一部抜粋なのでちゃんと動くように補完してください)

(DBのデータを配列に入れなおす処理、実は一発で全て取得する関数があるのでループを回す必要がなかったりしますが、分かりづらいと思うのでとりあえず今回は上記のような形と致しました)

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/05/21 12:20

    ご回答、本当に有難うございます。
    foreachでネストする手があったなんて…思いつかなかったです。
    大変参考になりした。

    キャンセル

link

checkベストアンサー

+1 

//勤怠状況参照
while($day = mysql_fetch_assoc($attendances)){

ですが、SQL が以下のようになっており、一度に全ユーザー分の出欠情報を取ってきてしまっています。 

select attendances from attendances_info

これを、以下のようにして、外側の while ループでいま注目してるユーザー $user_name についてのみの出欠情報を取ってくればうまく動くと思います。 

select attendances from attendances_info where user_name = '$user_name'

すみません、このような SQL を PHP でどう書くかはわからない（10年以上触ってないので忘れてしまった）ので、正しい書き方を調べてみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/05/20 18:34

    ご回答、大変うれしいです。
    やって見てうまくいきました！
    SQLで条件式が使えることは知ってますが、こんな風に使えるんですね！大変参考になりました。本当にありがとうございます！！

    キャンセル

15分調べてもわからないことは、teratailで質問しよう！

  • ただいまの回答率 88.38%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる
質問する

関連した質問

同じタグがついた質問を見る

SNSアカウントでログイン

teratailアカウントでログイン

思考するエンジニアのためのQ&Aサイト「teratail」について詳しく知る