aufräumen

.gitignore

@@ -6,6 +6,3 @@ runNumbersTest.exe
 numbers.o
 .vscode/launch.json
 .vscode/settings.json
-*.o
-*.exe
-runBintreeTest

bintree.c

@@ -12,26 +12,8 @@
 // if isDuplicate is NULL, otherwise ignores duplicates and sets isDuplicate to 1 (or to 0 if a new entry is added).
 TreeNode *addToTree(TreeNode *root, const void *data, size_t dataSize, CompareFctType compareFct, int *isDuplicate)
 {
-    TreeNode newNode = {data, NULL, NULL};
     
 
-    if(root == NULL)
-    {
-        return &newNode;
-    }
-
-    if (data < root->data)
-    {
-        root->left = addToTree(root->left, data, dataSize,compareFct, isDuplicate);
-    }
-    else if(data > root->data)
-    {
-       root->right = addToTree(root->right, data, dataSize,compareFct, isDuplicate);
-    }
-
-
-
-    return root;
 }
 
 // Iterates over the tree given by root. Follows the usage of strtok. If tree is NULL, the next entry of the last tree given is returned in ordering direction.
@@ -54,19 +36,27 @@ void clearTree(TreeNode *root)
     {
         clearTree(root->left);
         free(root->left);
-        root->left = NULL;
     }
-    if (root->right != NULL)
+    else if (root->right != NULL)
     {
         clearTree(root->right);
         free(root->right);
-        root->right = NULL;
     }
-    root = NULL;
+    root->data = NULL;
 }
 
 // Returns the number of entries in the tree given by root.
 unsigned int treeSize(const TreeNode *root)
 {
-    return root == NULL ? 0 : treeSize(root->left) + treeSize(root->right) + 1;
+    int counterL, counterR = 0;
+    if (root->left != NULL)
+    {
+        counterL = treeSize(root->left) + 1;
+    }
+    else if (root->right != NULL)
+    {
+        counterR = treeSize(root->right) + 1;
+    }
+
+    return counterL + counterR;
 }

highscores.txt

@@ -0,0 +1 @@
+player1;3999

makefile

@@ -44,12 +44,6 @@ unitTests: stack.o test_stack.c $(unityfolder)/unity.c
 numbersTests: numbers.o test_numbers.c $(unityfolder)/unity.c
 	$(CC) $(FLAGS) -I$(unityfolder) -o runNumbersTest test_numbers.c numbers.o $(unityfolder)/unity.c
 
-# --------------------------
-# bintree.c Tests
-# --------------------------
-bintreeTests: bintree.o test_bintree.c $(unityfolder)/unity.c
-	$(CC) $(FLAGS) -I$(unityfolder) -o runBintreeTest test_bintree.c bintree.o $(unityfolder)/unity.c
-
 # --------------------------
 # Clean
 # --------------------------

numbers.c

@@ -23,8 +23,6 @@ unsigned int checkArray(unsigned int *array, unsigned int len, unsigned int numb
     if (array[i] == number)
     {
       free = 0;
-      break;
-      
     }
   }
 
@@ -34,38 +32,33 @@ unsigned int checkArray(unsigned int *array, unsigned int len, unsigned int numb
 unsigned int *createNumbers(unsigned int len)
 {
   srand(time(NULL));
-  unsigned int *array = (unsigned int *)malloc(len * sizeof(unsigned int));
-  int randomNr, randomPos, filler;
-  
+  unsigned int *array = (unsigned int*)malloc(len * sizeof(unsigned int));
+  int randomNr, counter;
 
-  if (array == NULL)
+  if(array == NULL)
   {
-    return NULL; // Fehler
+    return NULL;
   }
 
   for (int i = 0; i < len; i++)
   {
-    array[i] = 0;
-  }
-  
-  for (int i = 0; i < len; i++)
-  {
+    counter = 0;
     do
     {
-      array[i] = (rand() % (2 * len))+ 1;
-    } while (!checkArray(array, i, array[i]));
+      if (counter == 9)
+      {
+        return NULL;
+      }
+      
+      randomNr = rand() % (2 * len + 1);
+      counter++;
+    } while (!checkArray(array, i, randomNr));
+
+    array[i] = randomNr;
+    printf("%u ", array[i]);
   }
 
-  randomPos = rand() % len;
-  randomNr = array[randomPos];
-
-  filler = randomPos;
-  while(filler == randomPos)
-  {
-    filler = rand() % len;
-  }
-  array[filler] = randomNr;
-  
+  printf("\n");
   return array;
 }
 
@@ -152,7 +145,7 @@ unsigned int getDuplicate(const unsigned int numbers[], unsigned int len)
   {
     temp[i] = numbers[i];
   }
-
+  
   // Sorting arr using mergesort
   mergeSort(temp, 0, len - 1);
 
@@ -161,7 +154,7 @@ unsigned int getDuplicate(const unsigned int numbers[], unsigned int len)
     duplicate = temp[i];
     if (duplicate == temp[i + 1])
     {
-
+      
       break;
     }
   }

test_bintree.c

@@ -1,80 +0,0 @@
-#include <stdio.h>
-#include <stdlib.h>
-#include "bintree.h"
-#include "unity.h"
-
-
-void sizeTest()
-{
-    TreeNode *root = (TreeNode *)malloc(sizeof(TreeNode));
-    TreeNode *node1 = (TreeNode *)malloc(sizeof(TreeNode));
-    TreeNode *node2 = (TreeNode *)malloc(sizeof(TreeNode));
-
-    int dataRoot = 2;
-    int dataNode1 = 1;
-    int dataNode2 = 3;
-
-    root->data = &dataRoot;
-    root->left = (TreeNode *)node1;
-    root->right = (TreeNode *)node2;
-
-    node1->data = &dataNode1;
-    node1->left = NULL;
-    node1->right = NULL;
-
-    node2->data = &dataNode2;
-    node2->left = NULL;
-    node2->right = NULL;
-
-    TEST_ASSERT_EQUAL_INT(3,treeSize(root));
-}
-
-void clearTest()
-{
-    TreeNode *root = (TreeNode *)malloc(sizeof(TreeNode));
-    TreeNode *node1 = (TreeNode *)malloc(sizeof(TreeNode));
-    TreeNode *node2 = (TreeNode *)malloc(sizeof(TreeNode));
-
-    int dataRoot = 2;
-    int dataNode1 = 1;
-    int dataNode2 = 3;
-
-    root->data = &dataRoot;
-    root->left = (TreeNode *)node1;
-    root->right = (TreeNode *)node2;
-
-    node1->data = &dataNode1;
-    node1->left = NULL;
-    node1->right = NULL;
-
-    node2->data = &dataNode2;
-    node2->left = NULL;
-    node2->right = NULL;
-
-    TreeNode *ptr = root;
-
-    clearTree(ptr);
-    TEST_ASSERT_EQUAL_INT(0,treeSize(root));
-}
-
-void setUp(void)
-{
-    // Falls notwendig, kann hier Vorbereitungsarbeit gemacht werden
-}
-
-void tearDown(void)
-{
-    // Hier kann Bereinigungsarbeit nach jedem Test durchgeführt werden
-}
-
-int main()
-{
-    UNITY_BEGIN();
-
-    printf("============================\nNumbers tests\n============================\n");
-
-    RUN_TEST(sizeTest);
-    RUN_TEST(clearTest);
-
-    return UNITY_END();
-}