Knex记录插入超出命令

时间:2017-11-27 05:57:29

标签: node.js postgresql asynchronous async-await knex.js

在我的测试中尝试使用Knex添加记录时,我看到了一些奇怪的异步行为。

点击POST /api/v1/chats路由时,会创建chat并将其添加到chat表中。此外,user id(其中有2个)传递给请求正文中的路由处理程序,新创建的chat的id用于向{{1}添加两个关联实体} table:user_chat{ user_id: 2, chat_id: 3 }。记录{4}和{4}的{ user_id: 4, chat_id: 3 }位于user且ID为3的事实。

但是,chat方法/查询没有以我期望的方式执行。从控制台可以看出,即从函数结束开始的第一次控制台日志发生一次之前,该函数调用开始时的控制台日志发生两次。

显然有一些异步调用在下一个控制之前没有得到解决。但我很难找到发生这种情况的地方。在我看来,这不应该是一个问题,因为我addUserChat await创建了第13行的user_chat记录,但显然我错了行为远离。

我很好奇我如何解决这个问题,以便例程按顺序运行,并且我的user_chat都按预期插入。

addUserChat: https://github.com/caseysiebel/lang-exchange/blob/master/src/server/db/queries/user_chat.js#L7

addUserChat: ( async (user_id, chat_id) => {
    console.log()
    console.log('====================================================================================================')
    console.log('in query')
    console.log('user_id', user_id)
    console.log('chat_id', chat_id)
    console.log()
    console.log('before await userChats')
    const user_chat = await userChats
        .insert({ user_id, chat_id })
        .returning('*')
    console.log('after await userChats')
    console.log('user_chat', user_chat);
    console.log()

    const data = await db('user_chat').select('*')
    console.log('data', data)
    console.log('****************************************************************************************************')
    console.log()

    return user_chat;
}),

控制台输出(https://gist.github.com/caseysiebel/262997efdd6467c72304ee783dadd9af#file-console-L5):

====================================================================================================
in query
user_id 2
chat_id 3

before await userChats

====================================================================================================
in query
user_id 4
chat_id 3

before await userChats
after await userChats
user_chat [ anonymous { id: 5, user_id: '4', chat_id: '3' } ]

after await userChats
user_chat [ anonymous { id: 6, user_id: '4', chat_id: '3' } ]

data [ anonymous { id: 1, user_id: '1', chat_id: '1' },
  anonymous { id: 2, user_id: '2', chat_id: '2' },
  anonymous { id: 3, user_id: '3', chat_id: '2' },
  anonymous { id: 4, user_id: '4', chat_id: '1' },
  anonymous { id: 5, user_id: '4', chat_id: '3' },
  anonymous { id: 6, user_id: '4', chat_id: '3' } ]
****************************************************************************************************

data [ anonymous { id: 1, user_id: '1', chat_id: '1' },
  anonymous { id: 2, user_id: '2', chat_id: '2' },
  anonymous { id: 3, user_id: '3', chat_id: '2' },
  anonymous { id: 4, user_id: '4', chat_id: '1' },
  anonymous { id: 5, user_id: '4', chat_id: '3' },
  anonymous { id: 6, user_id: '4', chat_id: '3' } ]
****************************************************************************************************

其他相关代码是POST /api/v1/chat路由处理程序:

router.post('/api/v1/chat', async (ctx) => {
    try {
        const { created_at , user_ids } = ctx.request.body;
        const chat_list = await queries.addChat({ created_at });
        const chat = chat_list[0];
        if (chat) {
            ctx.status = 201;
            ctx.body = {
                status: 'success',
                data: chat
            };
            try {
                console.log('user_ids', user_ids)
                await Promise.all(user_ids.map((user_id) => {
                    return user_chat_queries.addUserChat(user_id, chat.id)
                }));
            }
            catch (err) {
                ctx.status = 400;
                ctx.body = {
                    status: 'error',
                    message: err.chat || 'Sorry, an error has occured.'
                };
            }
        }
        else {
            ctx.status = 400;
            ctx.body = {
                status: 'error',
                message: 'Something went wrong.'
            };
        }
    }
    catch (err) {
        ctx.status = 400;
        ctx.body = {
            status: 'error',
            message: err.chat || 'Sorry, an error has occured.'
        };
    }
})

对呼叫所在的聊天路线进行测试

it('should add 2 user_chats', (done) => {
            console.log('00000000000000000000000000000000000000000000000000')
            chai.request(server)
                .post('/api/v1/chats')
                .send({
                    created_at: Date.now(),
                    user_ids: [ 2, 4 ]
                })
                .end((err, res) => {
                    should.not.exist(err);
                    res.status.should.equal(201);
                    res.type.should.equal('application/json');
                    res.body.status.should.eql('success');
                    const chat = res.body.data;
                    chat.should.include.keys('id', 'created_at');
                    let num_user_chats = 0;

                    console.log('&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&')
                    console.log('chat', chat)
                    console.log('chat.id', chat.id)
                    console.log('&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&')
                    knex('user_chat')
                        .select('*')
                        .then((data) => console.log('data', data))

                    knex('user_chat')
                        .where('user_id', 2)
                        .select('*')
                        .then((data) => console.log('data', data))

                    console.log('user_chat', user_chat);

                    done();
                });
        });

项目的所有代码:https://github.com/caseysiebel/lang-exchange

1 个答案:

答案 0 :(得分:3)

如果我的问题是正确的,那么您的问题是您不希望并行运行多个addUserChat个调用。

让它们并行运行的地方是:

await Promise.all(user_ids.map((user_id) => {
    return user_chat_queries.addUserChat(user_id, chat.id)
}));

要按顺序运行它们,您可以执行以下操作:

for (let user_id of user_ids) {
  await user_chat_queries.addUserChat(user_id, chat.id)
}

让它们按顺序运行的另一种更好的方法是使用事务。

EDIT理解@ Casey评论后的真正问题:

我认为您的userChats是一些预定义的查询构建器。因此,您使用相同的构建器一次又一次地使用所有单独的插入。

所以基本上在第一个用户ID上你的查询就像:

const user_chat = await userChats
    .insert({ user_id, chat_id }) // user_id = 2 
    .returning('*')

第二轮是:

const user_chat = await userChats
    .returning('*')
    .insert({ user_id, chat_id }) // user_id = 2 
    .insert({ user_id, chat_id }) // user_id = 4
    .returning('*');

现在,第二个查询实际上是在第一个查询发出之前构建的,因此两个查询都是相同的:

const user_chat = await userChats
    .returning('*')
    .insert({ user_id, chat_id }) // user_id = 2 
    .insert({ user_id, chat_id }) // user_id = 4
    .returning('*');

您可以通过记住为每个查询克隆构建器来解决此问题:

const user_chat = await userChats.clone()
    .insert({ user_id, chat_id })
    .returning('*')