Seltsames Verhalten für überlastete I/O-Operator in Klasse

Question

Ich schrieb ein kleines Musikprogramm unter Windows zum Spaß. Ich fand den Teil des Codes falsch. Dies ist der minimierte Code (MCVE) zur Wiedergabe des Problems:

#include <iostream> 
using std::istream; 
using std::cin; 
using std::cout; 
using std::cerr; // Debug output goes to stderr 
using std::endl; 

class Key { 
    public: 
    typedef signed char Pitch, Octave; // -128 to 127 
    typedef signed long Absolute; // -2147M to 2147M 
    Pitch p; 
    Octave o; 

    public: 
    Key(const Pitch& _p, const Octave& _o) : 
     p(_p), o(_o) {} 
    Key(void) : Key(Pitch(0), Octave(0)) {}; 
    Absolute abs(void) const { 
     return 12L * Absolute(o) + Absolute(p); 
    } 
    friend istream& operator>> (istream&, Key&); 
}; 

istream& operator>> (istream& is, Key& k){ 
    return (is >> k.p >> k.o); 
} 

Und hier ist das Laufteil:

Key k; 
cin >> k; 
cout << k.abs() << endl; 

Da der Eingang 0 0 wird der Ausgang soll 0 sein, aber die tatsächliche Ausgabe ist lächerlich groß (624). Ich habe versucht, diese

cerr << k.p << k.o << endl; 

Und der Ausgang ist 00 (kein Raum zwischen siehe Code), wie erwartet.

Answer 1

is >> k.p hat ein besonderes Verhalten, weil k.p Zeichentyp hat. Der Eingang 0 erzeugt das Zeichen '0', nicht den Wert 0. In technischer Hinsicht hat die Überladung von std::istream::operator>> dieses unterschiedliche Verhalten, wenn das Argument der rechten Hand einen Zeichentyp hat.

So Eingabe 0 0 auf einem ASCII-System würde Ausgabe 48 * 12 + 48 generieren, die zufällig 624 ist.

Wahrscheinlich möchten Sie in eine int-Variable einlesen und dann in den Bereich k.p konvertieren (mit einer Fehlerbehandlung bei Eingabe außerhalb des Bereichs).