İç içe if-else ifadeleri bana sorun veriyor

Question

Yazdığım kodun, bir char işaretçi listesinde a's, c's, t's ve g'nin miktarını sayması bekleniyor. Daha sonra, girilen char a, c, t veya g değilse, geçersiz harfi geçersiz listeye geçersizBase eklenmesi gerekir. Bunun yerine agtcpoop veri girdiğimde Geçersiz Baz yazdırır: ama kaka baskısı gerekiyorsa chars orada değil. Lütfen yardım et! Burada kodudur:

void countBase(char *p) 
{ 
    int aCount = 0, cCount = 0, tCount = 0, gCount = 0; 
    char invalidBase[100]; 
    int i, j=0; 
    while(*p != '\0') 
{ 
    if(*p == 'A' || *p == 'a') 
    { 
    aCount++; 
    } 
    else if(*p == 'C' || *p == 'c') 
    { 
    cCount++; 
    } 
    else if(*p == 'T' || *p == 't') 
    { 
    tCount++; 
    } 
    else if(*p == 'G' || *p == 'g') 
    { 
    gCount++; 
    } 
    else 
    { 
    invalidBase[j] = *p; 
    } 
    j++; 
    p++; 
} 

for(i = 0; invalidBase[i] != '\0'; i++) 
{ 
    printf("Invalid Base: %c\n", invalidBase[i]); 
} 

printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount); 
} 

Answer 1

Yoy belirsiz otomatik depolama süresi olan başlatılmamış değişkenlerin değerlerini kullanarak tanımsız davranış çağrılır.

geliştirmek için:

• Artım veri invalidBase depolandığı takdirde karşı j.
• Yazdırılacak uzunluğu belirlemek için j kullanın.

Geliştirilmiş kod: Bu birçok if-else tabloların yerine switch deyimi kullanabilirsiniz

void countBase(char *p) 
{ 
    int aCount = 0, cCount = 0, tCount = 0, gCount = 0; 
    char invalidBase[100]; 
    int i, j=0; 
    while(*p != '\0') 
    { 
     if(*p == 'A' || *p == 'a') 
     { 
     aCount++; 
     } 
     else if(*p == 'C' || *p == 'c') 
     { 
     cCount++; 
     } 
     else if(*p == 'T' || *p == 't') 
     { 
     tCount++; 
     } 
     else if(*p == 'G' || *p == 'g') 
     { 
     gCount++; 
     } 
     else 
     { 
     if(j < (int)(sizeof(invalidBase)/sizeof(*invalidBase))) /* avoid buffer overrun */ 
     { 
      invalidBase[j] = *p; 
      j++; 
     } 
     } 
     p++; 
    } 

    for(i = 0; i < j; i++) 
    { 
     printf("Invalid Base: %c\n", invalidBase[i]); 
    } 

    printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount); 
} 

Not. Bu konuda daha fazla düşünme sonra elimizde

void countBase(char *p) 
{ 
    int aCount = 0, cCount = 0, tCount = 0, gCount = 0; 
    char invalidBase[100]; 
    int i, j=0; 
    while(*p != '\0') 
    { 
     switch(*p) 
     { 
     case 'A': 
     case 'a': 
     aCount++; 
     break; 
     case 'C': 
     case 'c': 
     cCount++; 
     break; 
     case 'T': 
     case 't': 
     tCount++; 
     break; 
     case 'G': 
     case 'g': 
     gCount++; 
     break; 
     default: 
     if(j < (int)(sizeof(invalidBase)/sizeof(*invalidBase))) /* avoid buffer overrun */ 
     { 
      invalidBase[j] = *p; 
      j++; 
     } 
     break; 
     } 
     p++; 
    } 

    for(i = 0; i < j; i++) 
    { 
     printf("Invalid Base: %c\n", invalidBase[i]); 
    } 

    printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount); 
} 

Answer 2

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

void countBase(char *p) { 
    int aCount = 0, cCount = 0, tCount = 0, gCount = 0; 
    char invalidBase[100]; 
    int i, j=0; 
    while(*p != '\0') { 
     if(tolower(*p) == 'a') { 
      aCount++; 
     } 
     else if(tolower(*p) == 'c') { 
      cCount++; 
     } 
     else if(tolower(*p) == 't') { 
      tCount++; 
     } 
     else if(tolower(*p) == 'g') { 
      gCount++; 
     } 
     else { 
      // This is the correct place to increment j. 
      invalidBase[j++] = *p; 
     } 

     // The following line is wrong. You can only increment j if an invalid 
     // base was found and inserted on the array. 
     // j++; 
     p++; 
    } 

    // Your code lacked the "end of string" delimitation. 
    invalidBase[j] = '\0'; 

    for(i = 0; invalidBase[i] != '\0'; i++) { 
     printf("Invalid Base: %c\n", invalidBase[i]); 
    } 

    printf(" A: %i\n C: %i\n T: %i\n G: %i\n", aCount, cCount, tCount, gCount); 
} 

int main(){ 
    printf("Testing '%s'\n", "agtcpoop"); 
    countBase("agtcpoop"); 
    puts("***********************************"); 
    printf("Testing '%s'\n", "a"); 
    countBase("a"); 
    puts("***********************************"); 
    printf("Testing '%s'\n", ""); 
    countBase(""); 
    puts("***********************************"); 
    printf("Testing '%s'\n", "p"); 
    countBase("p"); 
    return 0; 
}