ich habe ein kleines Problem und weiß nicht wirklich weiter.
Ich lese verschiedene Dateien ein.
Diese haben eine unterschiedliche Namensgebung aber der Inhalt ist ähnlich.

for (i = 0; i < (int)strlen(strFilenamePrefix.c_str()) - 5; i++) {
      if (strncmp(&strFilenamePrefix[i], ".001.", 5) == 0) {
        strFilenamePrefix[i + 1] = '0';
        strFilenamePrefix[i + 2] = '0';
        strFilenamePrefix[i + 3] = '2';
      }
    }

    strcpy(strFileName1, (iniResultDir +  strFilenamePrefix +   strDateiFileSuffix).c_str());
	  ifstream ifss(strFileName1);

    if(ifss.is_open()){
      printf("file exists");

    }else{
     int j;
	  for (j = 0; j < (int)strlen(strFilenamePrefix.c_str()) - 5; j++) {
		  if (strncmp(&strFilenamePrefix[j], ".002.", 5) == 0) {
			strFilenamePrefix[j + 1] = '0';
			strFilenamePrefix[j + 2] = '0';
			strFilenamePrefix[j + 3] = '1';
		  }
	  }

    strcpy(strFileName1, (iniResultDir +  strFilenamePrefix + strDateiFileSuffixsecond).c_str());
    fp1 = (FILE*)fopen(strFileName1, "r");

    if ( fp1 == (FILE*)NULL )
    {
      sprintf(msg,"\tERROR: File %s doesn't exists...\n", strFileName);
       printf(msg);
       writeLog(msg);
       return false;
    }
    fclose(fp1);
    }

    return true;

Wenn ich diesen code weglasse
bekomme ich keine Fehlermeldung

for (j = 0; j < (int)strlen(strFilenamePrefix.c_str()) - 5; j++) {
if (strncmp(&strFilenamePrefix[j], ".002.", 5) == 0) {
strFilenamePrefix[j + 1] = '0';
strFilenamePrefix[j + 2] = '0';
strFilenamePrefix[j + 3] = '1';
}
}

Aber Warum, dann stimmt der Name der zu suchenden Datei nicht.

Hoffe jemand weiß was.
Danke

Dann ist 1. dein Code viel viel kürzer und 2. viel viel fehlerfreier.

Wenn du sagst Fehlerfreier, wo sind die Fehler ?Ausser das ich c mit c++ mische ?

Das ist der Fehler. Du denkst, das wäre harmlos, ist es aber nicht. Das sind wirklich schwerwiegendste Fehler, die du dadurch machst. Ein std::string ist eben kein nullterminiertes char-Array, noch nicht einmal ein einfacher Wrapper um eines. Du kannst nicht einfach Funktionen darauf loslassen, die blind annehmen, dass es sich um ein nullterminiertes char-Array handeln würde.

Wenn du sagst Fehlerfreier, wo sind die Fehler ?Ausser das ich c mit c++ mische ?

Das ist der Fehler.

zB könnte strncmp(&strFilenamePrefix[i], ".001.", 5) deine Debug Assertion auslösen. Weil man sowas mit std::string einfach nicht machen darf.

bool fileexist(char *filename)
{
    ifstream ifs(filename);

    if(ifs.is_open()){
      return true;

    }else{

    return false;
    }
}
andere Funktione:

strcpy(strFileName, (iniResultDir +  strFilenamePrefix + strDateiFileSuffix).c_str());
    bool erg = fileexist(strFileName);
    if (erg == false)
    {
        sprintf(msg,"\tERROR: File %s doesn't exists...\n", strFileName);
        printf(msg);
        writeLog(msg);
        return false;
    }

  return true;

Das Programm geht schief durch das return true. Ich möchte aber verstehen warum.
Auch wenn ich das Copy ändere und direkt zuweise geht es nicht.

bool fileexist(char *filename)
{
    ifstream ifs(filename);

    if(ifs.is_open()){
      return true;

    }else{

    return false;
    }
}

