代码中如何避免过多的if else

0 0

代码中如何避免过多的if else3

if()
{

}
else if()
{
}
else if()
{
}
else if()
{
}
else if()
{
}
else if()
{
}
......

这的有好几10个。如何对其进行优化。

企业应用

2012年8月24日 17:01

hyj0903
2
0 0 2

5个答案按时间排序按投票排序

0 0

这种时候我更喜欢使用MAP+反射的方式来避免使用if-else, 比如需要通过传入参数的值，来判断需要执行的操作，我更偏向选择下面的方式：

private static final Map<String, String> optionsMap;

	static {
		optionsMap = new HashMap<String, String>();
		optionsMap.put("delete", "doDeleteCat");
		optionsMap.put("modify", "doModifyCat");
		optionsMap.put("create", "doCreateCat");
		optionsMap.put("addCondMap", "doAddCondMap");
		optionsMap.put("delCondMap", "doDelCondMap");
	}

public void execute(RunData rundata, TemplateContext context) {
		rundata.getModuleInfo().setLayoutTemplate("emptyLayout.vm");
		boolean rs = false;
		String optionMethod = optionsMap.get(rundata.getParameters().getString("option"));
		if (StringUtils.isNotBlank(optionMethod)) {
			try {
				Method method = this.getClass()
						.getMethod(optionMethod, RunData.class);
				rs = (Boolean) method.invoke(this, rundata);
			} catch (Throwable e) {
				this.getLogger().error("invoke method:" + optionMethod + " failure!", e);
			}
		}
		context.put("status", rs ? STATUS_SECCESS : STATUS_FAILURE);
	}
	public boolean doCreateCat(RunData rundata) {
		long parentId = rundata.getParameters().getLong("parent_id", -1L);
		int nsId = rundata.getParameters().getInt("ns_id", -1);
		String catName = rundata.getParameters().getString("cat_name", "");
		return vtdCategoriesAdminAO.createVtdCategory(nsId, parentId, catName);
	}
.....

在这里，即使以后增加操作也只需要改变MAP不需要对调用代码进行任何修改

2012年8月27日 12:26

chiyx
48
0 0 3

添加评论

0 0

这个还是需要if到底是用来判断什么的。

if用来判断值的话，你这种情况可以用switch+enum来替换。

假如是判断对象的话，可以使用状态模式加以解决。

2012年8月24日 18:05

slam33661490
15
0 0 0

添加评论

0 0

如果你的if判断条件都是基本类型，那么你可以用switch-case语句代替，但是，这样篇幅也不小。
但是一般出现这种情况，多是业务需求的需要。如果让你在性能优化和业务需求重要性上二者选一的话，估计所有人都会选后者吧。

2012年8月24日 17:32

15101189864
30
0 0 1

添加评论

0 0

看你if 判断的是什么，理论上设计好，应该可以用 switch case 替换掉。

2012年8月24日 17:18

noedu
442
0 0 33

添加评论

0 0

1、if层次越少越好
如
if() {
} else if() {
}
看看能不能改成

if() {
    return;
}

if() {
    return;
}

2、提取方法
     把相关的if进行合并并提取方法

3、利用好多态替换if else

4、有没有可能如用enum替换掉有规则的字符串/数字等