aşırı strchr kullanma >>

Question

Bir tek (enum Symbol {e,a,b,c,d}; ile oluşturulan) Simge okumak için >> operatörünü aşırı çalışıyorum: bunun yerine ne olduğunu

istream & operator >> (istream & is, Symbol & sym) { 
    Symbol Arr[]={e,a,b,c,d}; 
    char ch; 
    is>>ch; 
    if (strchr("eabcd",ch)) 
    sym=Arr[ch-'e']; 
     else { 
     is.unget(); 
     is.setstate(ios::failbit); 
     } 
    return is; 
} 

Ama bu, bazı çöp (sayılar) okur < < aşırı yükümde yazdırmaya çalışırken bir segmentasyon hatasına yol açan, ne yapıyorum? Düzenleme: Oh, ve tabii ki iostream ve cstring dahil olmak üzere using namespace std; başlangıçta ekledim. ch olan

Answer 1

Burada bir kaç yanlış var. İlk önce, düzeltmeyi düzeltelim. Sadece her zaman parantez kullanın. Ne ile dizilmiş olduğunu görmek çok zor:

istream & operator >> (istream & is, Symbol & sym) { 
    Symbol Arr[]={e,a,b,c,d}; 
    char ch; 
    is>>ch; 
    if (strchr("eabcd",ch)) { 
     sym=Arr[ch-'e']; 
    } 
    else { 
     is.unget(); 
     is.setstate(ios::failbit); 
    } 
    return is; 
} 

Tamam harika. Şimdi, kullanıcı 'a' gibi bir şey girerse ne olur. strchr başarılı ve sym = Arr[ch - 'e'] yapıyorsunuz. Ancak bu durumda, -4. Bu bir yerlerde tamamen rastgele bir hafıza parçası, yani çöp alıyorsun.

const char* options = "eabcd"; 
if (const char* p = strchr(options, ch)) { 
    sym = Arr[p - options]; 
} 

Ama bu tür korkunç: aslında strchr kullanmak için böyle bir şey yapmak gerekiyordu.

switch (ch) { 
    case 'e': sym = e; break; 
    case 'a': sym = a; break; 
    ... 
    default: 
     is.unget(); 
     is.setstate(ios::failbit); 
} 

Ayrıca, is >> ch başarısız olabilir ve bu kontrol etmediğinizde: Sadece anahtarını kullanarak öneririm. Yapmanız gerekenler:

istream& operator>>(istream& is, Symbol& sym) { 
    char ch; 
    if (is >> ch) { 
     switch(ch) { ... } 
    } 
    return is; 
} 

Answer 2

ise 'a', ch - 'e' (97-101) aralık dışında dizi Arr erişmek yol açacaktır negatif bir sayı (-4), olacaktır. Bu tanımlanmamış davranışa yol açar.

Eğer sembolleri var yolu, bir switch deyimi kullanın gerekir:

Symbol Arr[]={a,b,c,d,e}; 

: Eğer Arr kullanmak istiyorsanız

switch (ch) 
{ 
    case 'a': 
     sym = a; 
     break; 

    case 'b': 
     sym = b; 
     break; 

    case 'c': 
     sym = c; 
     break; 

    case 'd': 
     sym = d; 
     break; 

    case 'e': 
     sym = e; 
     break; 

    default: 
    // Nothing to do 
    break; 
} 

, siz Arr tanımlamak gerekecektir Ardından diziye aşağıdaki gibi erişebilir ve