Kommentare gelöscht

bintree.c

@@ -2,144 +2,35 @@
 #include "stack.h"
 #include "bintree.h"
 
-
 //TODO: binären Suchbaum implementieren
-/*  * `addToTree`: fügt ein neues Element in den Baum ein (rekursiv),       Done
+/*  * `addToTree`: fügt ein neues Element in den Baum ein (rekursiv),
-    * `clearTree`: gibt den gesamten Baum frei (rekursiv),                  Done
+    * `clearTree`: gibt den gesamten Baum frei (rekursiv),
     * `treeSize`: zählt die Knoten im Baum (rekursiv),
     * `nextTreeData`: Traversierung mit Hilfe des zuvor implementierten Stacks. */
 
-
-    static TreeNode *root = NULL;
-
-
-    TreeNode        *addToTree      (TreeNode *root, const void *data, size_t dataSize, CompareFctType compareFct, int *isDuplicate);
-    void            *nextTreeData   (TreeNode *root);
-    void            clearTree       (TreeNode *root);
-    unsigned int    treeSize        (const TreeNode *root);
-
-    // self declared functions
-    TreeNode        *addToTreeRec   (TreeNode *currentNode, TreeNode *newNode, CompareFctType compareFct, int *isDuplicate);
-    void            clearTreeRec    (TreeNode *currentNode);
-    void            clearNode       (TreeNode *node);
-    void            treeSizeRec     (const TreeNode *currentNode, unsigned int *nodeCount);
-
-
 // Adds a copy of data's pointer destination to the tree using compareFct for ordering. Accepts duplicates
 // if isDuplicate is NULL, otherwise ignores duplicates and sets isDuplicate to 1 (or to 0 if a new entry is added).
-
-//      returned Value is new root
 TreeNode *addToTree(TreeNode *root, const void *data, size_t dataSize, CompareFctType compareFct, int *isDuplicate)
 {
-    // create a node
-    TreeNode *newNode;
 
-
-    newNode         = calloc(1, sizeof(TreeNode));
-    newNode->data   = calloc(1, dataSize);
-    newNode->left   = NULL;
-    newNode->right  = NULL;
-    memcpy(newNode->data, data, dataSize);
-
-
-    return addToTreeRec(root, newNode, compareFct, isDuplicate);
 }
 
-
-TreeNode *addToTreeRec(TreeNode *currentNode, TreeNode *newNode, CompareFctType compareFct, int *isDuplicate)
-{
-    if (currentNode == NULL)
-    {
-        isDuplicate = 0;
-        return newNode;
-    }
-    else if (compareFct(newNode->data, currentNode->data) < 0)
-    {
-        currentNode->left   = addToTreeRec(currentNode->left, newNode, compareFct, isDuplicate);
-    }
-    else if (compareFct(newNode->data, currentNode->data) > 0)
-    {
-        currentNode->right  = addToTreeRec(currentNode->right, newNode, compareFct, isDuplicate);
-    }
-    else
-    {
-        //duplicate
-        if (isDuplicate == NULL)
-        {
-            currentNode->left = addToTreeRec(currentNode->left, newNode, compareFct, isDuplicate);
-        }
-        else
-        {
-            *isDuplicate = 1;
-        }
-    }
-
-
-    return currentNode;
-}
- 
-
 // 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.
 // Use your implementation of a stack to organize the iterator. Push the root node and all left nodes first. On returning the next element,
 // push the top node and push all its left nodes.
-
-// Needs stack!!
 void *nextTreeData(TreeNode *root)
 {
 
 }
 
-
 // Releases all memory resources (including data copies).
 void clearTree(TreeNode *root)
 {
-    clearTreeRec(root);
+
 }
 
-
-void clearTreeRec(TreeNode *currentNode)
-{
-    if (currentNode != NULL)
-    {
-        clearTree(currentNode->left);
-        clearNode(currentNode);
-        clearTree(currentNode->right);
-    }
-}
-
-
-void clearNode(TreeNode *node)
-{
-    free(node->data);
-    node->data  = NULL;
-
-    node->left  = NULL;
-    node->right = NULL;
-
-    free(node);
-    node        = NULL;
-}
-
-
 // Returns the number of entries in the tree given by root.
 unsigned int treeSize(const TreeNode *root)
 {
-    unsigned int amountOfNodes = 0;
 
-    
-    treeSizeRec(root, &amountOfNodes);
-
-
-    return amountOfNodes;
 }
