ポインタ配列を使用した文字数の検索は、

Question

ですので、私は完全な答えを探していません。私はちょうどこれから始めるべきか分からない。私は名前の完全なポインタ配列を宣言しているコードを持っています。目的は、名前を検索し、使用されている各文字を数えるためのコードを書くことです。

/*                    
* Search through each character in s,           
* which is array containing n strings,           
* and update the global count array            
* with the correct character counts.           
* Note: check the examples to see            
* if the counts should be case             
* sensitive or case insensitive.            
*/ 
void letterCount(char * s[], int n){ 
    //Implement this function              
    int c = 0,x;      // This is what I've done and I 
    char p = 'a', j = 'z';   // don't know where im messing up. 
    while (s[c] != '\0') {   // I know I can't compare pointer 
    if (s[c] >= p && s[c] <= j){ // and integer. 
     x = *s[c] - 'a'; 
      count[x]++; 
     } 
     c++; 
    } 
} 

/*                    
* Initialize each value in the global           
* count array to zero.               
*/ 
void initializeCount(){ 
    //Implement this function              
    int i; 
    for (i = 0; i < 26; i++){  // Also not sure if this is correct. 
    count[i] = 0; 
    } 
} 

出力では、count[26]という配列に文字を使用する必要があります。 お願いします。

Answer 1

s[c]が文字ではない、それはpointerだ、あなたはpointerと有効でない文字pを比較している、

if (s[c] >= p && s[c] <= j) // here you are comparing address & char, which you shouldn't 

は、文字列の各文字を比較するための1つの以上のループを回転させます。

私はあなたがこれを持って期待し

void letterCount(char * s[], int n){ 
     //Implement this function              
     int c = 0,x,i;     // This is what I've done and I 
     char p = 'a', j = 'z';   // don't know where im messing up. 
     // assuming n is no of string, rotate main loop from 0 to n 
     while (c<n) {  
       for(i=0;s[c][i]!='\0';i++)   // I know I can't compare pointer 
         if (s[c][i] >= p && s[c][i] <= j){ // and integer. 
           x = s[c][i] - 'a'; 
           count[x]++; 
         } 
       c++; 
     } 
} 

ようにコードを変更します。