PHP E_NOTICE対処方法

Question

phpでsessionページをincludeするとsessyon_start()が2回呼ばれてE_NOTICEがでてしまい、出さない対処法として以下の2つしか思いつかないのですが、後者のエラー無視は避けて前者で書くべきなのでしょうか？

if(!isset($_SESSION)) session_start();

@session_start();

状況としては検索フォームページから検索ページを呼び出し、結果を求めてその値を持ったまま元の検索フォームページをincludeするという感じです。

以下の場合もたとえ値が入っていなくてもintvalで0に変換されるのだからそのままのほうが見やすいと思うのですが、E_NOTICEを潰すためには後者の書き方しか思いつかず、なんか気持ち悪いのです。

$x = intval($_POST['count']);

$x = 0;
if(isset($_POST['count'])){
  $x = intval($_POST['count']);
}

もっとスマートな書き方があったりしないでしょうか？
よろしくお願いします。

Answer 1

@でエラーを止めるのも、「何をしているのかわかった上で」使うなら有効な手段です。たとえば、file_get_contentsに失敗した場合にもE_WARNINGを投げますが、これを避ける方法は@しかありません。

なお、後者のissetチェックですが、PHP 7以降では??という演算子が用意されています。

//$_POST['count']がなくても何も起きない
$x = intval($_POST['count'] ?? 0);

Comments

[piyoon]

使用しているphpバージョンを書くべきでした。
現在５．６を使用しているのでShunsukeIzuiさんをベストアンサーにしました。
７を使うときには思い出して使おうと思います。

ありがとうございました。

Answer 2

私なら次のようにするかと思います。

$x = intval(filter_input(INPUT_POST, 'count'));

セッションのエラーに関しては後者でも特に問題はないかと思っています。

Comments

[piyoon]

@使ってもいいんですね。
いろいろなサイトを回っていると@はバグの温床（これは自分も納得です）として基本NGスタンスの意見が多く見られたのですが、代わりになる書き方を見ることがなかったため悩んでしまいました。

ありがとうございました。

[piyoon]

解決にはしたのですが、$_POST等ではなく普通の配列の場合issetでチェックするしか無いでしょうか？