文字数、単語数、行数の計数

Question

Visual Studio 2019でC++のコードを書いています．
入力テキスト中の文字数、単語数、行数を教える問題です．単語は、空白文字で区切られた非空白文字の列とします．
以下が入力テキストです．

NASA debuted a color picture from the Spirit rover
on Tuesday showing gray rocks peppering a Martian
lake bed awash in its natural hues of red pink
and orange Mission scientists said they were
bowled over by the spectacular quality of
the image taken with a dual camera system
called pancam that is mounted on a mast jutting
up from the rover I think my reaction has been
one of shock and awe said Jim Bell the team
member in charge of pancam Using special software
mission scientists can fly though the image zooming
in on rocks and other landscape features of interest
It is approximately the color that you would see
with your eyes if you were standing there Bell said
The resolution of course is pretty much what you
would see Pancam has 20 20 vision It is three to
four times better than any previous mission that has gone
to Mars in fact these pictures are the highest resolution
highest detailed pictures of Mars ever obtained They
are absolutely spectacular

以下のコードで実行すると，単語数：94と出力されます．
単語数：173と出力するには，どこを直したらいいでしょうか?
正しいコードを教えてください．お願いします．

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

// 定数定義
#define B 0 // 空白文字
#define N 1 // 通常文字

// プロトタイプ宣言
void updateWord(void);  // 単語数を更新する

// 大域変数
int word; // 単語数
int type; // 直前入力文字の種別
char ch;    // 入力文字

/********************************
メイン関数
********************************/
int main() {

	ifstream file;  // 入力ファイル  
	
	int letter; // 文字数
	int line; // 行数

	// 1. 文字数、単語数、行数を0とする。
	letter = 0;
	word = 0;
	line = 0;

	// 2. 「直前入力文字の種別」を「空白文字」とする。
	type = B;

	// 3. ファイルを開く。
	file.open("test.txt");

	// 4. ファイルから1文字読み込む。
	ch = file.get();

	// 5. ファイルの未尾に到達するまで以下の処理を繰り返す。 
	while (!file.eof()) {

		// 5-1. 文字数を1増やす。
		letter++;

		// 5-2. 単語数を更新する。
		updateWord();

		// 5-3. 読み込んだ文字が改行文字ならば、行数を1増やす。
		if (ch == '\n') {
			line++;
		}

		// 5-4. ファイルから1文字読み込む。
		ch = file.get();
	}

	// 6. 入力ファイルを閉じる。
	file.close();

	// 7. 文字数、単語数、行数を出力する。
	cout << "文字数：" << letter << endl;
	cout << "単語数：" << word << endl;
	cout << "行数：" << line << endl;
}

/********************************************
単語数を更新する
********************************************/
void updateWord(void) {

	

	// 1. 読み込んだ文字が空白文字ならば、以下の処理を行う。 
	if (isspace(ch)) {

		// 1-1. 「直前入力文字の種別」が「通常文字」ならば、単語数を1増やす。
		if (type == N) {
			word++;


			// 1-2. 「直前入力文字の種別」を「空白文字」とする。

			type = B;
		}

		// 2. そうでない場合、以下の処理を行う。 
		else {

			// 2-1. 「直前入力文字の種別」を「通常文字」とする。
			type = N;
		}
	}
}

Comments

[takasima20]

関数 updateWord の中を見直した方がいいと思う。

[退会済みユーザー]

関数 updateWordをどう直したらいいのかがわからないので教えてください。よろしくお願いします。

[episteme]

> // 2. そうでない場合、以下の処理を行う。

「そうでない」が「読みこんだ文字が空白ではない」なら、

if (isspace(ch)) {

// 1-1. 「直前入力文字の種別」が「通常文字」ならば、単語数を1増やす。
if (type == N) {
word++;

// 1-2. 「直前入力文字の種別」を「空白文字」とする。

type = B;
}
} // **** 1. の閉じカッコはココ。
// 2. そうでない場合、以下の処理を行う。
else {
// 2-1. 「直前入力文字の種別」を「通常文字」とする。
type = N;
}

[kaina]

丸投げでマルチポスト、その上回答もろくに読まないみたいですね。

https://dixq.net/forum/viewtopic.php?f=3&t=20806
https://noschool.asia/question/196382-1571002683
http://bbs.wankuma.com/index.cgi?mode=al2&namber=92631

Answer 1

とりあえず、変数 type の更新は関数 updateWord 内から main に移しましょう。
場所は、関数 updateWord を呼んだ次あたりに。
--- 追記 ---
たとえばmainのループをこんな感じにしたら

    while (!file.eof()) {
        letter++;
        updateWord();
        if (isspace(ch)) {
            type = B;
        else {
            type = N;
        }
        if (ch == '\n') {
            line++;
        }
        ch = file.get();
    }

関数内はこんな感じになりますね。

void updateWord(void) {
    if (isspace(ch)) {
        if (type == N) {
            word++;
        }
    }
}

Comments

[退会済みユーザー]

どのようにコードを書き直したらよいのでしょうか？教えてください．