ANTICHAT.XYZ    VIDEO.ANTICHAT.XYZ    НОВЫЕ СООБЩЕНИЯ    ФОРУМ  
Баннер 1   Баннер 2
Antichat снова доступен.
Форум Antichat (Античат) возвращается и снова открыт для пользователей. Здесь обсуждаются безопасность, программирование, технологии и многое другое. Сообщество снова собирается вместе.
Новый адрес: forum.antichat.xyz
Вернуться   Форум АНТИЧАТ > Программирование > С/С++, C#, Delphi, .NET, Asm
   
Ответ
 
Опции темы Поиск в этой теме Опции просмотра

мой первый VK Checker - просьба критиковать код
  #1  
Старый 30.08.2009, 12:30
Аватар для Дикс
Дикс
Познавший АНТИЧАТ
Регистрация: 16.04.2006
Сообщений: 1,488
Провел на форуме:
2209675

Репутация: 537


Отправить сообщение для Дикс с помощью ICQ
Arrow мой первый VK Checker - просьба критиковать код

http://www.sendspace.com/file/lly73w

написал чекер для ВК на C++
принимает список кукис cookie.txt, чекает их, битые переносит в invalid.txt

выкладываю с сорцами, надеясь что найдётся добрый человек, который покритикует код и укажет что можно улучшить. за хорошую помощь в улучшении кода готов заплатить

icq 8 6776690

остальные тестируйте на здоровье
 
Ответить с цитированием

  #2  
Старый 30.08.2009, 13:54
Аватар для razb
razb
Постоянный
Регистрация: 24.03.2009
Сообщений: 670
Провел на форуме:
2868783

Репутация: 414


Отправить сообщение для razb с помощью ICQ
По умолчанию

Бегло просмотрев твой код нашел море недочетов )
1) переполнение буфера.
2) один или два лишних циклов.
3) вместо массива строк в стеке (из-за которого кстати и переполнение в ф-ции file()) лучше юзать вектор или иной контейнер.
4) зачем реализацию выносить в хэдэр, они не для этого созданы.
5) от глобальных переменных можно тоже отказаться.
6) ну можно было хоть как нибудь сделать опционально выбирать файлы ))

п.с. возможно что то еще не доглядел ))
 
Ответить с цитированием

  #3  
Старый 30.08.2009, 14:53
Аватар для M_script_
M_script_
Постоянный
Регистрация: 29.10.2008
Сообщений: 306
Провел на форуме:
1819761

Репутация: 180
Отправить сообщение для M_script_ с помощью ICQ
По умолчанию

цикл
Код:
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..
 
Ответить с цитированием

  #4  
Старый 30.08.2009, 15:19
Аватар для Дикс
Дикс
Познавший АНТИЧАТ
Регистрация: 16.04.2006
Сообщений: 1,488
Провел на форуме:
2209675

Репутация: 537


Отправить сообщение для Дикс с помощью ICQ
По умолчанию

спасибо
вектора я пока ещё не изучал, учту
 
Ответить с цитированием
Ответ



Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Оценте мой первый сайт poker77 Обсуждение Ваших сайтов 16 10.01.2009 21:28
Мой первый скан LeverOne Защита ОС: вирусы, антивирусы, файрволы. 5 03.05.2006 07:26
МОЙ ПЕРВЫЙ ДЕФЕЙС =))) Xalegi Болталка 36 15.12.2005 13:21



Здесь присутствуют: 1 (пользователей: 0 , гостей: 1)
 


Быстрый переход




ANTICHAT.XYZ