哪种方法更适合有条件地分配？

Question

下面的代码只是初始化两个变量，具体取决于传递的cloudType，可以通过任何一种方法分配。 在JavaScript中，建议使用较少的代码行，因此我喜欢第二种方法，它也会删除重复。

方法一允许我灵活地在特定情况下添加操作，如果将来需要，因为第二种方法为我节省了大量字节也帮助我减少重复。

switch (cloudType) {
    case "aws":
        value = this._resetMessageDefaultValues.aws
        resetText = this._messages.current.aws
        break;
    case "private":
        value = this._resetMessageDefaultValues.private;
        resetText = this._messages.current.private
        break;
    case "azure":
        value = this._resetMessageDefaultValues.azure;
        resetText = this._messages.current.azure
        break;
}

要么

value = this._resetMessageDefaultValues[cloudType];
resetText = this._messages.current[cloudType]

Answer 1

无论如何，我更喜欢第二个代码。

如果您的cloudType变量与这两个其他对象的属性具有一对一的映射，为什么不说清楚？ 这也避免了重复的代码。

为安全起见，请将以下检查添加到您的代码中：

value = this._resetMessageDefaultValues[cloudType];
resetText = this._messages.current[cloudType]
if (typeof value == "undefined" || typeof resetText == "undefined") {
    // throw some error
}

这样您就会知道，当您的系统中添加了新的cloudType时，您应该将相关属性添加到其他对象中。

Answer 2

放大@Mohammad的答案

第二个代码已经是通过括号foo['bar']通过其键引用对象属性的行业标准。 如果要添加额外的操作，只需将下面的开关案例块放入。 仍然比你写的第一个解决方案节省了几个字节

value = this._resetMessageDefaultValues[cloudType];
resetText = this._messages.current[cloudType];

switch (cloudType) {
    case "aws": break;
    case "private": break;
    case "azure": break;
}

考虑到密钥真正直接与cloudType变量匹配，即使不是，我建议匹配它。