- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
/**
* @param int $day
* @return bool
*/
protected function isWorkingDay($day)
{
return $day % 8 <= 5;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+4
/**
* @param int $day
* @return bool
*/
protected function isWorkingDay($day)
{
return $day % 8 <= 5;
}
Cпособ определить, является ли указанный день (передается номер дня в неделе от 1 до 7) рабочим. Чем автора не устроил return $day <= 5 - непонятно..
Hubactep 11.07.2016 19:12 # 0
3_14dar 11.07.2016 21:46 # 0
feversocial 11.07.2016 22:30 # 0
3_dar 11.07.2016 19:44 # 0
bormand 11.07.2016 20:04 # +2
> @return bool
Ахуенный докстринг.
3_dar 11.07.2016 20:40 # 0
bormand 11.07.2016 20:43 # +3
3_14dar 11.07.2016 21:46 # 0
Soul_re@ver 11.07.2016 23:07 # +3
1024-- 12.07.2016 10:28 # 0
3_14dar 13.07.2016 17:12 # 0
Bobik 15.07.2016 00:47 # 0
НАХУЯ?
bormand 15.07.2016 00:50 # +1
> m_link = std::move(link);
И правда, нахуя?
Bobik 15.07.2016 00:56 # +1
А подход c std::move позволяет иметь столько же копирований, сколько и при приёме const string&, но быть более понятным в сигнатуре.
Soul_re@ver 15.07.2016 02:08 # +2
roman-kashitsyn 15.07.2016 10:34 # +3
Борманд, где твоё C++-чутьё? Так же можно сыкономить одно копирование, выжать каплю ПИРФОМАНСА, если в качестве входного параметра передаётся временный объект! Мув же почти бесплатный.
Ну и так меньше унылых констрефов писать.
3_dar 15.07.2016 22:35 # 0
Soul_re@ver 15.07.2016 22:40 # 0
3_dar 15.07.2016 22:49 # 0
метод ожидает string, и я думал const char* преобразуется в string, потом по значению передается в set_link
Soul_re@ver 15.07.2016 22:51 # 0
3_dar 15.07.2016 23:36 # 0
Sim_salapim 16.07.2016 00:03 # −3
Fike 14.07.2016 23:37 # 0