意図しない変数の内容 - PHPプロ！Q&A掲示板

2811

0P

意図しない変数の内容

質問日時 / 2010年7月16日 19:09 回答数 / 3件

Questioner: partagas Tweet

初心者です。フォームからの入力値（年月日）のチェックを行っています。「月」、「日」のチェック部分で$year、$monthがそれぞれ「1」になってしまい、エラーになります。定義以外で変数の中身をいじっている箇所は無いと思うのですが、原因がわかりません。

また、タイトル部分のif (isset($_POST['title']) && $_POST['title'])の右辺の必要性がわかりません。
(ソースは、このサイトの基礎講座を自分なりに手を加えたものです。)
お手数ですが、どなたかご教授お願いします。

<?php
if (isset($_POST['regit'])) {
//入力内容のチェック
$err_msg = array();
$today = array(date('Y'), date('n'), date('d'));
//全角→半角数字以外、値の範囲外はエラー
//年
if (isset($_POST['year'])) {
$year = mb_convert_kana($_POST['year'], 'a');
if (!is_numeric($year)) {
$err_msg[] = '「年」が数字ではない';
}
if ($year < $today[0] || $year > 2050) {
$err_msg[] = '「年」の入力値がおかしい';
}
} else {
$err_msg[] = '「年」が入力されていない';
}
//月
if (isset($_POST['month'])) {
$month = mb_convert_kana($_POST['month'], 'a');
if (!is_numeric($month)) {
$err_msg[] = '「月」が数字ではない';
}
if ($month < 1 || $month > 12) {
$err_msg[] = '「月」の入力値が範囲外';
} elseif ($year = $today[0] && $month < $today[1]) {
$err_msg[] = '「月」の入力値がおかしい';
}
} else {
$err_msg[] = '「月」が入力されていない';
}
//日
if (isset($_POST['day'])) {
$day = mb_convert_kana($_POST['day'], 'a');
if (!is_numeric($day)) {
$err_msg[] = '「日」が数字ではない';
}
if ($day < 1 || $day > 31) {
$err_msg[] = '「日」の入力値が範囲外';
} elseif ($year = $today[0] && $month = $today[1] && $day < $today[2]) {
$err_msg[] = '「日」の入力値がおかしい';
}
} else {
$err_msg[] = '「日」が入力されていない';
}
//タイトル
if (isset($_POST['title']) && $_POST['title']) {
$title = $_POST['title'];
} else {
$err_msg[] = 'タイトルを入力';
}

この質問への意見の募集は締め切られ、ポイントは既に配分されました。
意見を投稿することはできますが、ポイントを受け取ることはできません。

この質問に返信

ツリー一覧

意図しない変数の内容partagas

A01shimix27: } elseif ($year = $today[0] && $month <

A02signal$_POST['title'] これは【$_POST['title']】の値

A03partagasお二人とも有難うございました！最初の問題は単に

回答一覧

A01
answerershimix [7月18日 01:28]

27:        } elseif ($year = $today[0] && $month < $today[1]) {
41:        } elseif ($year = $today[0] && $month = $today[1] && $day < $today[2]) {

この部分は代入（=）じゃなくて比較（==）ですよね？

27:        } elseif ($year == $today[0] && $month < $today[1]) {
41:        } elseif ($year == $today[0] && $month == $today[1] && $day < $today[2]) {

>タイトル部分のif (isset($_POST['title']) && $_POST['title'])の右辺の必要性がわかりません。

では、右辺を削除してタイトルが未入力の場合にキチンと検知できるかどうか試してみてください。左辺のisset()がどういうものかはマニュアルを読めばわかると思いますが・・

　　http://jp.php.net/manual/ja/function.isset.php

A02
answerersignal [7月19日 10:29]

$_POST['title']

これは【$_POST['title']】の値が真となるかのチェック

しかし $_POST['title'] が "0" だった場合、偽となるため
タイトルが "0" は許されなくなります

$_POST['title'] === ""
正確には空文字ならエラーとなるはずです

スペースやらなにやらの対策を入れるとすれば
trim($_POST['title']) === ""
もしくは
strlen(trim($_POST['title'])) === 0

A03
answererpartagas [7月19日 18:38]

お二人とも有難うございました！

最初の問題は単に演算子のミスでしたか。おっちょこちょいでした。

二番目は、型のキャストをするので、$_POST['tiltle']は空文字と"0"のときFALSEとなり、条件全体としては、
NULL,空文字,"0"がFALSEとなるのですね。ただ、スペースは受け入れるので、それを回避するには、右辺を
trim($_POST['title']) === ""　や
strlen(trim($_POST['title'])) === 0
としなさいよ、と。

勉強になりました。有難うございました。
また、質問するかもしれませんが、その時はお付き合い頂ければと思います。

<<質問一覧へ