new fliegt mir um die Ohren

[Anfänger04]

Hallo!

Ich verstehe nicht, warum mir der Debugger ein Access Violation bei 0x00000000 bei diesem Code gibt:

struct Element
{
	unsigned char data[128];

	Element()
	{
		// Wenn ich das weglasse, passt alles
		memcpy(data, 0, sizeof(data));
	}
};

int main()
{
	Element elem; // Alles OK
	new Element(); // BUM ZAK AUTSCH UND AUS
}

Genauer gesagt passiert es beim new- Aufruf in main.
Wenn ich statisch anlege, gibts keine Probleme.
Wenn ich das memcpy im Konstruktor weglasse, gibts keine Probleme.

Aber was stimmt da nicht?

[Anfänger04]

Wenn ich vor und nach dem memcpy ein cout << "egalwas"; mache, dann stürzt das Programm auch beim statischen Anlegen ab!

Was ist da los?

[non~fricky]

Du kannst nicht einfach von Adresse 0 kopieren.

[Anfänger04]

Und noch was Komisches:

Mit ZeroMemory() treten garkeine Probleme auf!

WTF ?!

[Anfänger04@Schlauch]

non~fricky schrieb:

Du kannst nicht einfach von Adresse 0 kopieren.

Hä?

[dooooomi]

Dein memcpy() versucht Daten von Adresse 0 nach data zu kopieren. Adresse 0 "gehört" dir nicht, also gibt es eine Access Violation. Was du eigentlich machen möchtest ist ein memset().

[queer_boy]

memcpy kopiert daten von einem teil des speichers in den anderen. das ziel ist dabei der erste zeiger, die quelle der zweite. in deinem beispiel übergibst du "0" als quelle, d.h. memcpy versuchst, von stelle "0x00000000" daten zu kopieren. dafür muss es den zeiger dorthin dereferenzieren und das dereferenzieren eines null-zeigers ist undefiniert.

wenn du einen speicherbereich mit nullen füllen willst, dann verwende std::fill bzw. std::fill_n .

[Anfänger04@Checker]

Oh ... my ... god ...

memcpy != memset

SORRY BIN ERST AUFGESTANDEN

THX

[Nexus]

In C++ benutzt du statt char -Arrays besser std::string und statt C-Speicherfrickeleien std::fill() , std::copy() und Co.

[pumuckl]

btw: std::fill wäre die C++-variante des memset

[Anfänger04@THX]

Jo schon klar doch muss ich auf Performance achten, es geht um IPC und viele, viele Daten...

[Nexus]

pumuckl schrieb:

btw: std::fill wäre die C++-variante des memset

Stimmt, wurde ja auch erst zwei Mal erwähnt. :p

Anfänger04@THX schrieb:

Jo schon klar doch muss ich auf Performance achten, es geht um IPC und viele, viele Daten...

Okay. Und wie kommst du auf die Idee, memset() wäre performanter? Hast du das mit einem Profiler überprüft? Falls nicht, hüte dich vor vorschnellen Annahmen.

[is klar]

Anfänger04@THX schrieb:

Jo schon klar doch muss ich auf Performance achten, es geht um IPC und viele, viele Daten...

Ja, es ist ein beliebter Anfängerfehler zu meinen, man müsste auf Performance achten und dann die falschen Sachen "optimieren".

[Anfänger04@SRY]

Öh ich meinte die Verwendung von std::string.
Bei ein paar hunderttausend mal 128 Bytes pro Sekunde nehme ich lieber ein unsigned char Array, zuliebe der Nutzer.

[is klar]

Anfänger04@SRY schrieb:

Öh ich meinte die Verwendung von std::string.
Bei ein paar hunderttausend mal 128 Bytes pro Sekunde nehme ich lieber ein unsigned char Array, zuliebe der Nutzer.

Und woher weißt du, dass das langsam ist? Schon mal gemessen? Macht dein Programm fertig und versuche dabei den Code so schön wie möglich zu machen, ohne an Codeoptimierung zu denken, das kann der Compiler sowieso viel besser wie du. Nimm std::vector statt C-Arrays, da gibt es keinen messbaren Zeitunterschied.

[flamer^2]

als du