質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.47%

  • PHP

    20900questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

PHPでのエスケープ処理

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 999

YamamotoHiroki

score 69

PHPでエスケープ処理がうまくいきません。
エスケープ処理自体初めてなので間違っている箇所があれば指摘してほしいです。
ぐぐってもいまいちわからなかったですし
アップロードしたサイト上で確認するとどうやらclassが反映されていないことと
<input type="submit" value="内容を確認する">
をクリックしてもNotFoundになってしまうことです。

<section>

<h2>送信内容確認</h2>

<?php
$user1 = $_GET["name"];
$user2 = $_GET["mail"];
$user3 = $_GET["place"];

if ($_GET["select"] == 1 ) {
    $user4 = 1;
}elseif($_GET["select"] == 2 )
{
    $user4 = 2;
}elseif($_GET["select"] == 3 ){    
    $user4 = 3;
}elseif($_GET["select"] == 4 ){
    $user4 = 4;
}elseif($_GET["select"] == 5 ){
    $user4 = 5;
}
$user5 = $_GET["question"];

echo "<form action=&quot; contact-send.php &quot; method= &quot; get &quot;>
<table class=&quot; ta1 mb1em &quot;>
<tr>
<th>お名前※</th>
<td>".$user1."</td>
</tr>
<tr>
<th>メールアドレス※</th>
<td>".$user2."</td>
</tr>
<tr>
<th>ご住所</th>
<td>".$user3."</td>
</tr>
<tr>
<th>お問い合わせ項目※</th>
<td>".$user4."</td>
</tr>
<tr>
<th>お問い合わせ詳細※</th>
<td>".$user5."</td>
</tr>
</table>";


?>

<p class="c">
<input type="submit" value="内容を確認する">
</p>

</section>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

本題とは異なりますが変数には意味のある名前をつけましょう。
$user1 = $_GET["name"];
$user2 = $_GET["mail"];
$user3 = $_GET["place"];
$name = $_GET["name"];
$mail = $_GET["mail"];
$place = $_GET["place"];

== でも良いのですが、 変数の型がわかっている場合は === を使ってみてください。
http://php.net/manual/ja/language.operators.comparison.php
if ($_GET["select"] == 1 ) {
    $user4 = 1;
}
if ($_GET["select"] === 1 ) {
    $user4 = 1;
}

XSSのセキュリティホールがあります。htmlspecialchars()などでエンコードをしてください。
XSSといわれると分からない場合、例えばnameにHTMLのタグが渡されるとそのタグが文字列として表示されるのではなく、タグとして実行されてしまうのはお分かりいただけると思います。
http://viral-community.com/blog/xss-1835/
$user1 = $_GET["name"];
$user2 = $_GET["mail"];
$user3 = $_GET["place"];
//中略
$user5 = $_GET["question"];
$user1 = htmlspecialchars($_GET["name"]);
$user2 = htmlspecialchars($_GET["mail"]);
$user3 = htmlspecialchars($_GET["place"]);
//中略
$user5 = htmlspecialchars($_GET["question"]);


アップロードしたサイト上で確認するとどうやらclassが反映されていないことと 
<input type="submit" value="内容を確認する"> 
をクリックしてもNotFoundになってしまうことです。 
以下のようにしてみてください。
echo "<form action=\"contact-send.php\" method=\"get\">

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

①fromタグの部分は以下のように変更してみてください。
echo "<form action='contact-send.php' method='get'>
または
echo "<form action=\"contact-send.php\" method=\"get\">

②改行も同様に\n としてエスケープ処理をしてください。面倒なら以下のように
ヒアドキュメントにしても良いと思います。

print<<<EOF
<form action="contact-send.php" method= "get">
<table class="ta1 mb1em">
<tr>
<th>お名前※</th>
<td>$user1</td>
</tr>
<tr>
<th>メールアドレス※</th>
<td>$user2</td>
</tr>
<tr>
<th>ご住所</th>
<td>$user3</td>
</tr>
<tr>
<th>お問い合わせ項目※</th>
<td>$user4</td>
</tr>
<tr>
<th>お問い合わせ詳細※</th>
<td>$user5</td>
</tr>
</table>
EOF;

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/04/05 18:33

    ありがとうございます。
    ヒアドキュメントにすると何かデメリットはあるのですか?

    なければわざわざエスケープ処理する必要がないと思ったのですが。

    キャンセル

  • 2015/04/05 18:40

    今回のケースではデメリットはありませんが、繰り返し処理とかは出来ないのが
    デメリットです。最も、PHPのHTML出力については面倒なエスケープやヒアドキュメントよりも
    テンプレートエンジンを使用したほうが開発は捗るかとおもいます。

    キャンセル

関連した質問

同じタグがついた質問を見る

  • PHP

    20900questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。