##解決したいこと
共通化したデータベースの関数の登録処理を別のファイルで呼び出し実行したいが、関数を実行できずにデータベースにデータを登録できない。データが登録できるように実装したい。

##該当のソースコード
complete.php

<?php
  ini_set('display_errors', "On");

  if (empty($_SERVER["HTTP_REFERER"])) {
    header('Location: ./contact.php');
    exit;
  }
  session_start();
  require '../common/database.php';

  // XSS対策
  function h($s) {
    return htmlspecialchars($s, ENT_QUOTES, "UTF-8");
  }

  $name = h($_SESSION['name']);
  $kana = h($_SESSION['kana']);
  $tel = h($_SESSION['tel']);
  $email = h($_SESSION['email']);
  $body = h($_SESSION['contact']);
  var_dump($name);

  getDatabaseInsert($stmt);

?>

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <link rel="stylesheet" href="complete.css">
  <link rel="stylesheet" href="../common/header.css">
  <link rel="stylesheet" href="../common/footer.css">
  <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.4.1/jquery.min.js"></script>
  <script src="../public/js/header.js"></script>
  <script src="../public/js/header2.js"></script>
  <title>CAFE</title>
</head>
<body>
  <?php include("../common/header.php"); ?>
  <div class="complete-wrapper">
    <div class="container">
      <div class="title">
        <h2>お問い合わせ</h2>
      </div>
      <div class="form-contents">
        <p class="text">
          お問い合わせ頂きありがとうございます。<br>
          送信頂いた件につきましては、当社より折り返しご連絡差し上げます。<br>
          なお、ご連絡までに、お時間頂く場合もございますので予めご了承ください。<br>
        </p>
        <div class="top">
          <a href="../home/index.php">トップへ戻る</a>
        </div>
      </div>
    </div>
  </div>
  <?php include("../common/footer.php"); ?>
</body>
</html>

<?php

/* データベースへ接続 */
  function getDatabaseConnection() {
  try {
    $dbh = new PDO('mysql:host=localhost;dbname=cafe;charset=utf8mb4', 'root', 'root');
    $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
    return $dbh;
  } catch (PDOException $e){
      echo $e->getMessage();
      exit;
    }
  }

    // データベースの登録
  function getDatabaseInsert() {
    try{
      $dbh = getDatabaseConnection();
      $dbh->beginTransaction();
      $sql  = 'INSERT INTO contacts(name,kana,tel,email,body) VALUES(:name,:kana,:tel,:email,:body)';
      $stmt = $dbh->prepare($sql);
  
      $stmt->bindParam(':name', $name, PDO::PARAM_STR);
      $stmt->bindParam(':kana', $kana, PDO::PARAM_STR);
      $stmt->bindParam(':tel', $tel, PDO::PARAM_INT);
      $stmt->bindParam(':email', $email, PDO::PARAM_STR);
      $stmt->bindParam(':body', $body, PDO::PARAM_STR);
      $stmt->execute();
      $dbh->commit();
    } catch (PDOException $e) {
        $dbh->rollBack();
        echo 'データベースにアクセスできません！'.$e->getMessage();
    }
    return $stmt;
  }

  // データベースの取得
  function getDatabaseSelect() {
    try {
      $dbh = getDatabaseConnection();
      $dbh->beginTransaction();
      $sql = 'SELECT id,name,kana,tel,email,body FROM contacts';
      $stmt = $dbh->query($sql);
      $result = $stmt->fetchall(PDO::FETCH_ASSOC);
      $dbh->commit();
      $dbh = null;
    } catch (PDOException $e){
      $dbh->rollBack();
      echo $e->getMessage();
      exit;
    }
    return $result;
  }
?>

##試したこと
・そもそもフォームに入力したデータは送信されているのか？
var_dump($name)を実行し、が画像のようにデータは送信できている。

初歩的な質問かもしれませんが、ハマってしまい自力解決できなかったので、ご指摘いただけると幸いです。

hentaiman

2021/02/27 07:50

funtionの外側にある変数をfunctionの内側で使う為には引数で渡す必要があります基本なのでまずはマニュアル（https://www.php.net/manual/ja/functions.arguments.php）読んで自分のソースを修正しましょう var_dumpは中身があるはずの変数が書かれている直前に書いて確認してみましょう

退会済みユーザー

2021/02/27 08:16

＞＞funtionの外側にある変数をfunctionの内側で使う為には引数で渡す必要がありますというのは、コードのどこの部分を指しているのでしょうか？？

hentaiman

2021/02/27 08:21

getDatabaseInsert全部

退会済みユーザー

2021/02/27 08:26

getDatabaseInsert全部...なのですね。マニュアル読んでも今回の疑問をどのように解決すればいいのかわからずじまいです。。。

hentaiman

2021/02/27 08:36

回答に引数の使い方書いたので見比べて、自分でもfunctionを作って動作の違いを確認してみましょう

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

回答2件

ベストアンサー

引数無しのコード（質問に至った原因）

php
1$name = '寺ている';
2$kana = 'ダイジョウブカナ';
3function BADgetDatabaseInsert() {
4  echo "name:".$name."\n";
5  echo "kana:".$kana."\n";
6}
7BADgetDatabaseInsert();

Warning: Undefined variable $name in /var/www/html/func.php on line 5

name:
Warning: Undefined variable $kana in /var/www/html/func.php on line 6
kana:

引数有りのコード（解決法）

php
1function GOODgetDatabaseInsert($name, $kana) {
2  echo "name:".$name."\n";
3  echo "kana:".$kana."\n";
4}
5
6
7GOODgetDatabaseInsert($name, $kana);

name:寺ている

kana:ダイジョウブカナ

投稿2021/02/27 08:36

hentaiman

総合スコア6426

退会済みユーザー

2021/02/27 08:42

例文ありで非常にわかりやすいコードありがとうございます！無事に解決できました。

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

関数の引数と変数のスコープ
のサンプルコードをそれぞれ実行して、引数とスコープの概念を理解すると解決しそうです。

例

PHP
1
2function test(){
3   echo $a;
4}
5
6function testArg($text){
7   echo $text;
8}
9
10$a = "test";
11
12//何も表示されない。$aが定義されていない旨のnoticeが出る
13test();
14
15//testが表示される
16testArg($a)
17