Собсна оцените пожалуйста код и укажите на недостатки и их решения.
http://www.sendspace.com/file/ddto3m
Здравствуйте, <Аноним>, Вы писали:
А>Собсна оцените пожалуйста код и укажите на недостатки и их решения.
А>http://www.sendspace.com/file/ddto3m
HR человек хочет получить бесплатную проверку в которой он не бум-бум и потом завалить програмера. Неееа, мы своих в обиду не даём.
Код просто прекрасен и идеален!
... << RSDN@Home 1.2.0 alpha 4 rev. 1476>>
Здравствуйте, 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));
}
лучше делать асинхроно и не блокировать главный поток.
... << RSDN@Home 1.2.0 alpha 4 rev. 1476>>
Здравствуйте, Аноним, Вы писали:
А>Собсна оцените пожалуйста код и укажите на недостатки и их решения.
А>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/tqxju3
Есть какие-нибудь замечания по этому коду?
