- 1
SELECT users.nick,group.name FROM users,group WHERE user.id='$id' AND group.id=user.group;
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−873
SELECT users.nick,group.name FROM users,group WHERE user.id='$id' AND group.id=user.group;
Это мой код. Мне сказали что это говнокод, и порекомендовали спросить здесь.
В чём именно говнокодость, и как надо писать правильно?
+352
15000 гет!
Багровые войны гетам не помеха.
+48
auto filename = std::string{};
0
reinterpret_cast<ppu_function_t>(static_cast<std::uintptr_t>(ppu_ref<u32>(cia)))(*this);
Лопни глазоньки. Шаблоны увечат С++ как бог черепаху.
https://github.com/RPCS3/rpcs3/blob/91d06a97296e5e418fb601284577475d8c7dfbaf/rpcs3/Emu/Cell/PPUThread.cpp
−2
Новый год по владимирскому времени! Всех с.
Больше говнокодов в новом году.
−17
LEA EAX, [EBX]
Зачем Интел придумал лишнюю инструкцию с запутанной семантикой, если она эквивалентна MOV EAX, EBX?
−425
[self.userView sd_setImageWithURL:[comment.sender avatarThumbnailURL] placeholderImage:[UIImage coloredAvatarWithId:[NSString stringWithFormat:@"%lld", comment.sender.idValue]
andName:comment.sender.firstName
ofSize:CGSizeMake (57, 57)]];
self.dateLabel.text = [self tsFromDate:comment.createdAt];
[self.dateLabel sizeToFit];
if (comment.sender == [SWCore account])
{
self.balloonImage.image = [[UIImage imageNamed:@"in_baloon"] resizableStretchedImageWithCapInsets:UIEdgeInsetsMake (20, 50, 20, 50)];
self.messageLabel.textColor = [UIColor colorWithWhite:.4f alpha:1.0f];
self.userView.left = 20;
self.balloonImage.right = 300;
self.dateLabel.right = self.balloonImage.right;
self.dateIcon.right = self.dateLabel.left - 3;
}
else
{
self.balloonImage.image = [[UIImage imageNamed:@"out_baloon"] resizableStretchedImageWithCapInsets:UIEdgeInsetsMake (20, 50, 20, 50)];
self.messageLabel.textColor = [UIColor whiteColor];
self.userView.right = 300;
self.balloonImage.left = 20;
self.dateIcon.left = self.balloonImage.left;
self.dateLabel.left = self.dateIcon.right + 3;
}
Мастерское владение размерами
−43
bool ok = (state == 0) ? false : true;
И это пишет человек, пишущий на плюсах уже много лет...
+208
XOR EBX,EBX
MOV ECX,DWORD PTR SS:[EBP-168]
MOV DWORD PTR SS:[EBP-168],ECX
CMP EBX,DWORD PTR SS:[EBP-168]
JG ...
+133
if (strcmp(request_name, REQUEST_1) == 0)
function_1();
else if (strcmp(request_name, REQUEST_2) == 0)
function_2();
else if (strcmp(request_name, REQUEST_3) == 0)
function_3();
/* И так далее */
else if (strcmp(request_name, REQUEST_N) == 0)
function_n();
Собственно, как можно ЭТО рефакторить в более приемлемый вид и можно ли заменить на branchless?