Node.jsによるtable間のデータ共有方法について

Question

現在node.jsを用いて電話帳機能を持ったtodoリストの作成をしています。
DB内にtodoList[id , userName , todo , phone] , userInfo[id , usrename , phone]の２つのtableがあります。
内容としては
①userInfoにuserNameとphoneを登録しておく
②formよりuser名とtodoを入力しtodoListにデータを挿入する
③②の際userInfoに共通するuser名があった場合、formに入力されたusre名・todoに加えuserInfoから取得した電話番号をtodoListに追加する。
④todoListのデータが’/list’にてtodoリストとして表示されます。

todoListにuserNameとtodoを挿入し、userInfoから共通する電話番号を取得するところまではできたのですが、そのデータをtodoListに追加する方法がわかりません。以下コードです。

 1  app.post('/create' , (req , res) =>{
 2        connection.query(
 3            _//DB1にusername、todoを挿入_
 4            'insert into todoList(name ,todo) values(? ,?)',
 5            [req.body.addListUserName/*formのuser名に入力したdata*/,req.body.addListAction/*formのtodoに追加したdata*/],
 6            (error ,results) =>{
 7                connection.query(
 8                    'select phone from userInfo where name=?',
 9                    [req.body.addListUserName],
10                    (error , results) =>{
11                        console.log(results);
12                        connection.query(
13                            'update todoList set phone=? where name=?',
14                            [results , req.body.addListUserName],
15                            (error , results) =>{
16                                res.redirect('/list'/*todoリスト一覧*/);
17                            }
18                        );
19                });                  
20            });
21        });

11行目のconsole.logで確認すると電話番号の取得はできています。また、12行目以下の処理において１３行目のphone=?の？部分を数字に置き換えると、todoListのデータ更新もできますが、現在のコードではtodoListのphoneはnullとなってしまいます。どなたか解決方法をご教授いただければ幸いです。

Comments

[hoshi-takanori]

複数のデータベースを扱うなら connection も複数必要だと思いますが、もしかして複数のテーブルってことでしょうか？

[wowwo]

その通りです。記載が間違っておりました。正しくは複数のテーブルです。

[ybhQSpYbitSK]

DB1、DB2の表記は誤解を招くので、テーブル名でもう一度説明を見直して簡潔にしてください。「何を実現したいのか」「そのためにどういう手続や処理の手順を踏むのか」を明瞭に。

[hoshi-takanori]

select phone from userInfo … の結果 results を確認しましょう。たぶん [] または [{ phone: "123-456-7890" }] みたいなのが返ると思うので、[] ならユーザー未登録、そうでなければ phone の値を取り出せぼ良いかと。

[wowwo]

ご回答ありがとうございます。
select phone from userInfo … のresultsはおっしゃる通りの結果が返っています。
userInfoから取得してきた結果のresultsをtodoListのphoneに更新したいのですが、現在のコードではtodoListのphoneはnullとなってしまいます。

[ybhQSpYbitSK]

todoList の中で、同じ name の人がいなくて必ずゼロかイチだったらいいけど、そうなの？ insert into todoList したときのレコードの主キー（例えばidとか）を押さえておかないと、全く同じレコードに対する更新を後からなんてできないよ。

[hoshi-takanori]

つまり、
・results = [] ならユーザー未登録ならスキップ
・results = [{ phone: "123-456-7890" }] なら、そこから "123-456-7890" の部分を取り出して todoList にセット
という処理を書きましょう。

そして、spYD4RV7UIQmPhs さんのおっしゃる通り、現状のテーブル設計はあまりいけてないように見えるので、その辺も考えると良いとは思いますが、まずは上記の処理ができるようになってからですね。

[wowwo]

todoListの中はnameが同一の人がいない前提で作っていました。
お二人のおっしゃる通りidやテーブル設計に関してまだまだ勉強不足であり、課題が多いと再認識できました。ありがとうございます。