Code Fragment-减少嵌套层数，提前结束返回。

最新推荐文章于 2021-05-27 15:22:19 发布

原创最新推荐文章于 2021-05-27 15:22:19 发布 · 741 阅读

0 ·

CC 4.0 BY-SA版权

code fragment 专栏收录该内容

57 篇文章

订阅专栏

本文介绍了如何通过减少嵌套层数和优化条件语句来提高代码可读性，提供了两个具体示例进行对比分析，并讨论了在条件判断中去除不必要的else部分的重要性。

本文参考自《编写可读代码的艺术》，这本书，非常好！

本文主要介绍两种处理问题的方式，，

减少嵌套层数。嵌套的层数无疑增加了代码的复杂度。
尽早return，对于能够处理的判断，尽早的处理处理，而后期的修正也会使代码更直接明了。

一个例子，重构前

public synchronized Drawable getAvatar(Context context, Drawable defaultValue) {
    if (mAvatar == null) {
        if (mAvatarData != null) {
            Bitmap b = BitmapFactory.decodeByteArray(mAvatarData, 0, mAvatarData.length);
            mAvatar = new BitmapDrawable(context.getResources(), b);
        }
    }
    return mAvatar != null ? mAvatar : defaultValue;
}

上面的代码是很难一眼看清晰，而嵌套的使用，增加了代码的阅读复杂度。

重构之后：

public synchronized Drawable getAvatar2(Context context, Drawable defaultValue) {
	if (mAvatar != null) {
		return mAvatar;
	}

	if (mAvatarData == null) {
		return defaultValue;
	}

	Bitmap b = BitmapFactory.decodeByteArray(mAvatarData, 0, mAvatarData.length);
	mAvatar = new BitmapDrawable(context.getResources(), b);

	if (mAvatar == null) {
		return defaultValue;
	}

	return mAvatar;
}

另外对于if else中的return，没有意义进行else处理，如果符合条件，直接if就可以。如下：

private Contact getContactInfo(Contact c) {
    if (c.mIsMe) {
        return getContactInfoForSelf();
    } else if (Mms.isEmailAddress(c.mNumber) || isAlphaNumber(c.mNumber)) {
        return getContactInfoForEmailAddress(c.mNumber);
    } else {
        return getContactInfoForPhoneNumber(c.mNumber);
    }
}

经过重构以后：

private Contact getContactInfo(Contact c) {
	if (c.mIsMe) {
		return getContactInfoForSelf();
	}

	if (Mms.isEmailAddress(c.mNumber) || isAlphaNumber(c.mNumber)) {
		return getContactInfoForEmailAddress(c.mNumber);
	}

	return getContactInfoForPhoneNumber(c.mNumber);
}