forum.boolean.name

forum.boolean.name (http://forum.boolean.name/index.php)
-   Xors3D (http://forum.boolean.name/forumdisplay.php?f=126)
-   -   Написал c# враппер (http://forum.boolean.name/showthread.php?t=15832)

impersonalis 28.12.2011 22:57

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

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

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

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

pozitiffcat 28.12.2011 23:38

Ответ: Написал c# враппер
 
Цитата:

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

Dream 29.12.2011 16:22

Ответ: Написал c# враппер
 
передачи ссылок - актуально

pax 29.12.2011 17:11

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

Dream 29.12.2011 17:49

Ответ: Написал c# враппер
 
Цитата:

Сообщение от pax (Сообщение 215685)
Передача ссылок нужна только не для ссылочных типов.

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

impersonalis 29.12.2011 20:22

Ответ: Написал c# враппер
 
2 pax, SBJoker
Цитата:

Сообщение от pax (Сообщение 215685)
Передача ссылок нужна только не для ссылочных типов.

я вот про это:
Код:

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

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


Часовой пояс GMT +4, время: 10:45.

vBulletin® Version 3.6.5.
Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
Перевод: zCarot