-
-void treeSizeRec(const TreeNode *currentNode, unsigned int *nodeCount)
-{
-    if (currentNode != NULL)
-    {
-        treeSizeRec(currentNode->left, nodeCount);
-        *nodeCount++;
-        treeSizeRec(currentNode->right, nodeCount);
-    }
-}
-

bintreeTests.c

@@ -1,180 +0,0 @@
-#include <stdio.h>
-#include "unity.h"
-#include "bintree.h"
-
-#define MAX_TEST_NAME_LEN 10
-
-
-static int compareIntEntries(const void *arg1, const void *arg2)
-{
-    const int *entry1 = (const int *)arg1;
-    const int *entry2 = (const int *)arg2;
-
-    int result = *entry2 - *entry1;
-
-
-    return result;
-}
-
-
-// test if addToTree expands tree correctly
-//      by going down the path where the given pice of data is expected
-//      and checking if the pice of data is found there
-void test_addToTreeExpandsTreeCorrectly(void)
-{
-    TreeNode    *testRoot       = NULL;
-    int         *testIsDouble   = 0;
-    int         score1          = 12;
-    int         score2          = 6;
-    int         score3          = 18;
-    int         score4          = 3;
-    int         score5          = 9;
-    int         score6          = 15;
-    int         score7          = 21;
-
-
-    testRoot = addToTree(testRoot, &score1, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score2, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score3, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score4, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score5, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score6, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score7, sizeof(int), compareIntEntries, NULL);
-
-    // Checking the Tree without Doubles
-    TEST_ASSERT_NOT_NULL(testRoot);
-    TEST_ASSERT_EQUAL(score1, testRoot);
-    TEST_ASSERT_NOT_NULL(testRoot->left);
-    TEST_ASSERT_EQUAL(score2, testRoot->left);
-    TEST_ASSERT_NOT_NULL(testRoot->right);
-    TEST_ASSERT_EQUAL(score3, testRoot->right);
-    TEST_ASSERT_NOT_NULL(testRoot->left->left);
-    TEST_ASSERT_EQUAL(score4, testRoot->left->left);
-    TEST_ASSERT_NOT_NULL(testRoot->left->right);
-    TEST_ASSERT_EQUAL(score5, testRoot->left->right);
-    TEST_ASSERT_NOT_NULL(testRoot->right->left);
-    TEST_ASSERT_EQUAL(score6, testRoot->right->left);
-    TEST_ASSERT_NOT_NULL(testRoot->right->right);
-    TEST_ASSERT_EQUAL(score7, testRoot->right->right);
-
-    // Adding Double
-    testRoot = addToTree(testRoot, &score4, sizeof(int), compareIntEntries, NULL);
-
-    TEST_ASSERT_NOT_NULL(testRoot->left->left->left);
-    TEST_ASSERT_EQUAL(score4, testRoot->left->left->left);
-
-    // Trying to add Double while Doubles not Permitted
-    testRoot = addToTree(testRoot, &score7, sizeof(int), compareIntEntries, testIsDouble);
-
-    TEST_ASSERT_NULL(testRoot->right->right->left);
-    TEST_ASSERT_EQUAL(1, testIsDouble);
-
-
-    clearTree(testRoot);
-}
-
-
-// test if nextTreeData returns the next pice of data correctly
-//      needs Stack!!!
-void test_nextTreeDataReturnsNextDataCorrectly(void)
-{
-
-}
-
-
-// test if clear Tree frees all node.name and node memory AND sets them to zero
-//      aditionally tests if the memoryspaces have been cleared
-void test_clearTreeworksLikeExpected(void)
-{
-    TreeNode    *testRoot       = NULL;
-    int         score1          = 12;
-    int         score2          = 6;
-    int         score3          = 18;
-    int         score4          = 3;
-    int         score5          = 9;
-    int         score6          = 15;
-    int         score7          = 21;
-
-
-    // Fill Tree
-    testRoot = addToTree(testRoot, &score1, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score2, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score3, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score4, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score5, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score6, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score7, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score4, sizeof(int), compareIntEntries, NULL);
-
-   // Save all Adresses
-   TreeNode *node1  = testRoot->left;
-   TreeNode *node2  = testRoot->left->left;
-   TreeNode *node3  = testRoot->left->right;
-   TreeNode *node4  = testRoot->right;
-   TreeNode *node5  = testRoot->right->left;
-   TreeNode *node6  = testRoot->right->right;
-   TreeNode *node7  = testRoot->left->left->left;
-
-   clearTree(testRoot);
-
-    // Check if everything has been set to NULL
-    TEST_ASSERT_NULL(testRoot);
-    TEST_ASSERT_NULL(node1);
-    TEST_ASSERT_NULL(node2);
-    TEST_ASSERT_NULL(node3);
-    TEST_ASSERT_NULL(node4);
-    TEST_ASSERT_NULL(node5);
-    TEST_ASSERT_NULL(node6);
-    TEST_ASSERT_NULL(node7);
-}
-
-
-// tests if treeSize returns correct amount of nodes in Tree
-//      by using addToTree a given number of times and testing to see if 
-//      the treeSize matches the number of nodes added
-void test_treeSizeWorkingLikeExpected(void)
-{
-    TreeNode        *testRoot       = NULL;
-    int             nodeCount       = 7;
-    unsigned int    testTreeSize    = 0;
-    int             score1          = 12;
-    int             score2          = 6;
-    int             score3          = 18;
-    int             score4          = 3;
-    int             score5          = 9;
-    int             score6          = 15;
-    int             score7          = 21;
-
-
-    // Fill Tree
-    testRoot = addToTree(testRoot, &score1, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score2, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score3, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score4, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score5, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score6, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score7, sizeof(int), compareIntEntries, NULL);
-    testRoot = addToTree(testRoot, &score4, sizeof(int), compareIntEntries, NULL);
-
-    testTreeSize = treeSize(testRoot);
-
-    TEST_ASSERT_EQUAL(nodeCount, testTreeSize);
-
-    clearTree(testRoot);
-}
-
-
-// main, strings together all tests
-int main()
-{
-    UNITY_BEGIN();
-
-    printf("\n============================\nBinary Tree tests\n============================\n");
-    RUN_TEST(test_addToTreeExpandsTreeCorrectly);
-    // RUN_TEST(test_nextTreeDataReturnsNextDataCorrectly);
-    RUN_TEST(test_clearTreeworksLikeExpected);
-    RUN_TEST(test_treeSizeWorkingLikeExpected);
-
-    return UNITY_END();
-}
-

