Форум программистов, компьютерный форум CyberForum.ru

Покритикуйте код - C++

Восстановить пароль Регистрация
 
 
Рейтинг: Рейтинг темы: голосов - 46, средняя оценка - 4.80
Gepar
 Аватар для Gepar
1173 / 529 / 20
Регистрация: 01.07.2009
Сообщений: 3,512
02.10.2011, 23:28     Покритикуйте код #1
Есть класс Студенты (реализован через односвязный список), хотел бы услышать критику по поводу его улучшения, если кому не лень разбираться в столь поздний час Сам код естественно полностью рабочий и предупреждений тоже компилятор не выдаёт (если не считать в main в условии while, но там всё ок) так что хотелось бы услышать Ваши замечания если что можно сделать лучше.

Students.h
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
#ifndef STUDENTS_H
#define STUDENTS_H
 
#include <string>
using std::string;
 
#include <iostream>
using std::cout;
using std::cerr;
 
#include <iomanip>
using std::setw;
using std::left;
using std::right;
 
#include <stdexcept>
using std::out_of_range;
 
class Students
{
    struct ListItem
    {
        string fullname;
        string group;
        int year;
        int average;
        ListItem *Next;
        ListItem(string ="",int =0,int =0,string ="",ListItem* =0);
    };
 
    ListItem *Head;
    ListItem *Tail;
    ListItem *Current; //указатель на текущий элемент
    int count; // всего элементов
 
public:
    class iterator
    {
        Students::ListItem* current;
 
        //функция для проверки что current инициализирован
        void correct(){if(!*this) throw Students::Exception("access denied");}
 
        public:
        string first(){correct();return current->fullname;}
        string second(){correct();return current->group;}
        int third(){correct();return current->year;}
        int four(){correct();return current->average;}
 
        iterator() {current=0;}
 
        iterator(const Students &right){*this=right;}
        iterator(Students::ListItem* right){*this=right;}
        iterator& operator=(Students &right);
        iterator operator=(Students::ListItem* right);
 
        iterator& operator++();
        iterator operator++(int);
        bool operator==(const iterator& right) const;
        bool operator!=(const iterator& right) const;
        iterator* operator*(){return this;}
        operator bool(){return current!=0;}
    };
 
    class Exception
    {
        string str;
        public:
        Exception(string data) :str(data){};
        string what(){return str;}
    };
 
 
 
    Students(): Head(0), Tail(0), Current(0), count(0) {}
    Students(string data, int y, int a, string g);
    ~Students(){this->deleteAll();}
 
 
    void addToTail(string data, int y, int a, string g);
    void addToHead(string data, int y, int a, string g);
 
 
    void deleteFromHead(bool mode=1);//1 - с предупреждением(исключением) если список пуст
    void deleteFromTail();
    void deleteAll();
 
    //сортировка, принимает функцию для сравнения элементов
    void sort(bool cmp(string&,string&));
 
    //методы возвращающие итераторы
    iterator begin(){return Head;}
    iterator end(){return iterator(0);}
};
 
 
#endif
Students.cpp
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
#include "Students.h"
 
//////////////////////STUDENTS/////////////////
 
Students::ListItem::ListItem(string name,int y,int aver,string gr,ListItem* next)
{
    fullname=name;
    group=gr;
    average=(aver>=0 && aver<=100 ? aver : -1);
    year=(y>=1950 && y<=2012 ? y : -1);
 
    if(average==-1 || year==-1)
     throw Exception("wrong data");
    Next=next;
}
 
Students::Students(string data, int y, int a, string g)
{
    Head=Tail=new ListItem(data,y,a,g);
    Current=0;
    count=1;
}
 
 
void Students::addToTail(string data, int y, int a, string g)
{
    if (Head)
    {
        Tail->Next=new ListItem(data,y,a,g,0);
        Tail=Tail->Next;
        count++;
    }
    else
        Head=Tail=new ListItem(data,y,a,g);
}
 
 
void Students::addToHead(string data, int y, int a, string g)
{
    if (Head)
    {
        ListItem* temp=Head;
        Head=new ListItem(data,y,a,g);
        Head->Next=temp;
        count++;
    }
    else
        Head=Tail=new ListItem(data,y,a,g);
}
 
