Array-Element Kopieren

[Sven (Gast)]

Hallo,

ich habe ein Problem mit dem Kopieren von Array Werten an eine andere 
Position des Array. Bzw sollen diese einfach getauscht werden (in C++).

Was ich versucht habe, durch den Compiler geht, das Programm aber 
abstürzen lässt:

Arr[Zeile][Spalte + 1] = Arr[Zeile][Spalte];

Arr ist [800][600] groß.

Habt ihr eine Idee, wie ich diesen Tausch anders bewältigen kann?


MfG

[Sven (Gast)]

int temp;

temp=Arr[Zeile][Spalte + 1]
Arr[Zeile][Spalte]=temp;


würde es auch so gehen?

[Sven (Gast)]

Leider bekomme ich in beiden Fällen den Fehler: Segmentation Fault..

[Sebastian R. (sebastian_r569)]

Naja. Hast du beim Compiler alle Warnungen an? Wie äußert sich der 
Absturz?

a = temp;
a = b;
b = temp

Wäre das Tauschen mit Hilfsvariable.

Mein Favorit zum Tauschen zweier Variablen (oder Einträgen im Array) ist 
das hier:

a = a xor b;
b = b xor a;
a = a xor b;

Tauscht komplett ohne Hilfsvariable :D

[DPA (Gast)]

Liegen Zeile und Spalte denn im gültigen Bereich?

Also (Zeile >= 0 && Zeile < 800 && Spalte >= 0 && Spalte < 600-1).

Versuche eventuell ein Minimal beispiel zu erstellen, dass das Problem 
aufweist. Arrayzugriffe sind für sich genommen noch kein Problem.

valgrind und/oder ASAN können bei der Problemsuche auch helfen.
http://valgrind.org/docs/manual/quick-start.html
https://github.com/google/sanitizers/wiki/AddressSanitizer

[Sven (Gast)]

Sebastian R. schrieb:
> Hast du beim Compiler alle Warnungen an? Wie äußert sich der
> Absturz?

Ja, soweit ich das beurteilen kann. Der Absturz Äußert sich wie folgt:
Das Programm ist abgestürzt. Segementation fault.

Sebastian R. schrieb:
> a = temp;
> a = b;
> b = temp

müsste es nicht so sein?

temp= a;
a= b;
b=temp;

?

MfG

[Sven (Gast)]

DPA schrieb:
> Liegen Zeile und Spalte denn im gültigen Bereich?

Ja das ganze läuft in der Schleife:

for (element = 1; element <= Zeile * Spalte; element++)

sorry, das hätte ich noch dazu sagen sollen.

[Rufus Τ. F. (rufus)]

Sven schrieb:
> Der Absturz Äußert sich wie folgt: Das Programm ist abgestürzt.
> Segementation fault.

Würdest Du Dein Programm im Debugger laufen lassen, würde der Dir sagen, 
welcher Programmschritt dazu führt; Du könntest Dir an dieser Stelle 
Deine Variablen ansehen etc.


Du könntest ja mal Deinen Programmcode zeigen, nicht nur eine komplett 
nichtssagende Zeile.

[Sven (Gast)]

Die Frage habe ich auch missverstädnlich gestellt!
Es soll lediglich ein ArrayWert an eine andere Position geschrieben 
werden. z.b. eine Spalte weiter rechts

[Rufus Τ. F. (rufus)]

Sven schrieb:
> for (element = 1; element <= Zeile * Spalte; element++)

Daß in C (und verwandten Sprachen) das erste Arrayelement mit 0 
angesprochen wird, ist Dir irgendwie entgangen?

Wenn Du

char bla[10];

hast, darfst Du die Elemente darin nicht so ansprechen:

for (i = 1; i <= 10; i++)
  machwasmit(bla[i]);

Dann ist es kein Wunder, wenn Dein Programm auf die Schnauze fällt.

Richtig wäre

for (i = 0; i < 10; i++)
  machwasmit(bla[i]);

Das spricht bla[0] bis bla[9] an.

Das fehlerhafte Beispiel spricht bla[1] bis bla[10] an, bla[10] aber 
gibt es gar nicht ...

[Sebastian R. (sebastian_r569)]

Sven schrieb:
>
> müsste es nicht so sein?
>
> temp= a;
> a= b;
> b=temp;
>
> ?
>
> MfG


Äh. Ja. Natürlich. Irgendwas hab ich da verdreht.

[Heinz R. (Gast)]

>>Arr[Zeile][Spalte + 1] = Arr[Zeile][Spalte];

[Spalte + 1] bietet auch noch ne Chance in den letzten 
Schleifendurchgängen über Arraygrenzen hinauszulesen.

[Rufus Τ. F. (rufus)]

Heinz R. schrieb:
> bietet auch noch ne Chance in den letzten Schleifendurchgängen über
> Arraygrenzen hinauszulesen.

Natürlich.

Deswegen soll der Threadstarter ja auch seinen Code zeigen, und nicht 
nur nichtssagende (aber verdächtig riechende) Schnipselchen davon.

[DPA (Gast)]

Ich hoffe ja einfach mal, dass Zeile und Spalte bei:

Sven schrieb:
> Arr[Zeile][Spalte + 1] = Arr[Zeile][Spalte];

Und:

Sven schrieb:
> for (element = 1; element <= Zeile * Spalte; element++)


Nicht die selben Variablen sind.

[Sven (Gast)]

Rufus Τ. F. schrieb:
> Daß in C (und verwandten Sprachen) das erste Arrayelement mit 0
> angesprochen wird, ist Dir irgendwie entgangen?

nein, element hat weder mit der Frage, noch mit der Arrayzuweisung etwas 
zu tun. Es ist lediglich ein counter, der für andere Dinge benötigt 
wird.

DPA schrieb:
> Nicht die selben Variablen sind.

natürlich nicht

[Niklas G. (erlkoenig)]

Tauschen geht in C++ auch einfacher:

std::swap (Arr[Zeile][Spalte + 1], Arr[Zeile][Spalte]);

Das hat aber nichts mit dem Absturz-Problem zu tun. Da ist noch 
irgendwas im Programm falsch.
Füg einfach mal ein

assert (Zeile < std::size(Arr));
assert (Spalte+1 < std::size(Arr[0]));

ein, das hilft den Fehler einzugrenzen.

[Rufus Τ. F. (rufus)]

Sven schrieb:
> nein, element hat weder mit der Frage, noch mit der Arrayzuweisung etwas
> zu tun. Es ist lediglich ein counter, der für andere Dinge benötigt
> wird.

Ach.

Wie schön, daß wir das anhand des von Dir gezeigten Quelltextes 
nachvollziehen können.

Bist Du eigentlich an einer Lösung Deines Problemes interessiert, oder 
welchen Sinn hat dieser Thread hier?