第9回課題フィードバック

[qeita]

@yunichan

課題提出お疲れ様でした。チューターのヤギです。
ブックマークアプリの課題からブログをテーマに作ってみたのですね。トップのスライダーやログイン画面背景とJSをところどころ使って演出を工夫しており、通常の課題と違うアプローチを取られていて面白いですし、色々と挑戦している様が伺えました。

gs_user_tableテーブルのsqlが上がっていなかったので、とりあえず授業で扱ったテーブルを入れてみまして確認しましたところ、以下フィードバックになります。

• 以下ブログ詳細ページのコードなのですが、ページ下部のリンクをformタグで記述していますね(同様の記述はブログ編集ページでもあります)。サーバに何かしらパラメータを渡して送るもの(登録や更新といったもの)ならformで良いのですが、こちらではページ遷移として使われているため、formで遷移はされているのですがaタグで記述した方が良いと思います。

<form action="top.php?" method="post">
  <input type="submit" value="top" id="btn">
</form>
<form action="content.php?id=<?=$row["id"]+1?>" method="post">
  <input type="submit" value="next" id="btn">
</form>
<?php if($_SESSION["kanri_flg"]=="1"){ ?>
<form action="edit.php?id=<?=$row["id"]?>" method="post">
  <input type="submit" value="edit" id="edit_btn">
<?php }?>  
</form>  

• 先の点と同じ箇所になるのですが、「next」ボタンをクリックするとidに1加算したURLに遷移するようになっています(以下の箇所)。ただ、ブログ記事が常に連番になっているわけではないため、githubにあがっているsqlをインポートしたものだと、例えばid=2の次のid=3の記事は存在しないため、中身がブランクで表示されてしまいます。id=14の記事が最後ですが、これも「next」ボタンを押すと存在しないid=15の記事を見に行くようになっています。
  この辺りはデザイン側でどうにかするか(「next」ボタンをトリにする)、あるいはブログ記事テーブルのうち今表示している記事の次の記事のidを取得するようにしなくてはいけません。
  PHPでもなんとかできるかもですが、第10回授業の一覧検索でやられていたのを応用して、記事情報をjson出力して、それをajaxで取得し次の記事のidを参照できるようにするのもアリかなと思います。

<form action="content.php?id=<?=$row["id"]+1?>" method="post">

あとは画像もjsと同じく、imgとかimagesといったディレクトリを作って格納したらphpファイルと他ファイルを仕分けられて見やすくなるかと思います。

次回課題もいいもの作れるよう期待しております。