void Students::deleteFromHead(bool mode)
{
    if(Head->Next)
    {
        ListItem *temp=Head;
        Head=Head->Next;
        count--;
        delete temp;
    }
 
    else if (Head)
    {
        ListItem *temp=Head;
        Head=Tail=Current=0;
        delete temp;
        count=0;
    }
 
    else if(mode)
     throw Exception("list is empty");
}
 
void Students::deleteFromTail()
{
    if(Tail != Head)
    {
        ListItem *temp=Head;
        while(temp->Next!=Tail)
            temp=temp->Next;
 
    ListItem *toDelete=temp->Next;
    Tail=temp;
    temp->Next=0;
 
    delete toDelete;
    count--;
    }
 
    else if(Tail==0)
     throw Exception("list is empty");
 
    else
     Head=Tail=0;
}
 
 
void Students::deleteAll()
{
    while (Head)
     deleteFromHead(0);
 
    count=0;
}
 
void Students:: sort(bool cmp(string&,string&))
{
    ListItem* new_begin=NULL;
    ListItem* new_end=NULL;
    ListItem* sprev=NULL;
 
    for(ListItem *scur=this->Head;scur!=NULL;scur=this->Head)
    {
        ListItem *smin=NULL;
        ListItem *sminprev=scur;
        string min_name=scur->fullname;
        for(ListItem *gp=scur->Next;gp!=NULL;gp=gp->Next)
        {
            if(cmp(gp->fullname,min_name))
            {
                min_name=gp->fullname;
                smin=gp;
                sprev=sminprev;
            }
            sminprev=gp;
        }
        if(smin==NULL)
        {
            smin=scur;
        }
        else if(smin==scur->Next)
        {
            scur->Next=scur->Next->Next;
        }
        else
        {
            sprev->Next=smin->Next;
        }
        if(new_begin!=NULL)
        {
            new_end->Next=smin;
            new_end=smin;
        }
        else
        {
            new_begin=smin;
            new_end=smin;
        }
        if(smin==this->Head)
         this->Head=smin->Next;
    }
    this->Head=new_begin;
    this->Tail=new_end;
 
}
 
 
//////////////////////ITERATOR/////////////////
Students::iterator& Students::iterator::operator++()
{
    current=current->Next;
     return *this;
}
 
Students::iterator Students::iterator::operator++(int)
{
    iterator temp(*this);
    current=current->Next;
     return temp;
}
 
bool Students::iterator::operator==(const iterator& right) const
{
    return (current==right.current);
}
 
bool Students::iterator::operator!=(const iterator& right) const
{
    return !(*this==right);
}
 
Students::iterator Students::iterator::operator=(Students::ListItem* right)
{
    current=right;
    return *this;
}
 
Students::iterator& Students::iterator::operator=(Students &right)
{
    current=right.Current;
    return *this;
}
main.cpp
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
#include <iostream>
using std::cout;
using std::cin;
using std::endl;
using std::string;
#include <windows.h>
#include <string>
#include "Students.h"
#include "StudentsIterator.h"
#include "algorithm"
using std::for_each;
 
 
bool cmp( string &str1, string &str2 )
{
    return strcmp( str1.c_str(), str2.c_str() ) < 0;
}
 
 
void print(Students::iterator *ita)
{
    cout<<setw(15)<<left<<ita->first()
     <<setw(7)<<ita->second()
     <<setw(5)<<ita->third()
     <<setw(7)<<ita->four()<<"\n";
 
}
 
