这个函数有意义吗?

seu_cose 2008-07-30 02:24:13

for (int i = 0; i < everyServiceName.length; i++) {

publishService(everyServiceName[i]);
}

public void publishService(String serviceName) {

assertIsExistingService(serviceName);
......
}

private void assertIsExistingService(String serviceName) {
boolean foundIt = false;

for (int i = 0; i < everyServiceName.length; i++) {
if (serviceName.equals(everyServiceName[i])) {
foundIt = true;
break;
}
}
if (!foundIt) {
throw new IllegalArgumentException("Unrecognized service name "
+ serviceName + "!");
}
}
...全文
112 10 打赏 收藏 转发到动态 举报
AI 作业
写回复
用AI写文章
10 条回复
切换为时间正序
请发表友善的回复…
发表回复
polyic 2008-07-30
  • 打赏
  • 举报
回复
好像是没意义。
把你标出的函数删了,直接改成下面这样也不错的:

for (int i = 0; i < everyServiceName.length; i++) {

assertIsExistingService(everyServiceName[i]);
}


private void assertIsExistingService(String serviceName) {
boolean foundIt = false;

for (int i = 0; i < everyServiceName.length; i++) {
if (serviceName.equals(everyServiceName[i])) {
foundIt = true;
break;
}
}
if (!foundIt) {
throw new IllegalArgumentException("Unrecognized service name "
+ serviceName + "!");
}
}

除非你询问的publishService函数中的……有其他内容需要依赖接收到的String serviceName执行,不然挺没意义的(*^__^*) 嘻嘻……

public void publishService(String serviceName) {

assertIsExistingService(serviceName);
......
}

jingulang 2008-07-30
  • 打赏
  • 举报
回复
[Quote=引用 3 楼 bao110908 的回复:]

[/Quote]
zidasine 2008-07-30
  • 打赏
  • 举报
回复
不一定啊 上面不是判断了吗
serviceName.equals(everyServiceName[i])这个时候才是true
junjun1984 2008-07-30
  • 打赏
  • 举报
回复
?
ZangXT 2008-07-30
  • 打赏
  • 举报
回复
这段代码貌似很无聊,不知道谁写的,干什么的?
ZangXT 2008-07-30
  • 打赏
  • 举报
回复
[Quote=引用 4 楼 seu_cose 的回复:]
foundIt 似乎肯定true
[/Quote]
这不就是处理不为true的情况吗?
如果你能百分之百确定它为true,这方法就没有意义了.
seu_cose 2008-07-30
  • 打赏
  • 举报
回复
仔细看一下程序唉,,,大家看取值,foundIt 似乎肯定true,,但是这样有什么用呢
  • 打赏
  • 举报
回复
浴火_凤凰 2008-07-30
  • 打赏
  • 举报
回复
怎么就没有意义了啊?

62,635

社区成员

发帖
与我相关
我的任务
社区描述
Java 2 Standard Edition
社区管理员
  • Java SE
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
暂无公告

试试用AI创作助手写篇文章吧