PHP フォームから取得した値をデータベースへ登録

Question

前提・実現したいこと

PHPとSQLを組み合わせ、入力された値をデータベースへ登録するプログラムを開発しています

データベースへのデータ登録の際、4つある入力フォームのデータを1つの行に入れたいのですが、
4つのデータがそれぞれ別の行へ登録されてしまいます。

<?php
session_start();
require('dbconnect.php');

//取引の記録を行う

    $year = $db->prepare('INSERT INTO transaction SET year= ?');
    $year->execute(array($_POST['year']));

    $inex = $db->prepare('INSERT INTO transaction SET inex= ?');
    $inex->execute(array($_POST['inex']));

    $amount = $db->prepare('INSERT INTO transaction SET amount= ?');
    $amount->execute(array($_POST['amount']));

    $summary = $db->prepare('INSERT INTO transaction SET summary= ?');
    $summary->execute(array($_POST['summary']));

    $memo = $db->prepare('INSERT INTO memo SET memo= ?');
    $memo->execute(array($_POST['memo']));

    unset($_SESSION['transaction']);
?>

学習初心者ということもあり参考書や各種サイトを参考にしながら作成しましたが、何を試してもダメでもうお手上げ状態です。
よろしくお願いします。

Answer 1

execute()で実行しているので書いた通りになっています。
同じテーブルに対して挿入したいのでしたらSQL１つにまとめましょう。

$st = $db->prepare('INSERT INTO transaction (year,inex,amount,summary) VALUES (:year,:inex,:amount,:summary)');
$st->bindValue(':year',$_POST['year']);
$st->bindValue(':inex',$_POST['inex']);
$st->bindValue(':amount',$_POST['amount']);
$st->bindValue(':summary',$_POST['summary']);
$st->execute();

ただ、特にDBに対する変更をかける処理（CRUDで言えばR以外）はtry-catchをしてcommit/rollbackは必須です（できればRのときも）

蛇足：
ただ、DB設計大丈夫ですかね？
質問に書かれた通りだとすると主要項目と思われる４つが全てNULL OKになっているようですけど。
備考のような項目でなければNULL NGにしておいたほうが安全と思います。

Comments

[rc211v_74]

回答いただきありがとうございました。
やはり同テーブルに入れるにはSQLを1つにするしか無いんですね...また一つ勉強になりました
DBのNULL NGについては後でどうにかしようとして放っておいたままになっていたので修正しようと思います。

[m.ts10806]

解決されたようで何よりです。

＞やはり同テーブルに入れるにはSQLを1つにするしか無いんですね...また一つ勉強になりました

SQLは１文で１つ実行されますから、「どうしたいか」によってきちんと組んであげる必要があります（あとsetによるinsert文はあまり一般的ではない気がします。）

PHPでDB接続するときに是非参考にしてもらいたいのは下記のような記事です。
https://qiita.com/mpyw/items/b00b72c5c95aac573b71
基本形として覚えましょう。私が指摘したtry-catchによる対応にも言及されています。

>DBのNULL NGについては後でどうにかしようとして
実際のアプリケーション開発においては何よりも設計が先です。
DBが決まらないと詳細設計にはうつれませんし、詳細設計がかたまらないと実装にはうつれません。
特にDB利用の場合はDBがその後の全てを握っていると言っても過言ではありません。
重々検討の上、実装に進む必要があります。
特に運用始まってデータがどんどん蓄積されていくと後から変更するって結構負担が大きかったりします。
もちろん永遠に変更のないアプリケーションはありませんが、なるべく負担が少なくなるような形で設計しておくのが理想ですね。