javascript - 有什么更好的方法来设计句子生成器的内置统计函数？

Question

上下文:随机句子生成器

1. the function generateSentence() generates random sentencesreturned as strings (works fine)

2. the function calculateStats() outputs the number of uniquestrings the above function can theoretically generate (works fine also inthis mockup, so be sure to read the disclaimer, I don't want to waste your time)

3. the function generateStructure() and the words lists inDictionnary.lists are constantly growing as time passes

主生成器功能的快速模型:

function generateSentence() {
  var words = [];
  var structure = generateStructure();

  structure.forEach(function(element) {
    words.push(Dictionnary.getElement(element));
  });

  var fullText = words.join(" ");
  fullText = fullText.substring(0, 1).toUpperCase() + fullText.substring(1);
  fullText += ".";
  return fullText;
}

var Dictionnary = {
  getElement: function(listCode) {
    return randomPick(Dictionnary.lists[listCode]);
  },
  lists: {
    _location: ["here,", "at my cousin's,", "in Antarctica,"],
    _subject: ["some guy", "the teacher", "Godzilla"],
    _vTransitive: ["is eating", "is scolding", "is seeing"],
    _vIntransitive: ["is working", "is sitting", "is yawning"],
    _adverb: ["slowly", "very carefully", "with a passion"],
    _object: ["this chair", "an egg", "the statue of Liberty"],
  }
}

// returns an array of strings symbolizing types of sentence elements
// example : ["_location", "_subject", "_vIntransitive"]
function generateStructure() {
  var str = [];

  if (dice(6) > 5) {// the structure can begin with a location or not
    str.push("_location");
  }

  str.push("_subject");// the subject is mandatory

  // verb can be of either types
  var verbType = randomPick(["_vTransitive", "_vIntransitive"]);
  str.push(verbType);

  if (dice(6) > 5) {// adverb is optional
    str.push("_adverb");
  }

  // the structure needs an object if the verb is transitive
  if (verbType == "_vTransitive") {
    str.push("_object");
  }

  return str;
}

// off-topic warning! don't mind the implementation here,
// just know it's a random pick in the array
function randomPick(sourceArray) {
  return sourceArray[dice(sourceArray.length) - 1];
}

// Same as above, not the point, just know it's a die roll (random integer from 1 to max)
function dice(max) {
  if (max < 1) { return 0; }
  return Math.round((Math.random() * max) + .5);
}

在某些时候，我想知道它可以输出多少个不同的唯一字符串，我写了类似的内容(再次非常简化):

function calculateStats() {// the "broken leg" function I'm trying to improve/replace
  var total = 0;
  // lines below : +1 to account for 'no location' or 'no adverb'
  var nbOfLocations = Dictionnary.lists._location.length + 1;
  var nbOfAdverbs = Dictionnary.lists._adverb.length + 1;

  var nbOfTransitiveSentences = 
    nbOfLocations *
    Dictionnary.lists._vTransitive.length *
    nbOfAdverbs *
    Dictionnary.lists._object.length;
  var nbOfIntransitiveSentences =
    nbOfLocations *
    Dictionnary.lists._vIntransitive.length *
    nbOfAdverbs;

  total = nbOfTransitiveSentences + nbOfIntransitiveSentences;
  return total;
}

(旁注:不要担心命名空间污染、输入参数的类型检查或此类事情，为了示例清晰起见，假设这是在气泡中。)

重要免责声明:这不是为了修复我发布的代码。这是一个模型，它可以按原样工作。真正的问题是“随着 future 可能结构的复杂性的增长，以及列表的大小和多样性，计算这些类型的随机结构的统计数据的更好策略是什么，而不是我笨拙的策略calculateStats() 函数，该函数难以维护，可能处理天文数字*，并且容易出错？”

* 在真实的工具中，目前有 351 120 个独特的结构，对于句子来说......总数已经超过(10 的 80 次方)一段时间了。

Answer 1

由于您的句子结构发生了很大变化(在这个小示例中确实发生了变化，我无法想象它在实际代码中发生了多大变化)，我会做类似的事情:

首先，我需要以某种方式保存给定字典中存在的所有可能的句子结构...也许我会创建一个具有以下属性的 Language 对象:字典作为一个属性，我可以添加可能的句子结构(这部分可能可以优化并找到一种更程序化的方式来生成所有可能的句子结构，例如规则引擎)。 句子结构是什么意思？好吧，按照你的例子，我将把句子结构称为下一个:

[ 'location', 'transitive-verb', 'adverb', 'object' ] < - Transitive sentence
[ 'location', 'instransitive-verb', 'adverb' ] <- Intransitive sentence

您可能会找到一种生成此结构的方法...或对它们进行硬编码。

但是...为什么我认为这可以改进您计算统计数据的方式？因为您可以通过使用映射/归约操作来最小化每个句子数量的硬编码并使其更具可扩展性.

那么...如何？

想象一下，我们可以在全局范围内或通过对象或字典本身访问我们的结构:

// Somewhere in the code
const structures = [
  [ 'location', 'transitive-verb', 'adverb', 'object' ],
  [ 'location', 'instransitive-verb', 'adverb' ] 
];
...
// In this example I just passed it as an argument
function calculateStats(structures) {
  const numberOfCombinations = structures.reduce((total, structure) => {
      // We should calculate the number of combinations a structure has
      const numberOfSentences = structure.reduce((acc, wordType) => {
          // For each word type, we access the list and get the lenght (I am not doing safety checks for any wordType)
          return acc + Dictionary.lists[wordType].length
      }, 0);//Initial accumulator

      return total + numberOfSentences;
  }, 0); // Initial accumulator
  return numberOfCombinations;
}

因此，我们将利用迭代不同结构的能力，而不是对每种可能的组合进行硬编码，因此您基本上只需要添加结构，并且 calculateStats 函数不会增长。

如果您需要进行更复杂的计算，则需要更改 reducer 中使用的函数。

我对语法或句法分析的了解非常少，因此，拥有更多知识的您可能可以找到使其变得更简单或进行“更智能计算”的方法。

我自由地以 ES6 风格编写它，如果 reduce 对你来说是一种奇怪的动物，you can read more here或使用lodash/ramda/随便^^