Инкапсулировать без серьезного изменения..возможно ли? - C++

@124bit · Регистрация: 06.12.2009

C++

#include <iostream>
#include <stdio.h>
#include <stdlib.h>
using namespace std;
 
 
class list_block
{
    private:
 
        string data;
    public:
        list_block* next;
 
        string get_data() const {return data;}
        void set_data(string new_data){data=new_data;}
        void set_next(list_block* new_next) {next=new_next;}
 
        list_block(list_block* new_next) {set_next(new_next);}
};
 
 
class circle_list
{
    protected:
        list_block* first;
        list_block* second;
 
 
    public:
        circle_list();
        circle_list(string* strings, int n);
        circle_list(circle_list& list);
        ~circle_list();
 
        int size() const;
        void show() const;
        void push(string);  // chtob realizovat
                            //konstruktor s parametrom
                            //prishlos zasunut' push suda
};
 
 
 
 
 
 
circle_list::circle_list()
{
 
    first=new list_block(NULL);
    first->set_data("start");
    second=new list_block(first);
    second->set_data("end");
 
    first->set_next(second);
}
 
circle_list::circle_list(string* strings, int n)
{
    first=new list_block(NULL);
    first->set_data("start");
    second=new list_block(first);
    second->set_data("end");
 
    first->set_next(second);
    for (int i=0;i<n;i++) push(*(strings+i));
}
 
circle_list::circle_list(circle_list& list)
{
    first=new list_block(NULL);
    first->set_data("start");
    second=new list_block(first);
    second->set_data("end");
 
    first->set_next(second);
 
    list_block* block=list.first;
    int size=list.size()+2;
 
    for (int i=0;i<size-2;i++) block=block->next;
 
    while (block->get_data()!="start")
    {
        push(block->get_data());
        for (int i=0;i<size-1;i++) {block=block->next;}
    }
}
 
circle_list::~circle_list()
{
    list_block* block=first;
    list_block* block1=first;
    while (block->get_data()!="end")
    {
        block1=block->next;
        delete block;
        block=block1;
    }
    delete block;
    block=NULL;
    block1=NULL;
}
 
int circle_list::size() const
{
    list_block* block=first;
    if (block->next->get_data()=="end") return 0;
    int i=0;
 
    do
    {
        i=i+1;
        block=block->next;
    }
    while (block->next->get_data()!="end");
    return i;
}
 
void circle_list::show() const
{
    list_block* block=first->next;
    cout <<"Elementi:"<<endl;
    while (block->get_data()!="end")
    {
        cout << block->get_data() << endl;
        block=block->next;
    }
}
 
void circle_list::push(string st)
{
    list_block* new_block=new list_block(second);
    new_block->set_data(st);
    second=new_block;
    first->set_next(new_block);
}
 
 
 
 
 
class quene : public circle_list
{
    public:
        quene():circle_list(){};
        quene(string* strings, int n):circle_list(strings,n){};
        quene(quene& list):circle_list(list){};
 
        string deleter();
 
        void deleter(int n) {for (int i=0;i<n;i++) deleter();}
        void deleter_all() {deleter(size());}
        string push_pop(string st){push(st); return deleter();}
 
 
};
 
string quene::deleter()
{
    list_block* block=first;
    while (block->next->get_data()!="end")
        {
            block=block->next;
        }
    list_block* old_next=block->next;
    string res=block->get_data();
    block->next=block->next->next;
    block->set_data("end");
    delete old_next;
    return res;
}
 