highscores.txt

@@ -1 +1,2 @@
+player_name;9943
 player1;3999

makefile

@@ -38,6 +38,11 @@ $(program_obj_filesobj_files): %.o: %.c
 unitTests:
 	echo "needs to be implemented"
 
+stack: stack.c
+	$(CC) $(FLAGS) -c stack stack.c 
+
+test_stack: stack.o test_stack.c $(unityfolder)/unity.c
+	$(CC) $(FLAGS) -o runstackTests test_stack.c stack.o $(unityfolder)/unity.c
 # --------------------------
 # Clean
 # --------------------------

stack.c

@@ -10,24 +10,64 @@
 // Pushes data as pointer onto the stack.
 StackNode *push(StackNode *stack, void *data)
 {
+    StackNode *newNode = malloc(sizeof(StackNode));
+    if(newNode == NULL)
+    {
+        //printf("Fehler Bei Speicherallozierung!");
+        return NULL;
+    }
 
+    newNode->data = data;
+    newNode->next = stack;
+
+    return newNode;
 }
 
 // Deletes the top element of the stack (latest added element) and releases its memory. (Pointer to data has to be
 // freed by caller.)
 StackNode *pop(StackNode *stack)
 {
+    if(stack == NULL)
+    {
+        //printf("Fehlerhafte Adresse uebergeben");
+        return NULL;
+    }
 
+    StackNode *next = stack->next;
+
+    free(stack);
+
+    return next;
 }
 
 // Returns the data of the top element.
 void *top(StackNode *stack)
 {
+    if(stack == NULL)
+    {
+        //printf("Fehlerhafte Adresse uebergeben");
+        return NULL;
+    }
 
+    return stack->data;
 }
 
 // Clears stack and releases all memory.
 void clearStack(StackNode *stack)
 {
+    if(stack == NULL)
+    {
+        //printf("Fehlerhafte Adresse uebergeben");
+        return;
+    }
 
+    StackNode *currentNode = stack;
+    StackNode *nextNode;
+
+    while(currentNode != NULL)
+    {
+        nextNode = currentNode->next;
+        free(currentNode);
+        currentNode = nextNode;
+    }
 }

