The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Выпуск Red Hat Enterprise Linux 6.9"
Версия для распечатки Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Исходное сообщение [ Отслеживать ]
Подсказка: Для сортировки сообщений в нити по дате нажмите "Сортировка по времени, UBB".
. "Выпуск Red Hat Enterprise Linux 6.9" +3 +/
Сообщение от Аноним (-), 22-Мрт-17, 15:50 
> Дублирующиеся магические строчки: 329 и 337, 443 и 451, 850 883 и 908
> p = strappend("/run/systemd/sessions/", session);
> p = strappend("/run/systemd/sessions/", session);

Правильно. Нужно было создать единую функцию strappendRunSystemdSessions. Зато не "магически" и не "дублируется". Даешь также функцию strappendRunSystemdSeats и так далее. И после этого код, конечно же, станет читабельнее.

> я честно не могу понять, почему, например, на 850 результат процедуры get_files_in_directory записывается в переменную, названную r
> результат
> r
> r[esult]

Ну, видимо ты из тех, кто не использует i для счетчика, l для количества и т. д. Есть определенные символы, которые имеют однозначный смысл.

> Брезгует фигурными скобками

Это код-стайл такой. Аргументик-то так себе. Чисто вкусовщина. Как я и говорил выше, "а у вас пустая строчка после ифа отсутствует".

> На 238 пушка просто: /* We override the glibc assert() here. */. Молодец, Леня!

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

Слабо, очень слабо! Я ожидал что-нибудь типа option+15. А в ответ одни лишь "фигурная скобка стоит не там, пробел поставили не здесь, переменную назвали не так".

Ответить | Правка | Наверх | Cообщить модератору

Оглавление
Выпуск Red Hat Enterprise Linux 6.9, opennews, 21-Мрт-17, 18:16  [смотреть все]
Форумы | Темы | Пред. тема | След. тема



Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру