Re[7]: Химик, что там сейчас в Дельфе?

K>>type
K>>TVect=record
K>>x,y,z:double;
K>>function VectRad:double;//Длина вектора
K>>function VectRad2:double;//Квадрат длины
K>>function IsNil:boolean;//Равен ли нулю
K>>function MinVect:tvect;//Поменять все знаки
K>>end;

LB>С некоторых пор меня стали сильно удивлять такие имена переменных, функций и классов, для которых нужно тут же писать объясняющий комментарий.
LB>Кто такой VectRad? При чем тут радианы, если речь о длине? Или речь о какой-то радости? MinVect, судя по названию, должен возвращать какой-то минимизированный вектор

LB>Почему бы не сделать так, например:

LB>

LB>TVector = record
LB>  x, y, z: Double;
LB>  function GetLength: Double;
LB>  function GetSquareLength: Double;
LB>  function IsZeroLength: Boolean;
LB>  function InvertAllSigns: TVector;
LB>end;
LB>

LB>И выглядит каноничненько и комментарии тут сто лет не нужны

LB>Лучше дать хорошее имя и не экономить на его длине (никакого смысла в такой экономии давно нет), чем через пару месяцев расшифровывать свои же собственные сокращения.

You are right.
На форуме уже обсуждали, стоит ли переписывать старый дерьмокод, и я пришёл к выводу, что обычной такой код переписывать НЕ следует.

	От:	Khimik
	Дата:	27.01.19 08:09
	Оценка: