function2重定義回避について

Question

エラー文 functionの2重定義
Fatal error: Cannot redeclare h()

次のふたつのファイルでh function(htmlspcialchars)を
使いたくて２つとも定義しているのですが、当然ならが
エラーになってしまいます。

ネットで調べて、function existsを使ってif文で分岐してみたり、
h functionの場所を動かしてみたりしたいのですが、エラーになって
しまいます。

エラーをなくし、ふたつのファイルでh funcitonを使う方法を
ご教授いただけないでしょうか？

ファイル名 menber_add

<?php

function h($str)
{
	echo htmlspecialchars($str, ENT_QUOTES);
}

---

ファイル名 confirm

<?php
session_start();

function h($str)
{
	echo htmlspecialchars($str, ENT_QUOTES);
}

if(isset($err)){
	require_once('member_add.php');
	exit;
}

Answer 1

どちらも同じ内容の関数であるなら、片方は不要では？

Comments

[yoshipu]

全部のコードをお見せできなくて申し訳なかったのですが、関数がありませんとエラーになるため、片方だとできませんでした。

[ttyp03]

同じ内容の関数を重複定義していると今後のメンテナンスに影響が出てくるので、１つにした方が良いと提案しました。
その結果、エラーが発生するのであれば、ファイル構造に問題があるわけです。
共通関数的なものは例えばcommon.phpなどに集約して、maisumakunさんの回答のように、使いたいところでrequire_onceすれば良いのです。

[yoshipu]

ありがとうございます。require_once使っていきたいです。

Answer 2

いちばん適切な方法は、「どちらとも違うファイルの1箇所でhを定義して、使う側でそのファイルをrequire_onceする」ということになります。

Comments

[yoshipu]

おっしゃる通りです。ファイルサイズの節約にもなりよかったのですが、考えが僕にはおよびませんでした。。

Answer 3

menber_add.phpをこう変更しましょう。

<?php

if (!function_exists(h)) {
    function h($str)
    {
        echo htmlspecialchars($str, ENT_QUOTES);
    }
}

Comments

[yoshipu]

こちらで動きました。ありがとうございました。