編集履歴

回答編集履歴

加筆修正

2019/11/15 09:02

投稿

スコア0

test CHANGED Viewed

@@ -48,7 +48,7 @@
 そう思うのであれば、そもそも teratail でデバッグ依頼のようなことを投稿してはいけないのです。
-それを丁寧にやってくれる質問者さんが解決にたどり着いています。
+それを丁寧にやってくれる質問者さんが、経験豊富な回答者の情報を上手に引き出し、解決にたどり着いています。

リニューアル

2019/11/15 09:02

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -1,3 +1,113 @@
+まず、情報不足の段階で、解決につながるかどうか怪しい情報を提供し
+混乱させてしまったことについては申し訳ないと思っております。
+teratail など第三者に自分の関わるコードを評価してもらい
+解決につながるアドバイスを得る場合において、
+大事なことは、**質問者さんと回答者側とで**
+**同じコード、同じデータでテストし評価すること**です。
+開発中のコードを全部開示しろ、
+動かすのに必要なデータ全部開示しろ、ということではなく、
+要求された機能を満たすだけの最小限（ミニマル）なコードと、
+要求された機能をテストするのに必要な、リアルと関係ない仕様を満たすテストデータが必要で、
+解決に至る回答をつけようとしてくれる第三者の手元で試せるよう、
+質問者さんが頑張って情報提供しないと始まらないのです。
+それを怠ってしまうと、
+回答者側が再現できなかった部分を適当に解釈してコードを書き、
+実態に則さない状態でテストして「もしかしたらこれで解決するんじゃないだろうか」という状態の
+回答が寄せられます。
+（私が最初に書いたみたいに）
+質問者さんが直接動かしている実行環境のコードやデータを開示できないのは、実によくある話です。
+それでも teratail などを利用しようとするなら、
+開示できる範囲のミニマルなもの（コード、データ構造、テストデータ）を別途工夫して用意しないといけないのです。
+めんどうですか？ わずらわしいですか？ 手間暇かけなくないですか？
+そう思うのであれば、そもそも teratail でデバッグ依頼のようなことを投稿してはいけないのです。
+それを丁寧にやってくれる質問者さんが解決にたどり着いています。
+回答しようとする側は、
+足りない情報を自分の経験や知識に照らし合わせて補おうとしたり、
+ネット上の参考情報がないか調べる時間を使ったりと、
+質問者さんが考えるよりも多くの時間を（何人もの人が大なり小なり）費やしています。
+本業を抱えている人も多く、業務の合間を縫って手助けしようとしてくれているのに、
+情報が足りない、テスト用のデータがない部分を工夫するために余計に手間暇をかけて付き合おうとしています。
+質問者さん一人がめんどうでわずらわしいのではないのです。
+そして今回は、password_verify()の件以外に、
+文法エラーや、データベースアクセスにまつわる作法など、さまざまな
+「なぜここはこういう書き方をしているのだろう？」という箇所があり、
+実行環境の差異を騙して動かそうとしても、password_verify()の件じゃないところで引っかかり、
+回答する側としてはそこに手間暇がかかっています。
+そこを解決してからpassword_verify()の件に取り組みたい主旨で私は回答を書きました。
+それなのに、
+「※質問に関係のないことは情報追加・修正依頼に書かないでください。」などと言われては、
+デッドロックですよ。
+これでアドバイスしようかなと思っていた潜在的な回答者を追い出してしまっています。
+（情報セキュリティ方面の第一人者も戸惑っていましたし。）
+質問者と回答者側が同じテーブルにつかないと始まらない、
+そのために必要な指摘には柔軟に対応してください。
+なにもリアルなものを出してくれとは言っていません。
+第三者が検証・テストできる情報がほしいだけです。
+以下、リニューアル前の回答：
+---
 えーっと、回答につながるかどうかわかりませんが、
 整形ツールにかけた上で書き直してみた。

見直し

