The learn-vue from icesz

learn-vue's People

Contributors

Watchers

learn-vue's Issues

修改意见

整体运行没有什么大问题~ 所以我们来聊聊实现上的细节吧~

命名

那些加减乘除的符号实际上你可以命名为 operator (操作符)

那个运算结果的方法不应该叫做 add，应该叫做 calculate (运算)

错误提示

我建议你搞一个单独的 div 用来显示错误的，而不是 alert 出来，因为好烦hhhhhh，类似于

{
    data:{
        errorText:"",
    },
    methods:{
        someMethod(){
            if(somethingWrong){
                this.errorText = "put some error text here";
            }
        }
    }
}

if 嵌套太深

if 嵌套太深不太方便阅读，后期改起来也比较麻烦。
你可以改成类似于这样：

function someMethod(){
    // 判断xxxxx
    if(somethingWrong){
        this.errorText = "put some error text here";
        return;
    }
    // 判断xxxxx
    if(somethingElseWrong){
        this.errorText = "put some error text here";
        return;
    }
}

判断空值

其实你只是判断了输入的内容等不等于空字符串，如果我输入的内容是一个空格，那就有问题了。

判断时加上 trim() 方法是一个好习惯

if(this.numOne.trim()===""){
//....
}

建议不要使用 eval

在不得已的情况下不要使用 eval 方法，非常不安全。虽然你的做法和用法都没有错。
建议使用 switch

switch(this.currentOperator){
    case "+":
        this.result = Number(this.numOne)+Number(this.numTwo);
        break;
    case "-":
        // ...其余的
}