task01_vector Жилинская

[GoogleCodeExporter]

/zhilinskaya_valentina/task01_vector/

Original issue reported on code.google.com by zhilinskaya.valentina on 11 May 2013 at 5:34

[GoogleCodeExporter]

Напоминаю: в этом году нужно ВСЕМ (а не по 
вариантам) сделать и vector, и list.

Original comment by aivyu...@gmail.com on 11 May 2013 at 5:41

[GoogleCodeExporter]

0. Что-то я не пойму: у Вас 2 файла main.cpp и 
TVector.cpp.
В обоих описан шаблонный класс TVector. Куда 
смотреть и что вообще собирать.

1. Пробую собрать только main.cpp, получаю куча 
предупреждений компилятора:
yuliy@yuliy-osx:~/programming/mipt-hw/zhilinskaya_valentina/task01_vector > g++ 
-o ./x -Wall main.cpp 
main.cpp: In member function ‘void TVector<T>::push_back(const T&) [with T = 
int]’:
main.cpp:372:   instantiated from here
main.cpp:333: warning: comparison between signed and unsigned integer 
expressions
main.cpp: In member function ‘void TVector<T>::pop_back() [with T = int]’:
main.cpp:377:   instantiated from here
main.cpp:353: warning: comparison between signed and unsigned integer 
expressions
main.cpp: In member function ‘void TVector<T>::resize(int, T) [with T = 
int]’:
main.cpp:381:   instantiated from here
main.cpp:229: warning: comparison between signed and unsigned integer 
expressions
main.cpp:231: warning: comparison between signed and unsigned integer 
expressions
main.cpp:234: warning: comparison between signed and unsigned integer 
expressions
main.cpp:242: warning: comparison between signed and unsigned integer 
expressions
main.cpp:243: warning: comparison between signed and unsigned integer 
expressions
main.cpp:250: warning: comparison between signed and unsigned integer 
expressions
main.cpp: In member function ‘void TVector<T>::reserve(int) [with T = int]’:
main.cpp:383:   instantiated from here
main.cpp:263: warning: comparison between signed and unsigned integer 
expressions
main.cpp:267: warning: comparison between signed and unsigned integer 
expressions

2. Мало тестов. В том числе не вижу тестов на 
утечки памяти.

Original comment by aivyu...@gmail.com on 11 May 2013 at 5:45

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Кстати, Вы уже создавали issue по этой задаче. 
http://code.google.com/p/mipt-hw/issues/detail?id=184&colspec=ID%20Status%20Prio
rity%20Reporter%20Owner%20Summary

Закройте один из тасков. Смысл тасков в том, 
чтобы все обсуждения по задаче были в одном 
месте.

Original comment by aivyu...@gmail.com on 11 May 2013 at 5:47

[GoogleCodeExporter]

Я запуталась :D 
У меня не получается удалить два 
предыдущих(лишних) таска

Original comment by zhilinskaya.valentina on 11 May 2013 at 7:55

• Changed state: Fixed

[GoogleCodeExporter]

1. Тест на утечки нужно делать для ВСЕХ 
возможных сценариев. У Вас просто 
создается и удаляется вектор, к тому же 
пустой. А нужно поработать с этим вектором: 
подавлят/поудалять элемента, копировать и 
т.п.
Если после этого Created и Deleted совпали, можно с 
некоторой уверенностью говорить, что 
утечек нет.

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

Original comment by aivyu...@gmail.com on 19 May 2013 at 12:01

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Original comment by zhilinskaya.valentina on 19 May 2013 at 5:26

• Changed state: Fixed

[GoogleCodeExporter]

0. Warnings:
TVector.cpp: In member function ‘void TVector<T>::clear() [with T = leak]’:
TVector.cpp:344:   instantiated from here
TVector.cpp:67: warning: allocating zero-element array

1. Конст версии методов front/back должны 
возвращать конст ссылку. Иначе это иллюзия 
константности: отдаем ссылку, используя 
которую можно все сломать.

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

Задача сдана через 2 месяца после дедлайна. 
Остаются позорные 20%.

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

• Changed state: Done