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

下面的代码只是初始化两个变量,具体取决于传递的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]

Mohammad Deh.. 6

无论如何,我更喜欢第二个代码.

如果您的cloudType变量与这两个其他对象的属性具有一对一的映射,为什么不明确？这也避免了重复的代码.

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

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

这样您就会知道,当cloudType系统添加新内容时,您应该将相关属性添加到其他对象中.

无论如何,我更喜欢第二个代码.

如果您的cloudType变量与这两个其他对象的属性具有一对一的映射,为什么不明确？这也避免了重复的代码.

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

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

这样您就会知道,当cloudType系统添加新内容时,您应该将相关属性添加到其他对象中.