int main()
{
    string str[3]={"Inna","Masha","Julja"};
    quene VASA(str,3);
    cout << "...sozdaem klass VASA s devchenkami *Julja*,*Masha*,*Inna*"<< endl;
    VASA.show();
    cout << endl;
    getchar();
 //----
    quene VASA1(VASA);
    cout << "...sozdaem klass VASA_1 kotorij bliznec VASA i kortorij sebe skopiroval devchenok"<< endl;
    VASA1.show();
    cout << endl;
    getchar();
 //----
    cout << "__teper VASJU brosila " << VASA.push_pop("Julja")<<". Potomu chto on mudak!"
 
            <<" Zato Julja za kanikuli potolstela i teper za dvoih"<< endl;
    VASA.show();
    cout << endl;
    getchar();
  //----
    cout << "i u VASi_1 stalo bloshe devchonok chem u VASI.";
    VASA1.show();
    cout << endl;
    getchar();
  //----
    cout << "I chtob bilo chesno, VASJU_1 tozhe brosit " << VASA1.push_pop("Julja")<< endl;
    VASA1.show();
    cout << endl;
    getchar();
  //----
    string str1[0]={};
    quene Julik;
    cout << "...no tut pojavilsja Julik..holostjak"<< endl;
    Julik.show();
    cout << endl;
    getchar();
  //----
    cout << "__i Innochka stala s Julikom"<< endl;
    Julik.push("Innochka");
    Julik.show();
    cout << endl;
    getchar();
   //----
    cout << "..no potom vdrug ona ego brosila (("<< endl;
    Julik.deleter();
    Julik.show();
    cout << endl;
    getchar();
    //----
    cout << "__a potom opjat' vernulas'"<< endl;
    Julik.push("Innochka");
    Julik.show();
    cout << endl;
    getchar();
   //----
    cout << "..no potom opjat', zaraza, brosila (("
            << endl << "I Julik ostalsja sovsem odin" << endl;
    Julik.deleter_all();
    Julik.show();
    cout << endl;
    getchar();
   //----
    cout << "VIVOD PR2:  INNOCHKA BALBES!!!!"<< endl;
    system("pause");
    return 0;
}

