k001
k001
:...
Page Summary
07:53 pm :: code wtf
[info]spb-nick.livejournal.com :: (no subject) [+3]
[info]fenster.livejournal.com :: (no subject) [+2]
[info]ketmar :: (no subject)

April 2032
        1 2 3
4 5 6 7 8 9 10
11 12 13 14 15 16 17
18 19 20 21 22 23 24
25 26 27 28 29 30
k001 [userpic]
[info]k001
5/22/09 07:53 pm
  Leave a comment     8 comments     Add to Memories     Tell A Friend     Link  
code wtf

Наткнулся на кусок кода. Не очень нравится. Как бы это поизящнее написать, но чтобы и недлинно, и понятно? Что-то вечером в пятницу ничего не приходит в голову.

if (flags & LOADED)
       sprintf(status, "loaded");
if (flags & DIRTY)
       sprintf(status, "%s%sdirty", status, (strlen(status)) ? ", " : "");

Tags: , , ,
back to top
Comments
[info]spb-nick.livejournal.com
5/22/09, 06:07 pm
  Reply     Link     Thread  

sprintf(status,
"%s%s%s",
((flags & LOADED) ? "loaded" : ""),
((flags & (LOADED | DIRTY)) ? ", " : ""),
((flags & DIRTY) ? "dirty" : ""));

back to top
[info]dmarck
5/22/09, 06:24 pm
  Reply     Link     Parent     Thread  

Неверно, кстати: в случае только DIRTY будет запятая.

Но, да, в случае ровно двух состояний я б использовал один принт (кстати, snprintf)

back to top
[info]spb-nick.livejournal.com
5/22/09, 06:28 pm
  Reply     Link     Parent  

Да, конечно с запятой поторопился, как всегда :) И да, sprintf - зло :)

back to top
[info]k001
5/22/09, 06:40 pm
  Reply     Link     Parent  

> (кстати, snprintf)

там status объявлен как char[255] или что-то в этом роде, и функция короткая, видно, что всё хорошо. snprintf — если печатать всего много.

back to top
[info]fenster.livejournal.com
5/22/09, 07:57 pm
  Reply     Link     Thread  

Лично мне кажется, что чем понятнее — тем лучше. Зачем здесь вообще sprintf?

status[0] = '\0';
if (flags & LOADED)
strcpy(status, "loaded");
if (flags & DIRTY)
{
if (status[0])
strcat(status, ", ");
strcat(status, "dirty");
}

back to top
[info]fenster.livejournal.com
5/22/09, 07:57 pm
  Reply     Link     Parent     Thread  

Блин, отступы забыл сделать через   и они похерились. Впрочем, всё и так понятно.

back to top
[info]shep256.myopenid.com
6/14/09, 04:20 am
  Reply     Link     Parent  

Используйте <pre>, не убивайте себя nbsp-ами.

Стандартный код для говносайтов:
<code><pre>if (foo) {
something
}</pre></code>

Именно там начинается и заканчивается код. Иначе будут некрасивые пустые строки в браузере.

Почему для *говно*сайтов, потому что нормальный верстальщик добавил бы в CSS простое правило: code { white-space: pre; } и тег pre был бы уже не нужен.

А зачем вообще code - ну так положено, код обрамлять тегом таким. В светлом будущем правильные программы будут извлекать из этого полезную информацию. Например, браузер нарисует код с подсветкой синтаксиса.

back to top
[info]ketmar
5/23/09, 06:37 pm
  Reply     Link  

оставить как есть, только strlen убрать.

back to top
  Leave a comment     8 comments