task_04 geometry

[GoogleCodeExporter]

/baldin_dima/task04_geometry/

Original issue reported on code.google.com by dmitryba...@gmail.com on 1 Apr 2013 at 9:10

[GoogleCodeExporter]

класс TPolygon не реализован

Original comment by dmitryba...@gmail.com on 1 Apr 2013 at 9:11

[GoogleCodeExporter]

Теперь все классы реализованы.

Original comment by dmitryba...@gmail.com on 6 Apr 2013 at 2:38

[GoogleCodeExporter]

0. Warnings:
TGeometry.cpp: In function ‘int main()’:
TGeometry.cpp:330: warning: unused variable ‘cd’

1. Конструкцию вида:
  if ((x == p.GetX()) && (y == p.GetY()))
  return true;
  else return false;
можно упростить до:
  return (x == p.GetX()) && (y == p.GetY());

2. Если в какую-либо функцию всегда 
приходится передавать *this, то наверное 
логично эту фукнцию сделать методом 
соответствующего класса :)

3. В тестах Вы создаете объекты в куче. Потом 
не удаляете. Не вижу вообще ни одного delete в 
файле :) От этого утечки. Либо создавайте 
объекты на стеке (что наиболее логично), 
либо удаляйте.

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

Original comment by aivyu...@gmail.com on 6 Apr 2013 at 9:39

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Original comment by dmitryba...@gmail.com on 18 May 2013 at 5:46

• Changed state: Fixed

[GoogleCodeExporter]

1.  Все таки как Вы любите писать... кхм... 
странный код :). Например:
  if (Area(s.GetBeg(), s.GetEnd(), *this))                                    
                         return true;                                                                                                 
                 else return false;
Можно написать так:
  return Area(s.GetBeg(), s.GetEnd(), *this);

2. В TPolygon::Intersects непонятно зачем создаются 
экземпляры в куче. И потом не удаляются. 
Что, конечно, приводит к утечкам памяти. Это 
грубая ошибка.

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

Original comment by aivyu...@gmail.com on 19 May 2013 at 10:34

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Не заметил, когда исправлял. Закоммитил

Original comment by dmitryba...@gmail.com on 19 May 2013 at 11:25

• Changed state: Fixed

[GoogleCodeExporter]

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

Original comment by aivyu...@gmail.com on 19 May 2013 at 11:39

• Changed state: Done