編集履歴

質問編集履歴

不要と言われた書き換え前を削除し、インデントやスペースをなるべく統一

2017/08/05 09:44

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -10,418 +10,232 @@
 書き換え後の物に不備や、足りない機能があったら教えていただきたいです。
+8/6追記
+不要と言われた書き換え前を削除し、
+インデントやスペースをなるべく統一しました。
+一応下記のソースでエラーは出なくなりましたが、問題がありましたらご指摘ください。
-書き換え前login_check.php
+書き換え後login_check.php
 ```
 <?php
-  require_once 'MDB2.php';
-  require_once("function.php");
-  session_start();
-  header("Content-type: text/html; charset=utf-8");
-  //パラメーター取得
-  $id = $_POST['id'];
-  $password = $_POST['password'];
-  //ログイン判定
-  //DB接続
-  $db = MDB2::connect(DNS);
-  if (PEAR::isError($db)) {
-    die($db->getMessage());
-  }
-  //プレースホルダで SQL 作成
-  $sql = "SELECT * FROM USERS WHERE ID = ? AND IS_USER = 1;";
-  //パラメーターの型を指定
-  $stmt = $db->prepare($sql, array('text'));
-  //パラメーターを渡して SQL 実行
-  $rs = $stmt->execute(array($id));
-  $count = 0;
-  while ($row = $rs->fetchRow(MDB2_FETCHMODE_ASSOC)) {
-    $id = $row["id"];
-    $salt = $row["salt"];
-    $db_password = $row["password"];
-    $reset = $row["reset"];
-    $count++;
-   }
-  $db->disconnect();
-   //ログイン失敗
-   if ($count != 1) {
-    $_SESSION["error_status"] = 1;
-    header("HTTP/1.1 301 Moved Permanently");
-    header("Location: login.php");
-    exit();
-   }
-   //パスワードリセット対応
-   if ($reset == 1) {
-     $_SESSION["error_status"] = 1;
-    header("HTTP/1.1 301 Moved Permanently");
-    header("Location: password_reset.php");
-    exit();
-   }
-  //パスワード生成
-   $hash = strechedPassword($salt, $password);
-   if ($hash == $db_password) {
-    //ログイン成功
-   //セッション ID の振り直し
-   session_regenerate_id(true);
-    //セッションに ID を格納
-    $_SESSION['id'] = $id;
-    //CSRF のトークン作成
-    $_SESSION["token"] = get_csrf_token();
-    //DB接続
-    $db = MDB2::connect(DNS);
-    if (PEAR::isError($db)) {
-      die($db->getMessage());
-    }
-    //プレースホルダで SQL 作成
-    $sql = "UPDATE USERS  SET LAST_LOGIN_TIME = ? WHERE ID = ?";
-    //パラメーターの型を指定
-    $stmt = $db->prepare($sql, array('timestamp','text'));
-    //パラメーターを渡して SQL 実行
-    $stmt->execute(array(date('Y-m-d H:i:s'), $id));
-   $db->disconnect();
-    //リダイレクト
-    header("HTTP/1.1 301 Moved Permanently");
-    header("Location: welcome.php");
-   } else {
-    //ログイン失敗
-    $_SESSION["error_status"] = 1;
-    header("HTTP/1.1 301 Moved Permanently");
-    header("Location: login.php");
-   }
+ini_set('display_errors', 1);
+error_reporting(E_ALL);
+require_once("function.php");
+session_start();
+header("Content-type:text/html;charset=utf-8");
+//パラメーター取得
+$id = $_POST['id'];
+$password = $_POST['password'];
+//ログイン判定
+//DB接続 ※
+$dsn = 'mysql:dbname=portfolio;host=localhost;charset=utf8';
+$user = 'root';
+$password = '';
+$dbh = new PDO($dsn,$user,$password);
+$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
+//プレースホルダでSQL作成
+$sql= "SELECT * FROM USERS WHERE ID =? AND IS_USER =1;";
+//パラメータの型指定
+$stmt = $dbh->prepare($sql);
+//パラメータを渡してSQL実行
+$rs = $stmt->execute($id);
+$count=0;
+while($row=$rs){
+  $id = $row["id"];
+  $salt = $row["salt"];
+  $db_password = $row["password"];
+  $reset = $row["reset"];
+  $count++;
+}
+$dbh = null;
+//ログイン失敗
+if($count !=1){
+     $_SESSION["error_status"]=1;
+     header("HTTP/1.1 301 Moved Permanently");
+     header("Location: login.php");
+     exit();
+}
+//パスワードリセット対応
+if($reset == 1){
+     $_SESSION["error_status"] =1;
+     header("HTTP/1.1 301 Moved Permanently");
+     header("Location: login.php");
+     exit();
+}
+//パスワード生成
+$hash = stretchedPassword($salt,$password);
+if($hash == $db_password){
+//ログイン成功
+//セッション ID振り直し
+     session_regenerate_id(true);
+//セッションにID格納
+$_SESSION['id']=$id;
+//CSRFのトークン作成
+$_SESSION["token"]=get_csrf_token();
+//DB接続
+$dsn = 'mysql:dbname=portfolio;host=localhost;charset=utf8';
+$id = '';
+$password = '';
+$dbh = new PDO($dsn,$user,$password);
+$dbh->setAttrbute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
+//プレースホルダでSQL作成
+$sql = "UPDATE USERS SET LAST_LOGIN_TIME = ? WHERE ID = ?";
+//パラメータの型を指定
+$stmt = $db->prepare($sql,array('timestamp','text'));
+//パラメータを渡してSQL実行
+$stmt->execute(array(date('Y-m-d H:i:s'),$id));
+$dbh = null;
+//リダイレクト
+header("HTTP/1.1 301 Moved Permanently");
+header("Location: welcome.php");
+}else{
+     $_SESSION["error_status"]=1;
+     header("HTTP/1.1 301 Moved Permanently");
+     header("Location: login.php");
+}
+print_r($_SESSION);
 ?>
 ```
