Sauber Klasse?

ich würde es so programmieren:

foo foo:set (int x, int y)
{
    x_ = x;
    y_ = y;
}

DrGreenthumb

du weißt, dass es den -> operator gibt?

Mit Sicherheit weiß er das, aber (*this). sieht doch viel komplizierter und somit toller aus *troll*

Ich umgehe das Problem indem ich m_ benutze *weitertroll* :p

interpreter

1.) Wenn du schon überall this benutzen willst, dann wenigstens this-> und nicht dieses eklige (*this).
2.) Wieso inline asm? zum Posen?
3.) Naja, is halt ein Vektor, wie man ihn scho 100Mal gesehen hat. Allerdings könnte ich bei deinem Vektor sowas nicht machen:

v = 2.0f * v2;

nix da

dfgdf schrieb:

Mit Sicherheit weiß er das, aber (*this). sieht doch viel komplizierter und somit toller aus *troll*

Ich umgehe das Problem indem ich m_ benutze *weitertroll* :p

Türlich kenn ich den -> operator! Doch ich finde diese schreibweise irgendwie zutreffender. Naja egal.

Die Sache mit dem m_ habe ich mir abgewöhnt, da es sonst wieder nur FlameWars gibt!

war auf der,der fragt bezogen nicht auf nix da

Jester

op+, op* etc gehören nicht in die Klasse. Sollten freie Funktionen sein. Genauso einige andere Funktionen. Zum Beispiel angleWith... warum nicht als frei Funktion mit zwei Argumenten?

rüdiger

Wenn du schon Inline Assembler benutzt, dann kommentier auch was der macht. Das (*this) ist extrem unleserlich und daraus folgt schlecht. Privat kannst du das machen, aber wenn du dich bewerben willst, lass so ein Blödsinn weg.

Dann ist dein Code nicht const-correct und ich würde eine Menge hier inlinen.

Dann solltest du nicht alle Operatoren als Member implementieren (siehe http://www.gotw.ca/gotw/004.htm).

void würde ich an deiner Stelle nicht mehr explizit bei Parametern angeben!

Und du verletzt das OpenClosed Prinzip.

nix da

Jester schrieb:

op+, op* etc gehören nicht in die Klasse. Sollten freie Funktionen sein. Genauso einige andere Funktionen. Zum Beispiel angleWith... warum nicht als frei Funktion mit zwei Argumenten?

weil ich nicht umbedingt immer sowas schreiben wollte:

vector angle (0,0,0;
angle = angle2 (vector1, vector2);

nix da

kingruedi
Danke für die Hinweise, doch da muss ich etwas "nachharken":
1. Was genau ist nicht "const-correct".
2. Womit verstoße ich (neben der Globalen Variable) das OCP?
3. Die Sache mit dem void, was hat es damit auf sich?

naja das mit den Operatoren wusste ich nicht, aber man lernt nie aus und das mit inline kommt noch Danke für den Hinweis

interpreter

nix da schrieb:

kingruedi
Danke für die Hinweise, doch da muss ich etwas "nachharken":
1. Was genau ist nicht "const-correct".

Steht doch in dem Link.

2. Womit verstoße ich (neben der Globalen Variable) das OCP?

Die globale Var. hat damit nichts zu tun. Bei dir is alles public.
[/quote]

3. Die Sache mit dem void, was hat es damit auf sich?

Es ist überflüssig und sieht IMHO hässlich aus.

rüdiger

1. Member Funktionen, die das Objekt nicht verändern solltest du als const deklarieren. Siehe auch den Link

2. Die Variablen sind in der public Schnitstelle. Auch hier solltest du den Link lesen

3. Du kannst einfach void foo() anstelle void foo(void) schreiben, dass ist IMHO leserlicher. void als Parameter ist ein überbleibsel aus alten C Zeiten, da eine leere Parameterliste AFAIK beliebige Parameter bedeutete.

nix da

kingruedi schrieb:

2. Die Variablen sind in der public Schnitstelle. Auch hier solltest du den Link lesen

Naja ich wollte es eigentlich konfortabel halten und nicht immer sowas machen:

float _31 = vector.getX();

Das würde bei einer 4x4 Matrix schon sehr auf die Performance hauen, sofern ich mir das Logisch zusammen wurschtel, vorallem bei Matrizenmulitplikationen. Währe es zu Sachen des "Speeds" nicht logisch das OCP zu "sträken"? Denn ich weis nicht eine get_32() funktion bei einer Matrixklasse fände ich extrem bäh.

zum void: ich finde das bei Syntaxhighlight schöner und erkennlicher, aber jedem das seine, denkst du das ist ein Starkes Kriterium?

DrGreenthumb

kingruedi schrieb:

void als Parameter ist ein überbleibsel aus alten C Zeiten, da eine leere Parameterliste AFAIK beliebige Parameter bedeutete.

Das ist heute immer noch so. Wo da der Sinn ist, würde mich aber interessieren. Oder kommt das einem void func(...); gleich? .oO(geht das eigentlich? hab ich noch nie gesehen...)

@Nix da:
währe?? sträken?? weis ??
Du nix Deutsch

Walli

Also ich finde das inline-Asm für das Kreuzprodukt und den Betrag kannst du dir sparen. Wird nichts an Performance bringen bei so einfachen Berechnungen.

Christoph

nix da schrieb:

float _31 = vector.getX();
Das würde bei einer 4x4 Matrix schon sehr auf die Performance hauen, sofern ich mir das Logisch zusammen wurschtel, vorallem bei Matrizenmulitplikationen.

Nein. getX() ist eine inline-Funktion, daher generiert diese Zeile exakt den gleichen Code wie:

float _31 = vector.x;

boah, ich dachte "nix da" kennt sich aus.

Und dann macht er soviele Fehler.

leicht enttäuscht schrieb:

boah, ich dachte "nix da" kennt sich aus.

Und dann macht er soviele Fehler.

und Deutsch kann er auch nicht

rüdiger

@DrGreenthumb
auch in C++? Hab mir das () mal angewöhnt, weil volkard sonst nie meinen Code lesen wollte

Aber ich weiss nicht wie man auf die Liste zugreifen sollte, da va_start ja den letzten Parameter vor der Liste erwartet.