コードの添削をお願いしたいPHP

実現したいこと
このコードはどこがおかしいのでしょうか？
入力フォームのコード書いています。

５０５エラーがでます。分け合って、エラー箇所の表示に時間がかかるため、
添削お願いしたいのでが、よろしくお願いいたします

php
1<?php
2session_start();
3
4$errors = array();
5
6if(isset($_POST['submit'])) { 
7
8$name = $_POST['name'];
9$furigana = $_POST['furigana'];
10$email = $_POST['email']; 
11$tel = $_POST['tel'];
12$lineid = $_POST['lineid'];
13
14
15$time4 =$_POST['time4'];
16$timehh1 =$_POST['timehh1'];
17
18
19$age =$_POST['age'];
20$work  =$_POST['work'];
21$nunsyuu =$_POST['nunsyuu'];
22$mokuhyuu =$_POST['mokuhyuu'];
23$badmoney =$_POST['badmoney'];
24$workkeikenn =$_POST['workkeikenn'];
25$question =$_POST['question'];
26
27
28$subject  =  $_POST['subject'];
29$body = $_POST['body']; 
30
31
32
33
34
35$name = htmlspecialchars($name, ENT_QUOTES); 
36$furigana = htmlspecialchars($furigana, ENT_QUOTES); 
37$email = htmlspecialchars($email, ENT_QUOTES);
38$tel = htmlspecialchars($tel, ENT_QUOTES);
39$lineid = htmlspecialchars($lineid, ENT_QUOTES);
40
41$time4 = htmlspecialchars($time4, ENT_QUOTES);
42$timehh1 = htmlspecialchars($timehh1, ENT_QUOTES);
43
44
45$age = htmlspecialchars($age, ENT_QUOTES);
46$work = htmlspecialchars($work, ENT_QUOTES);
47$nunsyuu = htmlspecialchars($nunsyuu, ENT_QUOTES);
48$mokuhyuu = htmlspecialchars($mokuhyuu, ENT_QUOTES);
49$badmoney = htmlspecialchars($badmoney, ENT_QUOTES);
50$workkeikenn = htmlspecialchars($workkeikenn, ENT_QUOTES);
51$question = htmlspecialchars($question, ENT_QUOTES);
52
53$subject = htmlspecialchars($subject, ENT_QUOTES);
54$body = htmlspecialchars($body, ENT_QUOTES);
55
56
57if($name === "") { $errors['name'] ="名前が入力されていません。 ";
58}
59
60if($furigana === "") { $errors['furigana'] ="フリガナが入力されていません。 ";
61}
62if($email === "") { $errors['email'] ="メールアドレスが入力されていません。 ";
63}
64if(!email.match(/^([a-zA-Z0-9])+([a-zA-Z0-9._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9._-]+)+$/)) {
65
66  errmsg = errmsg + "メールアドレスをご確認下さい　";
67}
68
69if($tel === "") { $errors['tel'] ="電話番号が入力されていません。 ";
70}
71
72
73if($time4 === "") { $errors['time4'] ="第1希望：※月と日付を選択してください。 ";
74}
75if($timehh1 === "") { $errors['timehh1'] ="第1希望：※時間指定1を選択してください。 ";
76}
77
78
79}
80
81if(count($errors) === 0) {
82$_SESSION['name'] = $name; 
83$_SESSION['furigana'] = $furigana; 
84$_SESSION['email'] = $email; 
85$_SESSION['tel'] = $tel; 
86$_SESSION['lineid'] = $lineid;
87
88$_SESSION['time4'] = $time4; 
89$_SESSION['timehh1'] = $timehh1; 
90
91
92 
93$_SESSION['age'] = $age; 
94$_SESSION['work'] = $work; 
95$_SESSION['nunsyuu'] = $nunsyuu; 
96$_SESSION['mokuhyuu'] = $mokuhyuu; 
97$_SESSION['badmoney'] = $badmoney; 
98$_SESSION['workkeikenn'] = $workkeikenn;
99$_SESSION['question'] = $question; 
100
101
102$_SESSION['subject'] = $subject; 
103$_SESSION['body'] = $body; 
104
105header('Location:http://www.rensyuu.shop/rensyuu.shop/form2.php');
106  exit();
107   }
108}
109
110
111if(isset($_GET['action']) && $_GET['action'] === 'edit'){
112
113
114$name = $_SESSION['name'];
115$furigana = $_SESSION['furigana'];
116$email = $_SESSION['email'];
117$tel = $_SESSION['tel'];
118$lineid = $_SESSION['lineid'];
119
120$time4 = $_SESSION['time4'];
121$timehh1 = $_SESSION['timehh1'];
122
123
124$age = $_SESSION['age'];
125$work = $_SESSION['work'];
126$nunsyuu = $_SESSION['nunsyuu'];
127$mokuhyuu = $_SESSION['mokuhyuu'];
128$badmoney = $_SESSION['badmoney'];
129$workkeikenn = $_SESSION['workkeikenn'];
130$question = $_SESSION['questio'];
131
132
133$subject = $_SESSION['subject']; 
134$body = $_SESSION['body'];
135 } 
136?> 
137
138
139
140<!DOCTYPE html> 
141<html>
142
143<head>
144<meta content="text/html; charset=utf-8"/>
145
146
147 <title>お問い合わせ</title> 
148</head>
149<body>
150 
151 
152<?php echo "<ul>";foreach($errors as $value) {
153
154echo "<li>"; 
155echo $value; 
156echo "</li>"; 
157} 
158
159echo "</ul>";
160
161?>
162
163<form action ="form1.php" method ="post">
164
165
166             
167  以下追加↓          
168          
169
170 <th>名前：※</th><td><input type ="text" name =" name" value =" <?php if(isset($name)){ echo $name; } ?>"/></td > 
171 </tr>
172 <tr> 
173 
174
175
176 <th>フリガナ：※</th><td>
177           
178           
179<input type="text" name="furigana" value ="<?php if(isset($furigana)){ echo $furigana; } ?>"/></td >
180              
181</tr>
182 <tr>
183 <th>メールアドレス：※</th><td>
184 
185 
186<input type ="text"name ="email" value ="<?php if(isset($email)){ echo $email; } ?>"/></td >
187</tr>
188 <tr>
189
190
191 <th>電話番号：※</th><td>
192
193           
194<input type="text" name="tel" value ="<?php if(isset($tel)){ echo $tel; } ?>"/></td >
195            
196</tr>
197 <tr>
198
199
200 
201 <th> LINE ID：</th><td>
202           
203<input type="text" name="lineid" value ="<?php if(isset($lineid)){ echo $lineid; } ?>"/></td >
204             
205</tr>
206 <tr>
207
208
209 <div class="b c font25 span lt_004 w95 cm"><span>下記選択枠より、</span><span>ご希望の日時をご選択ください。</span></div>
210
211
212
213       
214
215
216<div class="select_date font22">
217
218  <table><tbody><tr class="time_list">
219    <td class="b">第1希望：<span class="fontS">※</span></td>
220    <td class="fontSS">
221      <table><tbody><tr class="time_list2">
222        <td>
223          <input type="date" id="input_date1" class="cal lt_004 hasDatepicker" name="time4" value="<?php if(isset($time4)){ echo $time4; } ?>"/></td >
224           
225
226        </td>
227        <td>
228          <div class="wrap">
229          
230
231            <select id="input_time1" name="timehh1">
232              <option value="時間指定1"></option>
233              <option value="10:00～11:00">10:00～11:00</option>
234              <option value="11:00～12:00">11:00～12:00</option>
235              <option value="12:00～13:00">12:00～13:00</option>
236              <option value="13:00～14:00">13:00～14:00</option>
237              <option value="14:00～15:00">14:00～15:00</option>
238              <option value="15:00～16:00">15:00～16:00</option>
239              <option value="16:00～17:00">16:00～17:00</option>
240              <option value="17:00～18:00">17:00～18:00</option>
241              <option value="18:00～19:00">18:00～19:00</option>
242              <option value="19:00～20:00">19:00～20:00</option>
243              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
244              10:00～11:00</option>
245              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
246              11:00～12:00</option>
247              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
248              12:00～13:00</option>
249              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
250              13:00～14:00</option>
251              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
252              14:00～15:00</option>
253              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
254              15:00～16:00</option>
255              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
256              16:00～17:00</option>
257              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
258              17:00～18:00</option>
259              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
260              18:00～19:00</option>
261              <?php if(isset($timehh1) && $timehh1==="時間指定1") { echo "selected" ;} ?>>
262              19:00～20:00</option>
263            </select>
264          </div>
265        </td>
266      </tr></tbody></table>
267    </td>
268  </tr></tbody></table>
269
270 
271
272
273</div>
274<div class="font16 c b">※は必須項目となります。</div>
275
276
277
278<div class="padding5"></div>
279<div class="font25 black b c span w95 cm lt_004">
280  <span>より明確なご提案を</span><span>させて</span><span>いただくため</span><br class="hidden400"><span>↓下記アンケートにも</span><span>任意で</span><span>お答えください。↓</span>
281</div>
282<div class="padding5"></div>
283
284
285年齢：<div>
286           
287           <input type="text" name="age" value="">
288         
289       </div>
290
291       <div>
292         職業：<div>
293        
294           <input type="text" name="work" value="">
295         </div>
296       </div>
297
298       <div>
299        年収：<div>
300        
301           <input type="text" name="nunsyuu" value="">
302         </div>
303       </div>
304
305
306       <div>
307         目標月収：<div>
308         
309           <input type="text" name="mokuhyuu" value="">
310         </div>
311       </div>
312
313
314
315       <div>
316        借金：<div>
317        
318           <input type="text" name="badmoney" value="">
319         </div>
320       </div>
321
322
323      
324 <div>
325        副業経験：</div>
326        <div>
327
328           <input type="text" name="workkeikenn" value="">
329         </div>
330      
331
332
333       <div>
334        その他質問：</div>
335        
336           <textarea name="question"></textarea>
337
338
339<th>お問い合わせの種類</th><td>
340
341 
342 <select name ="subject"> <option value ="お仕事に関するお問い合わせ">
343 <?php if(isset($subject) && $subject==="お仕事に関するお問い合わせ") { echo "selected" ;} ?>>
344  お仕事に関するお問い合わせ</option>
345  
346  <option value ="その他のお問い合わせ">
347   <?php if( isset( $subject) && $subject ==="その他のお問い合わせ") {echo "selected"; } ?>>その他のお問い合わせ </option></select>
348   </td>
349  
350  </tr>
351 <tr>
352 <th>お問い合わせ内容 </th>
353 <td><textarea name="body"cols ="40" rows ="10"><?php if(isset($body)){ echo $body; } ?></textarea></td></tr><tr>
354
355  <td colspan="2"><input type ="submit" name ="submit"value="確認画面へ"></td></tr> 
356</table>
357 </form> 
358</body> 
359</html>

