Извините, ничего не найдено.

impersonalis · 28.12.2011, 22:57

тред по диагонали читал, ибо
ЧТО Ж ВЫ ДЕЛАИТИ!!
1) Реализация функции-члена сразу после объявления в классе? Нет пути - юзай inline для подстановки.
2) Передача аргумента не по ссылке в реализации перегрузки копирования(рекурсия) /сравнения ? И где уже const-ы для методов и некоторых их аргументов?
3) Компоненты вектора в паблике?! Если у вас перегрузка сделана как функция поддержки, то объявите её дружественной классу - не уродуйте интерфейс!

4?) Заодно я не очень понял преимущество делать столько статических функций-членов. Большая часть из них вполне логически могла бы быть нестатическими.
Прошу прощения, если не прав и кого-то обидел.

А идея, наверно, неплохая - да.

апд:

только не говорите, что вы обсуждали C#-код, и C++ные придирки неактуальны

pozitiffcat · 28.12.2011, 23:38

апд: только не говорите, что вы обсуждали C#-код, и C++ные придирки неактуальны

да не впринципе ты в чем то прав, писать нужно грамотно, но нет смысла рефракторить ибо мертво
ЗЫ. а разве в шарпе есть inline и оно актуально?

Dream · 29.12.2011, 16:22

передачи ссылок - актуально

pax · 29.12.2011, 17:11

Передача ссылок нужна только не для ссылочных типов.

Dream · 29.12.2011, 17:49

Сообщение от pax

Передача ссылок нужна только не для ссылочных типов.

эээ... я думал все и так поймут

impersonalis · 29.12.2011, 20:22

2 pax, SBJoker

Сообщение от pax

Передача ссылок нужна только не для ссылочных типов.

я вот про это:

 Vector3 operator +(Vector3 one, Vector3 other)

не наю как на 4p (если не так - прошу извинить, и, по возможности, осветить механизм /можно в ЛС/),а на 2p, актуально (судя по телу, которое приведно)

Vector3 &operator +(const Vector3 &one,const Vector3 &other)