Re[2]: Прошу код ревью Convex Hull
От: greenpci  
Дата: 11.05.13 14:45
Оценка:
Здравствуйте, Trrrrr, Вы писали:

T>Исправь свой кодстайл, сразуже бросилось в глаза:


T> double compare_vectors(SharedPoint p1, SharedPoint p2, SharedPoint p3);

T>void ProcessPoint(SharedPointList list, SharedPoint p, SharedPoint next_p, bool is_top);
T>Функции то с большой буквы, то маленькие, почем б не назвать CompareVectors?
принимаю. Должен быть CompareVectors.

T>Аналогично с дефайнами, в одном месте

T>#ifndef line_h__
T>#define line_h__

T>в другом

T>#ifndef CONVEX_HULL_H__
T>#define CONVEX_HULL_H__

принимаю. В одном месте ручками писал, в дргом томато использовал.

T>а вообще #pragma once рулит


да, приму во внимание.
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.