![]() |
Многопоточный чеккер прокси под VK своими руками. Delphi
потребовалось, потому постарался по возможности создать "fine example of source code"
Уверен новичкам пригодится. Требуется: кинуть на форму Button1, Edit1, Label1 создать событие Form1.OnCreate затем полностью заменить данным кодом код модуля Unit1. ;) Код:
unit Unit1;В файл good.txt пишутся хорошие прокси. Пример может быть легко "переоборудован" практически в любую другую многопоточную программу по отсылке\приему\обработке http запросов. Если вы покажете изменения, которые, будучи внесены сделают его _реально_ более эффективным - велкам. |
в этом посте будет содержаться тот же самый код, только для консольной версии.
|
1. для класса лучше создавать отдельный модуль
2. есть WSAStartup а где WsaCleanUp? 3. дожидаться завершения потоков с помощью sleep в корне не верно. Статик поле - счетчик живых потоков. Либо пробегаться по массиву и проверять свойство Alive (я в дельфи не помню, в .NET'e это IsAlive) 4. for counter_1:=0 to threads-1 do MyThread[counter_1] := TMyThread.Create( True ); for counter_1:=0 to threads-1 do MyThread[counter_1].FreeOnTerminate := True; for counter_1:=0 to threads-1 do MyThread[counter_1].Priority := tpLower; эмм... а не проще ли создать объект, изменить свойства и добавить в массив? или сделать это в конструкторе? это навскидку, не углублялся. То что сразу бросилось в глаза |
сорцы хоть скинь, полные
|
AquaKlaster
сорцы полные. Программа состоит всего из 1 - этого - модуля. W!z@rD для наглядности 1-им модулем - лучше. WsaCleanUp - добавил, сделал вызов стартапа и клинапа в программе всего по 1 разу Конструктор - добавил счетчик живых потоков - добавил Буду рад комментам по увеличению эффкетивности (если такое возможно). |
>>Конструктор.. не проще, но прикольнее.
O_o WsaStartup/WsaCleanUp не дергай каждый раз, в каждом потоке. Startup перед запуском потока, CleanUp при завершении последнего потока. Смотри конструкторы/деструкторы http://www.rsdn.ru/forum/delphi/2927396.flat.aspx - рассматривается вопрос о том как определить работает поток или нет:? P.S. я на канале, помогу если не занят буду |
W!z@rD - спс за замечания)
ты прав, с конструктором веселее. И с WsaStartup тоже поправил уже, пока редактировал. Теперь вызываю его всего 1 раз, и 1 раз клин-ап. Сейчас встрою контроль по счетчику живых потоков и будет вообще красота >< |
не используй так называемые магические цифры...
"20000" константу делай из этого. MAX_BUF_LEN 'GET /index.php HTTP/1.1'+ #13#10 + 'Host: vkontakte.ru'+ #13#10#13#10; аналогично 'GET /index.php HTTP/1.1'#13#10'Host: vkontakte.ru'#13#10#13#10; не проще ли передавать прокси и учетку в поток? 1 поток - 1 проверка addr.sin_family:=AF_Inet; addr.sin_port:=htons(strtoint(port)); addr.sin_addr.S_addr:=inet_addr(PChar(ip)); with? (мне не хватает этого в дотнете) cs2.Enter; append(good); writeln(good,ip+':'+port); closefile(good); cs2.Leave; ппц, не пиши так =) закрытие файлов и освобождение ресурсов делай в блоке try...finally, там же можно сделать обработку исключений. "Set-Cookie", а "set-cookie" прийти не может? мб есть смысл переводить все в нижний регистр? форма будет висеть до тех пор пока не кончатся прокси? |
Цитата:
|
константу объявил, пусть будет - все удобнее.
'GET /index.php HTTP/1.1'+ #13#10 + 'Host: vkontakte.ru'+ #13#10#13#10; так и задумано. Если у юзера в запросе не 2 а 7,8 или даже 10 строк - очень удобно писать именно в таком формате. Цитата:
with : +3^4символьных слова в данном случае смотрятся лучше, чем +5 слов, образующие 3 дополнительные строки. Хотя вообще, согласен, удобная вещь. открытие\закрытие файлов и освобождение крит-секций в try...finally добавил. Обработку ошибок на connect (сразу говорю) считаю излишней, т.к. в случае, если коннекта нет, то никаких действий не просиходит и никаких "нештатных" ситуаций тоже. Означает же это что либо проксик совсем мертвый, либо что у юзера нет инета. В первом случае все норм, во втором - проблемы индейцев шерифа не волнуют. На WsaStartup тоже обработку ошибок не делал - лишний код. Если WsaStartup успешно, то все будет работать, а если нет, то юзер и сам догадается, что пора поставить XP вместо win-95 Set-Cookie - там приходит именно так. В нижнем регистре не приходит. Да и не суть важно за какую фразу "цепляться", главное чтобы она была в 100% нужных ответов и в минимуме (>>0) ненужных. Форма будет висеть пока её не закроют крестиком :) Если юзер снова нажмет кнопку - надпись на ней снова сменится на "Proceccing..." и программа безбажно отработает цикл еще раз. При закрытии формы - закрывается файл proxy и освобождаются крит-секции. Если даже завершение пройдет некорректно, это все равно не страшно. Контроль за фактом завершения программы по количеству незавершенных потоков (равному 0) добавил. Теперь программа пишет Ready только после того, как завершатся все созданные ею потоки. |
Цитата:
мне одному так кажется? ErrorNeo >>не проще ли передавать прокси и учетку в поток? 1 поток - 1 проверка у тебя поток сам выдергивает из списка прокси, не проще потоку прокси выдавать? и вынеси класс - поток в отдельный модуль. Намного проще читается |
Цитата:
Цитата:
|
W!z@rD насчет передачи данных в потоки - понял.
не знаю, насколько это будет эффективнее при значительных нагрузках, попробую - создам перегруженный конструктор с параметрами для передачи прокси в поток. Думаю выигрыш в скорости за счет полного отказа от 1 из крит-секций будет. Выносить в отдельный модуль не буду, у меня в 2 прогах он разбит на 2 модуля - и все равно мне такие небольшие коды удобнее читать одним модулем. ред. хотя нет. при 1000+ прокси вопрос будет в том, что быстрее - 1000+ раз выдернуть значение из файла с помощью крит-секции(или, если уж делать по-нормальному то не из файла а из массива памяти, куда все значения уже были предварительно считаны) или же создать\удалить 10к потоков. Я думаю первый метод намного быстрее. При этом +-5 секунд в случае, если требуется сделать <1000 Запросов - не важны. А вот если этот код будет переписан под что-либо "тяжеловесное", делающее 10-ки тысяч запросов и обрабатывающее их (одновременно в ~500 потоков) - тут думаю рациональнее таки выдирать данные из массива памяти. |
добавил "правильное" закрытие программы (в случае, если юзер решит закрыть её крестиком посреди процесса чека прокси):
при попытке закрытия программы она предварительно завершает все потоки, и только после этого закрывается сама. Кроме того сделал чтобы программа больше не "висела" ни на одном из этапов работы, добавив так не хватавший в этом коде application.processmessages() |
Цитата:
|
хм... чтото у меня даный код не рабоатет.. зделал..все.. запускаюю.. вылетает ошибка..
|
Цитата:
|
- почему socks4/5 не чекаются?
- нужен таймаут на Connect на recv и send можно выставить таймауты таким образом: iout := timeout * 1000; setsockopt(sk,SOL_SOCKET,SO_RCVTIMEO,@iout,4); // timeout recv iout/1000 sec setsockopt(sk,SOL_SOCKET,SO_SNDTIMEO,@iout,4); // timeout send iout/1000 sec |
вот ошибка
Цитата:
|
mazaxaka
[Warning]'и не мешают коду компилироваться. кроме того ты не указал строку, на которую ругается компилятор. у меня все компилируется, более того эту прогу юзает "по назначению" мой друг. Hellsp@wn на recv таймаут "реализован", хоть и несколько другим методом. Благодарю за +1 метод, попробую. Юзать как я понял надо: Код:
iout := timeout * 1000;Так же буду благодарен если кто-либо покажет кусочек кода с образцом таймаута для connect. Сокс 4\5 - попробую добавить. |
Програма то компилируется..
файл с прокси есть гуд тоже создал ставлю таймаут жму батом1 и нечего.. а ругается вот на ету Цитата:
Цитата:
|
Цитата:
Код:
LOCAL timeout: timevalКод:
varА имеет ли вообще смысл чекать прокси? Может быть лучше перед началом работы заносить все прокси в список или массив sockaddrin, а во время выполнения основного функционала (например проверка акков на валидность) занулять или удалять из списка нерабочие. |
0rs
ну я не собирался писать "многопоточный спамер по личкам ВК своими руками", потому написал безобидный пример) Кому надо - тот допишет сколько то строк сам. Спс за _примерный_ пример таймаута на коннект. Будет время погуглю как это правильно пишется и добавлю. mazaxaka видимо у тебя дельфи-7 - я писал на дельфи 6. Перенеси переменную counter_1 из глобальных в переменные функции Button1.click, и все заработает. |
Спс за сорцы.
Только один вопрос: почему ты используешь 2 крит. секции? Почему не одну? |
2 крит секции потому что работа идет с 2 файлами.
Не имеет смысла блокировать доступ к файлу good.txt из-за того, что идет работа с proxy.txt. А вот быстродействию кода это заметно повредило бы. |
| Время: 07:48 |