Get several values for the same option

Question

I have a program like this :

./server

Which has this usage :

Usage : 
-p  Port to use (4242 by default)
-x  Map width (20)
-y  Map height (20)
-n  Team name (name_team1 name_team2)
-c  Players per team
-t  Delay

I was able to parse all the options with this code :

int parse_cmd(int argc, char **argv, t_args *a)
{
    char ch;

    if (argv[1] && argv[1][0] != '-')
        usage();
    while ((ch = getopt(argc, argv, "p:x:y:n:c:t:")) != -1)
    {
        if (ch == 'p')
            a->port = atoi(optarg);
        else if (ch == 'x')
            a->x = atoi(optarg);
        else if (ch == 'y')
            a->y = atoi(optarg);
        else if (ch == 'n')
            a->teams = name_teams(optarg);
        else if (ch == 'c')
            a->size = atoi(optarg);
        else if (ch == 't')
            a->delay = atoi(optarg);
        else
            usage();
    }
    if (check_values(a) == FALSE)
        return (FALSE);
    return (TRUE);
}

But the thing is, for the -n option, I have to get the teams names like this :

./server -n team1 team2 team2

I just can't change the way it is.

Obviously I can do :

./server -n "team1 team2 team3"

And parse the teams, but it's for my firm and they don't want to put quotes around the team names, don't ask me why...

Any help in how can I get all the team names without using the quotes in shell?

score 4 · Answer 1 · answered Jul 02 '13 at 14:29

4

I think you have three differen posibilities:

use multiple '-n' parameters:
```
./server -n team1 -n team2 -n team3
```
use a character like ',' as divider inside optarg
```
./server -n team1,team2,team3
```
don't use getopt but parse argv for your own

answered Jul 02 '13 at 14:29

Ingo Leonhardt

9,435
2
24
33

hetepeperfan · Accepted Answer · 2013-07-02T18:27:19.117

You can also use optind. optint keeps track of the number of options encountered. optind points to the next index in argv[] that is encountered by getopt()

Therefore you can lookup in argv for whether there is a team. But for this to work you should omit the ":" in the optstring as in the next fragment "p:x:y:nc:t:" or decrement the value of optint before using it in the loop.

this is just a simple function to determine whether the loop must continue.

int 
is_team ( const char* team ) {
    if ( team == NULL)
        return 0;
    else if ( team[0] == '-' ) /*new argument*/
        return 0;
    else
        return 1;
}

And this is what you do when you encounter the 'n' option you can also use the colon in the optstring, but then the encounter option is also counted and then i = optind - 1 might also work

case 'n':
    { /*note this scope is significant*/
        int i;
        for ( i = optind ; is_team(argv[i]); ++i  ) {
            printf ( "team = %s\n", argv[i]);
            //argument = ++*optarg;
        }
    }
    break;

I hope this helps.

That did the work ! Thanks ! – Yanis Boucherit Jul 05 '13 at 13:37 — Yanis Boucherit, Jul 05 '13 at 13:37

score 0 · Answer 3 · answered Jul 02 '13 at 14:38

0

Use getopt in else if (ch == 'n') branch to parse all teams until next option occurs.

answered Jul 02 '13 at 14:38

kotlomoy

1,420
8
14

Get several values for the same option

3 Answers3

Linked