粗心是程序员的天敌！

如果我不告诉你下面的代码里面有什么错：

		for(int index = 0; index < 10; index++){
			Element e = new Element("type");
			String id = "id" + index;
			e.setAttribute("id", id);
			
			for(int innerIndex = 0; innerIndex < 5; index++){
				Element ie = new Element("url-pattern");
				String pattern = "type-url-pattern #" + innerIndex;
				
				ie.setText(pattern);
				e.addContent(ie);
			}
			
			userType.addContent(e);
		}

你能发现为什么会一直出OutOfMemory错么？

粗心，可以让再好的设计都成为空想！
谨以此文，记念我的粗心！
生活如是，粗心不好。


没能发现的兄弟姐妹们在下面留个名！

评论

29 楼 jltest 2009-04-08  

ijk啊。
真折腾到4层循环那也够复杂了。。

28 楼 kunee 2009-04-08  

函数，函数！！

看到多层循环或者多层IF else 就头晕

27 楼 ycmhn 2009-04-08  

发现了 里层循环的index++

26 楼 ray_linn 2009-04-07  

把第二层for单独放一个方法里

25 楼 kaipingk 2009-04-07  

这个算什么嘛， 在写程序的时候不要写了几百行才测试，一般写一个方法就测试一下，这种问题一下子就会发现！

24 楼 songze39 2009-04-07  

2个循环里面都是index++

23 楼 thinblue 2009-04-07  

命名的问题``

22 楼 minzaipiao 2009-04-07  

for(int innerIndex = 0; innerIndex < 5; index++){ 

index++; --->innerIndex++;

21 楼 faye.feelcool 2009-04-07  

代码大全，不错的。
我最早看的时候是2002年，当时是下了一个网上一些热心人翻译的第一版。自己用打印机打印出来，细细读了几遍受益匪浅。

最受用的就是“方法（或叫函数）”这个工具了。将问题层层分解，独立的功能就让他成为一个函数或方法，不要仅仅在需要公用的时候才想到函数，我们不仅需要给别人提供工具，更需要给自己提供工具。复杂是思维的大敌。

拆嵌套的最好的方法就是函数。循环做循环的事情，功能点做功能点的事情。如果是用函数的话是不容易发生此类问题的。不用IDE工具提醒，你一眼就能看的到。如果习惯够好的话，压根就写不出来楼主的情况的代码


for(int index = 0; index < 10; index++){ 
    Element e = new Element("type"); 
    String id = "id" + index; 
    e.setAttribute("id", id); 
    addChildContent(e); 
   
}

private void addChildContent(Element parent){
for(int innerIndex = 0; innerIndex < 5; innerIndex++){ 
        Element ie = new Element("url-pattern"); 
        String pattern = "type-url-pattern #" + innerIndex; 
         
        ie.setText(pattern); 
        parent.addContent(ie); 
    }
}

20 楼 absolute 2009-04-07  

楼主好好看看代码大全吧，你会豁然开朗的

19 楼 xzj127 2009-04-06  

无限循环！死机了吗？。。

     

18 楼 soaring 2009-04-06  

建议不要用这么容易混淆的变量，出了错误设个断点调试下很快就查出来了

17 楼 RednaxelaFX 2009-03-27  

同是论坛新手的飘过……

foreverandever 写道

其实用for(Object obj: Objects)这种方式是很不错的，只是有的时候代码就是要在这个Objects里面的第x个就要做不同的操作的时候，您会选择加一个计数器呢，或者是用常规的 for()去实现呢？

所以Ruby的Enumerable里有each_with_index，D语言的foreach语句也有可以得到index的版本；就怪Java语言太硬吧，哈哈（纯粹开玩笑

在Java里需要用到index时还是用传统的for语句合适。而IJK三位老人家也是从早期的高级编程语言开始就作为循环变量而被惯用的名字，倒也没啥不好的。只不过如果for循环是复制粘贴出来的话，漏改那么一两个变量是人之常情。所以说复制粘贴代码很邪恶……

PSP（个人软件过程）说应该养成code review的习惯。信不信PSP是一回事，写完代码之后先review过再运行或许比运行了再找错来得合算……

16 楼 foreverandever 2009-03-27  

night_stalker 写道

好吧，是我错了。请用index，innerIndex，innerOfInnerIndex……

ps：我决定以后用 index, jndex 和 kndex

以纪念 IJK 三位老人家……

我是论坛新手，也是JAVA新手。没有顶撞您的意思。我只是在上面的问题里面征求您的意见啊~

15 楼 night_stalker 2009-03-26  

好吧，是我错了。请用index，innerIndex，innerOfInnerIndex……

ps：我决定以后用 index, jndex 和 kndex

以纪念 IJK 三位老人家……

14 楼 foreverandever 2009-03-26  

night_stalker 写道

foreverandever 写道

可是当您在一个方法里面多次出现i, j, k， 您会不会觉得这些也会混 淆呢？

凡事不要绝对……i, j, k 一般只作整型循环变量。
非整型的循环变量往往不用i,j,k，如

for(Item item : itemsList)

如果无数的i, j, k和ii, jj, kk叠一起……我想是往往是程序结构有问题，写别的名字也好不了多少，抽出个函数更好吧？

其实用for(Object obj: Objects)这种方式是很不错的，只是有的时候代码就是要在这个Objects里面的第x个就要做不同的操作的时候，您会选择加一个计数器呢，或者是用常规的 for()去实现呢？

13 楼 night_stalker 2009-03-26  

foreverandever 写道

可是当您在一个方法里面多次出现i, j, k， 您会不会觉得这些也会混 淆呢？

凡事不要绝对……i, j, k 一般只作整型循环变量。
非整型的循环变量往往不用i,j,k，如

for(Item item : itemsList)

如果无数的i, j, k和ii, jj, kk叠一起……我想是往往是程序结构有问题，写别的名字也好不了多少，抽出个函数更好吧？

12 楼 foreverandever 2009-03-26  

night_stalker 写道

innerIndex这种写法，就是问题的根源。

它降低了可读性——和index容易混淆，还要打这么多字。

而且有i，j，k 这些公认的整型循环变量，为什么要用index？

是的啊。

可是当您在一个方法里面多次出现i, j, k， 您会不会觉得这些也会混 淆呢？

11 楼 night_stalker 2009-03-25  

innerIndex这种写法，就是问题的根源。

它降低了可读性——和index容易混淆，还要打这么多字。

而且有i，j，k 这些公认的整型循环变量，为什么要用index？

10 楼 luolonghao 2009-03-23  

关键还是要严谨，习惯问题啊