divyang4481 / mipt-hw

Automatically exported from code.google.com/p/mipt-hw
0 stars 0 forks source link

task13 Список (Ефимов) #107

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
\efimov_andrey\task13_List

Original issue reported on code.google.com by ae.insomniac on 11 Nov 2012 at 7:28

GoogleCodeExporter commented 9 years ago
1. Варнинги:

List.cpp: In member function ‘void TList::Delete(TNode*, TNode*)’:
List.cpp:126:9: warning: statement has no effect [-Wunused-value]
List.cpp: In member function ‘int TList::PopFirst()’:
List.cpp:133:7: warning: unused variable ‘v’ [-Wunused-variable]
List.cpp:136:2: warning: no return statement in function returning non-void 
[-Wreturn-type]
List.cpp: In member function ‘int TList::PopLast()’:
List.cpp:139:7: warning: unused variable ‘v’ [-Wunused-variable]
List.cpp:142:2: warning: no return statement in function returning non-void 
[-Wreturn-type]
List.cpp: In member function ‘TNode* TList::ExtractFirst()’:
List.cpp:152:2: warning: no return statement in function returning non-void 
[-Wreturn-type]
List.cpp: In member function ‘TNode* TList::ExtractLast()’:
List.cpp:155:2: warning: no return statement in function returning non-void 
[-Wreturn-type]

2. Где тесты, проверяющие корректность 
работы класса?

3. Отсутствует реализация для TList::operator==

4. Пишу код:
TList lst;
lst.Pushback(5);
Как я в дальнейшем могу изменить значение 
вставленного элемента?
Если это невозможно в предложенном в 
задаче интерфейсе, предложите и 
аргументируйте свой вариант.

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

Original comment by aivyu...@gmail.com on 25 Nov 2012 at 5:07

GoogleCodeExporter commented 9 years ago
more updates

1, пофиксил
2+3, создал
4, теперь методы вставки, принимающие числа, 
возвращают ссылку на новосозданный TNode.
принимающие TNode остались void'ами, потому что 
если они применяются, ссылка на него уже 
есть у вызвавшего.

Original comment by ae.insomniac on 19 Dec 2012 at 7:05

GoogleCodeExporter commented 9 years ago
whoops

Original comment by ae.insomniac on 19 Dec 2012 at 7:05

GoogleCodeExporter commented 9 years ago
Решение принято.

Original comment by aivyu...@gmail.com on 22 Dec 2012 at 10:21