个人项目优化小细节总结

最新推荐文章于 2021-05-27 11:49:46 发布

帅气大果果

最新推荐文章于 2021-05-27 11:49:46 发布

阅读量553

点赞数

分类专栏： Android

本文链接：https://blog.csdn.net/qq_34471736/article/details/77774455

版权

Android 专栏收录该内容

26 篇文章 0 订阅

订阅专栏

本文总结了接手二手项目后的优化经验，包括两部分：一是通过改进模糊不清的业务逻辑提高理解效率；二是将复杂的if else结构优化为更清晰的代码，提升代码可读性。这些小细节的优化有助于提升开发效率。

摘要由CSDN通过智能技术生成

作为开发人员，总是会遇到接手别人的二手项目，一般接手之后基本都会修改或者优化。这边也是，接手了一个二手项目，一些优化小细节总结下，也为自己以后优化多张张经验。积少成多嘛。

1.
优化前：

 boolean push = SPUtil.getBoolean(getApplicationContext(), ConstantsUtil.PUSH, true);
        if (push) {
            cbPush.setChecked(true);
        } else {
            cbPush.setChecked(false);
        }

也不知道想表达什么，反正当时还看了半天想着这业务逻辑，最后也是突然大悟

优化后：

boolean push = SPUtil.getBoolean(getApplicationContext(), ConstantsUtil.PUSH, true);

cbPush.setChecked(push);

2.优化前：

   new Handler().postDelayed(new Runnable() {
            @Override
            public void run() {
                if (TextUtils.isEmpty(openid) ) {

                    startActivity(new Intent(SplashActivity.this, ChooseLoginActivity.class));
                } else {

                    if (2 == loginType) {
                        startActivity(new Intent(SplashActivity.this, DseignerMainActivity.class));

                    } else if (4 == loginType) {
                        startActivity(new Intent(SplashActivity.this, ClerkMainActivity.class));

                    }  else {
                        startActivity(new Intent(SplashActivity.this, ChooseLoginActivity.class));
                    }


                }


                finish();
            }
        }, 500);

看到这么多if else 肯定很头疼把，来看看我优化后的

优化后:

       new Handler().postDelayed(new Runnable() {
            @Override
            public void run() {
                if (TextUtils.isEmpty(openid) ) {
                    startChooseLoginActivity();
                } else {
                    isLoginType(loginType);
                }
                finish();
            }
        }, 500);
    }

    private void isLoginType(int loginType){
        switch (loginType){
            case 2:
                startActivity(new Intent(SplashActivity.this, DseignerMainActivity.class));
                break;
            case 4:
                startActivity(new Intent(SplashActivity.this, ClerkMainActivity.class));
                break;
            default:
                startChooseLoginActivity();
                break;
        }
    }

    private void startChooseLoginActivity(){
        startActivity(new Intent(SplashActivity.this, ChooseLoginActivity.class));
    }