bool existsAllFiles (string strFilenamePrefix)
{

  char strFileName[MAX_PATH];
  FILE* fp;

    strcpy(strFileName, (iniSceneDir + strFilenamePrefix + strFirstFileSuffix).c_str());
    fp = (FILE*)fopen(strFileName, "r");
    if ( fp == (FILE*)NULL )
    {
        sprintf(msg,"\tERROR: File %s doesn't exists...\n", strFileName);
        printf(msg);
        writeLog(msg);
        return false;
    }
    fclose(fp);
    strcpy(strFileName, (iniSceneDir + strFilenamePrefix + strSecondFileSuffix).c_str());
    fp = (FILE*)fopen(strFileName, "r");
    if ( fp == (FILE*)NULL )
    {
        sprintf(msg,"\tERROR: File %s doesn't exists...\n", strFileName);
        printf(msg);
        writeLog(msg);
        return false;
    }
    fclose(fp);

    string str = strFilenamePrefix;
    string str2 = ".001.";
    str.replace(21,5,str2);
    strFilenamePrefix = str;

    strcpy(strFileName, (iniResultDir +  strFilenamePrefix + strthirdFileSuffix).c_str());

    ifstream ifs(strFileName);

    if(ifs.is_open()){
      printf("file exists");

    }else{

      FILE* create; 
      create = fopen(strFileName,"w"); 

      sprintf(msg,"\tWARNING: Emty File %s is created...\n", strFileName);
	    printf(msg);
	    writeLog(msg);
      fclose(create);
    }

    strcpy(strFileName, (iniResultDir +  strFilenamePrefix + strforthFileSuffix).c_str());
    bool erg = fileexist(strFileName);
    if (erg == false)
    {
        sprintf(msg,"\tERROR: File %s doesn't exists...\n", strFileName);
        printf(msg);
        writeLog(msg);
        return false;
    }

  return true;
}

Ja der Stack kann natürlich zerschossen werden, aber warum.
Das Programm überprüft die verschiedenen Datei ob diese existieren.

Aber bitte bitte bitte, schmeiss den Code weg und mach es ordentlich.
Da hast du zB eine Bibliothek für Filesystem Zugriff: http://www.boost.org/doc/libs/1_50_0/libs/filesystem/doc/index.htm

Und misch bitte bitte bitte nicht C und C++. Da kommt nur blödsinn raus.

ja ihr sagtet das ich ein mischmasch habe. Das ist mir klar.
Aber trotzdem möchte ich dennoch wissen warum es schief geht.

die habe ich auf char msg[255]

Grüße

ja ihr sagtet das ich ein mischmasch habe. Das ist mir klar.
Aber trotzdem möchte ich dennoch wissen warum es schief geht.

Stell dir einen Automechaniker vor. Jemand kommt mit einer selbstgebaute Kiste daher wo 100 millionen unterschiedliche Teile drin sind, sogar der Motor ist irgendwie aus anderen Teilen zusammengebaut.

Und der Automechaniker soll jetzt rausfinden warum das Auto manchmal nicht anspringt.

Die Lösung des Mechanikers ist: kauf dir ein ordentliches Auto.

Selbes hier.

uU ist msg mit 255 Zeichen zu kurz - je nachdem wie lange der Pfadname ist. uU ist writeLog fehlerhaft. Oder der Fehler liegt ganz woanders.

Das was du machen willst funktioniert so halt einfach nicht. Du musst den Code wegschmeissen und es richtig machen. Oder benutz einen Debugger und schau nach was genau den Fehler produziert.

Stell dir einen Automechaniker vor. Jemand kommt mit einem Foto von einer selbstgebaute Kiste daher wo 100 millionen unterschiedliche Teile drin sind, sogar der Motor ist irgendwie aus anderen Teilen zusammengebaut.

FTFY

@Threadersteller: Zum dritten Mal: Wenn du genaue Antworten willst, dann zeig auch den kompletten Code. Zu deinen kurzen Ausschnitten kann man keine genauen Analysen geben.