[The High Roller]

http://www.mediafire.com/?ivqgti8wlrb1b9e
This little application will allow you to add friends without doing any work. It'll add friends that should pertain to your interests/friends. You WILL be banned from adding friends if you add too many a day(50-70).

[ねこまっしぐら]

Haha, this is cool. But I think this is kinda odd. So I'm just adding random people? Or will it have mutual friends?

[The High Roller]

Haha, this is cool. But I think this is kinda odd. So I'm just adding random people? Or will it have mutual friends?

Adding random people and selling the account later.

[ねこまっしぐら]

Haha, this is cool. But I think this is kinda odd. So I'm just adding random people? Or will it have mutual friends?

Adding random people and selling the account later.

Ohhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhh. This would have been good for this thread.

[The High Roller]

Haha, this is cool. But I think this is kinda odd. So I'm just adding random people? Or will it have mutual friends?

Adding random people and selling the account later.

Ohhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhh. This would have been good for this thread.

You still have to register an account and your maxed to only 50 friends added per day.

[Bannedshee]

Did you make this? Because I don't see any credits.

[cxS]

Did you make this? Because I don't see any credits.

It's only 2 methods of code, how hard could it have been? :huh:

...

Code:

private void button_Click_1(object sender, EventArgs e)
{
    webBrowser1.ScriptErrorsSuppressed = true;
    webBrowser1.Navigate("https://www.facebook.com/find-friends/browser/");
    webBrowser1.DocumentCompleted += webBrowser1_DocumentCompleted;
}

void webBrowser1_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
{
    for (int i = 0; i < 700; i++)
    {
        if (webBrowser1.Document.All[i].Id != null)
            if (webBrowser1.Document.All[i].Id.StartsWith("u_"))
                webBrowser1.Document.All[i].InvokeMember("click");
    }
    System.Threading.Thread.Sleep(10000);
    webBrowser1.Navigate("https://www.facebook.com/find-friends/browser/");
}

On every button click you are adding the handler to the DocumentCompleted event method? Why...

Your loop:

Code:

for (int i = 0; i < 700; i++)
{
    if (webBrowser1.Document.All[i].Id != null)
        if (webBrowser1.Document.All[i].Id.StartsWith("u_"))
            webBrowser1.Document.All[i].InvokeMember("click");
}

I think it's safe to say that as soon as we find an ID that doesn't exist we won't find another after that, so why not place a break; in here at that time to avoid unnecessary checks to all the rest of the iterations? The issue here is that you're using 700 as a constant value as well hardcoded into the program. Why not make it dynamic? Check the number of values you can find?

I don't use fb though so I can't say. I'm assuming it's in sequential ascending order though for each of the results on that search page.

Code:

System.Threading.Thread.Sleep(10000);
webBrowser1.Navigate("https://www.facebook.com/find-friends/browser/");

Absolute worst thing you can do... Very abusive way of using Thread.Sleep().

[Prestige]

wrong section should be in the .net section C++ and C# are completely different

[cxS]

wrong section should be in the .net section C++ and C# are completely different

Unless you mean VisualC++.

[Bannedshee]

Did you make this? Because I don't see any credits.

It's only 2 methods of code, how hard could it have been? :huh:

...

Code:

private void button_Click_1(object sender, EventArgs e)
{
    webBrowser1.ScriptErrorsSuppressed = true;
    webBrowser1.Navigate("https://www.facebook.com/find-friends/browser/");
    webBrowser1.DocumentCompleted += webBrowser1_DocumentCompleted;
}

void webBrowser1_DocumentCompleted(object sender, WebBrowserDocumentCompletedEventArgs e)
{
    for (int i = 0; i < 700; i++)
    {
        if (webBrowser1.Document.All[i].Id != null)
            if (webBrowser1.Document.All[i].Id.StartsWith("u_"))
                webBrowser1.Document.All[i].InvokeMember("click");
    }
    System.Threading.Thread.Sleep(10000);
    webBrowser1.Navigate("https://www.facebook.com/find-friends/browser/");
}

On every button click you are adding the handler to the DocumentCompleted event method? Why...

Your loop:

Code:

for (int i = 0; i < 700; i++)
{
    if (webBrowser1.Document.All[i].Id != null)
        if (webBrowser1.Document.All[i].Id.StartsWith("u_"))
            webBrowser1.Document.All[i].InvokeMember("click");
}

I think it's safe to say that as soon as we find an ID that doesn't exist we won't find another after that, so why not place a break; in here at that time to avoid unnecessary checks to all the rest of the iterations? The issue here is that you're using 700 as a constant value as well hardcoded into the program. Why not make it dynamic? Check the number of values you can find?

I don't use fb though so I can't say. I'm assuming it's in sequential ascending order though for each of the results on that search page.

Code:

System.Threading.Thread.Sleep(10000);
webBrowser1.Navigate("https://www.facebook.com/find-friends/browser/");

Absolute worst thing you can do... Very abusive way of using Thread.Sleep().

True, but he still had to give credits. Oh well, he's banned now.