1

このプログラムを実行するたびに、セグメンテーション違反がスローされます。プログラムは単純です: 与えられた文字列の母音の数を数えます。これはプログラムです:

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

int vowelC(char *in, int c){
  //Declarations
  int i; //Dummy index
  char a; //Temp variable

  //Validate input
  if(!in || c == 0) return 1;

  //Main execution

  //Convert to all lower case
  while(in[i] != '\0'){
       a=in[i];
       if('A' <= a && a <= 'Z')
              a-= ('A' - 'a');
       in[i] = a;
       i++;
  }

  //Count the number of vowels
  while(!in[i]){
             if(in[i] == 'a' || in[i] == 'e' || in[i] == 'i' || in[i] == 'o' || in[i] == 'u')
                     c++;
  }
  return 0;                     
 }

void printUsage(void){
   printf("\n\nThis program will count the number of vowels from an entered character string.\n\t[-c <count>] [-h help]\n\n");
}

int main(int argc, char **argv){
  //Declarations
  int i; //Dummy index
  int j; //Dummy index
  int count = 0;
  int err;
  int strsize = 0;

  //Is there at least some user input?
  if(argc == 1){
        printUsage();
        return 1;
}

//Determine string size if there is more than two user arguments
for(i=2; i<argc; i++){
         strsize += strlen(argv[i]);
         if (argc > i+1)
            strsize++;
}

//Declare an array of appropriate length to hold the entered strings
char *in = (char *)malloc(strsize);

//Validate input
for(i=1; i<=(argc-2); i++){

         //Determine if the user requested usage
         if(strcmp("-h", argv[i])==0){
                         printUsage();
                         return 1;
         }

         else if(strcmp("-c", argv[i])==0){
                         //There must be at least one argument after a call 
                         if((i+1) == argc){
                                  printUsage();
                                  return 1;
                         }
                         38
                         //Run another loop to retrieve string inputs
                         for(i=2; i<argc; i++){
                                  for(j=0; j != '\0'; j++){//Determine if a sting is really a string
                                           if(isalpha(argv[i][j])){
                                                                   printUsage();
                                                                   return 1;
                                           }
                          }
                          strcat(in, argv[i]);
                          if(argc > (i+1))
                          strcat(in, " ");
                          }

         }

         else{//Unknown input
                        printUsage();
                        return -1;
         }

//For bracket
}

err = vowelC(in, count);
assert(!err);
printf("\n\nThe number of vowels counted is %d.\n\nPlease press enter to exit...", count);
getchar();
return 0;
//Main Bracket    
}

GDB は、セグメンテーション違反が で発生したことを報告しwhile(in[i] != '\0')ます。しかし、その理由は私にはわかりません。どんな洞察も高く評価されます。

4

5 に答える 5

3
int i; //Dummy index

初期化されていません。iこれは、デフォルト値に初期化されないため、ジャンク値を含めることができることを意味します。初期化されていない auto 変数の値は未定義です。

于 2013-10-04T06:14:45.200 に答える
1

ゼロではiない

while(in[i] != '\0'){
于 2013-10-04T06:15:27.397 に答える
0

あなたが使用している

while(in[i] != '\0')

しかし、「i」を初期化していません

'i' を 0 で初期化します

于 2013-10-04T06:17:31.703 に答える