The OpenNET Project / Index page

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



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

Оглавление

Microsoft открыл код классического файлового менеджера Windows, opennews (??), 09-Апр-18, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


92. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от Аноним (-), 10-Апр-18, 09:08 
> Довольно приятное оформление кода с явным разделением на блоки.

Это ты про функции на 200 строк? Серьёзно?
А смысл вот этого можешь объяснить? https://github.com/Microsoft/winfile/blob/master/src/wfdir.c...

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

101. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от пох (?), 10-Апр-18, 10:26 
> А смысл вот этого можешь объяснить? https://github.com/Microsoft/winfile/blob/master/src/wfdir.c...

здесь размер имени файла используется в качестве размера отрисовываемого item'а - а что рисуется  имя файла, данной функции нафиг не интересно знать. Если внезапно понадобится не только файлы - будет просто отрефакторить этот кусок. И да, память еще не бездонная, ее надо экономить, поэтому лишние переменные - лишние.

заняло пяток секунд, много меньше чем написать этот текст. Вы уверены что вы программист?

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

113. "Microsoft открыл код классического файлового менеджера Windo..."  –1 +/
Сообщение от Аноним (-), 10-Апр-18, 12:51 
> здесь размер имени файла используется в качестве размера отрисовываемого item'а

Причём его высоты? Или height уже как-то по-другому переводится? Нет, попробуй ещё раз.

> И да, память еще не бездонная, ее надо экономить, поэтому лишние переменные - лишние.

Сколько, говоришь, байт сэкономил? За определение макроса посреди функции, тем более макроса, который потом используется ровно один раз, руки надо  отрывать. Не хочешь создавать "лишнюю" переменную — используй ту же, код менее читаемым не станет.

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

123. "Microsoft открыл код классического файлового менеджера Windo..."  +1 +/
Сообщение от пох (?), 10-Апр-18, 14:12 
>> здесь размер имени файла используется в качестве размера отрисовываемого item'а
> Причём его высоты?

ну да. так и размер - по высоте. оно же не имя рисует, а поле для него готовит, ширина берется из ширины экранного элемента, на этом этапе длина не интересна.

>> И да, память еще не бездонная, ее надо экономить, поэтому лишние переменные - лишние.
> Сколько, говоришь, байт сэкономил? За определение макроса посреди функции, тем более макроса,

там каждый байт надо было экономить. Потому что памяти у той машины было - мегабайт. Один.

> который потом используется ровно один раз, руки надо  отрывать. Не

еще один опоздавший родиться.

> хочешь создавать "лишнюю" переменную — используй ту же, код менее читаемым
> не станет.

станет.

И его сложнее станет переделывать, если понадобится рисовать не только поле под файлы.

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

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

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




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

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