From 4fa891f21d7508bec4783d94a53d0c9691cea8ae Mon Sep 17 00:00:00 2001 From: PawelSapula Date: Mon, 25 May 2026 17:48:26 +0200 Subject: [PATCH] feat(Exchange): Set gainers/losers to percent change instead of change value. --- .../java/edu/ntnu/idi/idatt/model/Exchange.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/main/java/edu/ntnu/idi/idatt/model/Exchange.java b/src/main/java/edu/ntnu/idi/idatt/model/Exchange.java index 2680bc6..8ee1e18 100644 --- a/src/main/java/edu/ntnu/idi/idatt/model/Exchange.java +++ b/src/main/java/edu/ntnu/idi/idatt/model/Exchange.java @@ -114,7 +114,7 @@ public List findStocks(String searchTerm) { * *

* Returns the stocks that have done it the best - * in the latest week. + * (percent change) in the latest week. *

* * @param limit - Amount of stocks to be returned. @@ -122,8 +122,8 @@ public List findStocks(String searchTerm) { */ public List getGainers(int limit) { return stockMap.values().stream() - .filter(stock -> stock.getLatestPriceChange().compareTo(BigDecimal.ZERO) > 0) - .sorted(Comparator.comparing(Stock::getLatestPriceChange).reversed()) + .filter(stock -> stock.getLatestPriceChangePercent().compareTo(BigDecimal.ZERO) > 0) + .sorted(Comparator.comparing(Stock::getLatestPriceChangePercent).reversed()) .limit(limit) .toList(); } @@ -133,16 +133,16 @@ public List getGainers(int limit) { * *

* Returns the stocks that have done it the worst - * in value in the latest week. + * in percent change in the latest week. *

* * @param limit - Amount of stocks to be returned. - * @return A list of stocks sorted in inclining order. + * @return A list of stocks sorted in ascending order. */ public List getLosers(int limit) { return stockMap.values().stream() - .filter(stock -> stock.getLatestPriceChange().compareTo(BigDecimal.ZERO) < 0) - .sorted(Comparator.comparing(Stock::getLatestPriceChange)) + .filter(stock -> stock.getLatestPriceChangePercent().compareTo(BigDecimal.ZERO) < 0) + .sorted(Comparator.comparing(Stock::getLatestPriceChangePercent)) .limit(limit) .toList(); }