0 / 0 / 0
Регистрация: 07.12.2009
Сообщений: 18
|
|||||||||||||||||||||||||||||||
1 | |||||||||||||||||||||||||||||||
Кольцевой односвязный список05.12.2011, 22:42. Просмотров 2466. Ответов 5
Метки нет Все метки)
(
Реализовал класс для работы со списком студентов( использовал кольцевой односвязный список). Для навигации по классу используется итератор. Со временем данный класс будет сделан шаблонным.
Код рабочий, но хотелось бы услышать Ваши замечания, может что улучшить можно. Stud.h
list.h
iterator.h
Ну и напоследок файл test.cpp
0
|
|
05.12.2011, 22:42 | |
Односвязный кольцевой список Кольцевой односвязный список
Кольцевой односвязный список |
|
1492 / 1138 / 165
Регистрация: 05.12.2011
Сообщений: 2,279
|
|
05.12.2011, 23:02 | 2 |
int operator== (Student &);
int operator!= (Student &); int operator>= (Student &); int operator<= (Student &); int operator> (Student &); int operator< (Student &); операторы должны быть константными и принимать константные ссылки на студентов. Если есть возможность один оператор реализовать через другой, то наверно стоит этим воспользоваться. Например для реализации != можно воспользоваться уже имеющейся реализации ==: return !(lhs == rhs). C другими операторами прожно проделать то же самое. Почему вместо std::string используются указатели на чары? На это есть причина или просто по не знанию? Добавлено через 15 минут List(const INF &); - сомнительный конструктор. зачем такой нужен? bool empty(); - константная ф-ия int delElem(INF &); - сомнительно. Удалять лучше по итератору. Не очевидно, что тут удалится. Первый студент или все такие студенты. int searchElement(INF &); - константная ф-ия, возвращающая bool и принимающая константного студента. В стл такие ф-ии возвращают итераторы. то, что вернулось из ф-ии проверяется на равеноство с end итератором Iterator begin() Iterator end() // без константных версий ф-ии в некоторых контекстах список будет сложно или невозможно использовать. Нужны константные версии ф-ий и константные итераторы. template<typename T, typename E> - имхо лучше без второго аргумента делать, а потребовать, чтобы у типа Т был вложенный тип ValueType. Так выглядит целостнее чтоли. Внутни итератора его можно перетайпдефить для удобства, ну или использовать как есть, т.е. typename T::ValueType Имплемент не смотрел, ибо интерфейсы сильно хромают.
0
|
0 / 0 / 0
Регистрация: 07.12.2009
Сообщений: 18
|
|
05.12.2011, 23:06 [ТС] | 3 |
Насчёт константных операторов сравнения учту, спасибо. В условии к лабе указано что именно такие операторы сравнения должны присутствовать, тоесть особо нет смысла реализовать один через другой. Использование char* обусловлено тем, что для данного списка студентов будет разработан GUI.
0
|
1492 / 1138 / 165
Регистрация: 05.12.2011
Сообщений: 2,279
|
|
05.12.2011, 23:12 | 5 |
ну и что, что будет GUI. если нужны именно указатели на чары, то у std::string есть для этого ф-ия c_str. со стрингами будет меньше мороки и ошибок с управлением памяти. У вас уже сейчас в классе студента заложен баг из-за отсутствия своего а не дефолтного конструктора копирования. Дефолтный вам не подойдет. Кстити, у списка тоже нет конструктора копирования. Из-за этого будут проблемы.
На счет операторов: не соблюдение правил их написания может привести к ошибкам компиляции. например Student GetSomeMagicStudent(); Student student; ... if (student == GetSomeMagicStudent()) ... Не везде это скомпилится, потому что в оператор уйдет неконстантная ссылка на временный объект, что не разрешается.
1
|
Тематические курсы и обучение профессиям онлайн Профессия Разработчик на C++ (Skillbox) Архитектор ПО (Skillbox) Профессия Тестировщик (Skillbox) |
0 / 0 / 0
Регистрация: 07.12.2009
Сообщений: 18
|
|
05.12.2011, 23:53 [ТС] | 6 |
Ага, конструктор и вправду сомнительный, хотя это и есть констуктор копирования, точнее должен был им быть
![]() Функция int delElem(INF &) удаляет конкретный элемент списка, удаляется только один такой студент, хотя с несколькими одинаковыми не пробовал. Здесь итератор используется исключительно для навигации по списку, тоесть поиск впринципе можно прикрутить, а вот удаление я оставлю классу List. И, наверное таки нужно будет убрать Iterator end(). Всё таки список кольцевой а не линейный.
0
|
05.12.2011, 23:53 | |
Заказываю контрольные, курсовые, дипломные и любые другие студенческие работы здесь или здесь. Кольцевой односвязный список, сортировка Односвязный кольцевой список, реализовать Кольцевой односвязный список, удалить из него все отрицательные числа Сформировать односвязный кольцевой линейный список по файлу целых чисел Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |