Member 14798231 Ответов: 4

Бесконечный цикл. Не могу понять почему.


The problem is that infinite looping is taking place and i can't figure out why this is  happening. It happens when I enter the choice as 1 and after entering Name, Role and Subject, it goes infinite. I've created a class "Professor" with data members as 3 vectors of type string and an "int count" along with three member functions as "Recruit", "DisplayInfo" and "Remove".  I'm basically trying to build a data of college professors.
I tried debugging the code. After first iteration it enters the loop and the switch and then directly to default case without me choosing the value for "choice"


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

<pre>
#include<iostream>
#include<string>
#include<vector>
#include<algorithm>

using namespace std;

class Professor
{
private:
    vector<string> Name;
    vector<string> Role;
    vector<string>Subject;
    static int count1;
public:
    void Recruit()
    {
        string name,role,subject;
        cout<<"Enter the Name: ";
        getline(cin, name);
        cin.ignore();
        Name.push_back(name);

        cout<<"Enter the Role: ";
        getline(cin, role);
        cin.ignore();
        Role.push_back(role);

        cout<<"Enter the Subject to be assigned: ";
        getline(cin, subject);
        cin.ignore();
        Subject.push_back(subject);
        count1++;
    }
    void DisplayInfo()
    {
        //cout<<"\nName   Role   Subject\n";
        for(int i=0; i<count1; i++)
        {
            cout<<"Name: "<<Name[i]<<"\n";
            cout<<"Role: "<<Role[i]<<"\n";
            cout<<"Subject: "<<Subject[i]<<"\n";   
        }
        cout<<"\n\n";
    }
    void Remove()
    {
        string name, role, subject;
        cout<<"Enter the name of the Professor, his/her Role and the subject assigned to him/her: ";
        cin>>name>>role>>subject;
        for(int i=0; i<count1; i++)
        {
            if(Name[i]==name && Role[i]==role && Subject[i]==subject)
            {
                vector<string>::iterator iter1 = find(Name.begin(), Name.end(), name);
                
                if(iter1 == Name.end())
                {
                    cout<<"";
                }
                else{
                    Name.erase(iter1);
                }
                vector<string>::iterator iter2 = find(Role.begin(), Role.end(), role);
                if(iter2 == Role.end())
                {
                    cout<<"";
                }
                else{
                    Role.erase(iter2);
                }
                vector<string>::iterator iter3 = find(Subject.begin(), Subject.end(), subject);
                if(iter3 == Role.end())
                {
                    cout<<"";
                }
                else{
                    Role.erase(iter3);
                }
            }
        }
    }
};


int main()
{
    Professor p;
    int choice;
    do{
        cout<<"You have the following Choices:\n";
        cout<<"1) Recruit a Professor\n";
        cout<<"2) Display the Information about Professors\n";
        cout<<"3) Remove a Professor\n";
        cout<<"4) Quit\n";
        cout<<"\n\nEnter the choice:";

        cin>>choice;

        switch(choice)
        {
            case 1:
            {
                p.Recruit();
                break; 
            }
            case 2:
            {
                p.DisplayInfo();
                break;
            }
            case 3:
            {
                p.Remove();
                break;
            }
            case 4:
            {
                break;
            }
            default:
            {
                cout<<"Sorry Wrong Choice!!\n";
                break;
            }
        }
    }while(choice!=4);
    return 0;
}

4 Ответов

Рейтинг:
2

Patrice T

Цитата:
Проблема в том, что происходит бесконечное зацикливание, и я не могу понять, почему это происходит.

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

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

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

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

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

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

[Обновление]
Цитата:
Я попытался отладить код. После первой итерации он входит в цикл и переключатель, а затем непосредственно в случай по умолчанию без моего выбора значения для "выбора"

Обратите внимание на ценность choice, как он развивается по мере того, как код зацикливается.


Member 14798231

Я попытался отладить код. После первой итерации он входит в цикл и переключатель, а затем непосредственно в случай по умолчанию без моего выбора значения для "выбора"

Patrice T

Воспользуйся Улучшить вопрос чтобы обновить ваш вопрос.
Чтобы каждый мог обратить внимание на эту информацию.

Patrice T

Обратите внимание на ценность выбора.
вы нажали Enter ?

k5054

Что делает профессор::name[0] после добавления его в вектор?
Почему?
Также проверьте значение роли и субъекта.

Рейтинг:
2

Richard MacCutchan

