Чтение двойных слов и слов и выполнение действий на основе прочитанного

Question

Чтение двойных слов и слов и выполнение действий на основе прочитанного

0

Я пишу программу, которая принимает входные данные и форматирует их в балл как "Оценка имени". Вот что нужно сделать "Здесь формат Moonglow.

Текстовый файл состоит из слов. Если слово - это число, то это вопрос студента по вопросу, поэтому вы добавляете его на счет экзамена студента.
Если слово не является числом, но является словом "NAME", то следующим словом является имя студента (Moonglow использует только имена - фамилии являются корпоративными и безличными).
Если слово "AVERAGE", вы начинаете считывать цифры, пока не прочитаете слово, которое не является числом (или является концом файла). Вы усредняете все эти цифры и добавляете их в партитуру. Поскольку Moonglow немного рассеян, иногда число не следует "СРЕДНЕЕ". В этом случае вы игнорируете "СРЕДНЕЕ". "

Моя проблема заключается в том, что я каким-то образом вступил в бесконечный цикл и потратил часы, пытаясь это исправить. Это ОЧЕНЬ простая программа, и я не могу заставить ее работать!

#include <iostream>
#include <string>
#include <sstream>
#include <vector>
#include <stdlib.h>
using namespace std;

int main()
{
    string s, name = "test";
    string buffer;
    double qScore, eScore, totalExam = 0, grade = 0, numExam = 0, finalGrade, avg = 0;
    while (!cin.eof())
    {
        if (cin >> qScore)
        {
            if (cin.fail())
            {
                cin.clear();
            }

            else
            {
                grade += qScore;
            }
        }
        else if (cin >> s)
        {
            if (s == "NAME")
            {
                cin >> s;
                s = name;
            }
            else if (s == "AVERAGE")
            {
                while (cin >> eScore)
                {
                    numExam++;
                    totalExam += eScore;

                }
                cin.clear();
            }
        }
    }

    if (numExam == 0)
    {
        avg = 0;
    }
    else
    {
        avg = (totalExam / numExam);
    }
    finalGrade = avg + grade;
    cout << name << " " << finalGrade << endl;
    return 0;

}
// end of main

user3250013 29 янв. 2014, в 19:02

Источник

1

Для начала while (!eof()) неверно.
0x499602D2 29 янв. 2014, в 18:35
0

Должен ли я заменить его на while (cin >> qScore || cin >> s)?
user3250013 29 янв. 2014, в 18:39
0

Замените его на while (cin >> qScore && cin >> s) . Вы также можете сделать гораздо больше, чтобы исправить эту программу. Это только один.
0x499602D2 29 янв. 2014, в 18:43
0

Я вижу еще одну серьезную ошибку в моем коде - ничего не читается в «имя». Независимо от того, что я выполняю, имя не указано в выводе.
user3250013 29 янв. 2014, в 18:47
0

Тебе придется подождать, пока я работаю над твоим кодом. А пока я хочу знать, что вы предоставляете в качестве входных данных.
0x499602D2 29 янв. 2014, в 18:50
0

Я ценю помощь. Вот несколько примеров: 1: ИМЯ Фред 2: Starrlite СРЕДНИЙ Starrlite !!! ИМЯ Starrlite СРЕДНЕЕ 55 Starrlite !!! Starrlite СРЕДНИЙ Starrlite 5 6 7
user3250013 29 янв. 2014, в 18:58
0

Предполагается игнорировать любые слова, которые не являются СРЕДНЕМ или ИМЯ, за которыми следует одно имя. Любое число не после СРЕДНЕГО должно быть добавлено. Это странная лаборатория, цель состоит в том, чтобы преобразовать какой-то "византийский" формат в просто "<имя> <класс>"
user3250013 29 янв. 2014, в 18:59
1

Является ли 1: и 2: частью ввода?
0x499602D2 29 янв. 2014, в 19:06
0

Частично проблема в том, что вы всегда потребляете входные данные в шаблоне double, string. Ваш пример показывает, что это может быть больше похоже на строку, строку, строку ..., двойную, строку, строку ... Я бы вместо этого прочитал все как string (разделенную пробелом), а затем преобразовал бы в double или int .
Matt 29 янв. 2014, в 19:08
0

Почему вы делаете cin >> s затем s = name потом?
0x499602D2 29 янв. 2014, в 19:12
0

Кроме того, проблема в значительной степени конечного автомата, закодируйте его как таковой.
Matt 29 янв. 2014, в 19:14
0

1 и 2 являются отдельными входами. Как бы я конвертировать только числа в двойные?
user3250013 29 янв. 2014, в 19:23
0

@user3250013 user3250013 Смотрите мой ответ. Вы можете конвертировать только числа, посмотрев на символы, чтобы увидеть, если это число. Если я не забыл, и есть способ попытаться преобразовать и потерпеть неудачу, если это не число.
Matt 29 янв. 2014, в 19:32