2019/11/15 09:00

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -260,7 +260,7 @@
     $stmt = $pdo->query("SELECT * FROM user_tb");
-    while($row = $stmt -> fetch(PDO::FETCH_ASSOC)) {
+    while($row = $stmt->fetch(PDO::FETCH_ASSOC)) {  // ここいらは何をやっているかわかんないから放置してる
         $emails .= "[".$row["email"]."]";

見直し

2019/11/13 09:30

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -109,3 +109,183 @@
 }
 ```
+---
+signup.phpもガッチャガチャ引っ掻き回してみた。
+これもあくまで机上デバッグで文法エラーが出ないだろうというところまで詰めただけ。
+bindParamじゃなくbindValueを使ったり、
+try～catchの使い方を変えたり、
+`$pdo`を関数の引数に使って外にだしてもアクセスできるようにしたり、
+**全角空白でインデント**している箇所を直したり、
+まだ納得していない部分として、
+ra()で新しいユーザーIDを作っているみたいだけど、
+データベース上に登録済みのを生成しているかもしれないので、
+ユーザー登録する前に検証する処理を入れるべき。
+```php
+<?php
+function ra(){
+    $len="";
+    for( $i=0; $i<10; $i++ ){
+        $len.=substr(str_shuffle('abcdefghijklmnopqrstuvwxyz'), 0, 10);
+    }
+    return $len;
+}
+function newusers( $pdo, $userids ) {
+    $kuserid = ra();
+    if (strpos($userids, $kuserid) !== false) {
+        newusers();
+    } else {
+        $name = filter_input(INPUT_POST, 'name');
+        $email = filter_input(INPUT_POST, 'email');
+        $pass1 = filter_input(INPUT_POST, 'password');
+        $pass2 = filter_input(INPUT_POST, 'password2');
+        $niti = date("Y/m/d H:i:s");
+        $pdo->beginTransaction();
+        $stmt = $pdo->prepare("INSERT INTO user_tb (name, password,password2,userid,createddate,email) VALUES (:name, :password,:password2,:userid,:createddate,:email)");
+        $pass1s = password_hash($pass1, PASSWORD_DEFAULT);//ここでハッシュ化
+        $pass2s = password_hash($pass2, PASSWORD_DEFAULT);//ここでハッシュ化
+        $stmt->bindValue(':name', $name, PDO::PARAM_STR);
+        $stmt->bindValue(':password', $pass1s, PDO::PARAM_STR);
+        $stmt->bindValue(':password2', $pass2s, PDO::PARAM_STR);
+        $stmt->bindValue(':email', $email, PDO::PARAM_STR);
+        $stmt->bindValue(':userid', $kuserid, PDO::PARAM_STR);
+        $stmt->bindValue(':createddate', $niti, PDO::PARAM_STR);
+        $stmt->execute();
+        $pdo->commit();
+        $_SESSION['id'] = $kuserid;
+        exit("ok");  // ここでphpの処理が抜けちゃうけどいいのかい？
+    }
+}
+function fu001 ( $pdo, $emails ) {
+    $email = $_POST['email'];
+    if (strpos($emails,$email) !== false) {
+        exit( "emailerror" );
+    } else {
+        newusers($pdo, $userids);
+    }
+}
+$emails = "";
+$userids = "";
+try {
+    $pdo = new PDO('mysql:host=example.jp;dbname=user_data;charset=utf8','user','password',
+        array(PDO::ATTR_EMULATE_PREPARES => false, PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION)
+    );
+    $stmt = $pdo->query("SELECT * FROM user_tb");
+    while($row = $stmt -> fetch(PDO::FETCH_ASSOC)) {
+        $emails .= "[".$row["email"]."]";
+        $userids .= "[".$row["userid"]."]";
+    }
+    fu001($pdo, $emails);
+    //～～～省略～～～
+}catch(PDOException $e){
+　exit("spl");
+}
+```