From 2b07fcc3d020230d30874454763cda136e49fdd1 Mon Sep 17 00:00:00 2001 From: Collin Smith Date: Thu, 3 Sep 2020 03:12:04 -0700 Subject: [PATCH] Added logged warning when StatListGetter#get(short) returns null --- core/src/com/riiablo/attributes/StatListGetter.java | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/core/src/com/riiablo/attributes/StatListGetter.java b/core/src/com/riiablo/attributes/StatListGetter.java index 39c5f3fe..9226accc 100644 --- a/core/src/com/riiablo/attributes/StatListGetter.java +++ b/core/src/com/riiablo/attributes/StatListGetter.java @@ -2,7 +2,12 @@ package com.riiablo.attributes; import java.util.Iterator; +import com.riiablo.logger.LogManager; +import com.riiablo.logger.Logger; + public final class StatListGetter implements Iterable { + private static final Logger log = LogManager.getLogger(StatListGetter.class); + final StatList stats; final int list; final StatGetter tuple = new StatGetter(); @@ -35,7 +40,11 @@ public final class StatListGetter implements Iterable { /** @see StatList#indexOf(int, short) */ public StatGetter get(short stat) { final int index = indexOf(stat); - if (index < 0) return null; + if (index < 0) { + log.warn("stats({}) list({}) did not contain stat({})", stats, list, stat); + return null; + } + return tuple.set(stats, index); }