php、functionの使い方

Question

phpでfunctionを使い、簡単なif文を作成しているのですがデータが空だったら「数字を入力してください」とだして、それ以外だったらfunctionを呼び出すというプログラムなのですが、前の画面で受け取ったデータが定義されていないとエラーがでてしまいます。
またreturnで返した変数でもエラーが起こっており、そちらはまだ、returnを理解出来ていないのでエラーコードもわからない状態です。何卒ご回答のほどよろしくお願いいたします。

送る側のデータ
<form method="post" action="gengo.php">
西暦を入力してください<br/>
<input type="text" name="seireki"><br/>
<input type="submit" value="ok">
</form>

<?php

$seireki=$_POST['seireki'];

if($seireki==""){
  echo '数字を入力してください';
  exit();
}
else{

  function abc()
  {
    if(1868<=$seireki&&$seireki<=1911)
    {
      echo '明治';
    }
    if(1912<=$seireki&&$seireki<=1925)
    {
       echo '大正';
    }
    if(1926<=$seireki&&$seireki<=1988)
    {
       echo '昭和';
    }
    if(1989<=$seireki&&$seireki<=2019)
    {
       echo '平成';
    }
  }
  }
  abc();
?>

$seireki=$_POST['seireki'];

if($seireki==""){
  echo '数字を入力してください';
  exit();
}
else{
  $wareki=gengo($seireki);
  echo $wareki;

  function gengo($seireki)
  {
    if(1868<=$seireki&&$seireki<=1911)
    {
      $gengo='明治';
    }
    if(1912<=$seireki&&$seireki<=1925)
    {
      $gengo='大正';
    }
    if(1926<=$seireki&&$seireki<=1988)
    {
      $gengo='昭和';
    }
    if(1989<=$seireki&&$seireki<=2019)
    {
      $gengo='平成';
    }
    else
    {
      $gengo='令和';
    }
    return ($gengo);
  }
}


?>

Comments

[yuicho]

同じようなPHPコードが二つあります。
不要な方削除お願いします。

> 前の画面で受け取ったデータが定義されていないとエラーがでてしまいます。
> returnを理解出来ていないのでエラーコードもわからない状態です。
エラーの内容をそのまま貼ってもらうほうが良いと思います。

Answer 1

関数もクラスも変数とかも本質的には同じなのですが、
**「定義しておいて、適宜使うもの」**なので、
そもそもロジックの中に書くのは間違っています。
関数の中にロジックを書き、使うものです。

そしてやはり「変数のスコープ」「引数」「返り値」など基本的なことが抜けてるように思います。
「関数」と言っても、PHPの標準関数も「関数」で概念としては同じです。
基本的にはINPUTがあってOUTPUTがある（どちらかだけ、どちらもないこともある）
「何か値を渡したら（←INPUT：引数）、所定の処理をして、結果を返す（←OUTPUT：戻り値）」
のが関数です。

例えば「１０足して返す」ならこう。

<?php
function add10(int $a = 0):int
{
    return ($a + 10);
}

$sum = add10(15); //これくらいなら一度受け取らなくてもいいけど
echo $sum;

例えば「１０足した値を出力する」ならこう。

<?php
function add10(int $a = 0)
{
    echo ($a + 10);
}

add10(15);

みたいな感じ。
（引数・戻り値の型宣言などは別途調べてください。）

先に「変数のスコープ」のページを提示していますが、
関数の外で定義した変数は関数内で直接参照することはできません（「関数からは見えない」と表現したらいいかな）

前の画面で受け取ったデータが定義されていないとエラーが

こういうのはコピー＆ペーストするのが原則ですが、おおよそ$seirekiだと思いますが、これは変数のスコープによるものです。

まずは関数の定義する場所から。
コードの見通しをよくするところから始めてください。

Comments

[yudouhu]

なるほど、よくわかりました。
ありがとうございます。

Answer 2

公式マニュアルで基礎をしっかり学習してみてはいかがでしょうか。
関数
返り値

Answer 3

前の画面で受け取ったデータが定義されていないとエラー

についてはどんなエラーかわからないので何とも言えないですが、gengo()はifの外で定義したほうがいいです。
そもそもifの中で定義すること自体が可読性の観点からよろしくないですし、普通PHPでは関数定義は呼び出しの前後どちらでもいいのですが、ifの中で定義してしまうと呼び出す前に定義されてないとエラーになるようです。