3

投稿と投稿のタグを含むアプリを構築しようとしています。これらのためにpost、tagsとpost_tagテーブルがあります。tags事前に定義したタグがあり、アプリのどこかにフロントエンドでユーザーに提案されます。post_tagテーブルは、投稿とタグの IDを各行のペアとして保持します。

Express.js と postgreql と pg-promise を使用しています。

私が知る限り、作成後の操作にはトランザクション クエリが必要です。

また、ユーザーが投稿を作成したときにタグがtagsテーブルになかったかどうかを検出しtag_idて、その場で挿入できるようにするメカニズムも必要です。それ以外の場合は、テーブルの列と、テーブルの列をそれぞれ参照する必要があるためです。insertionpost_idtag_idpost_tagforeign key errorpost_tagpost_idtag_idpoststagsid

これは、私がこれまで使用して失敗した url 関数です。

privateAPIRoutes.post('/ask', function (req, res) {
    console.log('/ask req.body: ', req.body);
    // write to posts
    var post_id = ''
    var post_url = ''
    db.query(
        `
            INSERT INTO
                posts (title, text, post_url, author_id, post_type)
            VALUES
                ($(title), $(text), $(post_url), $(author_id), $(post_type))
            RETURNING id
        `,
        {
            title: req.body.title,
            text: req.body.text,
            post_url: slug(req.body.title),
            author_id: req.user.id,
            post_type: 'question'
        } // remember req.user contains decoded jwt saved by mw above.
    )
        .then(post => {
            console.log('/ask post: ', post);
            post_id = post.id
            post_url = post.post_url


            // if tag deos not exist create it here
            var tags = req.body.tags;
            console.log('2nd block tags1', tags);
            for (var i = 0; i < tags.length; i++) {
                if (tags[i].id == undefined) {
                    console.log('req.body.tags[i].id == undefined', tags[i].id);                        
                    var q1 = db.query("insert into tags (tag) values ($(tag)) returning id", {tag: tags[i].label})
                                .then(data => {
                                    console.log('2nd block tags2', tags);
                                    tags[i].id = data[0].id 


                                    // write to the post_tag
                                    db.tx(t => {
                                        var queries = [];
                                        for (var j = 0; j < tags.length; j++) {

                                            var query = t.query(
                                                `
                                                    INSERT INTO
                                                        post_tag (post_id, tag_id)
                                                    VALUES
                                                        ($(post_id), $(tag_id))
                                                `,
                                                {
                                                    post_id: post_id,
                                                    tag_id: tags[j].id
                                                }
                                            )
                                            queries.push(query);
                                        }   
                                        return t.batch(queries)
                                    })
                                        .then(data => {
                                            res.json({post_id: post_id, post_url: post_url})
                                        })
                                        .catch(error => {
                                            console.error(error);
                                        })
                                })
                                .catch(error => {
                                    console.error(error);
                                });
                }
            }
        })
        .catch(error => {
            console.error(error);
        })
});
4

1 に答える 1

5

db主な問題は、タスクまたはトランザクション内でルートレベルのオブジェクトを使用できないことです。トランザクション内で新しい接続を作成しようとすると、トランザクション ロジックが壊れます。このような場合に使用する必要がありますt.tx。ただし、あなたの場合、それがまったく必要であるとは思いません。

修正されたコード:

privateAPIRoutes.post('/ask', (req, res) => {
    console.log('/ask req.body: ', req.body);
    db.tx(t => {
        return t.one(
            `
        INSERT INTO
        posts (title, text, post_url, author_id, post_type)
        VALUES
        ($(title), $(text), $(post_url), $(author_id), $(post_type))
        RETURNING *
        `,
            {
                title: req.body.title,
                text: req.body.text,
                post_url: slug(req.body.title),
                author_id: req.user.id,
                post_type: 'question'
            } // remember req.user contains decoded jwt saved by mw above.
        )
            .then(post => {
                console.log('/ask second query: post[0]: ', post);
                console.log('/ask second query: tags: ', req.body.tags);
                console.log('/ask second query: tags[0]: ', req.body.tags[0]);

                // the key piece to the answer:
                var tagIds = req.body.tags.map(tag => {
                    return tag.id || t.one("insert into tags(tag) values($1) returning id", tag.label, a=>a.id);
                });

                return t.batch(tagIds)
                    .then(ids => {
                        var queries = ids.map(id => {
                            return t.one(
                                `
                                INSERT INTO post_tag (post_id, tag_id)
                                VALUES ($(post_id), $(tag_id))
                                RETURNING post_id, tag_id
                                `,
                                {
                                    post_id: post.id,
                                    tag_id: id
                                }
                            )
                        });
                        return t.batch(queries);
                    });
            });
    })
        .then(data => {
            // data = result from the last query;
            console.log('/api/ask', data);
            res.json(data);

        })
        .catch(error => {
            // error
        });
});

ここで重要なのは、タグ id-s を反復処理することです。設定されていないものについては、挿入を使用します。次に、配列を に渡して、それらをすべて解決しますt.batch。

その他の推奨事項:

  • one新しいレコード列を返す挿入を実行するときは、メソッドを使用する必要があります。
  • try/catchは、トランザクションで一度だけ使用する必要があります。これは、このライブラリだけでなく、promise の使用方法に関連しています。
  • クエリを外部 SQL ファイルに配置できます。クエリ ファイルを参照してください。

条件付き挿入をよりよく理解するには、SELECT->INSERTを参照してください。

于 2016-08-21T09:57:50.230 に答える