Vektor an Funktion übergeben und verändern

Andreas schrieb:

> void InitVec(TVector* Vector, int a, int b, int c)

Hier bekommst du einen Pointer auf das erste Element im Array.

>    *Vector[0] = a;

Die Arrayindizierung beinhaltet bereits das Dereferenzieren des 
Pointers.

     Vector[0] = a;

Der * Operator dereferenziert dir eine Adresse (Liefert also den Wert 
der an der Adresse steht)
Der [] macht das auch nur in Array Schreibweise. [2] dereferenziert dir 
das 2. Arrayelement ausgehend von der Startadresse (Und deiner Funktion 
wird ja die Startadresse deines Arrays übergeben). Coolerweise kann man 
beide Schreibweisen mischen solange man weiß was man macht ;-)

Peter II schrieb:
> Mach das ganze als Struct

Würd ich nicht machen. Solange die Datentypen all deiner Containerwerte 
gleich sind ist ein Array sicher eine gute Wahl.

L.g.

Halt einen Zeiger auf einen Zeiger übergeben, änderst Du dessen
Inhalt ...

1

void InitVec(TVector **Vector, int a)

2

{

3

   *Vector = a;

4

}

Wenn ich jetzt meine doch ziemlich eingerosteten C Kenntnisse 
zusammenklaube würde ich sagen Stefan Ernsts Variante müsste so korrekt 
sein.

Die eigentlich Frage ist aber warum muss der TO so ein krankes typedef 
machen.
Braucht er wirklich zigmal einen Variablentyp mit 3 Integer (z.B. 
3-dimensionale Koordinate im Raum), warum macht er dann nicht ein struct 
und benennt die einzelnen Variablen sauber.
Ich habe viele Jahre C programmiert, das ist schon einige Jahre her aber 
ich glaube so ein Typedef habe ich noch nie benötigt.

Andreas schrieb:

> Kannst du bitte begründen wieso die Definition bitte nicht sauber sein
> soll?

Weil Arrays in C Bastarde sind, die sich in einigen Bereichen anders 
verhalten als jede anderer Variablentyp. Krass ausgedrückt: Ein Array 
ist selten ein vernünftiger 'Datentyp'. Und genau so, als Datentyp, 
versuchst du hier ein Array zu verwenden. Verwende Arrays als das was 
sie gedacht sind: als einfaches Mittel zur Datenhaltung. Aber nicht um 
damit einen Datentyp aufzubauen. Das ist am Ende praktisch immer mehr 
Arbeit, als wie wenn man gleich eine saubere struct aufbaut.

1

typedef struct

2

{

3

  int x;

4

  int y;

5

  int z;

6

} TVector;

und du hast einen Datentyp der sich sauber verhält

1

void InitVec(TVector* Vector, int a, int b, int c)

2

{

3

   Vector->x = a;

4

   Vector->y = b;

5

   Vector->z = c;

6

}

7

8

TVector AddVec( TVector a, TVector b )

9

{

10

  TVector result = { a.x + b.x, a.y + b.y, a.z + b.z };

11

  return result;

12

}

13

14

int main()

15