1. Вы заявили: count1 как статичной она быть не должна. Это должна быть обычная переменная экземпляра.
2. Вы звоните cin.ignore() после каждого звонка в getline() в Recruit метод; они не нужны.
3. И более важным. Вы делать нужно позвонить cin.ignore после строки

cin>>choice;

в основной петле. Это очистит входной поток для первого getline приглашать Recruit.


Рейтинг:
0

OriginalGriff

Компиляция не означает, что ваш код верен! :смеяться:
Подумайте о процессе разработки как о написании электронного письма: успешная компиляция означает, что вы написали электронное письмо на правильном языке - например, на английском, а не на немецком, - а не то, что письмо содержало сообщение, которое вы хотели отправить.

Итак, теперь вы входите во вторую стадию разработки (на самом деле это четвертая или пятая, но вы перейдете к более ранним стадиям позже): тестирование и отладка.

Начните с рассмотрения того, что он делает, и как это отличается от того, что вы хотели. Это важно, потому что это дает вам информацию о том, почему он это делает. Например, если программа предназначена для того, чтобы позволить пользователю ввести число, а он удваивает его и печатает ответ, то если ввод / вывод был таким:

Input   Expected output    Actual output
  1            2                 1
  2            4                 4
  3            6                 9
  4            8                16
Тогда совершенно очевидно, что проблема заключается в бите, который удваивает его - он не прибавляет себя к себе или умножает его на 2, он умножает его на себя и возвращает квадрат входного сигнала.
Таким образом, вы можете посмотреть на код, и очевидно, что он находится где-то здесь:
int Double(int value)
   {
   return value * value;
   }

Как только у вас появится идея, что может пойти не так, начните использовать отладчик, чтобы выяснить, почему. Поместите точку останова в первую строку метода и запустите приложение. Когда он достигнет точки останова, отладчик остановится и передаст управление вам. Теперь вы можете запускать свой код построчно (так называемый "одноступенчатый") и просматривать (или даже изменять) содержимое переменных по мере необходимости (черт возьми, вы даже можете изменить код и повторить попытку, если вам это нужно).
Подумайте о том, что должна делать каждая строка кода перед ее выполнением, и сравните это с тем, что она действительно делала, когда вы использовали кнопку "Step over" для выполнения каждой строки по очереди. Он сделал то, что вы ожидали? Если да, то переходите к следующей строке.
Если нет, то почему? Чем это отличается?
Надеюсь, это поможет вам определить, в какой части этого кода есть проблема и в чем она заключается.
Это навык, и его стоит развивать, поскольку он помогает вам как в реальном мире, так и в развитии. И, как и все навыки, он только улучшается при использовании!


Рейтинг:
0

Rick York

Ваша организация кажется мне отсталой. На мой взгляд, имя, роль и предмет должны быть строками, А класс должен представлять одного профессора. Тогда у вас должна быть коллекция тех, которые могут быть вектором или списком. Эти коллекции имеют метод size (), поэтому вам не нужен член count. Ваша основная функция будет содержать экземпляр коллекции. Вы также можете создать класс, содержащий коллекцию и методы для отображения, удаления и добавления записей в коллекцию, или это могут быть простые функции, работающие с коллекцией профессоров. Вот одна из возможностей :

class Professor   // data for one professor
{
public:
    void Display()
    {
        cout << "Name   : " << Name << endln;
        cout << "Role   : " << Role << endln;
        cout << "Subject: " << Subject << endln;   
    }

private:
    std::string Name;
    std::string Role;
    std::string Subject;
};

class ProfCollection
{
public:
    Insert();
    Remove();
    Display();

private:
    std::vector< Professor >  Profs;
};
Следующим шагом будет реализация методов Insert, Remove и Display класса collection. Они будут похожи на то, что у вас есть, но отличаются, потому что они работают над коллекцией предметов профессора.


Member 14798231

таким образом, ProfCollection должен был бы унаследовать класс профессора, верно?

Rick York

Нет, вовсе нет. ProfCollection владеет коллекцией и управляет ею. Существует больше методов, необходимых для доступа к членам профессора, которые не были включены в решение.

Member 14798231

Как я должен провести операцию удаления? Я попытался сделать так, чтобы пользователь ввел имя, роль и предмет профессора, который будет удален, а затем цикл for для итерации через вектор "Profs" и оператор using an if. Но как я должен удалить это из вектора? функция erase() выдает ошибку