Форум АНТИЧАТ - мой первый VK Checker - просьба критиковать код

Форум АНТИЧАТ (https://forum.antichat.xyz/index.php)

- С/С++, C#, Delphi, .NET, Asm (https://forum.antichat.xyz/forumdisplay.php?f=24)

- - мой первый VK Checker - просьба критиковать код (https://forum.antichat.xyz/showthread.php?t=138294)

Дикс	30.08.2009 12:30

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

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

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

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

icq 8 6776690

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

razb	30.08.2009 13:54

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

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

M_script_

30.08.2009 14:53

цикл

Код:

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...

Дикс	30.08.2009 15:19

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

Время: 02:23