C Basic Head Command

Question

Estoy intentando recrear los comandos head, y tail de linux para mi clase de programación. Acabamos de empezar a usar C, por lo que soy nuevo en la idea de asignar memoria y punteros. Me pregunto por qué esto no funciona.

#include <stdio.h> 
#include <stdlib.h> 

int main(int argc,char **argv){ 

    /* Checks if correct amount of arguements */ 

    if(argc != 2 || argc != 4){ 
     printf("Usage: %s head <file> \n Or: head <file> -n <number of characters>", argv[0]); 
     exit(-1); 
    } 

    if(strcmp(argv[1], "-n" != 0)){ 
     char fileName[strlen(argv[1])] = argv[1]; 
    } 
} 

//Compile error on char fileName[strlen(argv[1])] = argv[1]; 

Cualquier información adicional también sería útil.

Answer 1

Lo primero es lo primero, su uso no coincide con la comprobación de sus argumentos. De acuerdo con el uso, se debe utilizar uno de:

head <filename> 
head <filename> -n <count> 

En otras palabras, es argv[1] siempre el nombre del archivo, argv[2] es la que se debe establecer a -n si hay más de dos argumentos.

En segundo lugar, a menos que desee utilizar Vlas (arrays de longitud variable), probablemente debería simplemente establecer un puntero al nombre de fichero con algo como:

char *fileName = argv[1]; 

No es necesario cambiarlo a todo (probablemente lo pasará al fopen, presumiblemente), por lo que es una pérdida tratar de hacer otra copia.

Además, su declaración if es incorrecta como or, debería ser and. Está garantizado que argc no será 2 o no será 4, ya que no puede ser ambos al mismo tiempo.

Me gustaría empezar con algo como:

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

static int usage (void) { 
    printf ("Usage: head <file>\n"); 
    printf (" or: head <file> -n <number of characters>\n"); 
    return -1; 
} 

int main (int argc,char *argv[]) { 
    char *fileName; 
    int lineCount; 

    // Checks if correct arguments 

    if ((argc != 2) && (argc != 4)) return usage(); 

    if ((argc == 4) && (strcmp(argv[2], "-n" != 0)) return usage(); 

    // Get file spec and line count 

    fileName = argv[1]; 

    lineCount = (argc == 2) ? 10 : atoi (argv[3]); // or strtol for purists 
    if (linecount < 0) lineCount = 0; 

    // Now go ahead and implement the logic for head. 

} 

Answer 2

Creo que es mejor escribir:

char fileName[strlen(argv[1])+1]; 
strcpy(fileName, argv[1]); 

o (si no whant para hacer una copia de cadena):

char* fileName = argv[1];