{

16

  TVector CircumPoint;

17

  TVector CenterPoint;

18

19

  InitVec( &CenterPoint, 2, 3, 0 );

20

21

  double Radius = 5.0;

22

  TVector Offset = { Radius, 0, 0 };

23

  CircumPoint = AddVec( CenterPoint, Offset );

24

25

  ...

TVector AddVec( TVector a, TVector b )
{
  TVector result = { a.x + b.x, a.y + b.y, a.z + b.z };
  return result;
}

result liegt dann auf dem Stack ...

Karl Heinz Buchegger schrieb:
> typedef struct
> {
>   int x;
>   int y;
>   int z;
> } TVector;
> und du hast einen Datentyp der sich sauber verhältvoid InitVec(TVector* Vector, 
int a, int b, int c)
> {
>    Vector->x = a;
>    Vector->y = b;
>    Vector->z = c;
> }

Dazu war ich zu faul zum Schreiben. Danke Karl Heinz.

Wenn man dem Ding jetzt noch einen sprechenden Namen gibt wie z.B. 
"CubicVektor", dann wüsste man sogar schon am Typ dass es sich um einen 
3-Dimensionalen Vektor (Vektor im geometrischen Sinne) handelt.

Joachim Drechsel schrieb:
> TVector AddVec( TVector a, TVector b )
> {
>   TVector result = { a.x + b.x, a.y + b.y, a.z + b.z };
>   return result;
> }
>
> result liegt dann auf dem Stack ...

macht ja nix.
Lass den Compiler optimieren, wenn der Compiler das kann hilf ihm mit 
einem 'inline' an der richtigen Stelle und das ist genauso effizient, 
wie wenn man das selbst ausschreibt.

Andreas schrieb:
> Diese Lösung hat halt den grossen Vorteil, dass man auf die Elemente
> einfach über Schleifen zugreifen kann. Wenn ich also später eine
> Dimension hinzufüge, dann muss ich an weniger stellen ändern.

Das ist doch eine Milchmädchenrechnung, Ein 4-dimensinaler vektor ist 
ein ganz neuer Datentyp, also sollte man da auch einen neuen definieren.
Mit deinem Array verlierst du jegliche Kontrolle wo jetzt welcher Wert 
drinsteht.
Was nützt dir deine for Schleife wenn du dann 4 statt 3 Werte zum 
initialisieren übergeben musst?

Joachim Drechsel schrieb:
> TVector AddVec( TVector a, TVector b )
> {
>   TVector result = { a.x + b.x, a.y + b.y, a.z + b.z };
>   return result;
> }
>
> result liegt dann auf dem Stack ...

Uuups das hatte ich nicht gesehen, hier sollte man eher dem Vector a den 
Vector b dazuaddieren. Dadurch dass man keinen returnwert definiert und 
Vector b als const deklariert würde das auch deutlich.

Karl Heinz Buchegger schrieb:
> Joachim Drechsel schrieb:
>> TVector AddVec( TVector a, TVector b )
>> {
>>   TVector result = { a.x + b.x, a.y + b.y, a.z + b.z };
>>   return result;
>> }
>>
>> result liegt dann auf dem Stack ...
>
> macht ja nix.
> Lass den Compiler optimieren, wenn der Compiler das kann hilf ihm mit
> einem 'inline' an der richtigen Stelle und das ist genauso effizient,
> wie wenn man das selbst ausschreibt.

Sorry Karl Heinz das verstehe ich jetzt nicht. Result liegt als lokale 
Variable auf dem Stack und wird ungültig wenn die Funktion beendet wird.

Andreas schrieb:
> ...siehst in dieser Defintion auch so einen Stilbruch?

ist schon besser.
Dadurch, dass du das Array in eine struct eingebettet hast, hast du es 
wieder zu einem 'normalen' Datentyp gemacht. Wie gesagt: Das 
Hauptproblem in C besteht darin, dass Arrays sich anders verhalten.

> Diese Lösung hat halt den grossen Vorteil, dass man auf die Elemente
> einfach über Schleifen zugreifen kann. Wenn ich also später eine
> Dimension hinzufüge, dann muss ich an weniger stellen ändern.

:-)
Ganz ehrlich.
Ich arbeite jetzt seit 30 Jahren im Bereich Computergrahik. Und am 
Anfang hab ich auch gedacht: Wenn da noch eine Dimension dazukommt ....
Die Anzahl der Fälle, in denen das tatsächlich notwendig war, konnte ich 
an 2 Fingern abzählen :-)

Andreas schrieb:
> ...siehst in dieser Defintion auch so einen Stilbruch?#define MAX_DIM 3
>
> typedef struct
> {
>   int32_t x[MAX_DIM];
> }TVector;

Das könnte man machen, hat aber den gravierenden Nachteil, daß man nicht 
weiss welche Dimension jetzt dieser Vektor tatsächlich hat. Um irgendwas 
damit tun zu können brauchst du zumindest die aktuelle Dimension.

Ich würde auch in der Struktur die Variablen benennen anstatt nur ein
Array zu verwenden, alleine schon wegen der Lesbarkeit (kann ja sein,
auch andere möchten das mal verwenden oder man schaut sich das nach
einem Jahr mal wieder an).

Udo Schmitt schrieb:

> Sorry Karl Heinz das verstehe ich jetzt nicht. Result liegt als lokale
> Variable auf dem Stack und wird ungültig wenn die Funktion beendet wird.

Wie gesagt: macht ja nix.
Ich gebe ja keinen Pointer darauf als Returnwert heraus

int twice( int j )
{
  int k = 2 * j;
  return k;
}

k liegt auf dem Stack und wird aus der Funktion herausgegeben. Kein 
Problem. Und mit Strukturen funktioniert das genauso.

> Uuups das hatte ich nicht gesehen, hier sollte man eher dem Vector a
> den Vector b dazuaddieren.

Kannst du natürlich machen. Das entspricht dann aber einem += Operator 
und keinem + Operator mehr.

> Dadurch dass man keinen returnwert
> definiert und Vector b als const deklariert

Die Notation, dass man Strukturen auch per Kopie übergeben kann, ist dir 
noch nicht wirklich geläufig, oder?

Karl Heinz Buchegger schrieb:
> Die Notation, dass man Strukturen auch per Kopie übergeben kann, ist dir
> noch nicht wirklich geläufig, oder?

Nein, nicht so. Reden wir jetzt von c oder c++?
Aber wie gesagt, c habe ich in den letzten 6-7 Jahren so gut wie nicht 
mehr benutzt, und c++ vorher nicht viel und dann gar nicht mehr.