From 15e520952f4c7437e9acb71b4cf99fc29ba80b79 Mon Sep 17 00:00:00 2001
From: Terry Kwan <terrykwan@TerrydeMacBook-Pro.local>
Date: Fri, 24 Nov 2023 13:06:15 +0100
Subject: [PATCH] =?UTF-8?q?terry:=20Logikl=C3=BCcken=20l=C3=B6sen=20in=20m?=
 =?UTF-8?q?ethoden=20getElement=20und=20setElement?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 CheckSparseVector.java | 14 +++++++-------
 SparseVector.java      |  9 ++++++++-
 2 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/CheckSparseVector.java b/CheckSparseVector.java
index 6f0219a..1dd0457 100644
--- a/CheckSparseVector.java
+++ b/CheckSparseVector.java
@@ -22,8 +22,6 @@ public class CheckSparseVector {
         otherVector1.setElement(9, 112.79);
         otherVector1.setElement(10, -212.79);
         otherVector1.setElement(12, 12);
-
-
         System.out.println("\n");
 
         SparseVector otherVector2 = new SparseVector(5);
@@ -62,19 +60,19 @@ public class CheckSparseVector {
         }
         System.out.println("\n");
 
-        SparseVector otherVector5 = new SparseVector(20); // ein leerer other. Vektor mit Länge 10
+        SparseVector otherVector5 = new SparseVector(20); // ein leerer other. Vektor mit Länge 20
         System.out.println("der Wert des otherVector5 an der Position 0 ist: "+ otherVector5.getElement(0)); // 0.0
         System.out.println("der Wert des otherVector5 an der Position 5 ist: "+ otherVector5.getElement(5)); // 0.0
         System.out.println("der Wert des otherVector5 an der Position 17 ist: "+ otherVector5.getElement(17));// 0.0
         System.out.println("der Wert des otherVector5 an der Position 19 ist: " + otherVector5.getElement(19));// 0.0
         System.out.println("die Länge des otherVector5 ist: " + otherVector5.getLength()); // 20
              try{ 
-        System.out.println(otherVector4.getElement(20));
+        System.out.println(otherVector5.getElement(20));
         } catch(Exception e){
         System.out.println(e);
         }
           try{ 
-        System.out.println(otherVector4.getElement(-1));
+        System.out.println(otherVector5.getElement(-1));
         } catch(Exception e){
         System.out.println(e);
         }
@@ -96,6 +94,8 @@ public class CheckSparseVector {
         } catch(Exception e){
         System.out.println(e);
         }
+        System.out.println("\n");
+
         // testen equal(), wenn die beiden Vektoren identisch sind → should be true, not anymore :)
         System.out.println(controlVector.equals(otherVector1));
         System.out.println("\n");
@@ -103,14 +103,14 @@ public class CheckSparseVector {
         controlVector.removeElement(5);
         System.out.println("der Wert des controlVector an der Position 5 ist: "+ controlVector.getElement(5)); // 0.0
         System.out.println("Die Länge nach remove ist: "+ controlVector.getLength()); // 10
+        System.out.println("\n");
 
         controlVector.removeElement(7);
         System.out.println("der Wert des controlVector an der Position 7 ist: "+ controlVector.getElement(7)); // 0.0
-
+        System.out.println("\n");
 
         controlVector.setElement(5, 100);
         System.out.println("der Wert des controlVector an der Position 5 ist: "+ controlVector.getElement(5)); //100.0
-
         System.out.println("\n");
 
         //  testen equals(other) --> nicht identisch
diff --git a/SparseVector.java b/SparseVector.java
index 95a8184..c4ac2a8 100644
--- a/SparseVector.java
+++ b/SparseVector.java
@@ -33,6 +33,13 @@ public class SparseVector {
     public void setElement(int index, double value) {
 
         removeElement(index);
+        
+        int maxIndex = this.getLength()-1;
+        
+        if (index > maxIndex) {
+            System.out.println("Fehler: Der Index " + index + " ist außerhalb des erlaubten Bereichs.");
+            return;
+        }
 
         if (this.head == null || this.head.index > index) {
             this.head = new Node(index, value, this.head);
@@ -68,7 +75,7 @@ public class SparseVector {
 
         Node now = this.head;
 
-        while (now != null && now.index < index) {
+        while (now != null) {
             if (now.index == index) {
                 return now.value;
             }