Students::iterator findElement(string toFind, Students::iterator begin, Students::iterator end)
{
    for(;begin!=end;++begin)
    {
        if (begin.first().find(toFind)!=(size_t)-1)
         return begin;
    }
    //если ничего не найдено - вернуть итератор указывающий за конец списка ( на NULL)
    return 0;
}
 
 
int main()
{
    SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
 
 
    Students test("Иванов Иван",1992,100,"КС-09");
    test.addToTail("ЛЛЛ ФФ",1991,100,"КС-09");
    test.addToTail("БББ ВВ",1992,100,"КС-09");
    test.addToHead("Иванов Василий",1993,100,"КС-08");
    test.addToHead("Иванов Петр", 1990,100,"КС-07");
    test.addToHead("ЯЯЯ ММ",1993,100,"КС-08");
    test.addToTail("ЮЮЮ ЛЛЛ",1994,100,"КС-07");
    test.sort(cmp);
 
    cout<<"All list(after sort):\n";
    for_each(test.begin(),test.end(),print);
    test.deleteFromHead();
    test.deleteFromTail();
    cout<<endl<<"After delFromHead + delFromTail:\n";
    for_each(test.begin(),test.end(),print);
 
    cout<<endl<<"All Students with name \"Иванов\" (searching from begin to end of list):\n";
    Students::iterator it=test.begin();
    while(it=findElement("Иванов",it,test.end()))
     print(*it++);
}
Комментариев только в коде совсем мало, но функции и переменные несут смысловую нагрузку так что надеюсь код не очень сложный в чтении получился.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Gepar
 Аватар для Gepar
1173 / 529 / 20
Регистрация: 01.07.2009
Сообщений: 3,512
11.10.2011, 10:40  [ТС]     Покритикуйте код #81
Ага, это оно с minigw работает, а вот с vs6.0 это ещё надо попотеть чтобы оно заработало, отслось догадаться что не так читая ошибки типа
Код
main.obj : error LNK2001: unresolved external symbol "public: class Students<int>::iterator __thiscall Students<int>::iterator::operator++(int)" (??Eiterator@?$Students@H@@QAE?AV01@H@Z)
Нет, ну вот как после таких ошибок можно любить VS

Добавлено через 3 минуты
Цитата Сообщение от talis Посмотреть сообщение
Слушайте, а где вы учитесь? Мне бы такого препода...
В ДНУ.

Цитата Сообщение от talis Посмотреть сообщение
Gepar, а так нельзя?
Не, задание же на листике было в виде кода, у него коварности хватило ещё подсунуть на некоторых листиках полностью рабочий код в задании на шаблоны (рабочий в плане компилирующийся) и заставить искать ошибки, ну я не нашёл и так и написал что ошибок нет, как оказалось и правда в некоторых вариантах ошибок нет, в моём в частности их не было.

Добавлено через 4 минуты
Вообще, по рассказам других преподавателей, препод этот по с++ (и по SQL на 4м курсе) где-то работает и нормально зарабатывает, а универ у него так чисто "для души" вот и издевается с нас такими заданиями
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 10:53     Покритикуйте код #82
Gepar, да нет. Если не брать задания типа "найдите ошибки в коде на листочке" - всё очень хорошо. Поверьте, когда препод ограничивается заданиями по высшей математике на одно вычисление (ну или максимум двумерными массивами - матрицами), и говорит "сделайте это функциями", "сделайте это классом", "прикрутите наследование как-нить" - это гораздо хуже.

Да и с листочком - если он даст скормить его компилятору и почитать его реакцию, то тоже хорошо. Можно изучить коварные подводные грабли, на самостоятельное знакомство с которыми обычно уходит несколько лет и несколько тысяч кружек чая бессонными ночами со слезящимися глазами.
aeshes
11.10.2011, 10:59
  #83

Не по теме:


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

Gepar
 Аватар для Gepar
1173 / 529 / 20
Регистрация: 01.07.2009
Сообщений: 3,512
11.10.2011, 11:03  [ТС]     Покритикуйте код #84
talis, ну преподша по паскалю и делфи и говорила что он может реально научить тому как оно есть так как он этими знаниями и правда зарабатывает на практике. С VS 6.0 я так понял с этими кодами ошибок я пойду сейчас к нему, ато я понятия не имею что ей не так-то, ведь пока код не шаблонным был она всё компилировала и не жаловалась.
Thinker
11.10.2011, 11:06
  #85

