Need help with listing files.

Ok Here's my code. It compiles fine, but it don't list me files indirectory.

In list.txt I have "C:\users\rain" but my program shows following output.

Code:

---

C:\Users\rain>D:\Development\Cplusplus\del\del2\Debug\del2.exe
C:\users\rain
File Found: rain

C:\Users\rain>

---

I tried changing "C:\users\rain" to "C:\users\rain\" but then output is like this:

Code:

---

The first file found is ╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠╠
ńŅ▀D..,

---

I can't find out why it shows only File Found: rain. I have many files and folders in "C:\users\rain" and none of them are called "rain"

Code:

---

#include "stdafx.h"

#include <windows.h>
#include <tchar.h>
#include <stdio.h>
#include <string>
#include <iostream>
#include "Shlwapi.h"


using namespace std;


void main()
{
  WIN32_FIND_DATA FindFileData;
  HANDLE hFind;

  //Read list file
  FILE *filein;
  long lSize;
  char * buffer;
  size_t result;
  //Get path to list.txt
            char szFileName[MAX_PATH];
    HINSTANCE hInstance = GetModuleHandle(NULL);
    GetModuleFileName(hInstance, szFileName, MAX_PATH);
        PathRemoveFileSpec(szFileName);
        PathAppend(szFileName,"\\list.txt");
        //Open file
  filein = fopen(szFileName, "r");
  //if (filein==NULL) {fputs ("File error",stderr); exit (1);}

    // obtain file size:
    fseek (filein , 0 , SEEK_END);
    lSize = ftell (filein);
    rewind (filein);

    // allocate memory to contain the whole file:
    buffer = (char*) malloc ((sizeof(char)*lSize)+1);

        // copy the file into the buffer:
    result = fread (buffer,1,lSize,filein);
        buffer[result] = '\0'; // make extra byte nul
        /*//Print some info
    printf ("%s\n",buffer);// list.txt
        printf ("%s\n",szFileName);//Path
    fclose(filein);
        */

        //token
        char *token;
        token = strtok(buffer,"\r\n");
          while( token != NULL )
      {
          // While there are tokens in "string"
          printf( "%s\n", token );
                        //List files
                        hFind = FindFirstFile(token, &FindFileData);
                        do{
                                _tprintf (TEXT("File Found: %s\n"),FindFileData.cFileName);
                        }while(FindNextFile(hFind, &FindFileData));
                        FindClose(hFind);

          // Get next token:
          token = strtok( NULL,"\r\n"); // C4996
      }
}

---

Try it with your error checking code enabled.

Also, show us the content of your input file, or how you generated it.

I'm surprised you didn't complain about void main, Salem! :o
rain-13: Aside from a non-working programs, there are fatal flaws in your code.
The first is void main. It's evil and should never be used: SourceForge.net: Void main - cpwiki
The second is malloc. You must free what you malloc. This isn't some kind of free ticket. If you're going to use C and malloc, you had better well learn to deal with the consequences. Use free in the appropriate spot.
And lastly, indent properly and consistently.

And why in the hell are you using C with C++? Start deciding which language you want to use.

This seems wrong:

Code:

---

token = strtok(buffer,"\r\n");    // \n already expands to whatever is appropriate for your platform

---

@Elysia
I don't see any C++ there, but he's mixing C standard functions with those of Win32 like it's going out of style. :eek:

If you look at the OP's code, there are certain lines that says it's C++:

#include <string>
#include <iostream>

using namespace std;

Quote:

---

Originally Posted by Elysia

If you look at the OP's code, there are certain lines that says it's C++:

#include <string>
#include <iostream>

using namespace std;

---

Oh. Missed those. I just went over the contents of main() and didn't see anything that looked like C++.

I had to add * at the end of path

Quote:

---

Originally Posted by Elysia

I'm surprised you didn't complain about void main, Salem! :o
rain-13: Aside from a non-working programs, there are fatal flaws in your code.
The first is void main. It's evil and should never be used: SourceForge.net: Void main - cpwiki
The second is malloc. You must free what you malloc. This isn't some kind of free ticket. If you're going to use C and malloc, you had better well learn to deal with the consequences. Use free in the appropriate spot.
And lastly, indent properly and consistently.

And why in the hell are you using C with C++? Start deciding which language you want to use.

---

I found codes from internet and put them together into one code, I didn't even know that I had C with C++. Which part i have C++ and which part is C? Is it bad if I have C with C++? If yes what replacements I need to do to get C only?

Before I didn't know about that void main problem.

Does that free(buffer); do what needed or do i need to free something other?

Code:

---

#include "stdafx.h"

#include <windows.h>
#include <tchar.h>
#include <stdio.h>
#include <string>
#include <iostream>
#include "Shlwapi.h"


using namespace std;


