FreaxNet (присоединяйтесь к разработке!)

[MUTOgen]

ну вот. а если "-p -h 127.0.0.1" то какая пауза?

Никакая. Синтаксическая ошибка в команде - пропущен операнд.

ну так вот я про то и толкую. можно было бы тыхнуть в run_error или же все таки увенчать все это дело абсолютным дефолтным нулем...

[Hater]

ну так вот я про то и толкую. можно было бы тыхнуть в run_error или же все таки увенчать все это дело абсолютным дефолтным нулем...

Дык я и говорю, что разработчики не поставили ещё никакой проверки.

[MUTOgen]

ну так вот я про то и толкую. можно было бы тыхнуть в run_error или же все таки увенчать все это дело абсолютным дефолтным нулем...

Дык я и говорю, что разработчики не поставили ещё никакой проверки.

а о чем тогда спорили?

[Hater]

ну так вот я про то и толкую. можно было бы тыхнуть в run_error или же все таки увенчать все это дело абсолютным дефолтным нулем...

Дык я и говорю, что разработчики не поставили ещё никакой проверки.

а о чем тогда спорили?

Можешь помочь проекту - поставить проверку. Там несложно - пару строчек добавить.

[MUTOgen]

На новую версию врядли тянет, но кое что поправил на свой взгляд. В частности со строк типа "FreaxNet 127.0.0.1" сразу кидает на run_error + добавил проверку на случай если при -p ничего не указано (хотя работает и без этого, но подумал пусть будет о.о). Дефолтом пауза 0.

[Hater]

+ добавил проверку на случай если при -p ничего не указано (хотя работает и без этого, но подумал пусть будет о.о). Дефолтом пауза 0.

Похоже, мы не договорились . Строка типа FreaxNet -p -h 127.0.0.1 ошибка! Пусть меня поправят, если я не прав.

[Hater]

Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?

[MUTOgen]

Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?

вот тут согласен, я б заменил эту проверку на проверку что-то типо if(argc==1)... А по поводу FreaxNet -p -h 127.0.0.1 - вроде такое должно по задумке работать...

[Hater]

Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?

вот тут согласен, я б заменил эту проверку на проверку что-то типо if(argc==1)... А по поводу FreaxNet -p -h 127.0.0.1 - вроде такое должно по задумке работать...

Почему? Этот ключ требует наличия операнда.

[Hater]

Например, ping -s google.ru

[MUTOgen]

Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?

вот тут согласен, я б заменил эту проверку на проверку что-то типо if(argc==1)... А по поводу FreaxNet -p -h 127.0.0.1 - вроде такое должно по задумке работать...

Почему? Этот ключ требует наличия операнда.

ну это опять же вопрос к автору... конечно можно и на хелп в таком случае переправлять справедливо.

[MUTOgen]

если принимать пропуженый аргумент опции -p за ошибку то тогда думаю что-то вроде этого должно быть

[Hater]

UTeHok
Ещё в файл cmd.c добавь после блока if(p) (75-я строка) такой код:

Код: Выделить всё

else
{
    IP[strlen(IP)-1]='\0';
    begin_port = 1;
    end_port = 3000;
}

Это исправление некорректной обработки файлов с хостами без указания портов.

Кстати, тут я поторопился. Ещё нужно смотреть на то, идёт ли это чтение из файла.

если принимать пропуженый аргумент опции -p за ошибку то тогда думаю что-то вроде этого должно быть

Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.

Что-то я раскритиковался . Больше не буду лезть в процесс написания.

[MUTOgen]

Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.

Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.

без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...

[Hater]

Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.

Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.

без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...

Не должно работать, т.к. пропущен ключ -h.

[MUTOgen]

Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.

Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.

без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...

Не должно работать, т.к. пропущен ключ -h.

а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена

[Hater]

Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.

Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.

без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...

Не должно работать, т.к. пропущен ключ -h.

а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена

Прошу прощения, возможно я не прав. Просто эта строка там действительно не нужна. Эту проблему нужно решать иначе. Это костыль, который работает в данном случае, но не будет работать в других (пример: 127.0.0.1 -p 2).

[UTeHok]

а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена

+1
Глядя на ваше обсуждение ошибок и недочетов FreaxNet, я познал истинный метод вылавливания багов из кода. Очень эффективный метод, должен вам признаться Все замечания и вылавленные баги будут учтены в следующей версии!

[MUTOgen]

Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.

Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.

без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...

Не должно работать, т.к. пропущен ключ -h.

а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена

Прошу прощения, возможно я не прав. Просто эта строка там действительно не нужна. Эту проблему нужно решать иначе. Это костыль, который работает в данном случае, но не будет работать в других (пример: 127.0.0.1 -p 2).

ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.

[Hater]

а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена

+1
Глядя на ваше обсуждение ошибок и недочетов FreaxNet, я познал истинный метод вылавливания багов из кода. Очень эффективный метод, должен вам признаться

И какой же?

[MUTOgen]

а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена

+1
Глядя на ваше обсуждение ошибок и недочетов FreaxNet, я познал истинный метод вылавливания багов из кода. Очень эффективный метод, должен вам признаться

поделитесь

[Hater]

ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.

Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.

[MUTOgen]

ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.

Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.

вы имеете ввиду сам ман или то что в процессе выполнения программы появляется в качестве хелпа?

[Hater]

ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.

Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.

вы имеете ввиду сам ман или то что в процессе выполнения программы появляется в качестве хелпа?

И то, и другое.

[UTeHok]

поделитесь

И какой же?

Отдаешь код двум (должно быть четное количество) энтузиастам-программистам на растерзание. Спустя совсем не продолжительное время всплывают такие нюансы и баги, о которых даже и подумать не мог, замечаешь совсем очевидные ошибки и просто опечатки.

Можно выпускать версию 0.063, просто учтя все, что вы тут написали, т.е. добавив проверки и т.д. На днях выложу.

[Hater]

Согласно ману (и run_error) имеет смысл следующая конструкция, смысл которой мне совершенно не понятен: FreaxNet -h 127.0.0.1 google.ru:1-80

[UTeHok]

Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.

Весь во внимании

Согласно ману (и run_error) имеет смысл следующая конструкция, смысл которой мне совершенно не понятен: FreaxNet -h 127.0.0.1 google.ru:1-80

Предлагаете убрать <host>?

USAGE: FreaxNet [Options] <IP_address>[:port]|[:begin_port-end_port]
OPTIONS: -h <host>
-f <file>
-p <pause>
EXAMPLES: FreaxNet -p 2 127.0.0.1
FreaxNet -h 127.0.0.1:16-256
FreaxNet -f hosts
See the man page for more options and examples %)

[MUTOgen]

По поводу мана... У меня с ним вообще старнности при установке вот скрин вобщем как он у меня выглядит:

P.S. а вообще было бы интересно узнать как пишутся маны. если имеются какие-то полезные ссылки поделитесь

[UTeHok]

По поводу мана... У меня с ним вообще старнности при установке вот скрин вобщем как он у меня выглядит:

У меня вот так

[Hater]

Предлагаете убрать <host>?
Цитата
USAGE: FreaxNet [Options]
OPTIONS: -h <host>
-f <file>
-p <pause>
EXAMPLES: FreaxNet -p 2 127.0.0.1
FreaxNet -h 127.0.0.1:16-256
FreaxNet -f hosts
See the man page for more options and examples %)

Нет. предлагаю убрать <IP_address>[:port]|[:begin_port-end_port]. И не только. Предлагаю посмотреть как это написано у других и переписать всё с нуля.