【PHP】PDOでexecute & fetchした値が、変数に格納されるときとされない時がある

Question

###前提・実現したいこと

データベースに新しい管理者のIDとパスワードを登録するために、PDOでDBに接続して新しいIDとパスワードをINSERTしようとしているのですが、
その際にSELECT文実行時にはなかった以下のエラーが発生してしまいます。

###発生している問題・エラーメッセージ

Notice: Undefined variable: php_skill2 in /home/users/0/******/***/****/admin/manage_done.php on line 39

上のエラーメッセージは、SELECT文実行時（下記ソース）では現れませんでした。
■connect_db.php

<?php

define("HOST","****");
define("DBNAME","****");
define("DBUSER","****");
define("DBPASS","****");

function pdo()
{
      $pdo = new PDO('mysql:host=' . HOST . ';dbname=' .DBNAME,DBUSER,DBPASS);
      $pdo->exec('set names utf8');
      $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
      return $pdo;
}
$pdo = pdo();
try{
$stmt = $pdo->prepare('SELECT login_no '
. ', login_id '
.', login_pass '
.' FROM テーブル名 '
.' WHERE login_id = ? '
.' AND login_pass = ? '
);
}catch(PDOException $e){
   $error = $e->getMessage();
}
$stmt->execute(array('ID名', 'パスワード名'));
$php_skill = $stmt -> fetchAll(PDO::FETCH_ASSOC);
foreach($php_skill as $k => $v){
	$result = $v['login_id'];
	$result2 = $v['login_pass'];
}
?>

###該当のソースコード

問題のソースコード

■manage_done.php

<?php
session_start();
//ログインボタンを押さないで
//このページにいる場合はログインページに戻す
if(empty($_SESSION['auth'])){
    header('Location:login.php');
    exit;
}
$php_skill2 ='';
require_once('connect_db.php');
$stmt2 = $pdo->prepare("INSERT INTO 
テーブル名
(login_id, login_pass) 
VALUES 
('".$_SESSION['id']."', '".$_SESSION['pass']."')");

try{
$stmt2->execute(array( $_SESSION['id'], $_SESSION['pass']));
}catch(PDOException $e){
$error = $e->getMessage();
}
try{
$php_skill2 = $stmt2 -> fetchAll(PDO::FETCH_ASSOC);
}catch(PDOException $e){
$error = $e->getMessage();
}
print_r($php_skill2); ←エラーが吐かれている箇所

foreach($php_skill2 as $k => $v){
	$result3 = $v['login_id'];
	$result4 = $v['login_pass'];
}
echo $result3;

?>

###試したこと
$_SESSION['id']にはechoした所ちゃんと値が渡っていたので、上記２つのコードで違いがあるとするとINSERTの書き方が間違えているのかと考えたのですが、どうかいたらいいのかよくわかりませんでした。

質問初めてなので至らない所があったら申し訳ありません。

ご教授お願い致します。

###補足情報(言語/FW/ツール等のバージョンなど)
FWは使っていません

Answer 1

$stmt2 = $pdo->prepare("INSERT INTO

テーブル名
(login_id, login_pass)
VALUES
('".$_SESSION['id']."', '".$_SESSION['pass']."')");

prepareで処理しているに直接変数を埋め込んじゃだめでしょ
VALUES(?,?)

のようにしてください

またINSERT下だけではfetchしてもデータは得られません
SELECT文を発行して下さい

Comments

[zoizoi]

回答ありがとうございます。

SELECT文を発行したら$php_skill2にデータが入り、DBに登録できました！

解決できたのでベストアンサーに選ばせていただきました。ありがとうございます！

Answer 2

単純に「変数$php_skill2がないよ」っていうエラーです。
しかしコードを見る限り、PHPにおいては問題ないコードにみえます。
つまり必ず$php_skill2が生成されるステップ、fetchAllのステップは通過すると思われます。
何か端折っているコードはないですか？

それからINSERTのSQL文ですが、executeで値を指定するなら、SQL文に値を埋め込む必要はないのでは？
SELECTと同様に?に置き換えて良いと思います。

Comments

[zoizoi]

回答ありがとうございます。

端折っている箇所がわからなかったので、PHPソース全文載せました。

INSERT文のSQLは？に置き換えようと思います。ありがとうございます！

Answer 3

エラー自体は、php_skill2っていう変数はどっから来たねん！
みたいな未定義の変数だよってエラーなので、変数の初期化を最初にすればいいと思います。

Comments

[zoizoi]

回答ありがとうございます。

$php_skill2 = '';

を先頭行に追加したところ、該当エラーは消えたのですが、結局 fetchALLした値が入っておらず空のままで、print_rしても何も返って来ません・・・。