int main()
{
  WIN32_FIND_DATA FindFileData;
  HANDLE hFind;

  //Read list file
  FILE *filein;
  long lSize;
  char * buffer;
  size_t result;
  //Get path to list.txt
            char szFileName[MAX_PATH];
    HINSTANCE hInstance = GetModuleHandle(NULL);
    GetModuleFileName(hInstance, szFileName, MAX_PATH);
        PathRemoveFileSpec(szFileName);
        PathAppend(szFileName,"\\list.txt");
        //Open file
  filein = fopen(szFileName, "r");
  //if (filein==NULL) {fputs ("File error",stderr); exit (1);}

    // obtain file size:
    fseek (filein , 0 , SEEK_END);
    lSize = ftell (filein);
    rewind (filein);

    // allocate memory to contain the whole file:
    buffer = (char*) malloc ((sizeof(char)*lSize)+1);

        // copy the file into the buffer:
    result = fread (buffer,1,lSize,filein);
        buffer[result] = '\0'; // make extra byte nul
        /*//Print some info
    printf ("%s\n",buffer);// list.txt
        printf ("%s\n",szFileName);//Path
    fclose(filein);
        */

        //token
        char *token;
        token = strtok(buffer,"\r\n");
          while( token != NULL )
      {
          // While there are tokens in "string"
          printf( "%s\n", token );
                        //List files
                        hFind = FindFirstFile(token, &FindFileData);
                        do{
                                _tprintf (TEXT("File Found: %s\n"),FindFileData.cFileName);
                        }while(FindNextFile(hFind, &FindFileData));
                        FindClose(hFind);

          // Get next token:
          token = strtok( NULL,"\r\n"); // C4996
      }
  free(buffer);
return        0;
}

---

Thanks for hints and tips.

Quote:

---

Originally Posted by rain-13

I found codes from internet and put them together into one code, I didn't even know that I had C with C++. Which part i have C++ and which part is C? Is it bad if I have C with C++? If yes what replacements I need to do to get C only?

---

It is true that you can compile C with C++. Many C++ programmers would scoff at such things, though.
It is also fine if you compile C with a C++ compiler, so long as you know that C++'s stricter type system demands you to do some things that aren't necessary in C.
About what is the C++ stuff: it's those things I quoted earlier. The headers and the using directive. Remove them and you get code that will compile as C.

Quote:

---

Before I didn't know about that void main problem.

---

Now you do ;)

Quote:

---

Does that free(buffer); do what needed or do i need to free something other?

---

That should be enough since it's the only thing you malloc.

How to remove * from path when I print it in console? * is last char of path.

I tried this:

Code:

---

char *temp;
strncpy (temp,token,strlen(token)-1);

---

but for some reason it makes y program crash. any other methods to get rid of last symbol of string?

And how to join temp and FindFileData.cFileName together into one string so I could check wether file is directory?

Code:

---

// del2.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"

#include <windows.h>
#include <tchar.h>
#include <stdio.h>
#include <string>
#include <iostream>
#include "Shlwapi.h"


using namespace std;


int main()
{
  WIN32_FIND_DATA FindFileData;
  HANDLE hFind;

  //Read list file
  FILE *filein;
  long lSize;
  char * buffer;
  size_t result;
  //Get path to list.txt
            char szFileName[MAX_PATH];
    HINSTANCE hInstance = GetModuleHandle(NULL);
    GetModuleFileName(hInstance, szFileName, MAX_PATH);
        PathRemoveFileSpec(szFileName);
        PathAppend(szFileName,"\\list.txt");
        //Open file
  filein = fopen(szFileName, "r");
  //if (filein==NULL) {fputs ("File error",stderr); exit (1);}

    // obtain file size:
    fseek (filein , 0 , SEEK_END);
    lSize = ftell (filein);
    rewind (filein);

    // allocate memory to contain the whole file:
    buffer = (char*) malloc ((sizeof(char)*lSize)+1);

        // copy the file into the buffer:
    result = fread (buffer,1,lSize,filein);
        buffer[result] = '\0'; // make extra byte nul
        /*//Print some info
    printf ("%s\n",buffer);// list.txt
        printf ("%s\n",szFileName);//Path
    fclose(filein);
        */

        //token
        char *token;
        token = strtok(buffer,"\r\n");
          while( token != NULL )
      {
          // While there are tokens in "string"
          printf( "%s\n", token );

                  char *temp;
                  strncpy (temp,token,strlen(token)-1);
                  //printf("%d\n",strlen(token)-1);
                  //List files
                        hFind = FindFirstFile(token, &FindFileData);
                        do{
                                _tprintf (TEXT("File Found: %s%s\n"),temp,FindFileData.cFileName);
                        }while(FindNextFile(hFind, &FindFileData));
                        FindClose(hFind);

          // Get next token:
          token = strtok( NULL,"\r\n"); // C4996
      }
        free(buffer);
return        0;
}

---