Member 14172302 Ответов: 3

Как мне получить строку слов в файле для интеграции с циклом while

Я использую visual studio 2017, и мне нужно, чтобы моя программа нашла
1. Напечатать все слова в файле
2. Найти среднюю длину всех слов
3. Найти минимальную и максимальную длину слова
4. Сколько слов находится в файле. В текстовом файле 79 слов, и каждая строка-это слово. Текстовый файл называется Test.txt и я не могу из-за любви ко мне заставить строку правильно интегрироваться с петлей.

Что я уже пробовал:

Всё. Приведенный ниже код должен работать, но он не работает со строкой должным образом. Может, кто-нибудь просто поможет? Это должно произойти примерно через 5 часов, и я в полной растерянности, потому что не знаю, чего еще может не хватать 

#include <fstream>
#include <iostream>
#include <string>

using namespace std;

int main() {
	ifstream file("Test.txt");
	string words;

	float wordCount = 0;
	float totalWordsCharacters = 0;
	while (file >> words) {
		wordCount++;
		totalWordsCharacters += words.length();
	}

	cout << "The average length of letters in each word is " << (totalWordsCharacters / 79) << endl;

	system("pause");
	return 0;
}

Richard MacCutchan

Не используйте типы float для целочисленных значений, так как это просто тратит впустую вычислительные циклы.

3 Ответов

Рейтинг:
1

Patrice T

Цитата:
Приведенный ниже код должен работать, но он не работает со строкой должным образом.

Тогда что-то не так в вашем коде.
Использование отладчика поможет вам понять, насколько неверен код.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.

Рейтинг:
0

CPallini

Ваш код не соответствует некоторым требованиям и особенности это некрасиво 79 магическое число С другой стороны, он, по-видимому, работает и работает хорошо (я протестировал его с помощью простого текстового файла в 5 строк).
Если вы все еще считаете, что это не работает, то, пожалуйста, предоставьте входной файл и укажите ожидаемый результат.

KarstenK

- плохая магия" ;-)

Рейтинг:
0

KarstenK

Возьмите два решения выше для реального, но я добавляю, что у вас нет какого-то кода для минимального и максимального значения, подобного этому:

int min = 10000;//some foolish start
// in the loop
if( length < min ) {
  min = length;
}

Rick York

Обычно я использую INT_MAX для начального значения. Это, конечно, достаточно глупо. ;)