The OpenNET Project / Index page

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



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

Оглавление

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

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


40. "Microsoft открыл код классического файлового менеджера Windo..."  +8 +/
Сообщение от Аноним (-), 09-Апр-18, 21:39 
Довольно приятное оформление кода с явным разделением на блоки. Это просто ты привык к современному гoвнокоду который выкатывают прямо из головы одной колбасой.
Ответить | Правка | Наверх | Cообщить модератору

49. "Microsoft открыл код классического файлового менеджера Windo..."  –2 +/
Сообщение от Кир (?), 09-Апр-18, 22:14 
А почему стиль оформления кода меняется в разных частях файла? Индусы не осилили astyle?
Ответить | Правка | Наверх | Cообщить модератору

55. "Microsoft открыл код классического файлового менеджера Windo..."  –1 +/
Сообщение от Аноним (-), 09-Апр-18, 22:45 
Не критично. Могу предположить, что в то время развитых средств и подходов для ревью кода не существовало.
Ответить | Правка | Наверх | Cообщить модератору

61. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от Michael Shigorinemail (ok), 09-Апр-18, 22:55 
> Не критично. Могу предположить

Да нет.  Вы просто привыкли, как и переобуваться в подскоке.

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

58. "Microsoft открыл код классического файлового менеджера Windo..."  –1 +/
Сообщение от Ivan_83 (ok), 09-Апр-18, 22:49 
1. Код слишко разрежен, там прямо так и скводит оплата за строки.
2. 3к+ строк в одном файле это явный перебор, скролить такое уже трудно, а на модули они растащить не смогли.
Ответить | Правка | К родителю #40 | Наверх | Cообщить модератору

63. "Microsoft открыл код классического файлового менеджера Windo..."  +2 +/
Сообщение от Аноним (-), 09-Апр-18, 23:17 
1. Самое то. Выкиньте уже ноутбук, пишите код как все нормальные люди - за полноценным монитором. А то повадились ставить пустые строки только между методами, да и то не всегда. Порой заглянешь в код современного "программиста" и глаза вытекают от попыток разобрать логику. for(int i=0;i<length;i++){..., if(a=b&&b<c+1||c==3){... и иже с ними. Про React-модули на несколько сотен строк без единой пустой строки-разделителя и говорить не стоит, сплошь и рядом.

2. Вы привели в пример один из ключевых файлов программы, если судить по названию. 3к+ строк - нормально. Есть небольшая неряшливость в отступах между методами, но в пределах терпимого.

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

71. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от Ivan_83 (ok), 10-Апр-18, 03:10 
Ты унылый троль: второй раз переходишь на личности, и каждый раз мимо.

Этот "ключевой" модуль легко можно было попилить на части, было бы желание.
Ещё раз повторяю: там слишком низкая концентрация кода, даже если взять 1080 и повернуть всё равно мало поможет.
3к строк может быть и нормально, если ты непризнанный гений и у тебя всё в голове легко умещается либо ты кодер который привык к однострочным патчам и не способен увидеть и понять что вокруг, для обычного человека это ппц как не удобно.

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

100. "Microsoft открыл код классического файлового менеджера Windo..."  +2 +/
Сообщение от пох (?), 10-Апр-18, 10:19 
> 1. Код слишко разрежен, там прямо так и скводит оплата за строки.

это нормальный, удобочитаемый код. Так примерно и было принято в те годы в MS (я видел другие исходники)

> 2. 3к+ строк в одном файле это явный перебор, скролить такое уже
> трудно, а на модули они растащить не смогли.

не было нормальных инкрементальных линкеров, а неинкрементный возился бы на тех процессорах часами - если не навернется в процессе "ой, память кончилась".
3k строк в одном ключевом файле никого, наоборот, не огорчало, пока они разбиты на относительно изолированные функции, нормальные ide, где не надо было скроллить, уже написали, это у вас они до сих пор ниасилены.
А вот открыть-подгрузить новый исходник как раз было не совсем бесплатным процессом.


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

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ообщить модератору

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

Гм... Вижу три разных оформления блоков в пределах одного экрана:
https://github.com/Microsoft/winfile/blob/master/src/wfdir.c...

Раз:


      if ((lpxdta) && (lpxdta->dwAttrs & ATTR_COMPRESSED))
      {
         rgbText = SetTextColor(hDC, RGB(0, 0, 255));
      }
      else
      {
         rgbText = SetTextColor(hDC, GetSysColor(COLOR_WINDOWTEXT));
      }

Два:


      if (bHasFocus)
         goto FocusOnly;

Три:


   if (lpLBItem->itemAction == ODA_FOCUS) {
      goto FocusOnly;
   }

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

124. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от пох (?), 10-Апр-18, 14:13 
> Гм... Вижу три разных оформления блоков в пределах одного экрана:

какой ужас, для современных кодошлепов...

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

122. "Microsoft открыл код классического файлового менеджера Windo..."  +1 +/
Сообщение от Аноним84701 (ok), 10-Апр-18, 14:05 
> Довольно приятное оформление кода с явным разделением на блоки. Это просто ты
> привык к современному гoвнокоду который выкатывают прямо из головы одной колбасой.

https://github.com/Microsoft/winfile/blob/master/src/wfdir.c


      } else {
         goto SelectFirst;
      }

   } else {

SelectFirst:


...

if (iMac == 0)
   {
       if (p)
       {
           LocalFree(p);
           p = NULL;
       }
       goto GDSDone;
   }

   lpSelItems = (LPINT) LocalAlloc(LMEM_FIXED, sizeof(INT) * iMac);
   if (lpSelItems == NULL)
      goto Fail;


...

if (lpLBItem->itemID == -1 || !lpStart || !lpxdta) {

      if (bHasFocus)
         goto FocusOnly;

      return;
   }

   if (lpLBItem->itemAction == ODA_FOCUS) {
      goto FocusOnly;
   }


...

if ((pwTabs = (WORD *)LocalAlloc(LPTR,
                                       sizeof(WORD) * MAX_TAB_COLUMNS)) == NULL) {

         lRetval = -1L;
         goto Done;
      }


Ага, шЫдевральное Spaghetti Microsoftia, но зато все разделено на блоки (иногда правда -- по случайному принципу)  …

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

174. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от Аноним (-), 11-Апр-18, 07:48 
Абсолютно нормальная практика - перейти по метке из кучи вложенных ifов. Настоящий макаронный код ты и не видел.
Ответить | Правка | Наверх | Cообщить модератору

188. "Microsoft открыл код классического файлового менеджера Windo..."  –1 +/
Сообщение от Аноним84701 (ok), 11-Апр-18, 13:53 
> Абсолютно нормальная практика овнокодеров, про структурное программирование слыхом не слыхавших -- перейти по метке из кучи вложенных ifов на каждый чих, по нескольку раз.

Поправил, не благодари.
> Настоящий макаронный код ты и не видел.

А будущее по аватарке предсказать возьмешься?

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

208. "Microsoft открыл код классического файлового менеджера Windo..."  +/
Сообщение от Саша (??), 13-Апр-18, 15:10 

> А будущее по аватарке предсказать возьмешься?

Все будет хорошо! И даже лучше!!!


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

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

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




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

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