knex的意外行为选择

时间:2017-11-21 23:19:27

标签: javascript sql postgresql knex.js

对于以下代码,我得到的结果有时是数组,有时是对象。我想接收数组,即使它是空的。

export const GetByPId = async (userId, pId) => knex('table1').where({ userId, pId }).select(
  'userId',
  'pId',
  'id',
  'created',
  'updated',
);

在我的Business对象中,我等待响应

static async LoadByPId(userId, pId) {
    const data = await GetByPId(userId, pId);
    console.log(`result ${JSON.stringify(data)}`);
}

一旦返回

[{ userId: 1, id: 1 ... - 我想要这个

并在下次返回时

{ userId: 1, id: 1 ... - 不想要这个

怎么回事?如何让它始终返回数组?

更新#1

现在它只返回一个结果。

更新#2

它变得越来越糟。

现在我的其他基本功能无法正常工作。 Knex仅适用于第一组参数,而其他所有参数均无效。

例如,如果快速服务器已重新启动并发送对userId:1和pId:1的请求,则它可以正常工作。如果我用相同的参数重复相同的请求,它的工作原理。但是如果我将参数(即userId或pId)更改为另一个有效集,则会失败。在尝试任何其他参数之前,我必须重新启动快速服务器。我在我的app和postman上测试了这个。

我的快递代码如下所示

router.post('/list', auth, async (req, res) => {
  try {
    const biz= await BizObj.LoadByPId(req.user.id, req.body.pId);
    res.json(biz);
  } catch (ex) {
    console.log(ex);
    res.status(400).json('Unauthorized');
  }
});

更新#4

如果我的knex配置是问题

development: {
    client: 'postgresql',
    connection: {
      database: 'somedb',
      user: 'SOMEUSER',
      password: '',
      timezone: 'UTC',
    },
    pool: {
      min: 2,
      max: 10,
    },
    migrations: {
      tableName: 'knex_migrations',
    },
  },

更新#5 单页代码(仅缺少快速设置)

在pg db / SomeObj

 id userId
 1  1
 2  2
 3  2

代码示例

import knex from 'knex';
import express from 'express';

const config = {
  development: {
    client: 'pg',
    connection: {
      database: 'somedb',
      user: 'SOMEUSER',
      password: '',
      timezone: 'UTC',
    },
    pool: {
      min: 2,
      max: 10,
    },
    migrations: {
      tableName: 'knex_migrations',
    },
  },
};

const knexed = knex(config.development);
const SQL = knexed('SomeObj');
const GetAll = async userId => SQL.where({ userId }).select(
  'id',
  'userId',
);
const GetById = async (userId, id) => SQL.where({ userId, id }).first(
  'id',
  'userId',
);

class SomeObj {
    constructor(data, userId) {
        this.userId = userId;
        this.id = data.id;
    }
    static async LoadAll(userId) {
        const data = await GetAll(userId);
        if (!data || data.length === 0) return null;
        return data.map(r => new SomeObj(r, userId));
    }
    static async Load(userId, id) {
        const data = await GetById(userId, id);
        if (!data) return null;
        return new SomeObj(data, userId);
    }
}

const router = express.Router();

router.post('/list', async (req, res) => {
  try {
    const res1  = await SomeObj.LoadAll(req.body.id); // works and returns array
    const res2 = await SomeObj.Load(req.body.id, 2); // fails and returns undefined
    res.json({ res1, res2 });
  } catch (ex) {
    res.status(401).json(ex);
  }
});

无法运行第二个查询。不知道我是否遗漏了一些简单的东西来关闭连接。

更新#6

我发誓knex正在弄乱我。每当我尝试某些东西(并且还原以确认更改是由于我的新输入)时,会有不同的响应。现在,res1和res2都返回第一个请求的正确结果,但第二个请求失败。

更新#7

Runkit示例:https://runkit.com/tristargod/runkit-npm-knex

它针对第一个请求运行,但对于快速服务器上的所有其他请求都失败。

更新#8

有关详细信息,请参阅https://github.com/tgriesser/knex/issues/2346#issuecomment-346757344。谢谢Mikael!

1 个答案:

答案 0 :(得分:2)

knex('table1')
 .where({ userId, pId })
 .select('userId', 'pId', 'id', 'created', 'updated')

应始终返回一系列结果。您正在做其他错误,但未在示例中显示。

示例代码:https://runkit.com/embed/kew7v2lwpibn

回应更新#7

tldr; Knex查询构建器是可变的,因此在重用它们时.clone()是必需的。 https://runkit.com/mikaelle/5a17c6d99cd063001284a20a

很好的例子,从中很容易发现问题

您多次重复使用同一个查询构建器,而不在查询之间进行克隆。如果您使用DEBUG=knex:*环境变量集运行代码,您会在第一次调用后看到构造的查询不正确。

const GetAll = async userId => SQL.clone().where({ userId }).select(
  'id',
  'userId',
);
const GetById = async (userId, id) => SQL.clone().where({ userId, id }).first(
  'id',
  'userId',
);