php7 エラーメッセージが表示されるので、何とかしたいです。

前提・実現したいこと

php７エラーが発生しました。調べても対処方法が得られません。教えてもらえないでしょうか？

発生している問題・

エラーメッセージ　Notice: A non well formed numeric value encountered in （プログラムアドレス）

該当のソースコード

ソースコード

<HTML> <form method="post"> 質問１　<br> <input type="radio" name="Q1" value=1000;>①yes<br> <input type="radio" name="Q1" value=2000;>②no<br> 質問２　<br> <input type="radio" name="Q2" value=100;>①yes<br> <input type="radio" name="Q2" value=200;>②no<br> <br> <input type="submit"> </form> PHP <?php $qtq = 10; $value1 = 0; $value2 = 0; if(isset($_POST['Q1'])){ echo $_POST['Q1']; $value1=$_POST['Q1']; } if(isset($_POST['Q2'])){ echo $_POST['Q2']; $value2=$_POST['Q2']; }

$qtq=$value1+$value2;
echo $qtq;

試したこと

「Notice: A non well formed numeric value encountered in」で、検索
ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

$value1=$_POST['Q1'];　変数のタイプが違っているのでエラーまでは、突き止めました。
問題は、変数のタイプを合わせる事が出来るのか？
または、$_POST['Q1']中身から、変数が取り出されるのか？
あたまのなかがごっちゃになっているので、説明が分からなくなりまっした。ごめんなさい。
要は、どのようにして$value1に代数を入力できますか？

ちなみに、エラーメッセージの下に計算された答えは表示されているみたい出す。
ここにより詳細な情報を記載してください。

m.ts10806

2020/01/08 00:16

マークダウンのcodeが正しく使えていないので質問編集して調整してください。

m.ts10806

2020/01/08 00:17

あと、調べるだけでは不十分で、ご自身でもいろいろ試す必要はあります。

行動規範の内容に同意します

回答3件

ベストアンサー

html
1<html>
2<head>
3  <meta charset="UTF-8" />
4</head>
5<body>
6  <form method="POST">
7    <p>質問１　<br />
8    <input type="radio" name="Q1" value="1000" />①yes<br />
9    <input type="radio" name="Q1" value="2000" />②no<br />
10    </p>
11    <p>質問２　<br />
12    <input type="radio" name="Q2" value="100" />①yes<br />
13    <input type="radio" name="Q2" value="200" />②no<br />
14    </p>
15    <p><input type="submit" /></p>
16  </form>
17</body>
18</html>

php
1<?php
2
3function h($s) {
4  return htmlspecialchars($s, ENT_QUOTES, "UTF-8");
5}
6
7$qtq = 10;
8$value1 = 0;
9$value2 = 0;
10
11if (isset($_POST['Q1'])) {
12  $value1 = $_POST['Q1'];
13  echo h($value1);
14}
15else {
16  echo '質問1を選択してください。';
17}
18
19if (isset($_POST['Q2'])) {
20  $value2 = $_POST['Q2'];
21  echo h($value2);
22}
23else {
24  echo '質問2を選択してください。';
25}
26
27if (is_numeric($value1) && is_numeric($value2)) {
28  $qtq = intval($value1) + intval($value2);
29  echo h($qtq);
30} else {
31  echo '...';
32}

$_POST[～]で受信したものはすべて文字列として扱われます。
文字列と文字列を「+」演算子にて処理する際、
数値でないものを処理すると、示されたようなエラーメッセージが表示されます。
そのため、intval()で加工したり、あるいは(int)$value1などと型キャストを明示します。

それと、今は想像がつかないかもしれないけど、$_POST[～]で受信したデータは、
細工すればHTMLフォームに書いていないデータを送り込むことも容易にできるため、
悪意を持ってhtmlやjavascriptを含むものを送り込まれたら、
そのままwebブラウザに出力してしまうecho $_POST['Q1']というのはやってはいけないです。
それを回避するために、htmlspecialchars()があります。

投稿2020/01/08 05:29

編集2020/01/08 05:36

退会済みユーザー

総合スコア0

回答はすでにされている通りですが、他に気になる点。

・入力された文字列をそのまま出力してはいけない
⇒ 「クロスサイトスクリプティング脆弱性（XSS脆弱性）」について学びましょう。
これは実用ではなく、練習問題としてページを作っただけだと思いますが、学習目的であればなおさら、最初からウェブページ作成の基本をきちんとやったほうがいいです。

・タグの属性値は引用符で囲んだ方が良い
⇒今回のように数字をそのまま書くようなケースは問題ないですが、これも基本をきちんとという意味で。

あと、なぜセミコロンを書いてしまったのかの原因を掘り下げておいた方が良いです。
間違いから学ばない人は成長しません。

投稿2020/01/08 00:31

otn

総合スコア84505

HTML
1
2<form method="post">
3質問１　<br>
4<input type="radio" name="Q1" value=1000>①yes<br>
5<input type="radio" name="Q1" value=2000>②no<br>
6質問２　<br>
7<input type="radio" name="Q2" value=100>①yes<br>
8<input type="radio" name="Q2" value=200>②no<br>
9<br>
10<input type="submit">
11</form>
12