Matrix class - spravnost kodu

[perry]

Zdravím,

začal jsem s C++ (přechod od C#, dříve jsem něco málo kutil v ANSI C). Začal jsem psát malou matematickou "knihovnu" pro použití v mých projektech (přeci jenom, je to základ).

Zajímalo by mě, zda je to dobře napsané a není-li to totální "prasečina", popř. co byste změnili

Matrix4x4.h

Děkuji za názory
_________________
Perry.cz

[if.then]

Pár rychlých postřehů neodborníka (i když jsem si taky už třídu pro matici napsal):
- inline se už tuším nemusí používat, moderní kompilery ho ignorují a inlinují dle libosti
- proč neděláš identity už v konstruktoru?
- kde máš operator[]?
- proč nemáš matici v poli?
- zdá se mi, že do téhle třídy chceš nacpat všechno - ale správný program není hotový tehdy, když není co přidat, ale když není co odebrat
- osobně preferuji mít class a externy v .h a všechny funkce a proměnné v .cpp, které neincluduješ. Tak se vyhneš vícenásobnému nalinkování (což je hrůza ). A taky je asi vhodnější #pragma once místo #ifndef...#define...#endif (je to jednodušší napsat ) - pokud tedy nepotřebuješ např. kontrolovat, zda je dodrženo pořadí includování apod.
_________________
For guns and glory, go to www.ceske-hry.cz.
For work and worry, execute VC++.

[perry]

- Ty inline jsou tam kvůli template... s nimi mi to nejde rozdělit do *.h a *.cpp.. viděl jsem sice řešení s extren keywordem, ale údajně to nepodporuje každý překladač.

- no, ne každou matici potřebuju jako identity ... ale stojí za zvážení, díky
- diky, na ten jsem jaksi zapomnel
- v poli to nemám, protože mi někdo říkal, že použít proměnné je rychlejší pro jednotkový přístup, než sapisovat v random pořadí do pole.. ale težko říct co je na tom pravdy
- nacpat všechno ? Zatím tam není nic, co bych nepotřeboval (kromě možná pár operátorů, ale ty radši píšu předem všechny)
- zvyk z ANSI C, kde #pragma nebylo

Jinak díky za postřehy
_________________
Perry.cz

[frca]

[perry]

Dobře.. přehodím to na pole...
_________________
Perry.cz

[nou]

[if.then]

Pole si vlastně v C můžeš představit jako x elementů přesně po sobě. x[i] je vlastně "*(x+i)". Takže rychlost je asi fakt dost podobná jako u prvků uložených bez pole...

BTW: A není to trošku blbost, dělat si template matici? Vždyť v hrách budeš na 99% používat jen float nebo jen double, proč si přidělávat práci?
_________________
For guns and glory, go to www.ceske-hry.cz.
For work and worry, execute VC++.

[rezna]

ja bych varoval pred #pragma once - je to M$ specific AFAIK ikdyz dneska i GCC to prelozi s nejakym prepinacem - a navic to ma taky sva uskali

[perry]

#pragma region tam je na uzavirani bloku (obdoba #region ze C#), má význam jen ve VC++

Ad templates: ve hrách je to jasný float (možná double), ale raději píšu věci co nejvíc univerzální, než abych je psal pořád dokola . Mohl bych to sice řešit přes #define, ale takhle mi to přijde nějak lepší

[Fila]

[rezna]

a hlavne bych se vyprnul na sabloni - psat nejuniverzalnejsi kod co to jde je super - ale az to opravdu potrebujes - takto si zbytecne do zacatku pridavas tuny balastu, ktery treba ani nikdy nebudes potrebovat

[VladR]

pragma #once je prasacina, to nepouzivaj. Raz by si to omylom mohol pouzit v robote a vyrobis si pruser.

[perry]

Tak tuny balastu mi nevadi (jediné co mi vadí, je ta matlanina, kdy nejde nějak čistě držet *.h a *.cpp strukturu)...
a co jsem četl, tak výkon šablonama netrpí... hlavni cil byl spis je vyzkoušel, protože jsem je pak chtěl aplikovat až budu psát stromy, kde mi přijdou jako podstatná a užitečná věc

Ad pole vs proměnné:
Když bych nechal proměnné a chtěl bych případně poslat matici do shaderu jako raw data, tak tam už výkon ztratím při konverzi na pole ? Nebo můžu vzít matici a poslat její obsah v proměnných do shaderu ve stylu

[VladR]

[VladR]