Говнокод #12143 — Си — Говнокод.ру

Си / Говнокод #12143

+141
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
```
//void wyslij(int pin, char mode[] = "open") { //было
void wyslij(int pin, int mode) { //стало
  if (pin != -1)
  {
    if (mode == "open")
    {

//...

  else if ( (mode == "touch") && ( (error != 1) || (olej_error == 1) ) )
```
Кусок дипломной работы польского студента, код для ардуино. Выцарапано отсюда: http://vimeo.com/47656204, примерно с 1:15.

Запостил: Xom94ok, 17 Ноября 2012

Tweet
Комментарии (31) RSS
- bormand 17.11.2012 16:50 # +5
  
  У меня все работает, что я не так делаю? http://ideone.com/bVvWJo.
  
  [кэп]Работать будет пока компилятор объединяет одинаковые константы в одну. Ну и пока кто-то не додумается передать туда не константу.[/кэп]
  Ответить
  - inkanus-gray 17.11.2012 18:29 # 0
    
    Зато как быстро работает! И не нужно вводить неинтуитивные числовые константы.
    Указатели-то побыстрее, чем строки, сравниваются.
    Ответить
    - bormand 17.11.2012 19:13 # 0
      
      Можно еще так - 'open', 'touc'. Но оно тоже компиляторозависимо (по стандарту компилятор может хранить такие литералы так, как ему захочется).
      Ответить
  - gb12335 18.11.2012 01:00 # +1
    
    Самое смешное будет, когда будет передана неконстантная строка.
    Ответить
    - defecate-plusplus 18.11.2012 01:03 # +2
      
      а твое устройство тоже не позволяет выделить и посмотреть белый текст в сообщении @Bormann?
      Ответить
      
      gb12335 18.11.2012 01:05 # 0
      
      Ну, я предполагаю, что неконстантная строка была получена откуда-то из другого источника.
      
      http://ideone.com/OjmQni
      
      Скажем, мы эти строчки считали из файла =)
      Ответить
      
      defecate-plusplus 18.11.2012 01:19 # +2
      
      > @bormand: Ну и пока кто-то не додумается передать туда не константу.
      Ответить
      
      gb12335 18.11.2012 01:20 # +1
      
      А если у нас межмудольное воздействие? Линкер, думаете, будет все константы грести в кучу?
      Ответить
      
      defecate-plusplus 18.11.2012 01:39 # +1
      
      строковые константы, по идее, имеют internal linkage, но нормальный оптимизирующий линкер их таки сгребет в одну кучу
      Ответить
      
      gb12335 18.11.2012 01:45 # −1
      
      Если не подставит на месте. Оптимизирующий компилер может маленькую строку вообще в регистры засунуть, не?
      Ответить
      
      defecate-plusplus 18.11.2012 01:52 # +1
      
      не слышал о таком
      все библиотечные функции, работающие со строками, работают с указателем - им нужен указатель в регистре
      код в топике тоже оперирует указателем
      Ответить
      
      bormand 18.11.2012 10:41 # +2
      
      То что строки упихаются в регистр - это конечно неправда, как с ней потом работать? Но вот кстати, gcc вполне справляется с strcmp("test", "test"), выкидывая его нахрен, и заменяя ноликом, а strlen("test") четверкой...
      Ответить
      
      bormand 18.11.2012 10:43 # +1
      
      Об этом я тоже писал: Работать будет пока компилятор объединяет одинаковые константы в одну.. Ну ок, если придираться к словам - то не только компилятор, но еще и линкер.
      Ответить
- inkanus-gray 17.11.2012 18:36 # +6
  
  В Си наконец-то появился оператор сравнения строк!
  Ответить
- defecate-plusplus 17.11.2012 18:52 # +4
  
  а что такое "вислый"?
  + "открыть", "потрогать"
  Ответить
  - Xom94ok 17.11.2012 19:08 # +1
    
    [captain]http://translate.google.ru/#pl/ru/wyslij[/captain]
    Но "вислый" мне больше нравится :-)
    Ответить