Показать ещё 11 комментариев

Теги:

c++

string

cout

cin

stringstream

1 ответ

Ещё вопросы

Должен ли я заменить его на while (cin >> qScore || cin >> s)?
Замените его на while (cin >> qScore && cin >> s) . Вы также можете сделать гораздо больше, чтобы исправить эту программу. Это только один.
Я вижу еще одну серьезную ошибку в моем коде - ничего не читается в «имя». Независимо от того, что я выполняю, имя не указано в выводе.
Тебе придется подождать, пока я работаю над твоим кодом. А пока я хочу знать, что вы предоставляете в качестве входных данных.
Я ценю помощь. Вот несколько примеров: 1: ИМЯ Фред 2: Starrlite СРЕДНИЙ Starrlite !!! ИМЯ Starrlite СРЕДНЕЕ 55 Starrlite !!! Starrlite СРЕДНИЙ Starrlite 5 6 7
Предполагается игнорировать любые слова, которые не являются СРЕДНЕМ или ИМЯ, за которыми следует одно имя. Любое число не после СРЕДНЕГО должно быть добавлено. Это странная лаборатория, цель состоит в том, чтобы преобразовать какой-то "византийский" формат в просто "<имя> <класс>"
Частично проблема в том, что вы всегда потребляете входные данные в шаблоне double, string. Ваш пример показывает, что это может быть больше похоже на строку, строку, строку ..., двойную, строку, строку ... Я бы вместо этого прочитал все как string (разделенную пробелом), а затем преобразовал бы в double или int .
Почему вы делаете cin >> s затем s = name потом?
Кроме того, проблема в значительной степени конечного автомата, закодируйте его как таковой.
1 и 2 являются отдельными входами. Как бы я конвертировать только числа в двойные?
@user3250013 user3250013 Смотрите мой ответ. Вы можете конвертировать только числа, посмотрев на символы, чтобы увидеть, если это число. Если я не забыл, и есть способ попытаться преобразовать и потерпеть неудачу, если это не число.

Matt · Accepted Answer · 2014-01-29T16-52-00.000Z

Я не буду писать для вас код (так как это явно проблема домашних заданий), но я буду предлагать некоторые предложения относительно того, как я буду делать это совершенно по-другому. Надеюсь, этот псевдокод поможет.

EDIT: Я пропустил логику для AVERAGE и как должен работать счет, я делаю некоторые исправления. (Обратите внимание, что с этими изменениями может быть лучше добавить новые состояния. Я бы рассмотрел Normal, Name, BeginAverage, Average, EndAverage. Но это зависит от вас.)

1) Определите эти состояния (возможно, используйте перечисление):

Обычный (вероятно, лучшее название для этого)
имя
В среднем

2) Вы хотите прочитать один "токен" и переход со следующими правилами. Обратите внимание, что токеном в этом случае будет строка, разделенная пробелами (например, "NAME", "AVERAGE", "Bob", "11", "23").

Если состояние Normal (это также начальное состояние)
- Если токеном является "NAME", перейдите к названию состояния
- Если токен "СРЕДНЕЕ", перейдите в состояние "Средняя"
- Если токен начинается с одного из этих символов "0", "1", "2", "9", преобразуйте его в двойное и добавьте его в свой счет
Если состояние Имя
- Скопируйте токен в свою переменную "имя", переход в нормальное состояние
Если состояние среднее
- Если вы получаете номер, добавьте его к временному счету и добавьте счетчик
- Если вы получите строку, которая не является "NAME", "AVERAGE" или числом, вернитесь в "Нормальное состояние"
- Если вы получите "НАЗВАНИЕ", перейдите в состояние "Имя"
- Прежде чем вводить это состояние, вам необходимо сбросить временный счетчик/счетчик, чтобы вычислить средний
- Прежде чем покинуть это состояние, вычислите среднее значение и добавьте его в общий балл

3) Для обработки ввода сначала прочитайте все в строковой переменной. Затем вы можете использовать stringstream для преобразования в double при необходимости. Просто прочитайте все строки (жетоны), как я описал, и обработайте их, как указано выше, до тех пор, пока вы не достигнете EOL.

"Код":

while (cin >> s)
{
    if (state == state_normal)
    {
        if (s == "NAME") { state = state_name; }
        else if (s == "AVERAGE")
        {
            state = state_average;
            // You are starting a new average, initialize these variables to 0
            scoresToAverage = 0;
            numberOfScores = 0;
        }
        else if (isNumber(s))
        {
            number = convertToNumber(s);
            score += number;
        }
    }
    if (state == state_name)
    {
        name = s;
        state = state_normal;
    }
    if (state == state_average)
    {
        bool calculateAverage = false;
        if (s == "NAME")
        {
             state = state_name;
             calculateAverage = true;
        }
        else if (s == "AVERAGE")
        {
             state = state_average;
             calculateAverage = true;
        }
        else if (isNumber(s))
        {
            number = convertToNumber(s);
            scoresToAverage += number;
            numberOfScores++;
        }
        else
        {
            state = state_normal;
            calculateAverage = true;
        }

        // If you are DONE averaging, calculate the average and add it to the total
        if (calculateAverage)
        {
            if (numberOfScores > 0)
            {
                score += (scoresToAverage / numberOfScores);
            }
            scoresToAverage = 0;
            numberOfScores = 0;
        }
    }
}