-書き換え後login_check.php
-```
-<?php
-require_once("function.php");
-session_start();
-header("Content-type:text/html;charset=utf-8");
-//パラメーター取得
-$id =$_POST['id'];
-$password=$_POST['password'];
-//ログイン判定
-//DB接続 ※
-$dsn='mysql:dbname=portfolio;host=localhost;charset=utf8';
-$id='';
-$password='';
-$dbh=new PDO($dsn,$user,$password);
-$dbh->setAttrbute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
-//プレースホルダでSQL作成
-$sql= "SELECT * FROM USERS WHERE ID =? AND IS_USER =1;";
-//パラメータの型指定
-$stmt =$db->prepare($sql,array('text'));
-//パラメータを渡してSQL実行
-$rs=$stmt->execute(array($id));
-$count=0;
-while($row=$rs->fetch(PDO::FETCH_ASSOC)){
-  $id=$row["id"];
-  $salt=$row["salt"];
-  $db_password=$row["password"];
-  $reset=$row["reset"];
-  $count++;
-}
-$dbh=null:
-//ログイン失敗
-if($count !=1){
-  $_SESSION["error_status"]=1;
-  header("HTTP/1.1 301 Moved Permanently")
-  header("Location: login.php");
-  exit();
-}
-//パスワードリセット対応
-if($reset == 1){
-     $_SESSION["error_status"] =1;
-     header("HTTP/1.1 301 Moved Permanently")
-     header("Location: login.php");
-     exit();
-}
-//パスワード生成
-$hash=stretchedPassword($salt,$password);
-if($hash == $db_password){
-//ログイン成功
-//セッション ID振り直し
-session_regenerate_id(true);
-//セッションにID格納
-$_SESSION['id']=$id;
-//CSRFのトークン作成
-$_SESSION["token"]=get_csrf_token():
-//DB接続
-$dsn='mysql:dbname=portfolio;host=localhost;charset=utf8';
-$id='';
-$password='';
-$dbh=new PDO($dsn,$user,$password);
-$dbh->setAttrbute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
-//プレースホルダでSQL作成
-$sql="UPDATE USERS SET LAST_LOGIN_TIME = ? WHERE ID = ?";
-//パラメータの型を指定
-$stmt = $db->prepare($sql,array('timestamp','text'));
-//パラメータを渡してSQL実行
-$stmt->execute(array(date('Y-m-d H:i:s'),$id));
-$dbh=null;
-//リダイレクト
-header("HTTP/1.1 301 Moved Permanently");
-header("Location: welcome.php");
-}else{
-  $_SESSION["error_status"]=1;
-  header("HTTP/1.1 301 Moved Permanently");
-  header("Location: login.php");
-}
-?>
-```