Не выполняются все функции

@diletant75 · Регистрация: 22.05.2016

Author24 — интернет-сервис помощи студентам

Здравствуйте. Почему не выполняются функции класса(вызывается функция 1 класса выполняется ввод , Enter, вызывается функция 2 а затем снова 1, выполняется ввод)? Как очистить память, чтоб при выборе 4 переключателя произвольные данные не появлялись?

Кликните здесь для просмотра всего текста

C++

#include <iostream>
 
using namespace std;
////////////////////////////////////
class publication
{
    private:
    char name;
    float cost;
    public:
    void putdata()
    {
       cout <<"\nVvedite nazvanie knigi\n"; cin>>name;
       cout <<"\nVvedite cenu knigi\n"; cin>>cost;
    }
    void getdata()
    {
       cout <<"\nNazvanie knigi\n"<<name;
       cout <<"\nCena knigi\n"<<cost;
    }
 
};
/////////////////////////////////////
class book
{
    private:
    int list;
    publication pub;
    public:
    void putdata()
    {
        pub.putdata();
        cout <<"\nVvedite colichestvo stranits "; cin>>list;
    }
    void getdata()
    {
        pub.getdata();
        cout <<"\nColichestvo stranits "<<list;
    }
};
/////////////////////////////////////
class type
{
    private:
    float vremiya_zapisi;
    publication pub;
    public:
    void putdata()
    {
        pub.putdata();
        cout <<"\nVvedite vremiya zapisi\n"; cin >> vremiya_zapisi;
    }
    void getdata()
    {
        pub.getdata();
       cout <<"\nVremiya zapisi\n"<<vremiya_zapisi;
    }
};
 
int main()
{
    int x,n,i;
    publication p1;
    book b1;
    type t1;
    cout <<"Hellow! Esli hotite vvesti\n publication najmite 1\n book najmite 2\n type najmite 3\nvivesti dannie na ekran -4 \n press "x" for exit"<<endl;
     cin >>n;
    for(i=0; i<5; i++)
     switch(n)
     {
          case 1:
          p1.putdata();
          break;
          case 2:
          b1.putdata();
          break;
          case 3:
          t1.putdata();
          break;
          case 4:
          p1.getdata(); b1.getdata(); t1.getdata();cout<<"\n\n%"<<i<<endl;
          break;
     }
    return 0;
}

@gledor · 17.09.2016, 20:12

Вы должны поместить ввод ответа от пользователя внутрь цикла for. Только тогда у Вас будут выполняться все ваши методы. Чтобы не было мусора,советую присваивать нулевое значение полям класса при создании объектов. Так же рекомендую использовать стандартный механизм наследования, а не делать включение базового класса в наследуемые. Переменная x не нужна. Имя публикации должно храниться в массиве. И советую перечитать седьмую, восьмую, девятую главы в вашей книге, из которой берете эти задания.

Я исправил это как мог, потому что все надо переделывать c нуля. А цикл for лучше заменить на бесконечный цикл while

C++

#include <iostream>
#include <cstring>
using namespace std;
////////////////////////////////////
class publication
{
    private:
    static const int MAX = 100;
    char name [ MAX ]; //Нам нужен массив для названия, а не единичный символ!
    float cost;
    public:
    publication( ): cost( 0.0 )// Это чтобы не было мусора
    {
        strcpy( name, "N/A" );
    }
 
    void putdata()
    {
       cout <<"\nVvedite nazvanie knigi\n"; cin>>name;
       cout <<"\nVvedite cenu knigi\n"; cin>>cost;
    }
    void getdata()
    {
       cout <<"\nNazvanie knigi\n"<<name;
       cout <<"\nCena knigi\n"<<cost;
    }
 
};
/////////////////////////////////////
 
class book
{
    private:
    int list;
    publication pub;
    public:
    book( ): list( 0 ) {} //Чтобы не было мусора
    void putdata()
    {
        pub.putdata();
        cout <<"\nVvedite colichestvo stranits "; cin>>list;
    }
    void getdata()
    {
        pub.getdata();
        cout <<"\nColichestvo stranits "<<list;
    }
};
/////////////////////////////////////
 
class type
{
    private:
    float vremiya_zapisi;
    publication pub;
    public:
    type( ): vremiya_zapisi( 0.0 ) {} //Чтобы не было мусора
    void putdata()
    {
        pub.putdata();
        cout <<"\nVvedite vremiya zapisi\n"; cin >> vremiya_zapisi;
    }
    void getdata()
    {
        pub.getdata();
       cout <<"\nVremiya zapisi\n"<<vremiya_zapisi;
    }
};
 
int main()
{
    int n, i;
       publication p1;
       book b1;
       type t1;
 
       for(i=0; i<5; i++)
       {
           cout <<"Hellow! Esli hotite vvesti\n publication najmite 1\n book najmite 2\n type najmite 3\nvivesti dannie na ekran -4 \n press "x" for exit"<<endl;
            cin >>n;
        switch(n)
        {
             case 1:
             p1.putdata();
             break;
             case 2:
             b1.putdata();
             break;
             case 3:
             t1.putdata();
             break;
             case 4:
             p1.getdata(); b1.getdata(); t1.getdata();cout<<"\n\n%"<<i<<endl;
             break;
        }
       }
       return 0;
}

@nmcf · 17.09.2016, 20:12

Для начала:

C++

1	char name[100];

	17.09.2016, 20:12