PHP 関数化について

Question

現在PHPの学習をテキストブックを使って学んでいます。
何周かテキストブックをして次に関数化する練習としてテキストブック
の内容を関数化しようと試みるのですがやり方がわかりません。

関数化する目的などは人によって違う事はわかりますが、例として下記の
サンプルスクリプトを関数化する場合どうするか教えて頂きたいです。
私が考えれる関数化
①$questionの配列を関数化する
②form/inputを関数化する
また、関数化した内容を使って他のコードを書いて頂けたらとても助かります。

<?php require '../header.php'; ?> <p>秘密の質問を選択してください。</p> <form action="select-foreach-output.php" method="post"> <select name="question"> <?php $question=[ '最初に見た映画の名前は？', '最初に飼ったペットの名前は？', '初めて買った車の名前は？', '卒業した小学校の名前は？', '小学校の担任の先生の名前は？', '生まれた市町村の名前は？' ]; foreach ($question as $item) { echo '<option value="', $item, '">', $item, '</option>'; } ?> </select> <p>質問の回答</p> <p><input type="text" name="answer"></p> <p><input type="submit" value="確定"></p> </form> <?php require '../footer.php'; ?>

Comments

[m.ts10806]

質問は編集できます。
コードはマークダウンを利用してください

[m.ts10806]

あとまず「考えられる関数化」を実現したコードを書いて提示してください。作業依頼になってます

Answer 1

①$questionの配列を関数化する

配列は関数化するのに適当ではありません。

Comments

[daiki1]

ご回答有り難うございます。

Answer 2

テキストを何回か読んでるということは、関数の構文自体は分かっていて、
「そもそも どこを関数に切り出したら良いのか分からない」 という話ですよね。

この短いコードでそんなことをするか？ というのは置いておくと、

• 処理に名前をつける

というのは わかりやすい指標だと思います。
コメントをつける単位で、その内容を関数にするとか。

例えば、

// output option tag
foreach (...) {
  echo('<option>'...);
}

この部分を、こうするとか。

outputOptionTag($questions);

Answer 3

現在PHPの学習をテキストブックを使って学んでいます。

何周かテキストブックをして次に関数化する練習としてテキストブック
の内容を関数化しようと試みるのですがやり方がわかりません。

やり方がわかりません、ここをもう少し掘り下げましょうね。

関数化することのメリット、デメリットを検討して、
本当に関数化する必要があるのかどうか、
改変する作業コスト、実装難度も考慮してくださいね。

やり方がわかりません、やり方が本当にわからないんですか？

function get_question() {
  return [
    '最初に見た映画の名前は？',
    '最初に飼ったペットの名前は？',
    '初めて買った車の名前は？',
    '卒業した小学校の名前は？',
    '小学校の担任の先生の名前は？',
    '生まれた市町村の名前は？'
  ];
}

これも単に配列を返すだけのユーザー定義関数です。
これだけではなんのメリットも感じられないでしょうが、
例えばここにデータベースへのアクセスを組み合わせて、
（固定の質問文ではなく）データベース上に格納した質問文データを返すこともできそうです。
もしくは、質問文をまとめたテキストファイルがあって、そこから読み込んでもいいし。

関数の呼び元では

$question = get_question();

するだけで、質問文データがどこからやってきたのかを意識しなくてもいいわけです。

Comments

[daiki1]

ご丁寧に有り難うございます。

Answer 4

これと決まったやり方はありません。

考え方としては「役割分担」としたほうが良いコードになります。
「役割分担」が考え方として難しければ「部品化」でもいいです。

「同じもの」を作るけど「ちょっと違う」ところを引数で分岐などをかけて対応します。
例えば、色だけが違う、中身だけが違う、on/offを切り替えるだけ、など。

入力フォームの各入力コントロールなんていい部品化の素材です。

nameの設定値が変わる
valueの設定値が変わる
classの設定値が変わる

など、それだけで結局作るのはinput type=text です。

selectタグの場合もnameとoptionの中身とデフォルト値(selected)だけが変わるだけで、そこを引数で対応すればひとつ関数作るだけで違うselectが作れますよね？

と言った具合に、「役割分担」「部品化」の観点から関数を切り出してみてください。

Comments

[m.ts10806]

ちなみに同じような質問は過去にも出ていますし、私も答えたことがあります。

[m.ts10806]

と思ったら直近だと同じ質問者の過去質問だった。伝わってなかったか。
https://teratail.com/questions/217539

[daiki1]

何度もすみません。
有り難うございました。

[m.ts10806]

追記修正依頼は見られているのでしょうか。
質問も回答もコメントも編集可能です。コードがマークダウン利用されてないと読む気が失せる回答者は多いです。正しく再現確認もできません。
また「すみません」「ありがとう」だけでは質問者の現状や理解度は全く伝わりませんので、具体的にコメントしてください。
回答者は誰も急いでいないので「とりあえずのお礼」は欲してないことがほとんどです。
欲しいのは質問者の理解と問題解決のプロセスです。

Answer 5

①も②も関数化という観点から外れています。
ひとまず今のコードを関数化するとスッキリするかもという観点から書いてみました。
このサンプルではSelectタグを出力する関数MakeSelectと、Selectタグを返却する関数MakeSelect2を作ってみました。
どちらの関数も引数にアイテムの変数を指定するようにしています。
MakeSelectは関数内でHTMLを出力するのに対し、MakeSelect2は呼び出し元で出力するようにしています。
今回の場合、個人的には2の方が用途として適していると思います。

<?php
$question=[
    '最初に見た映画の名前は？',
    '最初に飼ったペットの名前は？',
    '初めて買った車の名前は？',
    '卒業した小学校の名前は？',
    '小学校の担任の先生の名前は？',
    '生まれた市町村の名前は？'
];

$address=[
    '東京都',
    '埼玉県',
    '千葉県',
    '神奈川県',
];

function MakeSelect($items)
{
    $html[] = '<select name="question">';
    foreach ($items as $item) {
        $html[] = '<option value="' . $item . '">' . $item . '</option>';
    }
    $html[] = '</select>';
    echo join($html, PHP_EOL) . PHP_EOL;
}
function MakeSelect2($items)
{
    $html[] = '<select name="question">';
    foreach ($items as $item) {
        $html[] = '<option value="' . $item . '">' . $item . '</option>';
    }
    $html[] = '</select>';
    return join($html, PHP_EOL) . PHP_EOL;
}
?>
<?php require '../header.php'; ?>
<p>秘密の質問を選択してください。</p>
<form action="select-foreach-output.php" method="post">
<?php MakeSelect($question) ?>
<?=MakeSelect2($address) ?>
<p>質問の回答</p>
<p><input type="text" name="answer"></p>
<p><input type="submit" value="確定"></p>
</form>
<?php require '../footer.php'; ?>