Тухлый Александр (0xd34df00d)

*с++
За такой код в тырпрайзе руки отрывают? isMUC — булева переменная, есличо.
const int clientsIconsWidth = (!isMUC) (clientIcons.size () (iconSize + CPadding) — CPadding);

#1194508
from Azoth_primary, 6 months ago

← All messages

Replies (48)

Post a reply (xmpp)

@gelraen:
я бы за !isMUC * something точно отрывал
#1194508/1
from imax, 6 months ago
@nya:
за неявное приведение bool к int я бы точно оторвал ноги
#1194508/2
from Home, 6 months ago
@asphyx:
Нет, не отрывают
#1194508/3
from Home, 6 months ago
@asphyx:
@nya А чем тебе bool не int? ;) Приведение bool к int — штука весьма условная
#1194508/5
from Home, 6 months ago, in reply to /2
@asphyx:
А что ты думаешь тогда о конструкции !!some_int_var ? ;)
#1194508/6
from Home, 6 months ago
@nya:
@asphyx семантика страдает
#1194508/8
from Home, 6 months ago, in reply to /5
@nya:
@asphyx Ну это нормальная математическая конструкция
#1194508/9
from Home, 6 months ago, in reply to /6
@asphyx:
@nya Замени bool на int и перестанет страдать :)
#1194508/10
from Home, 6 months ago, in reply to /8
@jtootf:
по самые гланды
#1194508/11
from lagrange, 6 months ago
@asphyx:
@jtootf Ты злой
#1194508/12
from Home, 6 months ago, in reply to /11
@jtootf:
@asphyx и весьма коварный
#1194508/13
from lagrange, 6 months ago, in reply to /12
@0xd34df00d:
@jtootf :3
#1194508/14
from web, 6 months ago, in reply to /11
@0xd34df00d:
@gelraen Почему? Если true — значит, умножишь на ноль, будет ноль. Если false, то все ок.
#1194508/15
from web, 6 months ago, in reply to /1
@gelraen:
@0xd34df00d ну если выражение должно выдавать результатом true или false — то ок. а если число — то уге надо отрывать. потому как можно пропустить момент когда isMUC ВНЕАЗПНО станет int и побегать по граблям. ну и при чтении надо 0.5-1 лишних секунд на осмысливание того, что же, блять, этим автор хотел нам сказать
#1194508/16
from imax, 6 months ago, in reply to /15
@0xd34df00d:
@gelraen Ты прав, 0.5-1 лишних секунд — это слишком мало. Сюда надо нахуячить шаблонов.
#1194508/17
from web, 6 months ago, in reply to /16
@gelraen:
@0xd34df00d ты будешь гореть в специальном аду
#1194508/18
from imax, 6 months ago, in reply to /17
@0xd34df00d:
@gelraen DO WANT
#1194508/19
from web, 6 months ago, in reply to /18
@gelraen:
@0xd34df00d ок. а я пойду дальше про Fibre Channel читать
#1194508/20
from imax, 6 months ago, in reply to /19
@asphyx:
@gelraen О каких граблях речь?
#1194508/21
from Home, 6 months ago, in reply to /16
@gelraen:
@asphyx начиная картинками шириной в три монитора и заканчивая сегфолтом
#1194508/22
from imax, 6 months ago, in reply to /21
@0xd34df00d:
@asphyx Она кричит.
#1194508/23
from web, 6 months ago, in reply to /6
@kapsh:
ШТО
#1194508/24
from HolyShit, 6 months ago
@0xd34df00d:
@kapsh /15
#1194508/25
from web, 6 months ago, in reply to /24
@kapsh:
@0xd34df00d Не пойму только, почему ширина значка в муках нулевая. >__<
#1194508/26
from HolyShit, 6 months ago, in reply to /15
@0xd34df00d:
@kapsh Потому что у самой записи в списке контактов, отвечающей за мук, а не за участников мука, нет иконок клиента.
#1194508/27
from web, 6 months ago, in reply to /26
@nya:
@kapsh потомушто они в муках
#1194508/28
from Home, 6 months ago, in reply to /26
@kapsh:
@0xd34df00d А, ок.
#1194508/29
from HolyShit, 6 months ago, in reply to /27
@Cthulhu:
Я бы оторвал кстати за этот конкретный кусок ), ибо

const int clientsIconsWidth = (!isMUC) ? (clientIcons.size () * (iconSize + CPadding) — CPadding):0;

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

Хотя логичнее было бы написать
const int clientsIconsWidth = isMUC ? 0 : (clientIcons.size () * (iconSize + CPadding) — CPadding);
#1194508/30
from Miranda, 6 months ago
@nya:
@Cthulhu и вообще, чего экономить строки, лучше уж
const int clientsIconsWidth = 0;
if (isMUC) {
clientsIconsWidth += clientIcons.size() * (iconSize + CPadding) — CPadding;
}
#1194508/31
from Gajim, 6 months ago, in reply to /30
@Cthulhu:
@nya А зачем? Сам по себе такой код без проблем воспринимается человеком, покодившим некоторое время на с++. Пугать может разве что выходцев из паскаля )
Мне, например, в одну строчку воспринимать даже проще — там сразу глаза видят всю логику, а тут надо смотреть вниз, чо там в ифе, и нет ли else, и ваще
#1194508/32
from Miranda, 6 months ago, in reply to /31
@jtootf:
@nya const? +=?
#1194508/33
from lagrange, 6 months ago, in reply to /31
@asphyx:
@nya какой тут нафиг const?
#1194508/34
from Home, 6 months ago, in reply to /31
@nya:
@jtootf ну else { const ... = 0}
#1194508/35
from Gajim, 6 months ago, in reply to /33
@nya:
@asphyx в любом случае, зачем экономить строки? надо писать понятно и просто.
#1194508/36
from Gajim, 6 months ago, in reply to /34
@Cthulhu:
@nya Вообще, откуда взялся этот стереотип про непонятность компактного кода — хз. Я свое мнение уже написал выше, и думаю я не один такой )
#1194508/37
from Miranda, 6 months ago, in reply to /36
@asphyx:
@nya Я думаю, условный оператор — наиболее красивая конструкция для этого случая. Просто и понятно и строки экономятся :)
#1194508/38
from Home, 6 months ago, in reply to /36
@Cthulhu:
@asphyx Вот, точно не один )
#1194508/39
from Miranda, 6 months ago, in reply to /38
@nya:
@asphyx это в простых случаях да, int nya = isVisible ? MAGIC_VARIABLE_1 : MAGIC_VARIABLE_2
А если вложенные?
int nya = some() ? lol_func()? super_lol()+1: bool()(t()?-1:1) : TROLLFACE_VARIABLE
#1194508/40
from Gajim, 6 months ago, in reply to /38
@asphyx:
@nya Норм, я так делаю. Только скобки ставлю
#1194508/41
from Home, 6 months ago, in reply to /40
@Cthulhu:
@nya Мы какбе этот конкретный случай обсуждаем, а не вообще
#1194508/42
from Miranda, 6 months ago, in reply to /40
@nya:
@Cthulhu а конкретный случай — частность вообще
#1194508/43
from Gajim, 6 months ago, in reply to /42
@Cthulhu:
@nya Ну тогда я могу тебе сказать, что if .. else — говно, потому что если мне нужно написать
if (a==1) {
...
} else if (a==2) {
...
}
else if (a==3) {
...
}
и тд, и поэтому всем рулит свитч )
#1194508/44
from Miranda, 6 months ago, in reply to /43
@nya:
@Cthulhu и вообще, красивый код, который можно распевать — это хороший код
#1194508/45
from Gajim, 6 months ago, in reply to /44
@jtootf:
@Cthulhu паттерн-матчинг!
#1194508/46
from lagrange, 6 months ago, in reply to /44
@0xd34df00d:
@Cthulhu Я как бы в итоге так и сделал везде. Просто одна из пришедших мыслей была оформлена в этом посте.
#1194508/47
from Azoth_primary, 6 months ago, in reply to /30
@0xd34df00d:
@Cthulhu Только я триграфы в три строки пишу в среднем, а так да.
#1194508/48
from Azoth_primary, 6 months ago, in reply to /32
@0xd34df00d:
@nya Тогда будет жопа с областью видимости.
ДА ТЫ ЖЕ НИАСИЛИЛ ПЛЮСЦЫ,
#1194508/49
from Azoth_primary, 6 months ago, in reply to /35
@nya:
@0xd34df00d да мне похуй, у меня БЕТОН
#1194508/50
from Home, 6 months ago, in reply to /49

Тухлый Александр (0xd34df00d)

Subscribe

Replies (48)

Messages

Tags all

Search

Statistics