c++ probs mit zeigern!

Scho langsam dreh i ur durch. Immer Exception Handling, egal ob ich bei der Eingabe *, & oder nix verwend. Vielleicht könnts ihr einen Fehler finden.

Code:

FILE *stream;
char *auswahl = NULL;
char *datei = NULL;
cout << "Assoziatives Array" << endl;
cout << "------------------" << endl;
cout << "Datei eingeben: (Bei Enter test.txt)" << endl;
cin >> *auswahl;
strcpy(datei, "'");
strcat(datei,auswahl);
strcat(datei,"'");
if((stream=fopen(datei,"a+")) == NULL)
{
cout << "NULL";
}
else
{
cout << "NICHT NULL";
}

Plz hlp

Zitat von LakiStrike

char *datei = NULL;
...
cin >> *auswahl;
...

Problem ist, in nem NullPointer kann man nicht wirklich viel Speichern

Abgesehen davon wäre das einlesen dann so:

Code:

char *auswahl = (char *) malloc(1000); //1000 Bytes für den Pointer auswahl reservieren.

cin >> *auswahl;

.....

free(auswahl);

EDIT (Ringding): Sternchen vor auswahl hat gefehlt.

Zitat von Vir@s

EDIT (Ringding): Sternchen vor auswahl hat gefehlt.

Sorry, aber wenn du einen Stern vor auswahl machst schreibst du nur in die aktuelle Pointer location, wodurch nur 1 Zeichen einlesbar ist. Korrekt ist es ohne Stern!

(Abgesehen davon sollte man sowieso kein cin verwenden )

naja cin ist so ne 0815 methode (in meinen Augen, aber über Programmierstil lässt sich ja streiten ).
Bin mehr für char pointer + malloc function => scanf

Zitat von Neo-=IuE=-

scanf hat viele fehler

welche denn?

scanf hat keine Fehler, aber Schwachstellen, mit denen man leicht einen Buffer Overflow produziert (wenn man Strings einliest). Also es stimmt schon, dass man es nicht verwenden soll.

Ich würd sagen es kommt ganz auf das an was du machen willst, im formatiert einlesen ist scanf ungeschlagen, wo sonst kann man so einfach 2 integer einlesen, usw....

Prinzipiell für strings, chars, etc ist gets natürlich besser, funktioniert mit scanf aber auch