собственно это кольцевая очередь, на которой у меня построено уже 4 след лабы.Пошел сегодня сдавать - препод ткнул рукой в не инкапсулированный
public:
list_block* next;
из
class list_block в самом начале. И говорит, пока не перепишешь все так, чтоб он был в привате - не приму. А там куча операций присваивания и считывания через (->) из class circle_list и его сына quene.
Причем фрэнды использовать запретила. НА все мои возражения только кивает *овца((
Во общем, как малой кровью вбахать эту глупость в приват и иметь к ней нормальный доступ отовсюду? что не переписывать еще 5 таких лаб в хз скольких меcтах ?..

@volovzi · 08.02.2011, 01:47

124bit, нет, я имею в виду сделать так:

C++

class list {
private:
    class node {
        node * next;
        data_type data;
        ...
    };
...
};

@124bit · 08.02.2011, 02:00 **[ТС]**

не помогает. Доступ у функций почему-то не появляется
или ты имел ввиду что если просто list_block туда сунуть инкапсуляция нарушаться не будет, но
list_block* next; в приват не заносить?

C++

#include <iostream>
#include <stdio.h>
#include <stdlib.h>
using namespace std;
 
 
 
 
 
class circle_list
{
 
//---------------------------------------
    protected:
    class list_block
{
    private:
 
        string data;
        list_block* next;
    public:
 
//       list_block* next; ушел в приват
        string get_data() const {return data;}
        void set_data(string new_data){data=new_data;}
        void set_next(list_block* new_next) {next=new_next;}
 
        list_block(list_block* new_next) {set_next(new_next);}
};
 
//-----------------------------------------
 
    protected:
        list_block* first;
        list_block* second;
 
 
    public:
        circle_list();
        circle_list(string* strings, int n);
        circle_list(circle_list& list);
        ~circle_list();
 
        int size() const;
        void show() const;
        void push(string);  // chtob realizovat
                            //konstruktor s parametrom
                            //prishlos zasunut' push suda
};
 
 
 
 
 
 
circle_list::circle_list()
{
 
    first=new list_block(NULL);
    first->set_data("start");
    second=new list_block(first);
    second->set_data("end");
 
    first->set_next(second);
}
 
circle_list::circle_list(string* strings, int n)
{
    first=new list_block(NULL);
    first->set_data("start");
    second=new list_block(first);
    second->set_data("end");
 
    first->set_next(second);
    for (int i=0;i<n;i++) push(*(strings+i));
}
 
circle_list::circle_list(circle_list& list)
{
    first=new list_block(NULL);
    first->set_data("start");
    second=new list_block(first);
    second->set_data("end");
 
    first->set_next(second);
 
    list_block* block=list.first;
    int size=list.size()+2;
 
    for (int i=0;i<size-2;i++) block=block->next;
 
    while (block->get_data()!="start")
    {
        push(block->get_data());
        for (int i=0;i<size-1;i++) {block=block->next;}
    }
}
 
circle_list::~circle_list()
{
    list_block* block=first;
    list_block* block1=first;
    while (block->get_data()!="end")
    {
        block1=block->next;
        delete block;
        block=block1;
    }
    delete block;
    block=NULL;
    block1=NULL;
}
 
int circle_list::size() const
{
    list_block* block=first;
    if (block->next->get_data()=="end") return 0;
    int i=0;
 
    do
    {
        i=i+1;
        block=block->next;
    }
    while (block->next->get_data()!="end");
    return i;
}
 
void circle_list::show() const
{
    list_block* block=first->next;
    cout <<"Elementi:"<<endl;
    while (block->get_data()!="end")
    {
        cout << block->get_data() << endl;
        block=block->next;
    }
}
 
void circle_list::push(string st)
{
    list_block* new_block=new list_block(second);
    new_block->set_data(st);
    second=new_block;
    first->set_next(new_block);
}
 
 
 
 
 
class quene : public circle_list
{
    public:
        quene():circle_list(){};
        quene(string* strings, int n):circle_list(strings,n){};
        quene(quene& list):circle_list(list){};
 
        string deleter();
 
        void deleter(int n) {for (int i=0;i<n;i++) deleter();}
        void deleter_all() {deleter(size());}
        string push_pop(string st){push(st); return deleter();}
 
 
};
 
string quene::deleter()
{
    list_block* block=first;
    while (block->next->get_data()!="end")
        {
            block=block->next;
        }
    list_block* old_next=block->next;
    string res=block->get_data();
    block->next=block->next->next;
    block->set_data("end");
    delete old_next;
    return res;
}
 
int main()
{
    string str[3]={"Inna","Masha","Julja"};
    quene VASA(str,3);
    cout << "...sozdaem klass VASA s devchenkami *Julja*,*Masha*,*Inna*"<< endl;
    VASA.show();
    cout << endl;
    getchar();
 //----
    quene VASA1(VASA);
    cout << "...sozdaem klass VASA_1 kotorij bliznec VASA i kortorij sebe skopiroval devchenok"<< endl;
    VASA1.show();
    cout << endl;
    getchar();
 //----
    cout << "__teper VASJU brosila " << VASA.push_pop("Julja")<<". Potomu chto on mudak!"
 
            <<" Zato Julja za kanikuli potolstela i teper za dvoih"<< endl;
    VASA.show();
    cout << endl;
    getchar();
  //----
    cout << "i u VASi_1 stalo bloshe devchonok chem u VASI.";
    VASA1.show();
    cout << endl;
    getchar();
  //----
    cout << "I chtob bilo chesno, VASJU_1 tozhe brosit " << VASA1.push_pop("Julja")<< endl;
    VASA1.show();
    cout << endl;
    getchar();
  //----
    string str1[0]={};
    quene Julik;
    cout << "...no tut pojavilsja Julik..holostjak"<< endl;
    Julik.show();
    cout << endl;
    getchar();
  //----
    cout << "__i Innochka stala s Julikom"<< endl;
    Julik.push("Innochka");
    Julik.show();
    cout << endl;
    getchar();
   //----
    cout << "..no potom vdrug ona ego brosila (("<< endl;
    Julik.deleter();
    Julik.show();
    cout << endl;
    getchar();
    //----
    cout << "__a potom opjat' vernulas'"<< endl;
    Julik.push("Innochka");
    Julik.show();
    cout << endl;
    getchar();
   //----
    cout << "..no potom opjat', zaraza, brosila (("
            << endl << "I Julik ostalsja sovsem odin" << endl;
    Julik.deleter_all();
    Julik.show();
    cout << endl;
    getchar();
   //----
    cout << "VIVOD PR2:  INNOCHKA BALBES!!!!"<< endl;
    system("pause");
    return 0;
}

@volovzi · 08.02.2011, 02:31

ForEveR, ну, если ссылаешься на СБШ, то залезь внутрь того же вектора и посмотри, как там оно устроено.

124bit, да list_block* next закрытым не делать.

Добавлено через 2 минуты
ForEveR, а лучше не вектор смотреть, а сразу список. Там всё открыто.

@easybudda · 08.02.2011, 03:01

Сообщение от volovzi

да list_block* next закрытым не делать.

Те же грабли...

Сообщение от 124bit

препод ткнул рукой в не инкапсулированный
public:
list_block* next;
из
class list_block в самом начале. И говорит, пока не перепишешь все так, чтоб он был в привате - не приму.

Можно, конечно, попробовать измором взять, но шансов мало...

@volovzi · 08.02.2011, 03:17

easybudda, вот простой пример.

C++

template <typename Value>
class list {
public:
    typedef Value value_type;
 
private:
    struct node {
        explicit node (const value_type & value) : value(value), next(0) {}
        ~node () { delete next; }
    
        value_type value;
        node * next;
    };
    
public:
    list () : m_first(0), m_last(0) {}
    ~list () { delete m_first; }
    
    void push_back (const value_type & value) {
        if (m_first == 0) m_last = m_first = new node(value);
        else {
            m_last->next = new node(value);
            m_last = m_last->next;
        }
    }
    
private:
    node * m_first;
    node * m_last;
};

Где здесь грабли, и в чём состоит небезопасность?

@Mr.X · 08.02.2011, 03:47

Сообщение от volovzi

и в чём же разница между структурой и классом, кроме спецификаторов по-умолчанию?

В C++ существует такое понятие как POD-класс, что по сути означает Си-совместимую структуру, которая аналогична понятию записи в других языках. В таких структурах все данные сишных типов, и они открыты.
POD-объект может быть инициализирован с помощью агрегатного инициализатора (в фигурных скобках). Локальный POD-объект без инициализатора имеет неопределенное значение. При инициализации пустым инициализатором данные получают значение 0 или false.
В инструкции goto можно обходить объявления POD-объектов.

@volovzi · 08.02.2011, 12:26

Mr.X, за информацию спасибо, но к теме это не относится. Мы тут рассматриваем всё же не POD, а сиплюсплюсные классы.

@Mr.X · 08.02.2011, 12:48

Сообщение от volovzi

Mr.X, за информацию спасибо, но к теме это не относится. Мы тут рассматриваем всё же не POD, а сиплюсплюсные классы.

Я это к разговору об открытых данных классов. Согласно концепциям C++ они допустимы только в POD-классах, в других же - это дурной тон.

@volovzi · 08.02.2011, 13:31

Сообщение от Mr.X

Я это к разговору об открытых данных классов. Согласно концепциям C++ они допустимы только в POD-классах, в других же - это дурной тон.

Обоснуй. И прокомментируй это утверждение на примере класса std::pair или на примере исходников класса std::list.

@Mr.X · 08.02.2011, 14:18

Сообщение от volovzi

Обоснуй. И прокомментируй это утверждение на примере класса std::pair или на примере исходников класса std::list.

А чего тут обосновывать? Инкапсуляция – это один из основополагающих принципов ООП.
Что касается шаблона класса std::pair, то это классический пример аналога паскалевской записи.
Класс же std::list слишком сложен, чтобы его здесь обсуждать, да и обобщенное программирование – это совсем отдельная песня.

@volovzi · 08.02.2011, 21:16

Сообщение от Mr.X

А чего тут обосновывать?

Утверждение обосновывать.
Либо признать, что ляпнул не подумав. Тем более, что я уже привёл контрпример, его опровергающий.

Что касается шаблона класса std::pair, то это классический пример аналога паскалевской записи.

И что это означает?

Класс же std::list слишком сложен, чтобы его здесь обсуждать

А ты в него хотя бы заглядывал?
Вот определение основы узла списка из исходников СБШ:

C++

struct _List_node_base
  {
    _List_node_base* _M_next;   ///< Self-explanatory
    _List_node_base* _M_prev;   ///< Self-explanatory
 
    static void
    swap(_List_node_base& __x, _List_node_base& __y);
 
    void
    transfer(_List_node_base * const __first,
         _List_node_base * const __last);
 
    void
    reverse();
 
    void
    hook(_List_node_base * const __position);
 
    void
    unhook();
  };

Ничего сложного, и шаблонов здесь нет. Прокомментируй этот код.

...да и обобщенное программирование – это совсем отдельная песня.

С этого места поподробнее, пожалуйста. Я очень хотел бы узнать, чем отличается использование спецификаторов доступа в обобщённом и "обычном" программированиях.

@124bit · 08.02.2011, 23:11 **[ТС]**

VOLOVZI
так если я инкапсулирую подклас это ничего не изменит.. те же грабли, только она мне заявит, что подкласс некорректен

есть какие-нибудь другие легальные методы влезть в приват?

HighPredator · 08.02.2011, 23:34

Если так критично поставить указатель в приват, то сделайте так и заведите паблик функцию выдающую этот указатель

@volovzi · 09.02.2011, 00:45

124bit, не понял, что не изменит? Какие грабли? Я тут, блин, на три страницы распинаюсь в попытке объяснить суть.
И разберись уже с термином «инкапсуляция». Модификатор доступа private и инкапсуляция — это совершенно разные вещи.

@124bit · 09.02.2011, 08:35 **[ТС]**

Predator_2004
Так черт... это ж не весь код. У меня еще 5 лаб на этой основе. Мне кода огогого придется менять, если я ввиду функции записи и считывания. Задача как раз чтоб решить вопрос малой кровью-малыми изменениями
VOLOVZI я тебя понимаю, и с абсолютно с тобой согласен.
Но учитывая специфику ситуации(препода!) задача стоит не *сделать норм программу*, а *переделать программу выше так, чтоб тот указатель стоял в привате*. При ввведении подкласса, у меня сам класс выходит корректен, с точки зрения препода, а подкласс некорректен. Вот. Причем вариант *что-то объяснить* не канает

@volovzi 268 / 170 / 8 Регистрация: 14.03.2010 Сообщений: 501
	08.02.2011, 02:31	#18
	ForEveR, ну, если ссылаешься на СБШ, то залезь внутрь того же вектора и посмотри, как там оно устроено. 124bit, да list_block* next закрытым не делать. Добавлено через 2 минуты ForEveR, а лучше не вектор смотреть, а сразу список. Там всё открыто. 0

@easybudda Модератор 9958 / 5881 / 993 Регистрация: 25.07.2009 Сообщений: 11,119
	08.02.2011, 03:01	#19
	Сообщение от volovzi да list_block* next закрытым не делать. Те же грабли... Сообщение от 124bit препод ткнул рукой в не инкапсулированный public: list_block* next; из class list_block в самом начале. И говорит, пока не перепишешь все так, чтоб он был в привате - не приму. Можно, конечно, попробовать измором взять, но шансов мало... 0

@Mr.X 3056 / 1701 / 265 Регистрация: 03.05.2010 Сообщений: 3,867
	08.02.2011, 03:47	#21
	Сообщение от volovzi и в чём же разница между структурой и классом, кроме спецификаторов по-умолчанию? В C++ существует такое понятие как POD-класс, что по сути означает Си-совместимую структуру, которая аналогична понятию записи в других языках. В таких структурах все данные сишных типов, и они открыты. POD-объект может быть инициализирован с помощью агрегатного инициализатора (в фигурных скобках). Локальный POD-объект без инициализатора имеет неопределенное значение. При инициализации пустым инициализатором данные получают значение 0 или false. В инструкции goto можно обходить объявления POD-объектов. 0

@volovzi 268 / 170 / 8 Регистрация: 14.03.2010 Сообщений: 501
	08.02.2011, 12:26	#22
	Mr.X, за информацию спасибо, но к теме это не относится. Мы тут рассматриваем всё же не POD, а сиплюсплюсные классы. 0

@Mr.X 3056 / 1701 / 265 Регистрация: 03.05.2010 Сообщений: 3,867
	08.02.2011, 12:48	#23
	Сообщение от volovzi Mr.X, за информацию спасибо, но к теме это не относится. Мы тут рассматриваем всё же не POD, а сиплюсплюсные классы. Я это к разговору об открытых данных классов. Согласно концепциям C++ они допустимы только в POD-классах, в других же - это дурной тон. 0

@volovzi 268 / 170 / 8 Регистрация: 14.03.2010 Сообщений: 501
	08.02.2011, 13:31	#24
	Сообщение от Mr.X Я это к разговору об открытых данных классов. Согласно концепциям C++ они допустимы только в POD-классах, в других же - это дурной тон. Обоснуй. И прокомментируй это утверждение на примере класса std::pair или на примере исходников класса std::list. 0

@Mr.X 3056 / 1701 / 265 Регистрация: 03.05.2010 Сообщений: 3,867
	08.02.2011, 14:18	#25
	Сообщение от volovzi Обоснуй. И прокомментируй это утверждение на примере класса std::pair или на примере исходников класса std::list. А чего тут обосновывать? Инкапсуляция – это один из основополагающих принципов ООП. Что касается шаблона класса std::pair, то это классический пример аналога паскалевской записи. Класс же std::list слишком сложен, чтобы его здесь обсуждать, да и обобщенное программирование – это совсем отдельная песня. 0

@124bit 0 / 0 / 0 Регистрация: 06.12.2009 Сообщений: 17
	08.02.2011, 23:11 [ТС]	#27
	VOLOVZI так если я инкапсулирую подклас это ничего не изменит.. те же грабли, только она мне заявит, что подкласс некорректен есть какие-нибудь другие легальные методы влезть в приват? 0

HighPredator 5634 / 1949 / 369 Регистрация: 10.12.2010 Сообщений: 5,651 Записей в блоге: 3
	08.02.2011, 23:34	#28
	Если так критично поставить указатель в приват, то сделайте так и заведите паблик функцию выдающую этот указатель 0

@volovzi 268 / 170 / 8 Регистрация: 14.03.2010 Сообщений: 501
	09.02.2011, 00:45	#29
	124bit, не понял, что не изменит? Какие грабли? Я тут, блин, на три страницы распинаюсь в попытке объяснить суть. И разберись уже с термином «инкапсуляция». Модификатор доступа private и инкапсуляция — это совершенно разные вещи. 0

@124bit 0 / 0 / 0 Регистрация: 06.12.2009 Сообщений: 17
	09.02.2011, 08:35 [ТС]	#30
	Predator_2004 Так черт... это ж не весь код. У меня еще 5 лаб на этой основе. Мне кода огогого придется менять, если я ввиду функции записи и считывания. Задача как раз чтоб решить вопрос малой кровью-малыми изменениями VOLOVZI я тебя понимаю, и с абсолютно с тобой согласен. Но учитывая специфику ситуации(препода!) задача стоит не сделать норм программу, а переделать программу выше так, чтоб тот указатель стоял в привате. При ввведении подкласса, у меня сам класс выходит корректен, с точки зрения препода, а подкласс некорректен. Вот. Причем вариант что-то объяснить не канает 0

С++ для начинающих

Инкапсулировать без серьезного изменения..возможно ли? - C++