0

我一直在一个新项目中使用Folktale的Validation并且我发现它非常有用，但是我遇到了需要顺序验证的问题。我有一个配置对象，我需要执行以下验证：

  • 是一个对象吗？
  • 对象的密钥是否有效（它们是否出现在白名单中）？
  • 键的值是否有效？

每个验证都依赖于先前的验证 - 如果项目不是对象，验证其键是没有意义的（并且会出错），如果对象没有键，验证它们的值是没有意义的。实际上，如果验证失败，我想短路验证。

我最初的想法是使用Result而不是在其他地方定义和使用的 Validatio , but mixing the two types feels confusing, and I already havevalidateIsObject`。

我目前（工作但丑陋）的解决方案在这里：

import { validation } from 'folktale';
import { validateIsObject } from 'folktale-validations';
import validateConfigKeys from './validateConfigKeys';
import validateConfigValues from './validateConfigValues';

const { Success, Failure } = validation;

export default config => {
  const wasObject = validateIsObject(config);
  let errorMessages;
  if (Success.hasInstance(wasObject)) {
    const hadValidKeys = validateConfigKeys(config);
    if (Success.hasInstance(hadValidKeys)) {
      const hasValidValues = validateConfigValues(config);
      if (Success.hasInstance(hasValidValues)) {
        return Success(config);
      }
      errorMessages = hasValidValues.value;
    } else {
      errorMessages = hadValidKeys.value;
    }
  } else {
    errorMessages = wasObject.value;
  }
  return Failure(errorMessages);
};

我最初采用了使用嵌套matchWiths 的方法，但这更难阅读。

我该如何改进这个解决方案？

4

1 回答 1

1

您可以编写一个应用验证规则的助手，直到 aFailure返回。一个简单的例子：

const validateUntilFailure = (rules) => (x) => rules.reduce(
  (result, rule) => Success.hasInstance(result) 
    ? result.concat(rule(x)) 
    : result,
  Success()
);

我们concat用来合并两个结果。我们Success.hasInstance用来检查是否需要应用下一条规则。你的模块现在只有一行：

export default config => validateUntilFailure([ 
  validateIsObject, validateConfigKeys, validateConfigValues
]);

请注意，此实现在看到Failure. 递归实现可能是更实用的方法，但不会吸引所有人：

const validateUntilFailure = ([rule, ...rules], x, result = Success()) => 
  Failure.hasInstance(result) || !rule
    ? result
    : validateUntilFailure(rules, x, result.concat(rule(x)))

查看下面的示例以获取运行代码。有一个部分被注释掉了，它显示了如何运行所有规则，即使有失败。

const { Success, Failure } = folktale.validation;

const validateIsObject = (x) =>
  x !== null && x.constructor === Object
  	? Success(x)
	  : Failure(['Input is not an object']);

const validateHasRightKeys = (x) =>
  ["a", "b"].every(k => k in x) 
  	?  Success(x)
		:  Failure(['Item does not have a & b.']);

const validateHasRightValues = (x) =>
  x.a < x.b
  	? Success(x)
		: Failure(['b is larger or equal to a']);


// This doesn't work because it calls all validations on
// every item
/*
const validateItem = (x) =>
  Success().concat(validateIsObject(x))
           .concat(validateHasRightKeys(x))
           .concat(validateHasRightValues(x))
           .map(_ => x);
*/

// General validate until failure function:
const validateUntilFailure = (rules) => (x) => rules.reduce(
  (result, rule) => Success.hasInstance(result) 
    ? result.concat(rule(x)) 
    : result,
  Success()
);

// Let's try it out!
const testCases = [
  null,
  { a: 1 },
  { b: 2 },
  { a: 1, b: 2 },
  { a: 2, b: 1 }
];

const fullValidation = validateUntilFailure([
	validateIsObject, 
  validateHasRightKeys,
  validateHasRightValues
]);



console.log(
  testCases
    .map(x => [x, fullValidation(x)])
    .map(stringifyResult)
    .join("\n")
);

function stringifyResult([input, output]) {
  return `input: ${JSON.stringify(input)}, ${Success.hasInstance(output) ? "success:" : "error:"} ${JSON.stringify(output.value)}`;
}
<script src="https://cdnjs.cloudflare.com/ajax/libs/folktale/2.0.1/folktale.min.js"></script>

于 2018-01-10T16:34:46.360 回答