switch 方法只返回默认值

Question

我正在尝试在我的应用程序中处理日期格式的多个条件，并为此目的使用 switch 方法。该功能看起来不错，但问题是我的情况下没有任何东西不起作用。我知道条件有问题，但无法理解到底是什么。

谁能告诉我我的错误在哪里？

 var dateSettings = function (value) {

        var timezone = localStorage.getItem('timezone');
        timezone = JSON.parse(timezone);

        var timeFormat = localStorage.getItem('timeFormat');
        timeFormat = JSON.parse(timeFormat);

        var dateFormat = localStorage.getItem('dateFormat');
        dateFormat = JSON.parse(dateFormat);

        switch( value ) {
            case (timezone === true && timeFormat === true && dateFormat === false):
                return value = moment.utc(value).format('MMM dd, yyyy HH:mm');

            case (timezone === true && timeFormat === false && dateFormat === false):
                return value = moment.utc(value).format('MMM dd, yyyy HH:mm:ss');

            case (timezone === false && timeFormat === false && dateFormat === false):
                return value = moment(value).format('MMM dd, yyyy HH:mm:ss');

            case (timezone === false && timeFormat === false && dateFormat === true):
                return value = moment(value).format('YYYY-MM-DD HH:mm:ss');

            case (timezone === false && timeFormat === true && dateFormat === true):
                return value = moment(value).format('YYYY-MM-DD HH:mm');

            case (timezone === true && timeFormat === false && dateFormat === true):
                return value = moment.utc(value).format('YYYY-MM-DD HH:mm:ss');

            default:
                return value = moment.utc(value).format('YYYY-MM-DD HH:mm');
        }
    };

Answer 1

switch-case 与 if-else 语句不同 - 这是您实际应该使用的。

case 必须仅依赖于 value。

看看这个：http://www.w3schools.com/js/js_switch.asp

function switchCase(expression) {

    switch(expression) {
        case "a":
            console.log("This is an A");
            break;
        case "b":
            console.log("This is a B");
            break;
        default:
            console.log("DEFAULT");
    }
}

switchCase("a")
switchCase("banana")

您还返回了一个作业return value = ...，在我看来这不是很好的风格。

最好这样做：

value = 4;
return value;

这是你的样本，我会这样做：

var dateSettings = function(value) {

   var timezone = localStorage.getItem('timezone');
   timezone = JSON.parse(timezone);

   var timeFormat = localStorage.getItem('timeFormat');
   timeFormat = JSON.parse(timeFormat);

   var dateFormat = localStorage.getItem('dateFormat');
   dateFormat = JSON.parse(dateFormat);

   if (timezone === true && timeFormat === true && dateFormat === false) {
       value = moment.utc(value).format('MMM dd, yyyy HH:mm');
   } else if (timezone === true && timeFormat === false && dateFormat === false) {
       value = moment.utc(value).format('MMM dd, yyyy HH:mm:ss');
   } else if (timezone === false && timeFormat === false && dateFormat === false) {
       value = moment(value).format('MMM dd, yyyy HH:mm:ss');
   } else if (timezone === false && timeFormat === false && dateFormat === true) {
       value = moment(value).format('YYYY-MM-DD HH:mm:ss');
   } else if (timezone === false && timeFormat === true && dateFormat === true) {
       value = moment(value).format('YYYY-MM-DD HH:mm');
   } else if (timezone === true && timeFormat === false && dateFormat === true) {
       value = moment.utc(value).format('YYYY-MM-DD HH:mm:ss');
   } else {
       value = moment.utc(value).format('YYYY-MM-DD HH:mm');
   }
   return value;
};