POSIX threads

[OverRated]

...........

[MXP]

Why are they all receiving on the same buffer? Why not give each thread its own buffer?

[OverRated]

Even when i Declare Buffer inside the thread it still gives me invalid results. For soem reason buffer gets cut off and only holds the last 1/3 of the page.

[MXP]

Is buffer large enough to hold the whole page? Is there any more data to be read from the socket after the first read?

[OverRated]

Yes buffer is more than large enough to hold the whole page. Just needs to store the page in buffer and search for some text thats all

[MXP]

Is there more data to be read from the socket after you call recv()?

[OverRated]

no not until it loops again and sends different data.

[OverRated]

^ How does that help me O.o im on linux so win32 wont work. Unless i run it through wine but i rather not do that..

[Alvaro]

It's a bot.

[Alvaro]

Of course, sharing a global buffer across threads is not going to work.

This won't answer your question, but this code makes my eyes hurt:

Code: Select all

    for(i = 0; i < strlen(buffer) + 2;i++)

First of all, it looks like you may be reading past the end of the string in some cases. Also, unless the compiler is really smart, it will compute the length of the zero-terminated string in `buffer' every time. Similarly with using `strlen(word)'. Save those lengths in local variables at the beginning of the function, or just use std::string.

Also, that `search' function is messed up in general. Try `search("bababata", "babata")'.

You have to always check the results of system calls like `send' or `recv'. Network programming is very tricky, and all sorts of errors may happen. If you do detect an error, make sure to print out what the problem was, using errno. Sometimes `send()' and `recv()' will only send part of your message, and that's not even an error: It's part of normal operation in TCP. Also part of normal operation is getting an error with errno==EINPROGRESS or perhaps even errno==EAGAIN (with non-blocking sockets). I don't remember all the details, so check the documentation on exactly how it works. I like this book on the subject. If you can afford it or if you can find it in a library, read it.

[OverRated]

Code: Select all

int search(char* buffer,const char* word)
{
    unsigned int i = 0,
        j = 0;
    for(i = 0; i <= strlen(buffer);i++)
    {
        if(buffer[i] == word[j] && j != strlen(word))
            j++;
        else if(j == strlen(word))
            return (1);
        else
            j = 0;
    }
    return (0);
}

Fixed the search function. Ill buy that book probably. thanks.

[Alvaro]

You still didn't fix the multiple calls to strlen for the same string... You could check

Code: Select all

word[j] != '\0'

instead of

Code: Select all

j != strlen(word)

EDIT: And your code still thinks that the string "bababata" does not contain the substring "babata".

You could also just write this:

Code: Select all

#include <cstring>

bool search(const char* buffer, const char* word) {
  return std::strstr(buffer, word)!=0;
}


[jinguo.linux.gnu]

You still didn't fix the multiple calls to strlen for the same string... You could check

Code: Select all

word[j] != '\0'

instead of

Code: Select all

j != strlen(word)

EDIT: And your code still thinks that the string "bababata" does not contain the substring "babata".

You could also just write this:

Code: Select all

#include <cstring>

bool search(const char* buffer, const char* word) {
  return std::strstr(buffer, word)!=0;
}


Everyone on this website,can tell me about the different the POSIX pthread and the Boost thread?
When we programming a project on Linux or Unix which one i will choice?