make the check throw its exception

Author: madhephaestus

src/main/java/com/neuronrobotics/sdk/addons/kinematics/IVitaminHolder.java

@@ -3,32 +3,47 @@
 import java.util.ArrayList;
 
 public interface IVitaminHolder {
-	ArrayList<VitaminLocation> getVitamins() ;
+	ArrayList<VitaminLocation> getVitamins();
+
 	default void addVitamin(VitaminLocation location) {
-		for(VitaminLocation v:getVitamins()) {
-			if(v.getName().contentEquals(location.getName())) {
-				new RuntimeException("Vitamin Name "+v.getName()+"already exists");
+		if(hasVitamin(location)) 
+			throw new RuntimeException("Vitamin Name "+location.getName()+"already exists");
+		addVitaminInternal( location);
+	}
+
+	default boolean hasVitamin(VitaminLocation location) {
+		for (VitaminLocation v : getVitamins()) {
+			String name = v.getName();
+			String name2 = location.getName();
+			if (name.contentEquals(name2)) {
+				return true;
 			}
 		}
-		addVitaminInternal( location);
+		return false;
 	}
+
 	void addVitaminInternal(VitaminLocation location);
+
 	void removeVitamin(VitaminLocation loc);
-	default ArrayList<VitaminLocation> getVitamins(VitaminFrame frame){
+
+	default ArrayList<VitaminLocation> getVitamins(VitaminFrame frame) {
 		ArrayList<VitaminLocation> copy = new ArrayList<>();
-		for(VitaminLocation v:getVitamins()) {
-			if(v.getFrame()==frame)
+		for (VitaminLocation v : getVitamins()) {
+			if (v.getFrame() == frame)
 				copy.add(v);
 		}
 		return copy;
 	}
-	default ArrayList<VitaminLocation> getOriginVitamins(){
+
+	default ArrayList<VitaminLocation> getOriginVitamins() {
 		return getVitamins(VitaminFrame.LinkOrigin);
 	}
-	default ArrayList<VitaminLocation> getDefaultVitamins(){
+
+	default ArrayList<VitaminLocation> getDefaultVitamins() {
 		return getVitamins(VitaminFrame.DefaultFrame);
 	}
-	default ArrayList<VitaminLocation> getPreviousLinkVitamins(){
+
+	default ArrayList<VitaminLocation> getPreviousLinkVitamins() {
 		return getVitamins(VitaminFrame.previousLinkTip);
 	}
 }