stack.h

@@ -8,6 +8,11 @@ The latest element is taken from the stack. */
 #include <stdlib.h>
 
 //TODO: passenden Datentyp als struct anlegen
+typedef struct StackNode
+{
+    void *data;
+    struct StackNode *next;
+} StackNode;
 
 // Pushes data as pointer onto the stack.
 StackNode *push(StackNode *stack, void *data);

test_stack.c

@@ -0,0 +1,120 @@
+#include <stdio.h>
+#include <stdlib.h>
+#include "unity.h"
+#include "stack.h"
+
+void setUp(void) {
+    // Falls notwendig, kann hier Vorbereitungsarbeit gemacht werden
+}
+
+void tearDown(void) {
+    // Hier kann Bereinigungsarbeit nach jedem Test durchgeführt werden
+}
+
+
+
+void testeStackBeschreiben()
+{
+    //printf("=== Test: push() ===\n");
+
+    StackNode *stack = NULL;
+    int wert1 = 42;
+    stack = push(stack, &wert1);
+
+    int *topValue = (int *)(stack->data);
+
+    TEST_ASSERT_NOT_NULL(topValue);
+    TEST_ASSERT_EQUAL_INT(42, *topValue);
+
+
+    int wert2 = 12;
+    stack = push(stack, &wert2);
+
+    topValue = (int *)(stack->data);
+    int *secondValue = (int *)((stack->next)->data);
+
+    TEST_ASSERT_NOT_NULL(topValue);
+    TEST_ASSERT_EQUAL_INT(12, *topValue);
+    TEST_ASSERT_NOT_NULL(secondValue);
+    TEST_ASSERT_EQUAL_INT(42, *secondValue);
+
+
+    //printf("=== Ende Test: push() ===\n\n");
+    return;
+}
+
+void testepop()
+{
+    //printf("=== Test: pop() ===\n");
+
+    StackNode *stack = NULL;
+    int wert1 = 20;
+    int wert2 = 74;
+
+    stack = push(stack, &wert1);
+    stack = push(stack, &wert2);
+
+    stack = pop(stack);
+
+    int *topValue = (int *)(stack->data);
+
+    TEST_ASSERT_NOT_NULL(topValue);
+    TEST_ASSERT_EQUAL_INT(20, *topValue);
+
+    //printf("=== Ende Test: pop() ===\n\n");
+}
+
+void testetop()
+{
+    //printf("=== Test: top() ===\n");
+    
+    StackNode *stack = NULL;
+    int wert1 = 20;
+    int wert2 = 74;
+
+    stack = push(stack, &wert1);
+    stack = push(stack, &wert2);
+
+    int *topValue = top(stack);
+
+    TEST_ASSERT_NOT_NULL(topValue);
+    TEST_ASSERT_EQUAL_INT(74, *topValue);
+
+    //printf("=== Ende Test: top() ===\n\n");
+}
+
+void testeclearStack()
+{
+    //printf("=== Test: clearStack() ===\n");
+
+    StackNode *stack = NULL;
+    int wert1 = 20;
+    int wert2 = 74;
+
+    stack = push(stack, &wert1);
+    stack = push(stack, &wert2);
+
+    clearStack(stack);
+
+    //printf("Test: clearStack() aufgerufen. Speicher freigegeben.\n");
+    //printf("=== Ende Test: clearStack() ===\n\n");
+}
+
+int main()
+{
+    UNITY_BEGIN();
+
+    //printf("...");
+
+    RUN_TEST(testeStackBeschreiben);
+    RUN_TEST(testepop);
+    RUN_TEST(testetop);
+    RUN_TEST(testeclearStack);
+
+    /*testeStackBeschreiben();
+    testepop();
+    testetop();
+    testeclearStack();*/
+
+    return UNITY_END();
+}