![]() |
Модераторы: bsa |
![]() ![]() ![]() |
|
ferz721 |
|
|||
Шустрый ![]() Профиль Группа: Участник Сообщений: 60 Регистрация: 14.1.2011 Репутация: нет Всего: нет |
thanks.
посмотрите код, скажите где ошибки и + Присоединённый файл ( Кол-во скачиваний: 14 ) ![]() |
|||
|
||||
boostcoder |
|
|||
![]() pattern`щик ![]() ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 5458 Регистрация: 1.4.2010 Репутация: 20 Всего: 110 |
||||
|
||||
bsa |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 9185 Регистрация: 6.4.2006 Где: Москва, Россия Репутация: 85 Всего: 196 |
Раздражает необходимость качать и распаковывать файл.
посмотрел. дикое нарушение всех правил форматирования и оформления кода (один из вариантов тут). Дальше смотреть не стал. |
|||
|
||||
newbee |
|
|||
![]() Бревно ![]() ![]() Профиль Группа: Участник Сообщений: 703 Регистрация: 24.8.2011 Репутация: 3 Всего: 19 |
Плюсы наверное
![]() ТС. 1. Форматирование кривое. То есть оно в разных местах разное. 2. gender = 1; Пол лучше зафиксировать константами. 3. STUDENT(STUDENT & s) и STUDENT(STUDENT * s) можно выразить один через другой. 4. STUDENT::getExam сделай массив из трех результатов экзаменов. 5. STUDENT::getDate сделай getAge, getBirthday и getReceipt. Первое лучше вычислять через второе, как текущее_время - сохраненное_время (в качестве текущего_вреемени можно использовать и любое другое, если хочется узнать возраст человека, скажем, три года назад) 6. Какой ООП ты просишь посмотреть, там в коде просто пара структур. -------------------- You're face to face With man who sold the world |
|||
|
||||
bsa |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 9185 Регистрация: 6.4.2006 Где: Москва, Россия Репутация: 85 Всего: 196 |
Ты хотела сказать перечислением (enum)?
|
|||
|
||||
newbee |
|
|||
![]() Бревно ![]() ![]() Профиль Группа: Участник Сообщений: 703 Регистрация: 24.8.2011 Репутация: 3 Всего: 19 |
Есть множество способов введения констант. Пусть будет перечисление.
-------------------- You're face to face With man who sold the world |
|||
|
||||
bsa |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 9185 Регистрация: 6.4.2006 Где: Москва, Россия Репутация: 85 Всего: 196 |
Есть множество. И в С разницы практически нет между #define и enum. А вот в С++ есть. А просто константа... Нужно сильно погеморроиться, чтобы не допустить присваивания левого значения... |
|||
|
||||
ferz721 |
|
||||||
Шустрый ![]() Профиль Группа: Участник Сообщений: 60 Регистрация: 14.1.2011 Репутация: нет Всего: нет |
сколько считаеться кода и файлов много при котором лучше архивировать?
да, похоже их нет. |
||||||
|
|||||||
bsa |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Модератор Сообщений: 9185 Регистрация: 6.4.2006 Где: Москва, Россия Репутация: 85 Всего: 196 |
нисколько. данное действие крайне не рекомендуется. Во-первых, потому что люди не любят грузить всякую гадость. Во-вторых, просто лень. В-третьих, может не быть архиватора (ладно бы зип использовал). В-третьих, просто неудобно. Как ты можешь заметить, скачало только 4 участника форума. А если бы ты выложил текст просто так, то было бы на порядок больше просмотров. Поэтому, выкладываешь только те файлы, которых хочешь чтобы раскритиковали/похвалили. Более того, в большинстве случаев совсем не обязательно выкладывать полностью, достаточно лишь часть. Я сильно сомневаюсь, что народ срочно побежит твой код компилировать... Выкладывать архив следует тогда, когда тебя об этом попросят другие участники. |
|||
|
||||
borisbn |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 4875 Регистрация: 6.2.2010 Где: Ростов-на-Дону Репутация: 21 Всего: 135 |
Вот поэтому и не стал скачивать. Можно было и в сообщение вставить и на LWS выложить... ferz721, тебе bsa уже дважды сказал, что прикреплённый архив - не лучшее решение, а воз и ныне там... -------------------- Женщины отличаются от программистов тем, что у них чары состоят из стрингов |
|||
|
||||
baldina |
|
||||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 3433 Регистрация: 5.12.2007 Где: Москва Репутация: 15 Всего: 101 |
1. STUDENT(STUDENT * s) не нужен если есть STUDENT(STUDENT & s)
2. STUDENT(const STUDENT & s) 3. STUDENT(const STUDENT & s) не нужен, т.к. тривиален. компилятор сам его сделает 4. функции, определенные в классе не нужно объявлять inline, они такие по умолчанию 5. функции, не модифицирующие объект, должны быть объявлены с модификатором const
6. тривиальный (пустой) деструктор не нужен, компилятор сгенерирует его сам 7. signed int short это масло масляное. просто short 8. инициализация в конструкторе делается до тела
приведя класс в порядок мы получим весьма компактный код http://codepad.org/AHHyDYZ7 остальное в том же духе Это сообщение отредактировал(а) baldina - 11.11.2011, 13:48 |
||||
|
|||||
mes |
|
|||
любитель ![]() ![]() ![]() ![]() Профиль Группа: Участник Клуба Сообщений: 7954 Регистрация: 14.1.2006 Репутация: 79 Всего: 250 |
если в том же духе подойти, останутся структуры без всех этих ненужных в данном случае сетеров и гетеров.. да и экзамены не должны жить внутри студента.. |
|||
|
||||
baldina |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 3433 Регистрация: 5.12.2007 Где: Москва Репутация: 15 Всего: 101 |
т.е. от "задачи про ООП" ничего не останется?
![]() Добавлено через 1 минуту и 51 секунду полагаю, если отсутствие нескольких "полезных" конструкторов студент обоснует, это ему будет только в + Добавлено через 5 минут и 15 секунд мне кстати известен случай, когда студент подобным заниматься не стал, минималистский интерфейс объяснил, а на резонный вопрос преподавателя, что же материально и весомое останется для подтверждения знаний языка, студент на коленке набросал иерархию графических полиморфных объектов, попутно пояснив почему квадрат это не прямоугольник. препод был в отпаде. |
|||
|
||||
mes |
|
|||
любитель ![]() ![]() ![]() ![]() Профиль Группа: Участник Клуба Сообщений: 7954 Регистрация: 14.1.2006 Репутация: 79 Всего: 250 |
ну так ООП "разделяй и властвуй", а не "нагромаждай и мучайся".. так что то что получилось за уши к ООП притянуть можно ![]() хотя там и изначально ничего ООПешного не было.. ![]() |
|||
|
||||
baldina |
|
|||
![]() Эксперт ![]() ![]() ![]() ![]() Профиль Группа: Завсегдатай Сообщений: 3433 Регистрация: 5.12.2007 Где: Москва Репутация: 15 Всего: 101 |
||||
|
||||
![]() ![]() ![]() |
Правила форума "C/C++: Для новичков" | |
|
Запрещается! 1. Публиковать ссылки на вскрытые компоненты 2. Обсуждать взлом компонентов и делиться вскрытыми компонентами
Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, JackYF, bsa. |
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей) | |
0 Пользователей: | |
« Предыдущая тема | C/C++: Для новичков | Следующая тема » |
|
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности Powered by Invision Power Board(R) 1.3 © 2003 IPS, Inc. |