C / C++ / MFC

shaktikanta wrote:

I think the first one is faster.

ptse wrote:

Statement 2 should be faster.

_tcscpy(szDirpath,_T("C:\\BMPread_W\\TransitionEffects\\Images"));

FileCountInDir(szDirpath,&iFileCountInFolder,&iFileSize);

FileCountInDir(_T("C:\\BMPread_W\\TransitionEffects\\Images"), &iFileCountInFolder, &iFileSize);

for(i =0;i 0)|| ((*filesize) < _tcslen(ffd.cFileName)))

FindFilesInDirectory(szDirpath, Filelist);

void FileCountInDir(const TCHAR* dirpath, int * filescount, int* filesize);

char** CreateMemory(int rows, int stringsize) {
    int i;
    char** p;

    p = (char**) malloc (rows * sizeof (char*));
    if(p != NULL){
        for (i = 0; i < rows; i++){
            p[i] = (char*) malloc(stringsize * sizeof(char));
        }
    };
    return (p);
};

void FreeMemory(char** myArray, int row) {
    int i;
    if(myArray != NULL){
        for (i = 0; i < row; i++){
            free (myArray[i]);
        }
    };
    free(myArray);   // you forgot this
};

char** q;

q = CreateMemory(10, 256);  // create 10 rows each 256 chars
strcpy(q[0], "row 1\0");  // fill row 1
strcpy(q[9], "row 10\0");  // fill row 10
FreeMemory(q, 10); // release all the memory

Filelist[i] = (char*)malloc((iFileSize * sizeof(char))+1);

if (FileList[i] != 0) free (Filelist[i]);

free (myArray[i]);

1. Any other is just a variation of the above, which are all basically the same. The only 'difference' is the visual representation.
2. Hexadecimal is best as it makes it obvious what each bit represents, and is easy to visualise when flags are used in combination.
3. Only define a zero value if it has any specific meaning.