caxapa.ru :: clr

Mazur (10.09.2013 09:47, просмотров: 85) ответил Vit на default можно вписать перед верхним case и без break. clr_dsp_buf без () очень ёмко.

clr_dsp: // *.h //======================================================================== #define clr_dsp_buf memset(DSP_BUF, ' ', sizeof(DSP_BUF)); //======================================================================== //*.c clr_dsp_buf;

Ответить

- Когда рука тянется к "define" напиши "inline" :) Замените макрос функцией (можно inline). Макросы очень небезопасный механизм. Эта ваша ";" в макросе может неслабо "икнуться". - Скрипач(10.09.2013 10:32)
  - Сделал инлайн, получил что хотел и линейкой по рукам за это получать не буду. :) - Mazur(10.09.2013 11:46)
    - А с другой стороны. Почему за это ругаетесь, если скомпилировано и работает? Mazur(225 знак., 10.09.2013 11:49)
      - Повторяю в 100500 раз: надо не в дизассемблере ковыряться, а создавать работающую программу. Причём с вменяемым исходником. А эти ваши инлайны никакого отношения к созданию рабочей программы не имеют, то есть отвлекают от дела, причём самым SciFi(156 знак., 10.09.2013 12:37 - 12:40)
        
        Дело-то не в inline, дело в самом программисте. Или нет? - Bill(10.09.2013 13:05)
        
        Ну так я пытаюсь быть вежливым, поэтому не перехожу на личности :-) - SciFi(10.09.2013 13:06)
  - Да там этот макрос нафиг не нужен. Любой вменяемый человек знает, что такое "memset(buffer, ' ', sizeof(buffer));". SciFi(60 знак., 10.09.2013 10:51)
    - Указкой жестоко. Линейкой :) Скрипач(133 знак., 10.09.2013 10:58)
      - Если используется более чем в одном месте, то да, так как дублирование кода - это зло. Иначе - лишнее телодвижение. - SciFi(10.09.2013 11:03)
        
        Ибо HAL. При проектировании "сверху - вниз" никто не опускается до подсчета вызовов :) - Скрипач(10.09.2013 11:12)
        
        Я за такое проектирование: придумать интерфейс модуля, оформить в виде красивого хедера. Но тут же совсем другая история: просто ловим блох, так что это ни о чём. - SciFi(10.09.2013 11:14)
        
        Не всегда. Иногда код тот же самый, а объединять в функцию -- та самая преждевременная оптимизация, известно чем кончающаяся. Вообще такими штуками компилятор умеет заниматься. - fk0(10.09.2013 11:05)
        
        Я говорю про случаи, когда в одном экземпляре дублирующегося кода что-то исправил, а в другом - забыл. Поэтому и зло. - SciFi(10.09.2013 11:08)
        
        Бывает и обратное! В "оптимизированном" таким образом коде в общей для всех функции исправил... а оно не ко всем вариантам теперь подходит! - fk0(10.09.2013 11:19)
      - Да у него C вообще отобрать нужно. Слишком много вольности. В Паскале (с объектами) написал бы нормально, в ОО стиле, с методом clear и все дела. А C и C++ это два перегиба в разные стороны. В одном слишком мало нормальных средств и слишком много fk0(73 знак., 10.09.2013 11:03)
        
        Я уже писал где-то, что его нужно научить использовать printf. Зачем вообще этот "драйвер"? - Vladimir Ljaschko(10.09.2013 11:08)
        
        Printf низзя, sprintf можно ;) - Гyдвин(10.09.2013 11:23)
        
        Это нам нельзя ;) - Vladimir Ljaschko(10.09.2013 11:33)
        
        Да без этого вашего жырного printf любой проект влезает в Atmega8. А с printf перестаёт влезать в AT91SAM7X256. - fk0(10.09.2013 11:20)
        
        "Parole, parole, parole..." ;) - Vladimir Ljaschko(10.09.2013 11:22)
        
        Вот-вот. Вроде бы уговорили его переходить на Си, а он упорно кодит на ассемблере, только сишными буковками. Контрреволюционер :-) - SciFi(10.09.2013 11:10)
    - Любой профессионал библиотечных функций не использует! - fk0(10.09.2013 10:53)
- о ";" в макросах уже писали. без скобок выглядит как вызов функции с ошибкой (опиской), которая скорее дала бы warning, а не error - Vit(10.09.2013 10:06, ссылка)

Средства и методы разработки