Не по теме:

А у нас в первом семестре преподаватель super был, Паскаль правда учили. На втором семестре С/С++ должны были учить с другим преподавателем, который посещал занятия через раз, а когда приходил, то рассказывал что-то левое (про интернет-магазины и т.п.), вообще ни чему от него не научились. Хорошо, что у кого-то хоть лекции заумные были Всегда обожал строгих, умных и требовательных преподавателей, чтобы по три шкуры сдирал, но мало таких было... Зато с преподавателями по математическим дисциплинам очень повезло, большинство из МГУ:dance3:

talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 11:14     Покритикуйте код #86
Gepar, когда пойдёте к преподу с ошибками, упомяните что в MinGW в gcc такой-то версии всё компилируется нормально. VS 6 во-первых старая, во-вторых глючная. Ошибок в ней много. Код, который, скажем, gcc или icpc компилируют на ура, она не кушает. Иногда просто и лаконично - internal compiler error, а иногда такой бред...
aeshes
 Аватар для aeshes
437 / 200 / 13
Регистрация: 07.10.2011
Сообщений: 462
11.10.2011, 11:40     Покритикуйте код #87
Я помню, что в VS 6 нам препод говорил не разносить определение и реализацию шаблона по разным файлам h и cpp, а писать все в одном. Но это было давно, возможно я что-то путаю
HighPredator
 Аватар для HighPredator
5350 / 1733 / 320
Регистрация: 10.12.2010
Сообщений: 5,119
Записей в блоге: 3
11.10.2011, 16:00     Покритикуйте код #88
aeshes, ничего не путаете. Не все vs поддерживают экспорт шаблонов.

Добавлено через 25 секунд
По крайней мере я таких не видел.
Gepar
 Аватар для Gepar
1173 / 529 / 20
Регистрация: 01.07.2009
Сообщений: 3,512
11.10.2011, 17:32  [ТС]     Покритикуйте код #89
aeshes, я так и пробовал, препод тоже не смог угадать чего ей не нравится, но в этот раз ещё нашёл что ему на этот раз не нравится так что буду исправлять и нести в пн. Не понравилось преподу что у элементов списка (CStudent) нет set/get методов и данные в public (хотя как по мне так там обычная структура со строками да парой интов, чего туда ещё что-то добавлять, ну да просит так сделаю). И ещё он хочет чтобы я вынес итератор, мол Students::iterator это неудобно, как-будто создание отдельно итератора указанного типа удобнее и правильнее, в этом я его не понимаю, ну да я ещё подумаю над этим.

Добавлено через 3 минуты
И ещё сказал что лучше перегрузить оператор ! вместо приведения к bool, хотя как по мне так условия типа if(iterator) выглядят вполне логично.

Добавлено через 41 секунду
Ага, и ещё перетащить CStudent (элемент списка) отдельно от всего (и от .h файла списка и от cpp файла списка).
talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 18:35     Покритикуйте код #90
Цитата Сообщение от Gepar Посмотреть сообщение
И ещё он хочет чтобы я вынес итератор, мол Students::iterator это неудобно, как-будто создание отдельно итератора указанного типа удобнее и правильнее
Да ну бред же, бред сивой кобылы! Про STL ему скажите, что там так сделано. vector<int>::iterator, list<CStudent>::iterator и константные ещё. И так во всех контейнерах. И сделано так не от балды и не от нефиг делать.

На счёт того, что данные должны быть в private или protected - он прав.

Цитата Сообщение от Gepar Посмотреть сообщение
И ещё сказал что лучше перегрузить оператор ! вместо приведения к bool, хотя как по мне так условия типа if(iterator) выглядят вполне логично.
Не-а. Ни одно, ни другое. Вообще этого не надо. Итераторы не преобразуются к bool, и у них нет operator!.

Цитата Сообщение от Gepar Посмотреть сообщение
Ага, и ещё перетащить CStudent (элемент списка) отдельно от всего (и от .h файла списка и от cpp файла списка).
Ну это-то можно.
fasked
Эксперт C++
 Аватар для fasked