退会済みユーザー

2021/02/13 08:02

505が出ているのであれば、まずそれから解消して、そののちに切り分けを行い質問してください。ただ、現状では「全体がおかしいです」って回答が付くので、適切な教本で体系的な学習をした方が良いです。

m.ts10806

2021/02/13 08:05

１個１個指摘していくのは億劫になる内容です。ちゃんとしたレビューをお望みなら質問と回答という形ではおさまりません。

m.ts10806

2021/02/13 08:06 編集

PHPの中にそのままJavaScriptのコード混じってるみたいですし、これは厳しい。

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

回答3件

ツッコミどころが多数なので、ヒントのみ示します。

email.match()なんて、phpでjsは動かないよね。
それにメールアドレスを正規表現で検証するなら正確性はさておき、
filter_inputで検証フィルタにFILTER_VALIDATE_EMAILを使ってみるとか。
ここを改善すれば、とりあえず505エラーは突破できるかもしれませんし、ほかにも潜んでいるかもしれません。

if () { ... }
の{}の対応がとてもあやしいです。
phpの文法を理解してせめてphpのみの箇所だけでも、
適切にインデントしてかっこの対応ができているかチェックしてみてください。

php
1<?php
2session_start();
3
4$errors = array();
5
6if (isset($_POST['submit'])) { 
7
8    $name = $_POST['name'];
9    $furigana = $_POST['furigana'];
10    $email = $_POST['email']; 
11    $tel = $_POST['tel'];
12    $lineid = $_POST['lineid'];
13
14    $time4 =$_POST['time4'];
15    $timehh1 =$_POST['timehh1'];
16
17    $age =$_POST['age'];
18    $work  =$_POST['work'];
19    $nunsyuu =$_POST['nunsyuu'];
20    $mokuhyuu =$_POST['mokuhyuu'];
21    $badmoney =$_POST['badmoney'];
22    $workkeikenn =$_POST['workkeikenn'];
23    $question =$_POST['question'];
24
25    $subject  =  $_POST['subject'];
26    $body = $_POST['body']; 
27
28    //$name = htmlspecialchars($name, ENT_QUOTES); 
29    //$furigana = htmlspecialchars($furigana, ENT_QUOTES); 
30    //$email = htmlspecialchars($email, ENT_QUOTES);
31    //$tel = htmlspecialchars($tel, ENT_QUOTES);
32    //$lineid = htmlspecialchars($lineid, ENT_QUOTES);
33    
34    //$time4 = htmlspecialchars($time4, ENT_QUOTES);
35    //$timehh1 = htmlspecialchars($timehh1, ENT_QUOTES);
36
37    //$age = htmlspecialchars($age, ENT_QUOTES);
38    //$work = htmlspecialchars($work, ENT_QUOTES);
39    //$nunsyuu = htmlspecialchars($nunsyuu, ENT_QUOTES);
40    //$mokuhyuu = htmlspecialchars($mokuhyuu, ENT_QUOTES);
41    //$badmoney = htmlspecialchars($badmoney, ENT_QUOTES);
42    //$workkeikenn = htmlspecialchars($workkeikenn, ENT_QUOTES);
43    //$question = htmlspecialchars($question, ENT_QUOTES);
44    
45    //$subject = htmlspecialchars($subject, ENT_QUOTES);
46    //$body = htmlspecialchars($body, ENT_QUOTES);
47
48    if ($name === "") { 
49        $errors['name'] ="名前が入力されていません。 ";
50    }
51    if ($furigana === "") { 
52        $errors['furigana'] ="フリガナが入力されていません。 ";
53    }
54    if ($email === "") { 
55        $errors['email'] ="メールアドレスが入力されていません。 ";
56    }
57    //if (!email.match(/^([a-zA-Z0-9])+([a-zA-Z0-9._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9._-]+)+$/)) {  // <--- 正しいやり方を調べてね
58    //
59    //  errmsg = errmsg + "メールアドレスをご確認下さい　";
60    //}
61    if ($tel === "") { 
62        $errors['tel'] ="電話番号が入力されていません。 ";
63    }
64    if($time4 === "") { 
65        $errors['time4'] ="第1希望：※月と日付を選択してください。 ";
66    }
67    if ($timehh1 === "") { 
68        $errors['timehh1'] ="第1希望：※時間指定1を選択してください。 ";
69    }
70
71    if(count($errors) === 0) {
72        $_SESSION['name'] = $name; 
73        $_SESSION['furigana'] = $furigana; 
74        $_SESSION['email'] = $email; 
75        $_SESSION['tel'] = $tel; 
76        $_SESSION['lineid'] = $lineid;
77        $_SESSION['time4'] = $time4; 
78        $_SESSION['timehh1'] = $timehh1; 
79        $_SESSION['age'] = $age; 
80        $_SESSION['work'] = $work; 
81        $_SESSION['nunsyuu'] = $nunsyuu; 
82        $_SESSION['mokuhyuu'] = $mokuhyuu; 
83        $_SESSION['badmoney'] = $badmoney; 
84        $_SESSION['workkeikenn'] = $workkeikenn;
85        $_SESSION['question'] = $question; 
86        $_SESSION['subject'] = $subject; 
87        $_SESSION['body'] = $body; 
88        
89        header('Location:http://www.rensyuu.shop/rensyuu.shop/form2.php');
90        exit();
91    }
92}
93

