ANTICHAT — форум по информационной безопасности, OSINT и технологиям
ANTICHAT — русскоязычное сообщество по безопасности, OSINT и программированию.
Форум ранее работал на доменах antichat.ru, antichat.com и antichat.club,
и теперь снова доступен на новом адресе —
forum.antichat.xyz.
Форум восстановлен и продолжает развитие: доступны архивные темы, добавляются новые обсуждения и материалы.
⚠️ Старые аккаунты восстановить невозможно — необходимо зарегистрироваться заново.
 |
мой первый VK Checker - просьба критиковать код |
30.08.2009, 12:30
|
|
Познавший АНТИЧАТ
Регистрация: 16.04.2006
Сообщений: 1,488
Провел на форуме:
2209675
Репутация:
537
|
|
мой первый VK Checker - просьба критиковать код
http://www.sendspace.com/file/lly73w
написал чекер для ВК на C++
принимает список кукис cookie.txt, чекает их, битые переносит в invalid.txt
выкладываю с сорцами, надеясь что найдётся добрый человек, который покритикует код и укажет что можно улучшить. за хорошую помощь в улучшении кода готов заплатить
icq 8 6776690
остальные тестируйте на здоровье  |
|
|
30.08.2009, 13:54
|
|
Постоянный
Регистрация: 24.03.2009
Сообщений: 670
Провел на форуме:
2868783
Репутация:
414
|
|
Бегло просмотрев твой код нашел море недочетов )
1) переполнение буфера.
2) один или два лишних циклов.
3) вместо массива строк в стеке (из-за которого кстати и переполнение в ф-ции file()) лучше юзать вектор или иной контейнер.
4) зачем реализацию выносить в хэдэр, они не для этого созданы.
5) от глобальных переменных можно тоже отказаться.
6) ну можно было хоть как нибудь сделать опционально выбирать файлы ))
п.с. возможно что то еще не доглядел ))
|
|
|
|
30.08.2009, 14:53
|
|
Постоянный
Регистрация: 29.10.2008
Сообщений: 306
Провел на форуме:
1819761
Репутация:
180
|
|
цикл
Код:
while(strcmp(cookies[cookiesNum].c_str(), "")){ cookiesNum++; };
можно написать записать так
Код:
while(strcmp(cookies[cookiesNum++].c_str(), ""));
а можно вобще не использовать
Код:
string cookies[1000] = {""};
лучше использовать вектор для куков, но можно и массив оставить, только переполнение убрать
Код:
while(!fin.eof())
{
getline(fin, strings[n]);
// это убрать n++;
if(++n > 1000) break;
}
также тип возвращаемых значений во многих функциях я бы поменял с int на bool...
Последний раз редактировалось M_script_; 30.08.2009 в 14:56..
|
|
|
|
30.08.2009, 15:19
|
|
Познавший АНТИЧАТ
Регистрация: 16.04.2006
Сообщений: 1,488
Провел на форуме:
2209675
Репутация:
537
|
|
спасибо
вектора я пока ещё не изучал, учту
|
|
|
|
|
|
|
Здесь присутствуют: 1 (пользователей: 0 , гостей: 1)
|
|
|
|
Похожие темы
Линейный вид
