From 6b6e19ff1e74bb9c2ad3f5a1a8b035a8ffc21719 Mon Sep 17 00:00:00 2001
From: jonahk17 <jonahkwon@Jonahs-MacBook-Air-2.local>
Date: Wed, 18 Sep 2024 17:12:43 -0400
Subject: [PATCH] Static Map in Ports.java

---
 .../java/org/sciborgs1155/lib/FaultLogger.java    | 15 ++++++++-------
 src/main/java/org/sciborgs1155/robot/Ports.java   | 11 +++++++++++
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/sciborgs1155/lib/FaultLogger.java b/src/main/java/org/sciborgs1155/lib/FaultLogger.java
index b39f1bd3..83a6e14c 100644
--- a/src/main/java/org/sciborgs1155/lib/FaultLogger.java
+++ b/src/main/java/org/sciborgs1155/lib/FaultLogger.java
@@ -11,15 +11,14 @@
 import edu.wpi.first.wpilibj.DutyCycleEncoder;
 import edu.wpi.first.wpilibj.PowerDistribution;
 import java.util.ArrayList;
-import java.util.HashMap;
 import java.util.HashSet;
 import java.util.List;
-import java.util.Map;
 import java.util.Optional;
 import java.util.Set;
 import java.util.function.BooleanSupplier;
 import java.util.function.Supplier;
 import org.photonvision.PhotonCamera;
+import org.sciborgs1155.robot.Ports;
 
 /**
  * FaultLogger allows for faults to be logged and displayed.
@@ -191,7 +190,12 @@ public static void register(CANSparkBase spark) {
         () -> {
           for (FaultID fault : FaultID.values()) {
             if (spark.getFault(fault)) {
-              report(SparkUtils.name(spark), fault.name(), FaultType.ERROR);
+              report(
+                  String.format(
+                      "Name - %s ID- %s",
+                      Ports.idToName.get(spark.getDeviceId()), String.valueOf(spark.getDeviceId())),
+                  fault.name(),
+                  FaultType.ERROR);
             }
           }
         });
@@ -282,15 +286,12 @@ public static boolean check(CANSparkBase spark) {
    * @return If the spark is working without errors.
    */
   public static boolean check(CANSparkBase spark, REVLibError error) {
-    Map<Integer, String> nickname = new HashMap<>();
-    // nicknames for motors with corresponding ids
-    // ...
 
     if (error != REVLibError.kOk) {
       report(
           String.format(
               "Name - %s ID- %s",
-              nickname.get(spark.getDeviceId()), String.valueOf(spark.getDeviceId())),
+              Ports.idToName.get(spark.getDeviceId()), String.valueOf(spark.getDeviceId())),
           error.name(),
           FaultType.ERROR);
       return false;
diff --git a/src/main/java/org/sciborgs1155/robot/Ports.java b/src/main/java/org/sciborgs1155/robot/Ports.java
index 0cf90fe7..10ed25d4 100644
--- a/src/main/java/org/sciborgs1155/robot/Ports.java
+++ b/src/main/java/org/sciborgs1155/robot/Ports.java
@@ -1,6 +1,17 @@
 package org.sciborgs1155.robot;
 
+import java.util.HashMap;
+import java.util.Map;
+
 public final class Ports {
+
+  public static Map<Integer, String> idToName;
+
+  static {
+    idToName = new HashMap();
+    // idToName.put(id, name);
+  }
+
   public static final class OI {
     public static final int OPERATOR = 0;
     public static final int DRIVER = 1;