Перейти к содержимому


Фотография

Клуб Программистов


  • Авторизуйтесь для ответа в теме
Сообщений в теме: 171

#161 Arsenicum

Arsenicum

    Пыже-лансер

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 733

Отправлено 30 Март 2017 - 16:08

Чо?!


А так в первом if'е - правда ничего нет?

Или это ты выпустил кусок кода?


А в чем прикол предполагаемый физический смысл этого куска?


[вК=id7200] [ICQ=124648493]


#162 OXic

OXic

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 798

Отправлено 30 Март 2017 - 17:32

Я лишь путь поменял... строчки все идут последовательно, ничего не выкидывал...
А так да, -1 вернули - пофиг! два раза подряд close() - а что? а вдруг?..
 
Тут ещё так любят писать:
char a[MAX_SIZE];

memset(a, 0, sizeof(MAX_SIZE));
или
memset(a, 0, sizeof(sizeof(a)));
И всё нормально! Всё ревьювится и мерджится, а потом ещё девайсы прошивают...
TBD

#163 Arsenicum

Arsenicum

    Пыже-лансер

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 733

Отправлено 30 Март 2017 - 20:29

На тебя, что ли, баг релизный пришёл?)
А для чего там sizeof'ы внешние?)

[вК=id7200] [ICQ=124648493]


#164 Кузя

Кузя

    КЭП

  • Пользователи
  • PipPipPipPip
  • Cообщений: 11 710

Отправлено 30 Март 2017 - 20:30

Как разговор инопланетян подслушал

#165 OXic

OXic

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 798

Отправлено 30 Март 2017 - 20:31

На тебя, что ли, баг релизный пришёл?)

нет, я проверяющий и поучающий :)

 

А для чего там sizeof'ы внешние?)

потому что думалку люди не включают и, видимо, понятия не имеют, что функции делают на самом деле...


TBD

#166 Arsenicum

Arsenicum

    Пыже-лансер

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 733

Отправлено 30 Март 2017 - 20:36

Да даже если не думать, в сигнатуре memset'а написано, что там размер передаётся. Нафиг брать sizeof от уже имеющегося размера?
Или я что-то не так понял?)

[вК=id7200] [ICQ=124648493]


#167 OXic

OXic

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 798

Отправлено 30 Март 2017 - 20:47

1) Я вообще не понимаю, как народ пишет код и как изучают программированию...

2) Возможно, откуда-то тупо копируют. А самое плохое в том, что данные ошибки не отлавливает статический анализатор кода, потому что он надеется на адекватность программиста и думает, что он намеренно так написал, а я физически не могу пересмотреть всё. Когда же я написал об такой ошибке корейскому "куратору", то он не смог понять проблемы и сказал, что код же работает, а это значит, что всё хорошо  :wall:


TBD

#168 Arsenicum

Arsenicum

    Пыже-лансер

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 733

Отправлено 30 Март 2017 - 21:40

:))
Ну да, "работает - не чини" ©
У нас по процессу есть принудительный code review, до тех пор, пора разработчик не даст понять, что он понимает что и как пишет.

[вК=id7200] [ICQ=124648493]


#169 DanilaMaster

DanilaMaster

    КЭП

  • Пользователи
  • PipPipPipPip
  • Cообщений: 5 058

Отправлено 30 Март 2017 - 21:47

Как разговор инопланетян подслушал

 

это точно)))



#170 Zika

Zika

    КЭП

  • Пользователи
  • PipPipPipPip
  • Cообщений: 7 754

Отправлено 31 Март 2017 - 15:08

1) Я вообще не понимаю, как народ пишет код и как изучают программированию...

2) Возможно, откуда-то тупо копируют. А самое плохое в том, что данные ошибки не отлавливает статический анализатор кода, потому что он надеется на адекватность программиста и думает, что он намеренно так написал, а я физически не могу пересмотреть всё. Когда же я написал об такой ошибке корейскому "куратору", то он не смог понять проблемы и сказал, что код же работает, а это значит, что всё хорошо  :wall:

так у дофига людей такое отношение ко всему


пусть лошадь думает - у нее голова большая
Корректирую сингулярность

#171 OXic

OXic

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 798

Отправлено 04 Май 2017 - 08:13

N8mJKZGIE4g.jpg


TBD

#172 OXic

OXic

  • Пользователи
  • PipPipPipPip
  • Cообщений: 12 798

Отправлено 20 Июль 2017 - 05:14

_Z3wUOgv0dA.jpg


TBD




Количество пользователей, читающих эту тему: 0

0 пользователей, 0 гостей, 0 анонимных