セキュリティの関わる問題なので削除させていただきました。

Question

セキュリティの関わる問題なので削除させていただきました。

Answer 1

そもそもCakePHPであれば、

$this->request->clientIp(false);

で、接続元IPアドレスが取得できますよ。
つまり、getAccessIp()自体が必要ないのでは？
そもそもこれらのfunctionがControllerで書かれているのかModelで書かれているのかLibで書かれているか等不明ですが。

Comments

[amaguri]

Controllerのcommon.phpで使われています
回答ありがとうございます。今回知りたかったのが引数をどう使うか
$access_ipの変数　getAccessIpの中身を複数回処理するのに使いたい場合に
引数を使って処理をしかったのでこのご質問させていただきまし

[kunai]

わざわざ引数で渡さずとも、Controllerの中であればどの関数からでも
$this->request->clientIp(false);
で、接続元のIPアドレスが取得できます。
引数で渡すことにこだわられている理由がよくわかりませんが、何か理由があるのであれば教えてください。
ちなみに引数を使った場合については、同様の別質問で聞かれているところでお答えしました。

[amaguri]

IPアドレス以外でも
同様なことが起きており
入り番短いコードであったIPで質問させていただきました

Answer 2

そもそも function IpCheck() は function FigIpCheck() に内包されているので不要ですので
function IpCheck() は 呼び出さないでもいいです。

提示されているソースでの回答です。 本来のソースのではないです

Comments

[amaguri]

回答ありがとうございます。今回知りたかったのが引数をどう使うか
$access_ipの変数　getAccessIpの中身を複数回処理するのに使いたい場合に
引数を使って処理をしかったのでこのご質問させていただきました

[amaguri]

なのでどうしたら
$access_ip = CommonComponent::getAccessIp();
を
function getAccessIp() {
if (empty($_SERVER['HTTP_X_FORWARDED_FOR'])) {
return $_SERVER['REMOTE_ADDR'];
}

$ips = explode(",", $_SERVER['HTTP_X_FORWARDED_FOR']);
return trim(end($ips));
}
の中だけでかけるかを知りたいです