map, vector: improve handing of replaced elements

Author: chatziko

include/ADTMap.h

@@ -27,7 +27,7 @@ Map map_create(CompareFunc compare, DestroyFunc destroy_key, DestroyFunc destroy
 
 int map_size(Map map);
 
-// Αν υπάρχει κλειδί ισοδύναμο με key, αλλάζει την τιμή του σε value. Διαφορετικά προσθέτει το κλειδί key με τιμή value.
+// Προσθέτει το κλειδί key με τιμή value. Αν υπάρχει κλειδί ισοδύναμο με key, τα παλιά key & value αντικαθίσταται από τα νέα.
 //
 // ΠΡΟΣΟΧΗ:
 // Όσο το key είναι μέλος του map, οποιαδήποτε μεταβολή στο περιεχόμενό του (στη μνήμη που δείχνει) έχει μη ορισμένη συμπεριφορά.

modules/UsingADTSet/ADTMap.c

@@ -66,9 +66,13 @@ void map_insert(Map map, Pointer key, Pointer value) {
 
 	MapNode node = set_find(map->set, &search_node);
 	if(node != NULL) {
-		if(map->destroy_value != NULL)
+		if(key != node->key && map->destroy_key != NULL)
+			map->destroy_key(node->key);
+
+		if(value != node->value && map->destroy_value != NULL)
 			map->destroy_value(node->value);
 
+		node->key = key;
 		node->value = value;
 		return;
 	}

modules/UsingDynamicArray/ADTVector.c

@@ -64,7 +64,7 @@ void vector_set_at(Vector vec, int pos, Pointer value) {
 	assert(pos >= 0 && pos < vec->size);	// LCOV_EXCL_LINE (αγνοούμε το branch από τα coverage reports, είναι δύσκολο να τεστάρουμε το false γιατί θα κρασάρει το test)
 
 	// Αν υπάρχει συνάρτηση destroy_value, την καλούμε για το στοιχείο που αντικαθίσταται
-	if(vec->destroy_value != NULL)
+	if(value != vec->array[pos] && vec->destroy_value != NULL)
 		vec->destroy_value(vec->array[pos]);
 
 	vec->array[pos] = value;