[cdev] Задание ps

[Grigoriy A. Sitkarev]

Толик и Ваня.

Почему вы всё время отвечаете в чужих нитях обсуждений?

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

Когда будете готовы работать дальше, просигнальте. Я вас ещё озадачу,
потому что придумал как получить количество тиков в секунду (на самом
деле sysconf(_SC_CLK_TCK) может дать неверное значение, если ядро
пересобрано с CONFIG_HZ=1000 то sysconf(_SC_CLK_TCK) будет всё равно
возвращать 100) и нужно ещё массу всего править.

Вам надо сделать по-человечески вывод заголовка таблицы, печать
таблицы. Там нужно завернуть в цикл вывод полей процесса через таблицу
просмотра (lookup table). Сейчас она у вас развёрнута. Есть ещё ряд
мелких гадостей, например вам надо было сделать глобальную переменную
struct process *plist и она была бы у вас указателем на начало списка.
У вас её нет, поэтому вы функции list используете некрасивый счётчик
cnt. Выглядит нелепо и не идиоматично. Функциям надо дать более ясные
имена, я бы list() заменил на show_proc_list() или что-то в этом духе.
У вас программа должна читаться как проза и имена функций,
идентификаторов нужно выбирать продуманно, потому что это часть
документации программы.

--
Г.А.

15 марта 2010 г. 14:22 пользователь ka50 <astroori на mail.ru> написал:
> На данный момент выполнено:
>        * обработка некоторых ошибок
>        * реализована функция, позволяющая отображать любые поля по переданной
> в функцию маске
>        * с помощью вышеописанной функции реализованы флаги -l, -f, -a и их
> любые комбинации (с использованием разбора опций командной строки)
>
>
> Разработка временно приостановлена до сдачи экзамена по теории игр
> (18.03).
>
> KA50
>
>
> _______________________________________________
> cdev mailing list
> cdev на wiki.syktsu.ru
> http://wiki.syktsu.ru/cgi-bin/mailman/listinfo/cdev
>