Einzelnen Beitrag anzeigen
Ungelesen 30.06.10, 21:55   #12
kramido
Erfahrener Newbie
 
Registriert seit: Apr 2010
Beiträge: 172
Bedankt: 31
kramido ist noch neu hier! | 0 Respekt Punkte
Standard

Guter Stil in C(++): Initialisierung von Zeigern immer mit NULL um Fehlern vorzubeugen.

Code:
	
DIR *dp = NULL;
struct dirent *ep = NULL;
Wieso fragst du auf dp gleich NULL ab, brichst dann das Programm ab und fragst danach noch auf ungleich NULL ab? Ergibt keinen Sinn! Die zweite Bedingung ist immer erfüllt, sofern sie geprüft wird!

Code:
	if (dp == NULL) {
		perror ("opendir");
		exit(-1);
	}

		
	if (dp != NULL){
Außerdem könnte man als Stilhilfe noch angeben, dass man Konstanten stets auf die linke Seite des Vergleichs stellen sollte, damit keinen versehentliche Zuweisung geschieht. Sprich:
Code:
if(NULL != dp){
kramido ist offline   Mit Zitat antworten