Проект из нескольких файлов. В основном стоят директивы #define, во включаемых заголовочниках проверяется через #ifndef и не видится. Помогает только явное указание в свойствах проекта.
F_CPU объявлен через свойства проекта, используется в delay.h - не вызывает ошибок
USART_BUF_SIZE объявлен через директиву #define в main.c и используется в USART.h - вызывает ошибку.
Полная версия этой страницы: Почему не работает #define
Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему > Программирование
Цитата(aspID @ May 22 2012, 19:10) 
F_CPU объявлен через свойства проекта, используется в delay.h - не вызывает ошибок
USART_BUF_SIZE объявлен через директиву #define в main.c и используется в USART.h - вызывает ошибку.
USART_BUF_SIZE объявлен через директиву #define в main.c и используется в USART.h - вызывает ошибку.
Лучше define объявлять в заголовочном файле. Там ему место, если использовать надо в нескольких местах.
Да и стражи включения лучше использовать классическим образом.
Цитата(aspID @ May 22 2012, 18:10)
Проект из нескольких файлов. В основном стоят директивы #define, во включаемых заголовочниках проверяется через #ifndef и не видится. Помогает только явное указание в свойствах проекта.
F_CPU объявлен через свойства проекта, используется в delay.h - не вызывает ошибок
F_CPU объявлен через свойства проекта, используется в delay.h - не вызывает ошибок
Оба этих способа нормальны и допустимы.
.
Цитата(mdmitry @ May 22 2012, 22:48)
Лучше define объявлять в заголовочном файле.
Предлагаете сделать дополнительный .h ?
Цитата(mdmitry @ May 22 2012, 22:48)
Да и стражи включения лучше использовать классическим образом.
Здесь, к сожалению, не понял, что имели в виду. Кроме #pragma once еще дополнять конструкцию ifndef-define?
Цитата(_Артём_ @ May 22 2012, 23:04)
Поэтому при использовании в нескольких местах #include "main.c"
Компилятор делает в нескольких местах такую директиву? Или я чего-то неправильно понял
Я Вам сейчас одну умную вещь скажу, только не обижайтесь (с)
Дело в том, что каждый .c файл транслируется отдельно и совершенно независимо от других.
Поэтому когда дело доходит до трансляции USART.c в нём (и в USART.h) не будет видно определения USART_BUF_SIZE, сделанное только в main.c.
Лучше сделать, например, так: собрать все настройки проекта в один .h-файл (константы, определения ног, светодиодов, вкл/выкл различных опций программы и тому подобное) и включать его первым во все .c-файлы проекта.
Дело в том, что каждый .c файл транслируется отдельно и совершенно независимо от других.
Поэтому когда дело доходит до трансляции USART.c в нём (и в USART.h) не будет видно определения USART_BUF_SIZE, сделанное только в main.c.
Лучше сделать, например, так: собрать все настройки проекта в один .h-файл (константы, определения ног, светодиодов, вкл/выкл различных опций программы и тому подобное) и включать его первым во все .c-файлы проекта.
Цитата(aspID @ May 22 2012, 19:11)
Предлагаете сделать дополнительный .h ?
Тоже вариант: его можно менять в зависимости от надобности и включить в USART.h - будет как конфиг проекта.
Цитата(aspID @ May 22 2012, 19:11)
Здесь, к сожалению, не понял, что имели в виду. Кроме #pragma once еще дополнять конструкцию ifndef-define?
Просто чаще встречается такая конструкция(хотя кому-то может и наоборот):
Код
#ifndef usart_tx_rx_h__
#define usart_tx_rx_h__
#endif
#define usart_tx_rx_h__
#endif
Цитата(aspID @ May 22 2012, 19:11)
Или я чего-то неправильно понял
Не...это я что-то неправильно понял.
Цитата
USART_BUF_SIZE объявлен через директиву #define в main.c и используется в USART.h - вызывает ошибку.
А точно ошибку, а не warning? У меня откомпилировалось.
P.S. но как Си-проект, а вы хотите Си++?
Цитата(SSerge @ May 22 2012, 23:34)
Я Вам сейчас одну умную вещь скажу, только не обижайтесь (с)
Вроде спрашиваю не для того, чтоб обижаться 
Цитата(_Артём_ @ May 22 2012, 23:45)
Тоже вариант: его можно менять в зависимости от надобности и включить в USART.h - будет как конфиг проекта.
Вообще, предполагалось, что USART.h - самостоятельная и отдельная вещь, навроде библиотеки, которую в дальнейшем можно будет использовать в разных проектах БЕЗ внесения в нее правок и необходимости перекомпиляции
Цитата(_Артём_ @ May 22 2012, 23:45)
Просто чаще встречается такая конструкция(хотя кому-то может и наоборот):
Код
#ifndef usart_tx_rx_h__
#define usart_tx_rx_h__
...
#endif
#define usart_tx_rx_h__
...
#endif
Вообще,
Код
#pragma once
имеет то же назначение. Если компилятор без косяков этом месте
Хотя, бывают и особо мнительные случаи использования
Код
#pragma once
#ifndef usart_tx_rx_h__
#define usart_tx_rx_h__
...
#endif
#ifndef usart_tx_rx_h__
#define usart_tx_rx_h__
...
#endif
Цитата(_Артём_ @ May 22 2012, 23:45)
А точно ошибку, а не warning? У меня откомпилировалось.
Так ворнинг руками и выдается. Просто сделана "затычка", которая если не видит - предупреждает и переобъявляет. И в данном контексте раз не видит - значит, ошибка. [Моя ошибка имеется в виду, только пока до конца не понимаю, в чем]
Да,
А по поводу организации модулей могу предложить метод.
Каждый модуль состоит из 3 файлов:
module.c
module.h
module_conf.h - он включён инклюдом в module.h.
Таким макаром файлы
module.c
module.h
лежат в библиотеке в единственном экземпляре и подключаются ко всем проектам, в которых в них есть нужда, а
module_conf.h копируется в каждый проект и модифицируется под него (например задаёт размер фифо для уара и кол-во активных уартов ну и т.д. и т.п)
Код
#pragma once
нынче можно сказать стала обыденностью и поддерживается всеми современными компиляторами.А по поводу организации модулей могу предложить метод.
Каждый модуль состоит из 3 файлов:
module.c
module.h
module_conf.h - он включён инклюдом в module.h.
Таким макаром файлы
module.c
module.h
лежат в библиотеке в единственном экземпляре и подключаются ко всем проектам, в которых в них есть нужда, а
module_conf.h копируется в каждый проект и модифицируется под него (например задаёт размер фифо для уара и кол-во активных уартов ну и т.д. и т.п)
Цитата(aspID @ May 22 2012, 19:57)
Вообще, предполагалось, что USART.h - самостоятельная и отдельная вещь, навроде библиотеки, которую в дальнейшем можно будет использовать в разных проектах БЕЗ внесения в нее правок и необходимости перекомпиляции
Цитата(aspID @ May 22 2012, 19:57)
Так ворнинг руками и выдается. Просто сделана "затычка", которая если не видит - предупреждает и переобъявляет. И в данном контексте раз не видит - значит, ошибка. [Моя ошибка имеется в виду, только пока до конца не понимаю, в чем]
Ну раз хотите как "библиотеку" то можно так: создаёте файл допустим usart_cfg.h:
Код
#pragma once
#define USART_BUF_SIZE 32
#define USART_BUF_SIZE 32
Файл usart_cfg.h вставляете в USART.h(include "usart_cfg.h") - так он будет виден в main.c и USART.c с одним и тем же USART_BUF_SIZE.
Но перекомпилироваться всё равно будет...
Цитата(aspID @ May 22 2012, 19:57)
Вообще,
имеет то же назначение. Если компилятор без косяков этом месте
Код
#pragma once
имеет то же назначение. Если компилятор без косяков этом месте
В общем - да, смысл тот же.
Цитата(demiurg_spb @ May 22 2012, 20:11)
лежат в библиотеке в единственном экземпляре и подключаются ко всем проектам, в которых в них есть нужда, а
module_conf.h копируется в каждый проект и модифицируется под него (например задаёт размер фифо для уара и кол-во активных уартов ну и т.д. и т.п)
module_conf.h копируется в каждый проект и модифицируется под него (например задаёт размер фифо для уара и кол-во активных уартов ну и т.д. и т.п)
Может сразу template.
Цитата(demiurg_spb @ May 22 2012, 20:11)
Каждый модуль состоит из 3 файлов:
module.c
module.h
module_conf.h - он включён инклюдом в module.h.
module.c
module.h
module_conf.h - он включён инклюдом в module.h.
И чтобы сократить количество файлов которые нужно редактировать можно завести один файл io_modules_conf.h, включить его во все module.h - и все настройки в нём сразу и менять.
надо же... даже все ваши идеи понял! 
Действительно, хорошие мысли! Надо взять на вооружение...
Действительно, хорошие мысли! Надо взять на вооружение...
QUOTE (aspID @ May 22 2012, 19:57)
Вообще,
имеет то же назначение. Если компилятор без косяков этом месте
Назначение то же, но работает не всегда так же. Не все компиляторы корректно обрабатывают разноименные ссылки (жесткие и/или символьные) на один и тот же файл. В принципе - да, это будет косяк компилятора.
CODE
#pragma once
имеет то же назначение. Если компилятор без косяков этом месте
Цитата(aspID @ May 23 2012, 00:33)
Действительно, хорошие мысли! Надо взять на вооружение...
Вот что получилось с утреца: USART_conf.h Код
#ifdef _COMMON_CONFIG_PRESENT_
#include "config.h"
#else
// Put parameters here
#endif
config.h #include "config.h"
#else
// Put parameters here
#endif
Код
#define _COMMON_CONFIG_PRESENT_
#define ENABLE_BIT_DEFINITIONS
#define USART_BUF_SIZE 32 // Cyrillic SMS len 70
В дальнейшем так и буду поступать, спасибо! #define ENABLE_BIT_DEFINITIONS
#define USART_BUF_SIZE 32 // Cyrillic SMS len 70
Цитата(aspID @ May 23 2012, 06:35)
Код
#ifdef _COMMON_CONFIG_PRESENT_
#include "config.h"
#else
// Put parameters here
#endif
#include "config.h"
#else
// Put parameters here
#endif
Код
#define _COMMON_CONFIG_PRESENT_
#define ENABLE_BIT_DEFINITIONS
#define USART_BUF_SIZE 32 // Cyrillic SMS len 70
В дальнейшем так и буду поступать, спасибо! #define ENABLE_BIT_DEFINITIONS
#define USART_BUF_SIZE 32 // Cyrillic SMS len 70
А Вы только в config.h определили #define _COMMON_CONFIG_PRESENT_? Тогда проверка не имеет смысла, всегда ветка else выполняется, так как символ не определен ранее.
И посмотрите, пожалуйста, как написаны заголовочные файлы, например, для Вашего компилятора. Посмотрите, что ещё подключается автоматически с файлом описания применяемого микропроцессора. Такой подход весьма распространен и, на мой взгляд, удобен.
Цитата(mdmitry @ May 23 2012, 15:04)
А Вы только в config.h определили #define _COMMON_CONFIG_PRESENT_?
Нет, только в опциях проекта. В файле config.h сделана проверка и #error, если вдруг _COMMON_CONFIG_PRESENT_ не определен. Иначе у препроцессора возникает неоднозначность, какой из .c файлов проекта первый обнаружит этот файл: если библиотечный, то "сие не есть гуд", а вот если main.c - хорошо. Поэтому, добивался однозначности.
Нажмите для просмотра прикрепленного файла - Вот так.
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.