// done reading input, just print!

isNumber берет строку и может просто проверить, что первый символ - это число ('0', '1',...). Это может быть более привлекательным и смотреть на каждый символ для чисел/десятичных знаков.

convertToNumber берет строку и преобразует в double. Могут использовать stringstreams для преобразования или просто старые atoi.

Это довольно полезно до сих пор. Всего один вопрос. Что вы имеете в виду, используя enum для состояний?
Предполагая, что вы не знаете, что такое enum , cprogramming.com/tutorial/enum.html Таким образом, вы можете создать новый тип enum котором каждое возможное значение является различным состоянием. В качестве альтернативы вы можете использовать int и определять const , например, 0 - первое состояние, 1 - второе, 2 - третье. Это (в некоторой степени) то, что enum делает для вас.
Я посмотрел перечисления и успешно их использовал ... Я верю. Тем не менее, теперь мой вывод всегда показывает имя, но оценка ВСЕГДА 0, независимо от того, что ввод. Может быть, я неправильно конвертирую? atof (s.c_str ()) правильно для преобразования в строку правильно?
Я не совсем уверен, не увидев ваш новый код. Возможно, пришло время стереть старый отладчик и взглянуть на значение s ? Вы на самом деле сохраняете конвертированное значение в итог?
У меня есть, чтобы проверить на первый символ s между 0 и 9, если (s [0] == '0' || .......... и так далее ...)
так ли вы использовать отладчик , чтобы убедиться, что , if это когда - нибудь правда? Вы проверяли значение s в этих случаях? Значение eScore ? Вы добавляете eScore к общему количеству? Так много переменных, я не могу помочь. Вы можете добавить свой новый код к своему вопросу или опубликовать ссылку на pastebin.com . Кроме того, вы можете проверить, является ли символ числом просто (s[0] >= '0' && s[0] <= '9') , но это всего лишь оптимизация.
прости, но где мне найти декодер? Я здесь новенький. И я попытаюсь (s [0]> = '0' && s [0] <= '9') попробовать только для оптимизации (мне это нравится гораздо больше)
Вы перепутали случаи «isNumber» для «нормального» и «среднего» состояний. Для «нормального» вам нужно добавить оценку, для «среднего» вы можете просто ничего не делать / игнорировать число (строки 23 против 39-41). Примечание: вы добавляете grade , которая всегда равна 0 в конце, вам это не нужно. А изучение того, как использовать отладчик, - это совершенно другое обсуждение, не совсем подходящее для комментариев к моему ответу. Возможно, посмотрите на это самостоятельно? bing.com/...
Проблема в том, что после AVERAGE встречаются числа, которые появляются ПОСЛЕ добавления и усреднения. Если числа встречаются до чтения AVERAGE, они добавляются в оценку. Затем оценка добавляется в avg для finalGrade.
Я неправильно прочитал «СРЕДНЮЮ» часть проблемы (она вроде плохо сформулирована), смотрите исправления.
Так что теперь у меня есть этот pastebin.com/wbck11j0 Имя не читается. Насколько я знаю, я сделал то, что вы сказали, я должен. Хммм Также я согласен, что описание проблемы было трудно для меня, чтобы следовать.
if «s в while петли должны быть else if , switch(...) {case...} else if , или еще лучше, используйте switch(...) {case...} . Вы должны действительно научиться использовать отладчик, потому что это было бы довольно ясно. Кроме того, вы должны научиться мысленно шагать по коду, чтобы точно понять, что он должен делать. Также в состоянии «СРЕДНИЙ» вы пропускаете случай, когда вашим токеном снова является слово «СРЕДНИЙ», или это любое другое нечисловое слово.
вот что у меня сейчас. Вот мой цикл ввода сейчас. pastebin.com/H5QmMqWR В настоящее время ВСЕ выходные данные имеют значение «0» независимо от того, что я делаю в качестве ввода. Я понимаю, что делает каждая часть кода, но я просто не могу заставить ее работать должным образом. С тех пор я начал читать, как использовать отладчики, но боюсь, что пока еще не достаточно опытен с ними.
Я выяснил, что я делал неправильно все время. Я никогда не инициализировал свой enum к состоянию Normal, поэтому ничего не выполнялось. Я действительно ценю всю помощь, которую вы мне оказали. И я изучил новую концепцию C ++ (перечисления), прежде чем мы сделали в классе, поэтому я очень благодарен!