getlineがうまく動作しない

Question

現在c++で多重定義について勉強しています。挿入演算子と抽出演算子を多重定義しようとしています。挿入演算子はコードに書いてある通りignoreで邪魔な文字を無視してgetlineでメンバ変数に文字を代入しようとしています。
しかし、これを実行すると最初の変数areaCodeには代入されるのですがそのあとの変数には代入されません。環境はVSの最新版です。
デバックで見たのですがのexchangeとlineには'\0'とフという文字が入っていました。
これをうまく操作させるにはどのようにすればよいのでしょうか。

また、このコードはある参考書のサンプルコードになります。

#include <iostream>
#include < string >
using namespace std;


class PhoneNum {
	friend ostream& operator<<(ostream&, const PhoneNum&);
	friend istream& operator>>(istream &, PhoneNum&);
private:
	char areaCode[4];
	char exchage[4];
	char line[5];
};

ostream& operator<<(ostream& output, const PhoneNum& num)
{
	output << "(" << num.areaCode << ") "
		<< num.exchage << "-" << num.line;

	return output;
}

istream& operator>>(istream &input, PhoneNum &num)
{
	input.ignore();

	input.getline(num.areaCode, 4);
	input.ignore(2);
	input.getline(num.exchage, 4);
	input.ignore();
	input.getline(num.line, 5);

	return input;
}

int main() {
	PhoneNum phone;

	cout << "[phone]形式で電話番号を"
		<< "入力してください:\n";
	
	cin >> phone;
	cout << "入力された電話番号は: \n"
		<< phone << '\n';
	return 0;

}

Answer 1

getline はデリミタを指定しないと '\n' まで読み込もうとします。
読み込み先の領域が小さすぎると failbit が立ってエラーとなり、
それ以降の読み込みはできません。

お勧めはしませんが、次のようにすると読み込めます。

istream& operator>>(istream &input, PhoneNum &num)
{
    input.ignore();  // skip '('
    input.getline(num.areaCode, 4, ')');
    input.ignore();  // skip ' '
    input.getline(num.exchage, 4, '-');
    input.getline(num.line, 5);
    return input;
}

Comments

[daiking]

こんにちは。回答ありがとうございます。
上のコードで実際に動きました。ありがとうございます。

Answer 2

こんにちは。

istream::getlineはバッファサイズを超えた場合はfailとなるようです。バッファオーバーフローだろうと思います。
エラーをリセットしないと継続読み込みに失敗します。
input.getline(...);の後でinput.clear();することで読めるとは思います。

しかし、積極的にバッファオーバーフロー判定を使うのはあまり好ましくない使い方のように感じます。
一度std::stringへ1行読み出してから解析するとスマートに書けることが多いです。

Comments

[daiking]

こんにちは。オーバーフローではなくデリミタを指定していなかったことが原因だとわかりました。ありがとうございました。

Answer 3

僕ならこう↓書く

#include <iostream>
#include <sstream>
#include <string>
#include <algorithm>

class PhoneNum {
  friend std::ostream& operator<<(std::ostream&, const PhoneNum&);
  friend std::istream& operator>>(std::istream &, PhoneNum&);
private:
  char areaCode[4];
  char exchage[4];
  char line[5];
};

std::ostream& operator<<(std::ostream& output, const PhoneNum& num) {
    output << "(" << num.areaCode << ") "
        << num.exchage << "-" << num.line;
    return output;
}

std::istream& operator>>(std::istream &input, PhoneNum &num) {
  std::string inputline;
  // 一行読みこんで
  std::getline(input, inputline); 
  // '0'～'9' 以外を' 'に置き換えて
  std::replace_if(inputline.begin(), inputline.end(),
                  [](char ch) { return ch < '0' || ch > '9'; },
                  ' ');
  // areaCode, exchange, line を抽出する
  std::istringstream(inputline) 
    >> num.areaCode
    >> num.exchage
    >> num.line;

    return input;
}

int main() {
    PhoneNum phone;

    std::cout << "[phone]形式で電話番号を"
        << "入力してください:\n";

    std::cin >> phone;
    std::cout << "入力された電話番号は: \n"
        << phone << '\n';
    return 0;
}

Comments

[daiking]

回答ありがとうございます。replace_ifは使ったことがないので調べてみようと思います。