などと、インデント量を吟味して書かないと、かっこの対応すら見失います。

$_POSTから受け取ったフォーム送信データを早々にhtmlspecialchars()で加工していますが、
ファイルに保存したりデータベースに格納するときにHTML出力用の加工が加わっていると、
いざ検索するときにヒットしなくなります。
検索する単語もhtmlspecialchars()で加工すれば回避できそうに見えますが、
そもそもHTML出力する直前で加工して出力しさえすれば、
HTML出力用に加工したデータを変数で引っ張り回す必要などありません。

よくある手として、
PHP で h() を書くなら一緒に echo しよう。 - こせきの技術日記
なんてのもあります。
いわゆる「h関数」ですが、いっそechoも組み合わせて出力させちゃえばいいって考え方もアリです。

htmlの文法をあれこれ無視した記述があり、
とても指摘しきれません。

option要素のselectedって、<option value="10:00～11:00" selected>10:00～11:00</option>の位置に出現しないと効果がないため、

php
1<option value="10:00～11:00"<?php
2  if (...) { 
3    echo ' selected';
4}
5?>>10:00～11:00</option>

みたいになります。

とか。
用事があるので、打ち切ります。

投稿2021/02/13 08:58

退会済みユーザー

総合スコア0

ベストアンサー

ざっくり書き直しておきます

