Add the ability to select multiple tasks at once

src/db.c

@@ -156,6 +156,8 @@ void delete_task(int id) {
 
 
 void get_task(int id, task_t* t) {
+    t->id = -1;
+
     sqlite3* db = get_db();
     sqlite3_stmt* stmt;
 
@@ -185,7 +187,7 @@ void get_task(int id, task_t* t) {
     sqlite3_finalize(stmt);
     sqlite3_close(db);
 
-    // t.id will be (-1) if not found
+    // t->id will be (-1) if not found
 }
 
 int list_tasks() {
@@ -224,5 +226,24 @@ int list_tasks() {
 }
 
 int get_new_task_id() {
-    return rand()%10000;
+    task_t t;
+
+    int new_id = rand() %MAX_TASK_ID;
+    t.id = new_id;
+
+    int i=0;
+    while (t.id != -1 && i < NEW_TASK_ID_MAX_RETRIES) { // If t.id==-1, task is not found so the id is sage to use
+        new_id = rand() %MAX_TASK_ID;
+        get_task(new_id, &t);
+        i++;
+    }
+
+    if (i == NEW_TASK_ID_MAX_RETRIES) {
+        printf(RED BOLD "Il semblerait que vous ayez un nombre de tâches se rapprochant de %d.\n" RESET, MAX_TASK_ID);
+        printf(YELLOW "Il peut être judicieux de supprimer les tâches effectuées pour libérer de la place (et des identifiants).\n");
+        printf("Vous pouvez également modifier la limite du nombre d'identifiants et recompiler le projet.\n" RESET);
+        return -1;
+    }
+
+    return new_id;
 }

src/include/config.h

@@ -4,4 +4,7 @@
 #define VERSION "0.0.0"
 #define DB_FILE "takl.sqlite3"
 
+#define MAX_TASK_ID 10000 // max is set to MAX_TASK_ID-1
+#define NEW_TASK_ID_MAX_RETRIES 10000 // number of retries before giving up
+
 #endif

src/include/db.h

@@ -31,6 +31,7 @@ void delete_task(int id);
 
 /*
 Copie la tâche [id] dans *t si elle existe
+t.id vaut -1 si non trouvé
 */
 void get_task(int id, task_t* t);
 

src/main.c

@@ -15,9 +15,9 @@ void help(char* name) {
     printf("\tFormat de la date:\n");
     printf("\t\tRelatif: min+%%d, h+%%d, j+%%d\n");
     printf("\t\tAbsolu:  dd/mm (pas de changement d'année pour l'instant\n");
-    printf("\tinfo <id>\n");
+    printf("\tinfo <id1> <id2> ...\n");
-    printf("\tdone <id>\n");
+    printf("\tdone <id1> <id2> ...\n");
-    printf("\trm   <id>\n");
+    printf("\trm   <id1> <id2> ...\n");
 }
 
 
@@ -84,6 +84,9 @@ int add(int argc, char* argv[]) {
     }
 
     task_t t = create_task(argv[2], due_to);
+    if (t.id == -1) {
+        return 1;
+    }
     
     if (add_task(t) != 0) {
         printf("Erreur lors de l'ajout à la base de données\n");
@@ -102,20 +105,22 @@ int info(int argc, char* argv[]) {
         return 1;
     }
 
-    int id = strtol(argv[2], NULL, 10);
+    for (int i=2; i < argc; i++) {
+        int id = strtol(argv[i], NULL, 10);
 
-    task_t t;
+        task_t t;
-    t.id = -1;
+        t.id = -1;
-    get_task(id, &t);
+        get_task(id, &t);
 
-    if (t.id == -1) {
+        if (t.id == -1) {
-        printf(BOLD YELLOW "Tâche inexistante. Vérifiez que l'identifiant est bien correct\n" RESET);
+            printf(YELLOW "Tâche " BOLD "[%s]" RESET YELLOW " inexistante. Vérifiez que l'identifiant est bien correct\n" RESET, argv[i]);
-        return 1;
+            return 1;
+        }
+
+        print_task(t);
+        free(t.text);
     }
 
-    print_task(t);
-
-    free(t.text);
     return 0;
 }
 
@@ -126,20 +131,22 @@ int mark_done(int argc, char* argv[]) {
         return 1;
     }
 
-    int id = strtol(argv[2], NULL, 10);
+    for (int i=2; i < argc; i++) {
+        int id = strtol(argv[i], NULL, 10);
 
-    task_t t;
+        task_t t;
-    t.id = -1;
+        t.id = -1;
-    get_task(id, &t);
+        get_task(id, &t);
 
-    if (t.id == -1) {
+        if (t.id == -1) {
-        printf(BOLD YELLOW "Tâche inexistante. Vérifiez que l'identifiant est bien correct\n" RESET);
+            printf(YELLOW "Tâche " BOLD "[%s]" RESET YELLOW " inexistante. Vérifiez que l'identifiant est bien correct\n" RESET, argv[i]);
-        return 1;
+            return 1;
+        }
+
+        t.done = true;
+        update_task(t);
     }
 
-    t.done = true;
-    update_task(t);
-
     return 0;
 }
 
@@ -151,19 +158,21 @@ int delete(int argc, char* argv[]) {
         return 1;
     }
 
-    int id = strtol(argv[2], NULL, 10);
+    for (int i=2; i < argc; i++) {
+        int id = strtol(argv[i], NULL, 10);
 
-    task_t t;
+        task_t t;
-    t.id = -1;
+        t.id = -1;
-    get_task(id, &t);
+        get_task(id, &t);
 
-    if (t.id == -1) {
+        if (t.id == -1) {
-        printf(BOLD YELLOW "Tâche inexistante. Vérifiez que l'identifiant est bien correct\n" RESET);
+            printf(YELLOW "Tâche " BOLD "[%s]" RESET YELLOW " inexistante. Vérifiez que l'identifiant est bien correct\n" RESET, argv[i]);
-        return 1;
+            return 1;
+        }
+
+        delete_task(t.id);
     }
 
-    delete_task(t.id);
-
     return 0;
 }