From 9f56840374d48b4bdb5f6e61fec74ce274fec313 Mon Sep 17 00:00:00 2001 From: hizumiaoba <56146205+hizumiaoba@users.noreply.github.com> Date: Wed, 15 Sep 2021 15:55:32 +0900 Subject: [PATCH] perf: using stream and foreach instead --- src/com/ranfa/lib/Scraping.java | 35 ++++++++++++--------------- src/com/ranfa/main/DelesteRandomSelector.java | 2 +- 2 files changed, 16 insertions(+), 21 deletions(-) diff --git a/src/com/ranfa/lib/Scraping.java b/src/com/ranfa/lib/Scraping.java index cfac2da..f0b650a 100644 --- a/src/com/ranfa/lib/Scraping.java +++ b/src/com/ranfa/lib/Scraping.java @@ -104,10 +104,9 @@ public class Scraping { if(attribute.equals(NONSELECTED)) { res = data; } else { - for(int i = 0; i < data.size(); i ++) { - if(data.get(i).getAttribute().equals(attribute)) - res.add(data.get(i)); - } + data.stream() + .filter(element -> element.getAttribute().equals(attribute)) + .forEach(res::add); } return res; } @@ -132,10 +131,9 @@ public class Scraping { if(difficulty.equals(NONSELECTED)) { res = data; } else { - for(int i = 0; i < data.size(); i++) { - if(data.get(i).getDifficulty().equals(difficulty)) - res.add(data.get(i)); - } + data.stream() + .filter(element -> element.getDifficulty().equals(difficulty)) + .forEach(res::add); } return res; } @@ -151,25 +149,22 @@ public class Scraping { return getOnlyLevelSongs(data, level); ArrayList res = new ArrayList(); if(isLess) { - for(int i = 0; i < data.size(); i++) { - if(data.get(i).getLevel() < level) - res.add(data.get(i)); - } + data.stream() + .filter(element -> element.getLevel() < level) + .forEach(res::add); } else if (isMore) { - for (int i = 0; i < data.size(); i++) { - if(data.get(i).getLevel() > level) - res.add(data.get(i)); - } + data.stream() + .filter(element -> element.getLevel() > level) + .forEach(res::add); } return res; } private static ArrayList getOnlyLevelSongs(ArrayList data, int level) { ArrayList res = new ArrayList(); - for(int i = 0; i < data.size(); i++) { - if(data.get(i).getLevel() == level) - res.add(data.get(i)); - } + data.stream() + .filter(element -> element.getLevel() == level) + .forEach(res::add); return res; } diff --git a/src/com/ranfa/main/DelesteRandomSelector.java b/src/com/ranfa/main/DelesteRandomSelector.java index a01461d..164ca28 100644 --- a/src/com/ranfa/main/DelesteRandomSelector.java +++ b/src/com/ranfa/main/DelesteRandomSelector.java @@ -37,7 +37,7 @@ import com.ranfa.lib.Song; import com.ranfa.lib.TwitterIntegration; import com.ranfa.lib.Version; -@Version(major = 1, minor = 2, patch = 1) +@Version(major = 1, minor = 2, patch = 2) public class DelesteRandomSelector extends JFrame { private static ArrayList selectedSongsList = new ArrayList(); -- 2.11.0