私が書いたソースコード(php)のレビューをしていただきたい。

Question

私が書いたソースコードのレビューをしていただきたい。

以下のようなwebページを作りました。使用言語は「php」です。入門書を読みながら、なんとか目的は達成していますが、うまく設計できていないと考えています。オープン系の言語は触れる機会がないため、考え方・設計のヒントやアドバイスを有識者の方から頂きたくこのような投稿をさせていただきました（私は金融系の業務SEで主にCOBOLを使用しメインフレーム開発に従事しています）。

###＜今回作成したwebページ＞
#####①テキストボックス内に図鑑番号を入力し「検索」をクリック

#####②図鑑番号をキーにデータベースから内容を取得し表示

###＜私が書いたソースコード＞
#####①index.php
######（初回表示されるページ）

<?php require '../header.php'; ?>
<?php require 'poke_search.php'; ?>
<?php require '../footer.php'; ?>

「header.php」はhtmlヘッダーを記述
「footer.php」はhtmlフッターを記述

######①-1「poke_search.php」のソースコード
#######（初回表示されるページ）

図鑑番号
<form action="poke_search_result.php" method="post">
<input type="text" name="Poke_No">
<input type="submit" value="検索">
</form>

######②「poke_search_result.php」のソースコード
#######（「検索」クリック後に表示されるページ）

<?php require '../header.php'; ?>
<?php require 'poke_search.php'; ?>
<table>
<tr>
	<th>図鑑番号</th>
	<th>ポケモン名</th>
        ...
	<th>合計</th>
</tr>
<?php
$pdo=new PDO('mysql:host=localhost;dbname=pokemon;charset=utf8','abc','XXXXXXXXX');
$sql=$pdo->prepare('select * from pokedex where Poke_No=?');
$sql->execute([$_REQUEST['Poke_No']]);
foreach ($sql as $row) {
	echo '<tr>';
	echo '<td>', $row['Poke_No'], '</td>';
        ...
	echo '<td>', $row['Total'], '</td>';
	echo '</tr>';
	echo "\n";
}
?>
<?php require '../footer.php'; ?>

###聞いてみたい点
「poke_search.php」を「index.php」,「poke_search_result.php」にて上述のような形で使用しましたが、一般的でしょうか？このwebページでは、毎度「poke_search.php」がサーバから返される形になっています。上手く検索結果欄だけ更新するようにはどのようにすればよいでしょうか。

Comments

[m.ts10806]

設計はコード書く前にするものでは。

[takasima20]

この規模のやつならこんなもんじゃないの? しいていうなら、Poke_No の入力内容は残した方がいいかも。

Answer 1

質問がゆるいので何が聞きたいのかよく分かってませんが、入門レベルのコードで「一般的か？」と聞かれると、何もかもが一般的ではないです。

php において、複数ページに渡る処理を行う場合はロジックとビューの分離を行います。また、DB 処理が発生するものは、(小さなシステムでも)なんらかのフレームワークを使用することがよくあります。

大方針としてそういった分離のできる設計(データの持ち方)をすると良いです。

で、以下の部分を適当に解釈して回答します。

このwebページでは、毎度「poke_search.php」がサーバから返される形になっています。上手く検索結果欄だけ更新するようにはどのようにすればよいでしょうか。

サーバから返さないようにするのであれば、初回データ取得の段階で、全データをクライアントに渡し、JavaScript で検索処理を実現することになります。
適当なデータ構造をブラウザ側に渡し、それを JavaScript で加工し、表示するイメージですね。

検索処理をサーバサイドで実現するのであれば、JavaScript の ajax という機能を使用します。こちらは都度必要なデータをサーバに確認に行き、とってきたデータを JavaScript で表示するという機能です。

Comments

[sq_kida]

不明瞭な質問に対し丁寧なご説明ありがとうございます。精進します。

[退会済みユーザー]

ここ、面白いので参考まで。
https://github.com/kamranahmedse/developer-roadmap