- guest 17.11.2012 19:40 # +2
  
  Так вот почему польские игры такие плохие.
  Ответить
  - Psionic 19.11.2012 03:38 # 0
    
    Painkiller/BulletStorm?
    Ответить
- Govnocoder#0xFF 18.11.2012 17:19 # +2
  
  > int mode
  > mode == "open"
  > числовая переменная сравнивается со строкой
  > Cи
  WTF_am_I_reading.jpg
  Ответить
- movaxbx 18.11.2012 22:20 # +2
  
  Лучше так http://ideone.com/s4t0n8
  Ответить
  - bormand 18.11.2012 22:26 # +5
    
    Сорри за маты: http://ideone.com/TtXkAN. Но для демонстрации надо было в рифму.
    Ответить
    - TarasB 18.11.2012 22:32 # 0
      
      пизда равна джигурде?
      Ответить
      
      bormand 18.11.2012 22:36 # +2
      
      Ага. Стандарт не оговаривает в каком виде хранятся multi-character literals. В данном случае авторы компилятора решили хранить последние 4 байта литерала (в нашем случае "уй" и "да" в кодировке utf-8).
      Ответить
      
      TarasB 18.11.2012 22:44 # +4
      
      http://ideone.com/9c0SD9
      
      Бля, у тебя кавычки одинарные.
      Наебал, наебал...
      Ответить
      
      LispGovno 19.11.2012 07:37 # −1
      
      А почему в UTF-8? Это особнность multi-character literals в gcc?
      Ответить
      
      defecate-plusplus 19.11.2012 07:52 # +2
      
      дефолтная кодировка под *nix ведь, на которой и работает идеоне
      Ответить
      
      LispGovno 19.11.2012 08:52 # −1
      
      То есть если я напишу в линухе: 'ху', то получу UTF8, если "ху", то однобайтовую кодировку, а если L"ху", то UTF32?
      
      А если я напишу в винде: 'ху', то получу однобайтовую кодировку, если "ху", то однобайтовую кодировку, а если L"ху", то UTF16?
      Ответить
      
      defecate-plusplus 19.11.2012 09:14 # +4
      
      исходный кот записан в какой-то конкретной кодировке
      компилятор всё, что встретит русского в '' или "" будет считать, как тупо последовательность байт, которую записал программист
      и ему будет всё равно, это был UTF-8 или бНОПНЯ, или cp-866
      в частности, русский текст исходного кода на никсовом ideone был UTF-8, и потому в программе он ведет себя как UTF-8
      
      а если ты запишешь L"", это будет const wchar_t array[N];
      Ответить
      
      LispGovno 19.11.2012 09:48 # −1
      
      Начинает проясняться.
      
      ""- в какой кодировке написал, то и будет, понятно.
      А если написать L"", то gcc придется провести перекодировку в wchar_t. Студия соответственно перекодирует в UTF16, а gcc в UTF32. Так вот как студия и gcc определят исходную кодировку, ведь чтобы перекодировать - её нужно знать. Ключами компиляции как-то указывается?
      И ещё получается, если исходник в кодировке UTF16 под виндой или UTF32 под линухом, то мы в "" получаем wchar_t, но зариентерпреткастенный в обычный char[N]?
      Ответить
      
      defecate-plusplus 19.11.2012 10:03 # +2
      
      подозреваю, что при L"unknown multibyte", если совсем неизвестно, используется пользовательская локаль, либо определенный utf, если компилятор понимал, что файл-то юникодный (типа есть BOM в начале)
      кстати, поддерживает ли gcc 4.x BOM - не знаю
      вот 3.х, сука, точно не поддерживает - периодически студия сохраняет русско-содержащие файлы в utf-8, и потом у gcc возникают непреодолимые трудности - как раз тот случай, если видишь эту ошибку в первый раз, то не понимаешь, что случилось
      Ответить
    - roman-kashitsyn 19.11.2012 00:45 # +3
      
      Занятное эхо
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

Си / Говнокод #12143

Комментарии (31) RSS

Добавить комментарий