4924 / 2504 / 180
Регистрация: 07.10.2009
Сообщений: 4,306
Записей в блоге: 1
11.10.2011, 20:05     Покритикуйте код #91
Цитата Сообщение от talis Посмотреть сообщение
На счёт того, что данные должны быть в private или protected - он прав.
Лучше всего, конечно, пять звездочек, то есть в private.

Цитата Сообщение от talis Посмотреть сообщение
Да ну бред же, бред сивой кобылы! Про STL ему скажите, что там так сделано. vector<int>::iterator, list<CStudent>::iterator и константные ещё. И так во всех контейнерах. И сделано так не от балды и не от нефиг делать.
А я вот вообще не вижу особого смысла в итераторах для специализированного контейнера. Учитывая, что функция сортировки является методом данного контейнера, то все остальные алгоритмы так же должны добавляться в методы. А для перебора элементов контейнера использоваться простой цикл от 0 до size.
talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 20:11     Покритикуйте код #92
Да нет, почему. Если итератор реализован верно (не проверял), то те же for_each или count_if, или find / find_if переписывать смысла нет.

Добавлено через 1 минуту
Некоторые алгоритмы STL не получится использовать, потому что у на фактически ForwardIterator - список-то односвязный.
fasked
Эксперт C++
 Аватар для fasked
4924 / 2504 / 180
Регистрация: 07.10.2009
Сообщений: 4,306
Записей в блоге: 1
11.10.2011, 20:12     Покритикуйте код #93
Цитата Сообщение от talis Посмотреть сообщение
то те же for_each или count_if, или find / find_if переписывать смысла нет.
В таком случае нет смысла переписывать и сортировку и вообще специализированный контейнер создавать нет смысла. Либо делать обобщенный контейнер - шаблонный - и для него уже STL совместимый итератор.
talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 20:16     Покритикуйте код #94
Сортировка переписывалась-то из тех соображений, что std::sort кушает RandomAccessIterator, а в условиях односвязного списка делать его было несколько нецелесообразно.

Добавлено через 1 минуту
Цитата Сообщение от fasked Посмотреть сообщение
Либо делать обобщенный контейнер - шаблонный - и для него уже STL совместимый итератор.
Стоп, ТС вчера же это и делал. Шаблон в смысле.
fasked
Эксперт C++
 Аватар для fasked
4924 / 2504 / 180
Регистрация: 07.10.2009
Сообщений: 4,306
Записей в блоге: 1
11.10.2011, 20:41     Покритикуйте код #95
Цитата Сообщение от talis Посмотреть сообщение
Стоп, ТС вчера же это и делал. Шаблон в смысле.
Каюсь, целиком читать лень было тему.
Тогда вопрос к ТС, почему класс до сих пор называется Students?
talis
11.10.2011, 20:46
  #96

Не по теме:

Этот вопрос имеется у всех за исключением препода Препод стенает, что iterator сделан как nested type.

Gepar
 Аватар для Gepar
1173 / 529 / 20
Регистрация: 01.07.2009
Сообщений: 3,512
11.10.2011, 23:43  [ТС]     Покритикуйте код #97
Цитата Сообщение от talis Посмотреть сообщение
Да ну бред же, бред сивой кобылы! Про STL ему скажите, что там так сделано. vector<int>::iterator, list<CStudent>::iterator и константные ещё. И так во всех контейнерах. И сделано так не от балды и не от нефиг делать.
Ну я об этом и говорил показывая свой main и что создавать отдельно от класса итератор указывая его типа
C++
1
iterator<int> it;
выглядит неуклюже, ну да я тут подумаю, возможно придётся вынести класс если он не захочет так компилироватся в vs6.0.


Цитата Сообщение от talis Посмотреть сообщение
Да нет, почему. Если итератор реализован верно (не проверял), то те же for_each или count_if, или find / find_if переписывать смысла нет.
Итератор реализован правильно, эти алгоритмы работают с ним нормально.
Цитата Сообщение от talis Посмотреть сообщение
Этот вопрос имеется у всех за исключением препода
Не, препод тоже сказал что список Students int'ов не очень будет выглядеть, ну да сам виноват, нечего задания по кусочкам давать, откуда же мне знать что на шаблоны задание включает редактирование первого задания

