parser beta 0.1 #1

Closed
nekitmilk wants to merge 3 commits from nekitmilk_add_getopt into dev
2 changed files with 22 additions and 22 deletions
Showing only changes of commit 0bc97e71fe - Show all commits

Binary file not shown.

View File

@ -14,7 +14,8 @@
// -10 - Пользователь выбрал тип атаки порт сканнинг, но не ввел нужные параметры
// -20 - Пользователь выбрал тип атаки флуд, но не ввел нужные параметры
// -100 - неизвестная ошибка
// -101 - неизвестная опция, следует предложить вызвать флаг помощи
// -101 - неизвестная опция или потерян аргумент, следует предложить вызвать флаг помощи
// -600 - пользователь ввел токен, но не id или наоборот
// Какие-то еще коды?
int my_check_params(int argc, char** argv) {
@ -60,20 +61,20 @@ int my_check_params(int argc, char** argv) {
opts.telegram_token = optarg;
break;
case 'h':
Review

Вообще все выводы сунуть в my_diag в принципе его можно написать, my_check_params ничего не выводит, а только статус код возвращает

Вообще все выводы сунуть в my_diag в принципе его можно написать, my_check_params ничего не выводит, а только статус код возвращает
Review

Я временно эти cout'ы поставил, просто чтобы уже видеть как это работает, потом это можно будет вынести в my_diag в соответствии со статус кодами (как только приступим его создавать)

Я временно эти cout'ы поставил, просто чтобы уже видеть как это работает, потом это можно будет вынести в my_diag в соответствии со статус кодами (как только приступим его создавать)
std::cout << "Usage: " << argv[0] << " [options]\n"
<< "Required:\n"
<< " -a, --attack TYPE Type of attack (scan|flood)\n"
<< " -d, --domain DOMAIN Target domain\n"
<< " -i, --ip IP Target IP\n"
<< " -p, --port PORT Port. Required only for flood type!\n"
<< "Optional:\n"
<< " -l, --log FILE Log file\n"
<< " -t, --telegram ID Telegram ID\n"
<< " -b, --token TOKEN Telegram bot token\n";
// std::cout << "Usage: " << argv[0] << " [options]\n"
// << "Required:\n"
// << " -a, --attack TYPE Type of attack (scan|flood)\n"
// << " -d, --domain DOMAIN Target domain\n"
// << " -i, --ip IP Target IP\n"
// << " -p, --port PORT Port. Required only for flood type!\n"
// << "Optional:\n"
// << " -l, --log FILE Log file\n"
// << " -t, --telegram ID Telegram ID\n"
// << " -b, --token TOKEN Telegram bot token\n";
status = 0;
break;
case '?':
Review

my_diag

my_diag
std::cerr << "Unknown option!\n.--help for info\n";
// std::cerr << "Unknown option!\n.--help for info\n";
status = -101;
break;

А сюда можно попасть?

А сюда можно попасть?

В default? По идее нет, но это просто на всякий случай, если вдруг по какой-то причине ни одно из условий не было выполнено, то мы код -100 вернем

В default? По идее нет, но это просто на всякий случай, если вдруг по какой-то причине ни одно из условий не было выполнено, то мы код -100 вернем
default:
@ -84,34 +85,33 @@ int my_check_params(int argc, char** argv) {
// Проверка обязательных параметров
if (status != 0)
if (status != 0 && status != -101)
{
if (opts.attack_type != "flood" && opts.attack_type != "scan") {
Review

my_diag

my_diag
Review

В этих условиях cout'ы временны, останется только код

В этих условиях cout'ы временны, останется только код
std::cerr << "Error: Missing required parameters!\n--help for more info\n";
// std::cerr << "Error: Missing required parameters!\n--help for more info\n";
status = -1;
}
else if (opts.attack_type == "scan" && (opts.domain.empty() || opts.ip.empty())) {
Review

my_diag

my_diag
std::cerr << "Error: Missing required parameters!\n--help for more info\n";
// std::cerr << "Error: Missing required parameters!\n--help for more info\n";
status = -10;
}
Review

Если у нас при типе атаки flood порт задан, а ip и domain нет, то это тоже неверная комбинация

Если у нас при типе атаки flood порт задан, а ip и domain нет, то это тоже неверная комбинация
else if (opts.attack_type == "flood" && (opts.domain.empty() || opts.ip.empty() || opts.port.empty())) {
Review

my_diag

my_diag
std::cerr << "Error: Missing required parameters!\n--help for more info\n";
// std::cerr << "Error: Missing required parameters!\n--help for more info\n";
status = -20;
}
else if ((!opts.telegram_id.empty() && opts.telegram_token.empty()) || (opts.telegram_id.empty() && !opts.telegram_token.empty())) {

my_diag

my_diag
status = -600;
}
else if (opts.attack_type == "scan" && !opts.domain.empty() && !opts.ip.empty()) {
std::cerr << "Error: Missing required parameters!\n--help for more info\n";
// std::cerr << "Error: Missing required parameters!\n--help for more info\n";

my_diag

my_diag
status = 1;
}
else if (opts.attack_type == "flood" && !opts.domain.empty() && !opts.ip.empty() && !opts.port.empty()) {
Review

Если есть telegram_id, то должен быть и token и наоборот

Если есть telegram_id, то должен быть и token и наоборот
Review

добавил

добавил
std::cerr << "Error: Missing required parameters!\n--help for more info\n";
// std::cerr << "Error: Missing required parameters!\n--help for more info\n";
status = 2;
}
// Какие-то еще коды?
Review

Переносы

Переносы
Review

убрал

убрал
}
return status;
}