2010-11-20 14 views
0

gcc 4.5.1 c89Lesen von Textzeilen aus Datei

Ich verwende den folgenden Code, um eine Textzeile aus einer Konfigurationsdatei einzulesen. Die Konfigurationsdatei ist im Moment klein und wird mit neuen Feldern erweitert. Und ich kann ziemlich genau entwerfen, wie die Konfigurationsdatei selbst aussehen wird. So habe ich es in der folgenden Weise:

config.cfg

# Configuration file to be loaded 

# protocol to use either ISDN, SIP, 
protocol: ISDN 

# run application in the following mode, makecall or waitcall 
mode: makecall 

ich den Doppelpunkt als eine Möglichkeit haben, für die Art der Konfiguration zu suchen, die benötigt werden. Ich frage mich nur, ob es einen besseren Weg gibt, dies zu tun?

Mein Code, den ich verwendet habe, ist wie folgt:

static int load_config(FILE *fp) 
{ 
    char line_read[LINE_SIZE] = {0}; 
    char *type = NULL; 
    char field[LINE_SIZE] = {0}; 
    char *carriage_return = NULL; 

    /* Read each line */ 
    while(fgets(line_read, LINE_SIZE, fp) != NULL) { 
     if(line_read != NULL) { 
      /* Ignore any hashs and blank lines */ 
      if((line_read[0] != '#') && (strlen(line_read) > 1)) { 
       /* I don't like the carriage return, so remove it. */ 
       carriage_return = strrchr(line_read, '\n'); 
       if(carriage_return != NULL) { 
        /* carriage return found so relace with a null */ 
        *carriage_return = '\0'; 
       } 

       /* Parse line_read to extract the field name i.e. protocol, mode, etc */ 
       parse_string(field, line_read); 
       if(field != NULL) { 
        type = strchr(line_read, ':'); 
        type+=2; /* Point to the first character after the space */ 

        if(strcmp("protocol", field) == 0) { 
         /* Check the protocol type */ 
         printf("protocol [ %s ]\n", type); 
        } 
        else if (strcmp("mode", field) == 0) { 
         /* Check the mode type */ 
         printf("mode [ %s ]\n", type); 
        } 
       } 
      } 
     } 
    } 

    return TRUE; 
} 

/* Extract the field name for the read in line from the configuration file. */ 
static void parse_string(char *dest, const char *string) 
{ 
    /* Copy string up to the colon to determine configuration type */ 
    while(*string != ':') { 
     *dest++ = *string++; 
    } 
    /* Insert nul terminator */ 
    *dest = '\0'; 
} 

Antwort

1

sollte einfacher sein:

static int load_config(FILE *fp) 
{ 
    int r=0; 
    char line[LINE_SIZE], field[LINE_SIZE], type[LINE_SIZE], dummy[LINE_SIZE]; 

    /* Read each line */ 
    while(fgets(line, LINE_SIZE, fp)) 
    { 
    if(strchr(line,'\n')) *strchr(line,'\n')=0; 
    if(3==sscanf(line,"%[^: ]%[: ]%s,field,dummy,type)) 
     ++r,printf("\n %s [ %s ]",field,type); 
    } 
    return r; 
} 
1

Wenn Sie entwerfen können, was die Konfigurationsdatei aussehen wird, wie ich für XML gehen würde, es mit Expat Parsen. Es ist schmerzlos.

1

Die Standardantwort auf einfaches Parsing-Problem ist die Verwendung von lex und yacc.

Da Sie jedoch die Form der Konfigurationsdatei frei festlegen können, sollten Sie eine der zahlreichen Bibliotheken verwenden, die verschiedene Konfigurationsdateiformate implementieren, und nur dieses verwenden.

http://www.google.com/search?q=configuration+file+parser

http://www.nongnu.org/confuse/ zum Beispiel, scheinen Ihre Bedürfnisse angemessen zu decken, aber einen Blick auf verschiedene andere Optionen nehmen, die auch einfacher sein könnte.

+0

Danke, ich werde sie mir ansehen. In der Zwischenzeit können Sie potenzielle Probleme mit meinem Quellcode sehen. – ant2009

+0

Die Frage war C89, Lex, Yacc, ... sind nicht Teile davon. – user411313