Fixed order of operations in Insert, added serialization and more unittests

schoeffelbe/pr03.py

@@ -107,7 +107,7 @@ def partitionIterative(arr : MemoryArray, l : Literal, h : Literal, mode=0):
 
     # Carefull that we do not use a reference. I suppose python would return one here if we just assign without value>Literal cast.
     # At least this helped fix weird issue
-    pivotValue : Literal = Literal(arr[h].value)
+    pivotValue : Literal = arr[h]
     i : MemoryCell = MemoryCell(l.pred())
   
     for j in mrange(l, h):
@@ -115,7 +115,7 @@ def partitionIterative(arr : MemoryArray, l : Literal, h : Literal, mode=0):
             i += Literal(1)                 # increment index of smaller element
             swap(arr, int(i), int(j))
   
-    swap(arr, int(i.succ()), int(h))
+    swap(arr, i.succ().value, h.value)
     return i.succ()
 
 def LEGACY_quickSort(z: MemoryArray, l: Literal = Literal(0), r: Literal = Literal(-1), mode=0):
@@ -185,6 +185,6 @@ if __name__ == '__main__':
         print(filename)
         toSort = MemoryArray.create_array_from_file(filename)
         timeMS(quickSortIterative, toSort, Literal(0), toSort.length().pred(), mode=1)
-        # print(toSort)
+        print(toSort)
 
     print("Kann durch die Modifikation eine besser Laufzeit als nlog(n) erreicht werden? Nein! nlog(n) ist das Minimum. Durch die Änderung kann aber der Worst-Case fall von n^2 für z.B. bereits vorsortierte Arrays oder Arrays mit vielen Duplikaten vermieden werden.")

schoeffelbe/priorityQueue.py

@@ -89,12 +89,13 @@ class PriorityQueue:
         
         i = self.size
         self.heap[i].set([entry])
-        self.size += Literal(1)
         
         while i > Literal(0) and self.heap[self.parent(i)].value[0] < self.heap[i].value[0]:
             self.swap(i, self.parent(i))
             i = self.parent(i)
 
+        self.size += Literal(1)
+    
     def pop(self):
         if self.isEmpty():
             raise IndexError("Queue is empty!")
@@ -119,6 +120,36 @@ class PriorityQueue:
     def __len__(self):
         return self.size
 
+    def __str__(self):
+        entries = []
+        for i in mrange(self.size.value):
+            entry = self.heap[i].value[0]
+            if entry.getItem() != MAX_VALUE:
+                entries.append(str(entry))
+        return "[" + ", ".join(entries) + "]"
+
+def testQueueRandom(number: int):
+    import random
+    import string
+
+    pq = PriorityQueue(Literal(101))
+
+    entries = []
+    for _ in range(number):
+        value = ''.join(random.choices(string.ascii_uppercase + string.digits, k=3))
+        priority = random.randint(1, 100)
+        entry = HeapEntry(value, priority)
+        entries.append(entry)
+        pq.insert(entry)
+
+    print(pq)
+    for entry in entries:
+        print(entry)
+
+    while not pq.isEmpty(): 
+        print(pq.pop())
+
+
 
 if __name__ == '__main__':
     # Proof of Concept
@@ -153,6 +184,11 @@ if __name__ == '__main__':
     pq.insert(HeapEntry("A", 1))
     pq.insert(HeapEntry("C", 3))
     pq.insert(HeapEntry("B", 2))
-    print(pq.pop())
-    print(pq.pop())
-    print(pq.pop())
+    pq.insert(HeapEntry(42, 4))
+    pq.insert(HeapEntry(42, 1))
+    pq.insert(HeapEntry("C", 2))
+    print(pq)
+    while not pq.isEmpty(): 
+        print(pq.pop())
+
+    testQueueRandom(100)