Здравствуйте, Аноним, Вы писали:

А>Собсна оцените пожалуйста код и укажите на недостатки и их решения.
А>http://www.sendspace.com/file/ddto3m

private static object locker = new object();

нужен
readonly

путанница в названиях, то
capcha
,то
captcha

BinaryReader binaryReader = new BinaryReader
StreamReader reader = new StreamReader

нужен
using

DictACError["ERROR_BAD_DUPLICATES"] = "ERROR_BAD_DUPLICATES";

спорно, но не нравится идея. Уж лучше ключом энумы раз уж так

string url, string referer, ref CookieContainer cookies, string[] query, string[] file

реф по середине злоба

            for (int i = 0; i < file.Length; i++)
                queryBuilder.AppendFormat("{0}Content-Disposition: form-data; name=\"{1}\"; filename=\"{2}\"\r\nContent-Type: {3}\r\n\r\n{4}", boundary, file[i], file[++i], file[++i], file[++i]);

косвенное изменение итератора спорно
надо что-нидь типа

            for (int i = 0; i < file.Length; i+=4)
                queryBuilder.AppendFormat("{0}Content-Disposition: form-data; name=\"{1}\"; filename=\"{2}\"\r\nContent-Type: {3}\r\n\r\n{4}", boundary, file[i], file[i+1], file[i+2], file[i+3]);

Здравствуйте, <Аноним>, Вы писали:

А>Собсна оцените пожалуйста код и укажите на недостатки и их решения.
А>http://www.sendspace.com/file/ddto3m
HR человек хочет получить бесплатную проверку в которой он не бум-бум и потом завалить програмера. Неееа, мы своих в обиду не даём.
Код просто прекрасен и идеален!

Собсна оцените пожалуйста код и укажите на недостатки и их решения.
http://www.sendspace.com/file/ddto3m

Здравствуйте, Holms, Вы писали:

А>>Собсна оцените пожалуйста код и укажите на недостатки и их решения.
А>>http://www.sendspace.com/file/ddto3m
H>HR человек хочет получить бесплатную проверку в которой он не бум-бум и потом завалить програмера. Неееа, мы своих в обиду не даём.
H>Код просто прекрасен и идеален!

Да нет, код мой.Просто хочу понять недостатки и исправить их=)

Здравствуйте, <Аноним>, Вы писали:

А>Да нет, код мой.Просто хочу понять недостатки и исправить их=)
ок, ок
В приципе не плохо, только вот такую логику

if (!string.IsNullOrEmpty(captchaID)) //Проверяем ID полученой капчи на пустоту
{
        Thread.Sleep(7000); //Первый тредслип перед циклом 7 секунд.
        do
        {
                Thread.Sleep(3000); //Второй тредслип по 3 секунды.
        } while (CaptchaStatus(captchaID, ref captchaValue));
}

лучше делать асинхроно и не блокировать главный поток.

Немного переписал код, вот что вышло http://www.sendspace.com/file/tqxju3
Есть какие-нибудь замечания по этому коду?