PHP
1<?php
2$submit =   filter_input(INPUT_POST,'submit');
3$name =     filter_input(INPUT_POST,'name');
4$furigana = filter_input(INPUT_POST,'furigana');
5$email =    filter_input(INPUT_POST,'email',FILTER_VALIDATE_EMAIL); 
6$tel =      filter_input(INPUT_POST,'tel',FILTER_VALIDATE_REGEXP,["options"=>["regexp"=>"/^\d+$/"]]);
7$lineid =   filter_input(INPUT_POST,'lineid');
8$errors=[];
9
10function h($val){
11  return htmlspecialchars($val);
12}
13
14if(!is_null($submit)){
15  if($name === "")     $errors['name'] ="名前が入力されていません。 ";
16  if($furigana === "") $errors['furigana'] ="フリガナが入力されていません。 ";
17  if($email===false)   $errors['email'] ="メールアドレスをチェックして。 ";
18  if($tel===false)     $errors['tel'] ="電話番号をチェックして。 ";
19  if(count($errors)>0){
20    echo "<ul>";
21    array_walk($errors,function($x){
22      echo "<li>$x</li>\n";
23    });
24    echo "</ul>";
25  }else{
26    echo "OK!<br>\n";
27  }
28}
29?>
30
31<form method ="post">
32<table>
33<tbody>
34<tr>
35<th>名前：※</th>
36<td><input type ="text" name =" name" value ="<?=h($name)?>" required></td>
37</tr>
38<tr>
39<th>フリガナ：※</th>
40<td><input type="text" name="furigana" value ="<?=h($furigana)?>" required></td >
41</tr>
42<tr>
43<th>メールアドレス：※</th>
44<td><input type ="text"name ="email" value ="<?=h($email)?>" required></td >
45</tr>
46<tr>
47<th>電話番号：※</th>
48<td><input type="text" name="tel" value ="<?=h($tel)?>" required></td >
49</tr>
50<tr>
51<th> LINE ID：</th><td>
52<input type="text" name="lineid" value ="<?=h($lineid)?>"></td >
53</tr>
54</tbody>
55</table>
56<input type="submit" name="submit" value="send">
57</form>

投稿2021/02/13 17:55

yambejp

総合スコア116724

タグの中にタイプミスがありますが、大丈夫ですか？
空白が足りないです。
name="body"cols
になっています。

ただ、それだけで505エラーが出るかは分かりません。
普通にエラー番号を受けとめると、検証しているクライアントのHTTPリクエストは大丈夫かなと？想像してしまいます。（ブラウザが古くて、サーバーが受け付けなくなった古いタイプのリクエストを送っているなど。phpやhttpdの設定でエラーを厳し目に出す環境はあったような気もします）

もし単純なタイプミスを直しても、同じ現象が続く場合、エラーを返しているサーバーやがエラーを受け取っているクライアント（フラウザなど）が何かなどの動作環境がわかれば、もう少し見当がつく人がでてくるかもしれませんね。

505 HTTP Version Not Supported
サポートしていないHTTPバージョン。リクエストがサポートされていないHTTPバージョンである場合に返される。

投稿2021/02/13 08:21