データベースをチェックするコードの関数化がうまくできません

Question

データベースをチェックするコードを関数化しようとしているのですが、うまくいきません。

関数化しないときは、うまく動作します。

関数化する前の文（想定通りの動作をします）

try {
	global $wpdb;
	$users_table = $wpdb->prefix . "users"; //ユーザーテーブルを定義
	$session_ID = $_SESSION["ID"]; //セッションIDを定義
	$check_DB_users_table = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $users_table WHERE ID = %d", "$session_ID" ), ARRAY_A );
} catch( PDOException $e ) { // PDOException の場合のみ
	wp_die('データベースエラーが発生しました。');
} catch( Exception $e ) { // PDOException でキャッチできなかった例外はこっち
	wp_die('何らかのエラーが発生しました。');
}

echo $check_DB_users_table['user_email'] //ユーザーテーブルの指定されたIDに登録されたメールアドレスが出力されます。

上の文を関数化した結果、うまくいかない例

function check_DB_users_table_by_id() {
  try {
    global $wpdb;
    $users_table = $wpdb->prefix . "users"; //ユーザーテーブルを定義
    $session_ID = $_SESSION["ID"]; //セッションIDを定義
    $check_DB_users_table = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $users_table WHERE ID = %d", "$session_ID" ), ARRAY_A );
  } catch( PDOException $e ) { // PDOException の場合のみ
    wp_die('データベースエラーが発生しました。やり直してください。');
  } catch( Exception $e ) { // PDOException でキャッチできなかった例外はこっち
    wp_die('何らかのエラーが発生しました。やり直してください。');
  }
}

//関数を実行
check_DB_users_table_by_id():

echo $check_DB_users_table['user_email'] //何も出力されない

関数化する前はうまくいくコードが、関数化したら動かなくなってしまう原因がわからず、頭がこんがらがっています。。

どなたか、原因がわかる方がいらっしゃいましたら、道しるべを頂けましたら幸いです。

Comments

[kei344]

まだ質問が「受付中」になっていますが、「ベストアンサー」を選び「解決済」にされてはいかがでしょうか。

[yayak]

失礼いたしました！解決済みに変更します。

Answer 1

function check_DB_users_table_by_id() {
  try {
    global $wpdb;
    $users_table = $wpdb->prefix . "users"; //ユーザーテーブルを定義
    $session_ID = '1'; //セッションIDを定義
    $check_DB_users_table = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $users_table WHERE ID = %d", "$session_ID" ), ARRAY_A );
  } catch( PDOException $e ) { // PDOException の場合のみ
    wp_die('データベースエラーが発生しました。やり直してください。');
  } catch( Exception $e ) { // PDOException でキャッチできなかった例外はこっち
    wp_die('何らかのエラーが発生しました。やり直してください。');
  }
return $check_DB_users_table;
}

echo check_DB_users_table_by_id()['user_email'];とか。

Comments

[yayak]

上記のようにしないと動作しない理由なども、今回の皆様の回答のおかげで理解が深まりました。
本当に有難うございます。
もっともっと勉強します！

Answer 2

WPはいまいちわからないのですが、普通に

check_DB_users_table_by_id()内でローカル変数$check_DB_users_tableに
値を設定しても、スコープ外から参照できないだけでは？

• $check_DB_users_tableを引数で処理するとか
• $check_DB_users_tableをグローバルにするとか
• $check_DB_users_tableをreturnして外で受けるとか

hoge($a);
function hoge(&$b=null){
 $b=1;
}
print $a;

Comments

[yayak]

リターンして外で受け取りましたところ、想定していた動作になりました。
本当に有難うございます。
感謝感激です。

Answer 3

関数の中の$check_DB_users_tableは関数が終了すると破棄されるのでそれが正しい動作です。

Comments

[yayak]

その通りすぎて赤面してベッドでじたばたしております。
ご回答、心より感謝いたします。
有難うございます。