为什么我得到一个"不能为非可空字段返回null"做突变时出错?

时间:2018-04-25 11:53:35

标签: javascript node.js express graphql apollo-server

我在服务器端尝试(Apollo)GraphQL并且一直有一个可能很愚蠢的问题。我尝试注册用户,但不断收到下面链接图片中显示的错误。问题是什么?忽略非常简单的身份验证流程,因为我正在测试GraphQl

enter image description here

以下是相关的代码段:

模式

export default `

type User {
    id: ID!
    name: String!
    email: String!
}

type Query {
    allUsers: [User]
  currentUser: User
}

type Mutation {
    createAccount(name: String!, email: String!, password: String!): User
    loginUser(email: String!, password: String!): User
    updatePassword(email: String!, password: String!, newPassword: String!): User
    deleteAccount(email: String!, password: String!): User
}

`

解析器

createAccount: async (
  parent,
  { name, email, password },
  { User },
  info
) => {
  try {
    // Check for invalid (undefined) credentials
    if (!name || !email || !password) {
      return 'Please provide valid credentials';
    }

    // Check if there is a user with the same email
    const foundUser = await User.findOne({ email });

    if (foundUser) {
      return 'Email is already in use';
    }

    // If no user with email create a new user
    const hashedPassword = await bcrypt.hash(password, 10);
    await User.insert({ name, email, password: hashedPassword });

    const savedUser = await User.findOne({ email });

    return savedUser;
  } catch (error) {
    return error.message;
  }
},

1 个答案:

答案 0 :(得分:6)

解析器的最大问题是,在任何数量的场景中,您都会返回一个字符串,而不是返回User个对象。您的架构指定createAccount可以返回Usernull(如果它是User!,它将是不可为空的,然后null将不是有效的类型)。

当您在解析器中返回一个字符串时,因为它期望一个对象,它会将其强制转换为一个对象,然后开始在该对象上查找User属性(如name和{ {1}})。这些属性不存在,并且因为它们是email对象上的非null属性,为它们返回null / undefined会导致错误。

你的解析器可能只是抛出它需要抛出的任何错误。然后,它们将作为响应中User数组的一部分返回。例如:

errors

现在,如果您有重复的电子邮件,响应将如下所示:

// Check if there is a user with the same email
const foundUser = await User.findOne({ email })

if (foundUser) throw new Error('Email is already in use')

// If no user with email create a new user
const hashedPassword = await bcrypt.hash(password, 10);
await User.insert({ name, email, password: hashedPassword });

const savedUser = await User.findOne({ email });

return savedUser;

如果要操纵客户端上的错误显示方式,则应使用Apollo服务器中间件的{ "data": { "createAccount": null }, "errors": [ { "message": "Email is already in use", "locations": [ { "line": 4, "column": 3 } ], "path": [ "createAccount" ] } ] } formatError配置选项。使用自定义错误也是一种很好的做法,允许您添加formatResponse等自定义属性,以便更轻松地识别客户端的错误类型。

最后,无需检查解析程序中是否定义了名称,电子邮件或密码 - 您的架构已将这些输入标记为非null,这意味着GraphQL将自动返回错误(如果有的话)不见了。