task01_TList Савинова

[GoogleCodeExporter]

/savinova_anna/task01_TList/

Original issue reported on code.google.com by anna.sav...@phystech.edu on 20 May 2013 at 7:32

[GoogleCodeExporter]

Вижу 2 файла: TList.cpp и TList2.cpp
Который смотреть?

Original comment by aivyu...@gmail.com on 26 May 2013 at 12:36

[GoogleCodeExporter]

Смотрю файл TList.cpp, т.к. он собирается:

0. iterator::operator++() возвращает константную 
ссылку. Это неудобно, т.к. может быть 
желание ее еще поменять или присвоить 
неконст экземпляру итератора.

1. Не вижу постфиксных версий operator++/--

2. Не реализован iterator::operator->

3. Нелогично реализовывать у итератора 
operator+, т.к. в случае списка он работает за O(n).

4. operator= тоже у Вас возвращает конст ссылку 
на себя. Это неудобно.

5. front/back возвращают почему-то именно int.

Решение не принято.

Original comment by aivyu...@gmail.com on 26 May 2013 at 12:43

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Original comment by anna.sav...@phystech.edu on 26 May 2013 at 4:21

• Changed state: Fixed

[GoogleCodeExporter]

1. А вот постфиксная версия operator++/-- не может 
возвращать ссылку. Т.к. она возвращает 
ссылку на локальную переменную. После 
выхода из метода, этого объекта не 
существует. Что приводит к грубой ошибке и 
проезду по памяти.

Исправьте эту ошибку и задача будет 
принята.

Original comment by aivyu...@gmail.com on 26 May 2013 at 8:02

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

сделано.

Original comment by anna.sav...@phystech.edu on 26 May 2013 at 8:08

• Changed state: Fixed

[GoogleCodeExporter]

Решение принято.

Оценка: 0.2 балла.

Original comment by aivyu...@gmail.com on 26 May 2013 at 8:20

• Changed state: Done