teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

コード微修正

2017/10/12 03:40

投稿

miyahan
miyahan

スコア3095

answer CHANGED
@@ -1,20 +1,16 @@
1
1
  メソッドチェーン(`$xxx->yyy->zzz()` みたいに `->` でつないでいく書き方)の途中の null 判定は難しいと思います。**その前に判定を入れる**しかないとかなーと。
2
2
 
3
- 表示する内容がないのにテーブルのヘッダだけ表示しているのもおかしいので、一般的にはこんなつくりになると思います。
3
+ 表示する内容がないのにテーブルのヘッダだけ表示しているのもおかしいので、一般的にはこんな感じのつくりになると思います。
4
4
 
5
5
  ```php
6
- <?php
7
- $values = $app->getValues();
8
- ?>
9
-
10
6
  <div>
11
- <?php if ($values): ?>
7
+ <?php if ($reservation = $app->getValues()): ?>
12
8
  <table>
13
9
  <thead>
14
10
  <tr><th>お名前</th><th>ご予約日</th><th>コース</th><th>確認・編集</th></tr>
15
11
  </thead>
16
12
  <tbody>
17
- <?php foreach($values->users as $user): ?>
13
+ <?php foreach($reservation->users as $user): ?>
18
14
  <tr>
19
15
  <td data-label="お名前"><?= h($user->name);?></td>
20
16
  <td data-label="ご予約日"><?= h($user->date); ?></td>