Добавлено через 2 минуты
В общем завтра уже займусь моим классом сегодня лень, препод ещё предлагал если так нравится красота вида Students::iterator то чтобы я вынес класс-итератор а потом дефайном в классе Students его как-то хитро объявил и тогда вызовы из main
C++
1
Students::iterator it;
мол будут работать, но я не понял как это можно сделать, что же я тогда через define должен определять по средине класса.
talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 23:46     Покритикуйте код #98
Цитата Сообщение от Gepar Посмотреть сообщение
выглядит неуклюже, ну да я тут подумаю, возможно придётся вынести класс если он не захочет так компилироватся в vs6.0.
Да нет, почему. Stl же в vs6 есть. Можете посмотреть, как там сделано. Я правда не знаю, что ей надо.

Цитата Сообщение от Gepar Посмотреть сообщение
нечего задания по кусочкам давать, откуда же мне знать что на шаблоны задание включает редактирование первого задания
Хе-хе, вот вы и попались. Я вам с самого начала говорил, что студент - студентом, а список - он и в африке список, и так и надо было его называть.

Добавлено через 1 минуту
Цитата Сообщение от Gepar Посмотреть сообщение
если так нравится красота вида Students::iterator то чтобы я вынес класс-итератор а потом дефайном в классе Students его как-то хитро объявил и тогда вызовы из main
"Давайте сделаем самокат, прикрутим к нему сиденье и моторчик. И коляску справа. И мотоциклом назовём."
Gepar
 Аватар для Gepar
1173 / 529 / 20
Регистрация: 01.07.2009
Сообщений: 3,512
11.10.2011, 23:48  [ТС]     Покритикуйте код #99
Цитата Сообщение от talis Посмотреть сообщение
Да нет, почему. Stl же в vs6 есть. Можете посмотреть, как там сделано. Я правда не знаю, что ей надо.
Ну препод сказал что студия старая вот может и не умеет обрабатывать вложенные классы в шаблонах и их последующее объявление вне класса.

Цитата Сообщение от talis Посмотреть сообщение
Хе-хе, вот вы и попались. Я вам с самого начала говорил, что студент - студентом, а список - он и в африке список, и так и надо было его называть.
Ну и ладно, мне не долго позаменять Students на List, особенно если это делать не вручную по слову Но это чуть позже, нужно доделать не шаблонную версию класса чтобы преподу она нравилась.

Добавлено через 1 минуту
Цитата Сообщение от talis Посмотреть сообщение
"Давайте сделаем самокат, прикрутим к нему сиденье и моторчик. И коляску справа. И мотоциклом назовём."
Ну так это же он предложил, я то причём, мне и так нравиться А Вы поняли как он хочет это сделать? Я что-то не понял как это я дефайном его должен там влеплять? Хотя это я больше из интереса спрашиваю, препод такой что если ты ему доказываешь свою точку мнения то он насчёт стиля ещё как-то соглашается чтобы и так было, вот насчёт set/get методов для CStudent там например не поспоришь, а здесь я если что его переговорю.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
11.10.2011, 23:52     Покритикуйте код
Еще ссылки по теме:

C++ Покритикуйте код
Покритикуйте мою игру C++
C++ Графы. Покритикуйте код

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
talis
 Аватар для talis
789 / 541 / 37
Регистрация: 11.05.2010
Сообщений: 1,298
Записей в блоге: 1
11.10.2011, 23:52     Покритикуйте код #100
Скажите вашему Гудвину, что где-то там, за горами и пустыней, уже приняли C++11. Это к слову о vs6.

Добавлено через 50 секунд
Я про дефайн тоже не очень понял. Типа, #define Students::iterator iterator - так как-то, похоже
Yandex
Объявления
11.10.2011, 23:52     Покритикуйте код
Ответ Создать тему
Опции темы

Текущее время: 05:12. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru