char**, pointeur et fonction

Bonjour,

arguments est passé par valeur à la fonction make_arguments_table, ce qui fait que les allocations mémoire qui sont faits dans la fonction le sont sur une copie locale de arguments ! Il faut passé l'adresse de arguments pour que ce pointeur soit directement utilisé et modifié (ou passer sa référence, ce qui revient au même puisque le passage par référence n'est rien d'autre qu'un passage par adresse avec une syntaxe de passage par valeur !).

PS: char** x et char* x[] c'est la même chose. Eviter d'utiliser ces 2 syntaxes en même temps, ça donne l'impression que c'est compris comme étant différents.

#include <stdio.h>
#include <string.h>
#include <malloc.h>

void make_arguments_table(int argc,char**,char***);

int main(int argc,char** argv)
{
   char** arguments;
   make_arguments_table(argc,argv,&arguments);
   printf("main:arguments    = %X\n",arguments);
   printf("main:arguments[0] = %X\n",arguments[0]);
   return 0;
}
   
void make_arguments_table(int argc,char** argv,char*** new_argv)
{
   int arguments_count = argc - 2;
   int counter = 2;
   int len = 0;
   int i=0;
       
   *new_argv = (char**)malloc(arguments_count * sizeof(char*));
   if (*new_argv == NULL)
   {
      printf("Lack of memory space (1)\n");
      return;
   }
   printf("make_arguments_table:arguments    = %X\n",*new_argv);
         
   for (i=0 ; i<arguments_count ; ++i)
   {
      *new_argv[i] = (char*)malloc((strlen(argv[i + 2]) + 1) * sizeof(char));
      if (*new_argv[i] == NULL)
      {
         printf("Lack of memory space (2)\n");
         return;
      }
      printf("make_arguments_table:arguments[%d] = %X\n",i,*new_argv[i]);
      strcpy(*new_argv[i],argv[i + 2]);
   }
}

Ce qui donne (programme test) :

test a b c
make_arguments_table:arguments    = 4107D0
make_arguments_table:arguments[0] = 4107C0
make_arguments_table:arguments[1] = 4107B0
main:arguments    = 4107D0
main:arguments[0] = 4107C0

Jean-François

Ca marche comme ça !

#include <stdio.h>
#include <string.h>
#include <time.h>
#include <malloc.h>
#include <windows.h> // Sleep()

void make_arguments_table(int argc,char** argv,char**& new_argv);
void display_help();

int main(int argc, char** argv)
{
   // No enough arguments
   if (argc < 3)
   {
      display_help();
      return (0);
   }
       
   // Wrong first argument (time hh:mm)
   if (strlen(argv[1]) != 5)
   {
      display_help();
      return (0);
   }
   if
   (
      (argv[1][0] < '0' || argv[1][0] > '2')
      ||
      (argv[1][1] < '0' || argv[1][1] > '9')
      ||
      (argv[1][2] != ':')
      ||
      (argv[1][3] < '0' || argv[1][3] > '5')
      ||
      (argv[1][4] < '0' || argv[1][4] > '9')
   )
   {
      display_help();
      return (0);
   }
       
   // Read the time
   int h=0;
   int m=0;
   time_t time_since_1970;
   struct tm* local_time;
       
   // Hour and minute from chars to integers
   h = (10 * (((int)argv[1][0]) - '0')) + (((int)argv[1][1]) - '0');
   m = (10 * (((int)argv[1][3]) - '0')) + (((int)argv[1][4]) - '0');
   if (h < 0 || h > 23 || m < 0 || m > 59)
   {
      display_help();
      return (0);
   }
       
   // Wait for the right time and execute the commands
   char** arguments;
   make_arguments_table(argc,argv,arguments);
   while (1)
   {
      time(&time_since_1970);
      local_time = localtime(&time_since_1970);
      printf("Scheduled: %02d:%02d\tLocal: %02d:%02d:%02d\n"
            ,h,m,local_time->tm_hour,local_time->tm_min,local_time->tm_sec);

      if (local_time->tm_hour == h && local_time->tm_min == m)
      {
         printf("time reached\n");
         int counter = 0;
         while (counter < argc - 2)
         {
            printf("execute %s\n",arguments[counter]);
            counter++;
         }
         return (0);
      }
      Sleep(5000); // wait 5 seconds before checking again
   }
   return (0);
}
   

void make_arguments_table(int argc,char** argv,char**& new_argv)
{
   int arguments_count = argc - 2;
   int counter = 2;
   int len = 0;
   int i = 0;
       
   new_argv = (char**)malloc(arguments_count * sizeof(char*));
   if (new_argv == NULL)
   {
      printf("Lack of memory space (1)\n");
      return;
   }
         
   for(i=0 ; i<arguments_count ; i++)
   {
      new_argv[i] = (char*)malloc((strlen(argv[i + 2]) + 1) * sizeof(char));
      if (new_argv[i] == NULL)
      {
         printf("Lack of memory space (2)\n");
         return;
      }
      strcpy(new_argv[i],argv[i + 2]);
   }
}

void display_help()
{
   printf("usage: stask <hh:mm> <command_1> <command_2> ... <command_n>\n");
   printf("Schedule a task.\n");
   printf("  <hh:mm>\t: Specify when to execute the command.\n");
   printf("\t\t  hh = hour   (00 <= hh <= 23).\n");
   printf("\t\t  mm = minute (00 <= mm <= 59).\n");
   printf("  <command>\t: Command to execute (use quotes if it includes spaces).\n");
   printf("\t\t  You can put several commands.\n\n");
   printf("Exemple:\n$ stask 07:00 \"wxvlc /home/username/playlist.m3u\" \"echo OK > stask.log\"\n\n");
}

Ce qui donne :

stask 17:10 format "copy *.* z:\\" dir
Scheduled: 17:10        Local: 17:09:21
Scheduled: 17:10        Local: 17:09:26
Scheduled: 17:10        Local: 17:09:31
Scheduled: 17:10        Local: 17:09:36
Scheduled: 17:10        Local: 17:09:41
Scheduled: 17:10        Local: 17:09:46
Scheduled: 17:10        Local: 17:09:51
Scheduled: 17:10        Local: 17:09:56
Scheduled: 17:10        Local: 17:10:01
time reached
execute format
execute copy *.* z:\
execute dir

Jean-François

Jean-François