私はネストをしてはいけないと教わってきました。ネストとはifの中にifとか、forの中にforとかです。もちろん座標計算などではforのネストが必要になることもありますので一概にダメとは言えませんが、基本的にはネストはやめましょうということです。
また今回の例はあくまで一例です。
<h2>今回のサンプル</h2>
今回リファクタリングする例です
<div class="remark-highlight"><pre class="language-java"><code class="language-java">public List&#x3C;SellCsvDto> getSellCsv() {
 List&#x3C;Sell> sells = this.sellRepository.findAll();
 List&#x3C;SellCsvDto> lists = new ArrayList&#x3C;>();
 for(Sell sell : sells){
 for(Detail detail : sell.getDetails()){
 SellCsvDto csv = SellCsvDto.builder()
 .sellId(sell.getSellId().getId())
 .name(sell.getSellerName())
 .campaign(sell.getDiscountAmt())
 .amt(sell.getAmount())
 .way(sell.getWay())
 .phoneNumber(sell.getPhoneNumber())
 .bookId(detail.getBookIdStr())
 .count(detail.getCount())
 .build();
 lists.add(csv);
 }
 }
 return lists;
}
</code></pre></div>
<ul>
<li>SellがDetailをListで持っています
<ul>
<li>伝票と明細のような関係です</li>
</ul>
</li>
<li>SellCsvDtoは伝票情報と明細情報をひとまとめにしたデータ構造になっています
<ul>
<li>Sellのリストを全てSellCsvDtoに変換する</li>
</ul>
</li>
</ul>
これをこうしました
<div class="remark-highlight"><pre class="language-java"><code class="language-java">public List&#x3C;SellCsvDto> getSellCsv() {
 List&#x3C;Sell> sells = this.sellRepository.findAll();
 List&#x3C;SellCsvDto> dtos = new ArrayList&#x3C;>();
 for(Sell sell : sells) {
 dtos.addAll(this.toSellCsvDtoList(sell));
 }
 return dtos;
}
</code></pre></div>
もちろんメソッドを新しく作成しているのですが
Sellを1件ずつSellCsvDtoのリストに変換して、用意しているSellCsvDtoのリストに全追加しているということはわかりやすくなったかと思いますこのようにメソッド抽出することで読みやすくすることができます
toSellCsvDtoListメソッドはメソッド抽出しているので同じ内容です
<div class="remark-highlight"><pre class="language-java"><code class="language-java">private List&#x3C;SellCsvDto> toSellCsvDtoList(Sell sell){
 List&#x3C;SellCsvDto> dtos = new ArrayList&#x3C;>();
 for(Detail detail : sell.getDetails()){
 SellCsvDto csv = SellCsvDto.builder()
 .sellId(sell.getSellId().getId())
 .name(sell.getSellerName())
 .campaign(sell.getDiscountAmt())
 .amt(sell.getAmount())
 .way(sell.getWay())
 .phoneNumber(sell.getPhoneNumber())
 .bookId(detail.getBookIdStr())
 .count(detail.getCount())
 .build();
 dtos.add(csv);
 }
 return dtos;
}
</code></pre></div>
<ul>
<li>SellからDetailを1件ずつループしていき、その分だけSellCsvDtoを作成してリストに詰め込んでいる。</li>
</ul>
メソッド抽出の規模が大きくなってくるとクラス分割なども手としてあります。IDEによってはポチポチっと簡単にできるので便利です

リファクタリング | ネストしたループ