мой первый VK Checker - просьба критиковать код

FORUMS

MEMBERS

RECENT POSTS

УСТРАНЕНИЕ КОНКУРЕНТОВ. БЛОКИРОВКА ДОМЕНОВ, БЛОКИРОВКА ИНСТАГРАМ/ТЕЛЕГРАМ И ДРУГОЕ. ПРОВЕРЕННЫЙ СЕЛЛЕР.

НОВЫЕ ТОРГОВАЯ НОВОСТИ ЧАТ

Скрыть

		ANTICHAT > ПРОГРАММИРОВАНИЕ > С/С++, C#, Rust, Swift, Go, Java, Perl, Ruby
мой первый VK Checker - просьба критиковать код

Опции темы

Поиск в этой теме

Опции просмотра

мой первый VK Checker - просьба критиковать код

30.08.2009, 12:30

Дикс

Познавший АНТИЧАТ

Регистрация: 16.04.2006

Сообщений: 1,488

С нами: 10562786

Репутация: 537

мой первый VK Checker - просьба критиковать код

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

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

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

icq 8 6776690

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

𝕏 Twitter Reddit Telegram Копировать ссылку

30.08.2009, 13:54

razb

Постоянный

Регистрация: 24.03.2009

Сообщений: 670

С нами: 9017461

Репутация: 414

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

п.с. возможно что то еще не доглядел ))

30.08.2009, 14:53

M_script_

Постоянный

Регистрация: 29.10.2008

Сообщений: 306

С нами: 9227388

Репутация: 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

С нами: 10562786

Репутация: 537

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

« Предыдущая тема | Следующая тема »

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