久久r热视频,国产午夜精品一区二区三区视频,亚洲精品自拍偷拍,欧美日韩精品二区

您的位置:首頁技術(shù)文章
文章詳情頁

java for循環(huán)的時候增加循環(huán)體的長度是不是不太好的?

瀏覽:140日期:2023-10-23 17:57:45

問題描述

for(CityDataVO cityItem: citys){boolean flag =false;for(ProvinceDataVO proItem : list){ xxxxxx flag = true; break;}if(!flag){ ProvinceDataVO province = new ProvinceDataVO(); province.setProvinceId(cityItem.getProvinceId()); province.setProvinceName(cityItem.getProvinceName()); province.setReportNum(cityItem.getCount()); List<CityDataVO> vo = new ArrayList<>(); vo.add(cityItem); province.setCityData(vo); list.add(province);} }

比如上面的代碼,如果我沒有進(jìn)入第二層的for循環(huán)里,我就會對list進(jìn)行一個add的操作。這里應(yīng)該是沒有問題的,因為我在第一個for循環(huán)體里操作的是第二個for循環(huán)的size()。我記得有一個情景是for(){...}我直接在...操作了for的size(),這樣好像是不行的,當(dāng)初好像有一個什么辦法的,我暫時想不到了,哪位大神幫我回憶回憶呵??

問題解答

回答1:

不要用 for(:){},用 for(int i = 0; i < list.size(); i++),小心不要搞成死循環(huán)

回答2:

在for each,也就是你的代碼里 for(a : as)的循環(huán)中,不能對被循環(huán)集合進(jìn)行增加或刪除操作,否則會報ConcurrentModificationException.在這段代碼里是沒有問題的。因為你改變list的操作都是在內(nèi)層循環(huán)之外做的。

看你的需求其實就是按省統(tǒng)計各市的數(shù)據(jù)。有條件使用Java 8的話,可以看看Stream的GroupBy方法。可以大大簡化代碼。

一些不相太干的問題,set方法設(shè)置一個list的方式是不太合適的。

如果是有業(yè)務(wù)邏輯的對象,應(yīng)該對內(nèi)部結(jié)構(gòu)進(jìn)行包裝后以業(yè)務(wù)領(lǐng)域的概念提供接口,而不應(yīng)該直接暴露內(nèi)部的集合。

即便是數(shù)據(jù)傳遞對象,也不應(yīng)該提供集合屬性的set方法。一般而言,私有集合變量的生命周期應(yīng)該由它的父對象管理。外界通過外面包裝對象的add或remove方法操作私有的集合。如果需要提供集合方式的get方法。需要考慮是否要進(jìn)行copy或者使之不可變。

回答3:

不能在foreach里操作list(add,remove等操作)

回答4:

希望直接跳出兩層循環(huán),無非是兩種辦法:

用break label的語法(這個我從沒用過,可以Google一下);

就是你用的辦法,設(shè)個標(biāo)志位。

另外,city的復(fù)數(shù)是cities而不是citys。

回答5:

這寫得啰嗦了。我的話先這樣寫:

for (CityDataVO cityItem: citys){ if (validateCityItem(cityItem, list)) {continue; } List<CityDataVO> vo = new ArrayList<>(); vo.add(cityItem); ProvinceDataVO province = new ProvinceDataVO(); province.setProvinceId(cityItem.getProvinceId()); province.setProvinceName(cityItem.getProvinceName()); province.setReportNum(cityItem.getCount());province.setCityData(vo);list.add(province);}...private boolean validateCityItem(CityDataVO cityItem, List<ProvinceDataVO> list) { for(ProvinceDataVO proItem : list){if (...) { return true;} }return false;}

你的原代碼有幾個需要優(yōu)化的地方:

盡量避免雙重循環(huán),內(nèi)循環(huán)一般都應(yīng)該提取出來;

對同一個變量的操作,代碼行盡可能集中在一起,這樣閱讀起來更自然;

變量命名應(yīng)當(dāng)表現(xiàn)其業(yè)務(wù)含義和主要類型,比如 List 對象盡量都用 xxxList 方式命名。這我懶得給你改了。

回答6:

不明白你到底問什么..是if進(jìn)不去嗎?

回答7:

提問前先組織好自己的思路,你想問的是什么?你希望得到什么回答?不然其他人幫不了你回答。

標(biāo)簽: java
相關(guān)文章:
主站蜘蛛池模板: 抚顺县| 神农架林区| 上蔡县| 永靖县| 察雅县| 龙井市| 花莲县| 金门县| 繁昌县| 泽州县| 吉木萨尔县| 周至县| 连州市| 遵义县| 舒城县| 墨竹工卡县| 乌兰县| 青川县| 阿城市| 崇州市| 承德县| 平南县| 酉阳| 安乡县| 高台县| 庆云县| 巴塘县| 临漳县| 瑞安市| 云梦县| 甘肃省| 黄大仙区| 鹤岗市| 吉木乃县| 互助| 望都县| 兰州市| 淅川县| 黄浦区| 临漳县| 定边县|