diff --git a/imageloader/src/main/java/fr/uha/gabalier/controller/ImageController.java b/imageloader/src/main/java/fr/uha/gabalier/controller/ImageController.java index f689ef8..5722cc6 100644 --- a/imageloader/src/main/java/fr/uha/gabalier/controller/ImageController.java +++ b/imageloader/src/main/java/fr/uha/gabalier/controller/ImageController.java @@ -8,7 +8,7 @@ import java.awt.event.ActionEvent; import java.awt.event.ActionListener; import java.util.List; -public class ImageController extends Controller> implements ActionListener { +public class ImageController extends Controller> implements ActionListener { private int index; diff --git a/imageloader/src/main/java/fr/uha/gabalier/core/ImageLoader.java b/imageloader/src/main/java/fr/uha/gabalier/core/ImageLoader.java index f16dbfe..cf5883c 100644 --- a/imageloader/src/main/java/fr/uha/gabalier/core/ImageLoader.java +++ b/imageloader/src/main/java/fr/uha/gabalier/core/ImageLoader.java @@ -8,6 +8,7 @@ import java.awt.*; import java.io.File; import java.util.ArrayList; import java.util.List; +import java.util.Optional; import java.util.concurrent.CancellationException; import java.util.concurrent.ExecutionException; @@ -27,9 +28,9 @@ public class ImageLoader extends SwingWorker, Integer> { private void loadImages(File current) throws NullPointerException { if (current.isFile()) { - final Image icon = ImageLib.createScaledImage(current, this.iconWidth, -1); - if (icon != null) { - images.add(icon); + final Optional icon = ImageLib.createScaledImage(current, this.iconWidth, -1); + if (icon.isPresent()) { + images.add(icon.get()); publish(images.size()); } return; @@ -43,7 +44,7 @@ public class ImageLoader extends SwingWorker, Integer> { } @Override - protected List doInBackground() throws Exception { + protected List doInBackground() { loadImages(root); return images; } @@ -51,7 +52,7 @@ public class ImageLoader extends SwingWorker, Integer> { @Override protected void process(List data) { if (this.isCancelled()) return; - app.setStatus(String.valueOf(data.get(data.size() - 1)) + " images loaded"); + app.setStatus(data.get(data.size() - 1) + " images loaded"); } @Override diff --git a/imageloader/src/main/java/fr/uha/gabalier/util/ImageLib.java b/imageloader/src/main/java/fr/uha/gabalier/util/ImageLib.java index cd962ee..7e3fe22 100644 --- a/imageloader/src/main/java/fr/uha/gabalier/util/ImageLib.java +++ b/imageloader/src/main/java/fr/uha/gabalier/util/ImageLib.java @@ -5,19 +5,20 @@ import java.awt.*; import java.awt.image.BufferedImage; import java.io.File; import java.io.IOException; +import java.util.Optional; public final class ImageLib { - public static Image createScaledImage(File file, int iconWidth, int iconHeight) { + public static Optional createScaledImage(File file, int iconWidth, int iconHeight) { final BufferedImage img; try { if ((img = ImageIO.read(file)) != null) { - return img.getScaledInstance(iconWidth, iconHeight, Image.SCALE_DEFAULT); + return Optional.of(img.getScaledInstance(iconWidth, iconHeight, Image.SCALE_DEFAULT)); } } catch (IOException e) { e.printStackTrace(); } - return null; + return Optional.empty(); } private ImageLib() {} diff --git a/imageloader/src/main/java/fr/uha/gabalier/view/View.java b/imageloader/src/main/java/fr/uha/gabalier/view/View.java index 6ef11ea..2c90deb 100644 --- a/imageloader/src/main/java/fr/uha/gabalier/view/View.java +++ b/imageloader/src/main/java/fr/uha/gabalier/view/View.java @@ -7,7 +7,7 @@ import javax.swing.*; public class View extends JPanel { private final T model; - private Controller controller; + private Controller controller; protected View(T model) { this.model = model; @@ -17,7 +17,7 @@ public class View extends JPanel { return model; } - protected Controller getController() { + protected Controller getController() { return controller; } diff --git a/simple-pathfinding/src/main/java/fr/uha/gasser/pathfinding/algorithm/AStarWorker.java b/simple-pathfinding/src/main/java/fr/uha/gasser/pathfinding/algorithm/AStarWorker.java index 1e8cd14..a9fc50b 100644 --- a/simple-pathfinding/src/main/java/fr/uha/gasser/pathfinding/algorithm/AStarWorker.java +++ b/simple-pathfinding/src/main/java/fr/uha/gasser/pathfinding/algorithm/AStarWorker.java @@ -16,12 +16,13 @@ public class AStarWorker extends SwingWorker, List> { private final Node start; private final Node goal; private final Grid target; - private List path; + private List path = Collections.emptyList(); public AStarWorker(Node start, Node goal, Grid target) { this.start = start; this.goal = goal; this.target = target; + Thread.currentThread().setName("AStarWorker"); } @Override