refactor: include only in header files

Makefile

@@ -43,7 +43,7 @@ set_version: ${LIB} ./set_version.c
 test: ${LIB} $(addprefix build/, ${TEST_OBJECTS})
 	mkdir --parents ./bin
 	${CC} ${CC_FLAGS} -o ${TEST_EXECUTABLE} $(addprefix build/, ${TEST_OBJECTS}) ${LIB_CC_FLAGS}
-	./${TEST_EXECUTABLE}
+	./${TEST_EXECUTABLE} ${ARGS}
 
 .PHONY: lint
 lint:

lib/array_list.h

@@ -4,8 +4,6 @@
 #include <stdbool.h>
 #include <stdlib.h>
 
-#include "linked_list.h"
-
 #define ARRAY_LIST_INITIAL_CAPACITY 10
 
 struct array_list {

lib/character.c

@@ -1,10 +1,5 @@
 #include "character.h"
 
-#include <stdbool.h>
-#include <stdlib.h>
-
-#include "string.h"
-
 void character_append(char* string, const char character) {
   size_t length = string_get_length(string);
   character_append_at(string, character, length);

lib/character.h

@@ -4,6 +4,8 @@
 #include <stdbool.h>
 #include <stdlib.h>
 
+#include "string.h"
+
 /**
  * @brief Append a character to a string, assuming string points to an array
  * with enough space.

lib/convert.c

@@ -1,13 +1,5 @@
 #include "convert.h"
 
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "character.h"
-#include "mathematics.h"
-#include "stdbool.h"
-#include "string.h"
-
 char* convert_character_to_string(const char character) {
   char* string = malloc(sizeof(char*) * 2);
   if (string == NULL) {

lib/convert.h

@@ -1,6 +1,14 @@
 #ifndef __CONVERT__
 #define __CONVERT__
 
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "character.h"
+#include "mathematics.h"
+#include "stdbool.h"
+#include "string.h"
+
 char* convert_character_to_string(const char character);
 
 char convert_character_to_digit(const char character);

lib/dictionary.c

@@ -1,9 +1,5 @@
 #include "dictionary.h"
 
-#include <stdlib.h>
-
-#include "string.h"
-
 struct dictionary *dictionary_initialization() {
   struct dictionary *dictionary = malloc(sizeof(struct dictionary));
   dictionary->items = malloc(sizeof(struct dictionary_item *) * DICTIONARY_INITIAL_CAPACITY);

lib/dictionary.h

@@ -4,7 +4,7 @@
 #include <stdbool.h>
 #include <stdlib.h>
 
-#include "linked_list.h"
+#include "string.h"
 
 #define DICTIONARY_INITIAL_CAPACITY 10
 

lib/filesystem.c

@@ -1,16 +1,5 @@
 #include "filesystem.h"
 
-#include <dirent.h>
-#include <errno.h>
-#include <fcntl.h>
-#include <pwd.h>
-#include <stdbool.h>
-#include <stdio.h>
-#include <stdlib.h>
-#include <unistd.h>
-
-#include "string.h"
-
 int filesystem_read(char *path, char **file_content, off_t *file_size) {
   int file_descriptor = open(path, O_RDONLY);
   if (file_descriptor == -1) {

lib/filesystem.h

@@ -1,7 +1,16 @@
 #ifndef __FILESYSTEM__
 #define __FILESYSTEM__
 
+#include <dirent.h>
+#include <errno.h>
 #include <fcntl.h>
+#include <pwd.h>
+#include <stdbool.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <unistd.h>
+
+#include "string.h"
 
 /**
  * @brief Read the content of a file.

lib/linked_list.c

@@ -1,10 +1,5 @@
 #include "linked_list.h"
 
-#include <stdbool.h>
-#include <stdlib.h>
-
-#include "stack.h"
-
 struct linked_list *linked_list_initialization() {
   struct linked_list *list = malloc(sizeof(*list));
   if (list == NULL) {

lib/linked_list.h

@@ -1,10 +1,12 @@
 #ifndef __LINKED_LIST__
 #define __LINKED_LIST__
 
+#include <stdbool.h>
 #include <stdlib.h>
 
+#include "stack.h"
+
 struct linked_list {
-  // first node of the list
   struct linked_list_node *head;
 
   size_t length;

lib/mathematics.c

@@ -1,7 +1,5 @@
 #include "mathematics.h"
 
-#include <stdbool.h>
-
 bool mathematics_equals(const float number1, const float number2) {
   return (number1 - number2) < MATHEMATICS_FLOAT_PRECISION;
 }

lib/queue.c

@@ -1,8 +1,5 @@
 #include "queue.h"
 
-#include <stdio.h>
-#include <stdlib.h>
-
 struct queue *queue_initialization() {
   struct queue *queue = malloc(sizeof(*queue));
   if (queue == NULL) {

lib/queue.h

@@ -1,6 +1,7 @@
 #ifndef __QUEUE__
 #define __QUEUE__
 
+#include <stdio.h>
 #include <stdlib.h>
 
 // FIFO = First In First Out

lib/stack.c

@@ -1,8 +1,5 @@
 #include "stack.h"
 
-#include <stdio.h>
-#include <stdlib.h>
-
 struct stack *stack_initialization() {
   struct stack *stack = malloc(sizeof(*stack));
   if (stack == NULL) {

lib/stack.h

@@ -1,6 +1,7 @@
 #ifndef __STACK__
 #define __STACK__
 
+#include <stdio.h>
 #include <stdlib.h>
 
 // LIFO = Last In First Out

lib/string.c

@@ -1,13 +1,5 @@
 #include "string.h"
 
-#include <stdbool.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "character.h"
-#include "convert.h"
-#include "dictionary.h"
-
 size_t string_get_length(const char* string) {
   size_t length = 0;
   while (string[length] != '\0') {

lib/string.h

@@ -2,8 +2,13 @@
 #define __STRING__
 
 #include <stdbool.h>
+#include <stdio.h>
 #include <stdlib.h>
 
+#include "character.h"
+#include "convert.h"
+#include "dictionary.h"
+
 /**
  * @brief Return the length of a string (excluding '\0').
  *
@@ -100,7 +105,7 @@ char* string_reverse(const char* string);
  *
  * @param string1
  * @param string2
- * @return true if the strings are equals, false otherwise
+ * @return true if the strings are equals, false otherwise.
  */
 bool string_equals(const char* string1, const char* string2);
 
@@ -108,7 +113,7 @@ bool string_equals(const char* string1, const char* string2);
  * @brief Check if the string is a integer.
  *
  * @param string
- * @return true if the string is a integer, false otherwise
+ * @return true if the string is a integer, false otherwise.
  */
 bool string_get_is_integer(const char* string);
 
@@ -143,7 +148,7 @@ char* string_concatenate(char* string1, char* string2);
  * @brief Check if a string contains only unique characters.
  *
  * @param string
- * @return true if string contains only unique characters, false otherwise
+ * @return true if string contains only unique characters, false otherwise.
  */
 bool string_get_has_unique_characters(const char* string);
 
@@ -162,7 +167,7 @@ char* string_substring(const char* string, size_t index_start, size_t index_end)
  *
  * @param string
  * @param substring
- * @return true if the string contains the substring, false otherwise
+ * @return true if the string contains the substring, false otherwise.
  */
 bool string_get_is_substring(const char* string, const char* substring);
 
@@ -188,7 +193,7 @@ char* string_get_last_occurence_of_character(const char* string, char character)
  *
  * @param string
  * @param prefix
- * @return true if the string starts with the substring, false otherwise
+ * @return true if the string starts with the substring, false otherwise.
  */
 bool string_starts_with(const char* string, const char* prefix);
 
@@ -197,7 +202,7 @@ bool string_starts_with(const char* string, const char* prefix);
  *
  * @param string
  * @param prefix
- * @return true if the string ends with the substring, false otherwise
+ * @return true if the string ends with the substring, false otherwise.
  */
 bool string_ends_with(const char* string, const char* prefix);
 

lib/terminal.c

@@ -1,16 +1,5 @@
 #include "terminal.h"
 
-#include <stdbool.h>
-#include <stdint.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "character.h"
-#include "dictionary.h"
-#include "linked_list.h"
-#include "queue.h"
-#include "stack.h"
-
 char* terminal_input() {
   char character;
   size_t length = 1;

lib/terminal.h

@@ -1,8 +1,12 @@
 #ifndef __TERMINAL__
 #define __TERMINAL__
 
+#include <stdbool.h>
+#include <stdint.h>
+#include <stdio.h>
 #include <stdlib.h>
 
+#include "character.h"
 #include "dictionary.h"
 #include "linked_list.h"
 #include "queue.h"

test/array_list_test.c

@@ -1,12 +1,5 @@
 #include "array_list_test.h"
 
-#include <assert.h>
-#include <stdint.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-
 void array_list_test() {
   struct array_list *list = array_list_initialization();
   assert(list->size == 0);

test/array_list_test.h

@@ -1,6 +1,13 @@
 #ifndef __ARRAY_LIST_TEST__
 #define __ARRAY_LIST_TEST__
 
+#include <assert.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+
 void array_list_test();
 
 #endif

test/character_test.c

@@ -1,12 +1,5 @@
 #include "character_test.h"
 
-#include <assert.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-#include "test.h"
-
 void character_test() {
   character_append_test();
   character_append_at_test();

test/character_test.h

@@ -1,6 +1,13 @@
 #ifndef __CHARACTER_TEST__
 #define __CHARACTER_TEST__
 
+#include <assert.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+#include "test.h"
+
 void character_test();
 
 void character_append_test();

test/convert_test.c

@@ -1,12 +1,5 @@
 #include "convert_test.h"
 
-#include <assert.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-#include "test.h"
-
 void convert_test() {
   convert_character_to_string_test();
   convert_character_to_digit_test();

test/convert_test.h

@@ -1,6 +1,13 @@
 #ifndef __CONVERT_TEST__
 #define __CONVERT_TEST__
 
+#include <assert.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+#include "test.h"
+
 void convert_test();
 
 void convert_character_to_string_test();

test/dictionary_test.c

@@ -1,11 +1,5 @@
 #include "dictionary_test.h"
 
-#include <assert.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-
 void dictionary_test() {
   struct dictionary *dictionary = dictionary_initialization();
   assert(dictionary->length == 0);

test/dictionary_test.h

@@ -1,6 +1,12 @@
 #ifndef __DICTIONARY_TEST__
 #define __DICTIONARY_TEST__
 
+#include <assert.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+
 void dictionary_test();
 
 #endif

test/linked_list_test.c

@@ -1,12 +1,5 @@
 #include "linked_list_test.h"
 
-#include <assert.h>
-#include <stdint.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-
 void linked_list_test() {
   linked_list_initialization_test();
   linked_list_add_in_head_test();

test/linked_list_test.h

@@ -1,6 +1,13 @@
 #ifndef __LINKED_LIST_TEST__
 #define __LINKED_LIST_TEST__
 
+#include <assert.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+
 void linked_list_test();
 
 void linked_list_initialization_test();

test/mathematics_test.c

@@ -1,9 +1,5 @@
 #include "mathematics_test.h"
 
-#include <assert.h>
-
-#include "libcproject.h"
-
 void mathematics_test() {
   mathematics_absolute_value_test();
   mathematics_pow_test();

test/mathematics_test.h

@@ -1,6 +1,10 @@
 #ifndef __MATHEMATICS_TEST__
 #define __MATHEMATICS_TEST__
 
+#include <assert.h>
+
+#include "libcproject.h"
+
 void mathematics_test();
 
 void mathematics_absolute_value_test();

test/queue_test.c

@@ -1,11 +1,5 @@
 #include "queue_test.h"
 
-#include <assert.h>
-#include <stdint.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-
 void queue_test() {
   queue_initialization_test();
   queue_push_test();

test/queue_test.h

@@ -1,6 +1,12 @@
 #ifndef __QUEUE_TEST__
 #define __QUEUE_TEST__
 
+#include <assert.h>
+#include <stdint.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+
 void queue_test();
 
 void queue_initialization_test();

test/stack_test.c

@@ -1,11 +1,5 @@
 #include "stack_test.h"
 
-#include <assert.h>
-#include <stdint.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-
 void stack_test() {
   stack_initialization_test();
   stack_push_test();

test/stack_test.h

@@ -1,6 +1,12 @@
 #ifndef __STACK_TEST__
 #define __STACK_TEST__
 
+#include <assert.h>
+#include <stdint.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+
 void stack_test();
 
 void stack_initialization_test();

test/string_test.c

@@ -1,12 +1,5 @@
 #include "string_test.h"
 
-#include <assert.h>
-#include <stdio.h>
-#include <stdlib.h>
-
-#include "libcproject.h"
-#include "test.h"
-
 void string_test() {
   string_get_length_test();
   string_to_uppercase_test();

test/string_test.h

@@ -1,6 +1,13 @@
 #ifndef __STRING_TEST__
 #define __STRING_TEST__
 
+#include <assert.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include "libcproject.h"
+#include "test.h"
+
 void string_test();
 
 void string_get_length_test();

test/test.c

@@ -1,6 +1,4 @@
-#include <stdbool.h>
-#include <stdio.h>
-#include <string.h>
+#include "test.h"
 
 bool assert_string_equal(const char *actual, const char *expected) {
   if (strcmp(expected, actual) != 0) {

test/test.h

@@ -2,6 +2,8 @@
 #define __TEST__
 
 #include <stdbool.h>
+#include <stdio.h>
+#include <string.h>
 
 bool assert_string_equal(const char *actual, const char *expected);