Merge pull request #45 from BrekiTomasson/patch-1

Update VillagerUtil.java
OverlordsIII · Oct 14, 2022 · 8bd84d2 · 8bd84d2
2 parents 1c3361f + ae953a0
commit 8bd84d2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/io/github/overlordsiii/villagernames/util/VillagerUtil.java b/src/main/java/io/github/overlordsiii/villagernames/util/VillagerUtil.java
@@ -52,12 +52,12 @@ private static String upperFirstLetter(String string){
 
     private static String pickRandomName(NamesConfig namesConfig) {
         List<String> names = namesConfig.getNameList();
-        int index = random.nextInt(names.size());
+        int index = random.nextInt(names.size() - 1);
         if (usedUpNames.size() > names.size()/2) {
             usedUpNames.clear();
         }
         if (usedUpNames.contains(names.get(index))) {
-            index = random.nextInt(names.size());//Partial random, but you could possibly choose a name in the list again
+            index = random.nextInt(names.size() - 1); // Partial random, but you could possibly choose a name in the list again
         }
         usedUpNames.add(names.get(index));
         return names.get(index);