Externes #define ?

[Datenkeimling (Gast)]

Kann ich

#define XY

in eine externe Datei.h legen und dieses XY in zwei Dateien code1.c und 
code2.c verwenden? Ich habe es mit include Datei.h in beiden .c Dateien 
versucht. Wurde trotzdem nicht geunden.

[PittyJ (Gast)]

Bevor wir rätseln: zeig uns den Code / die Dateien.

[mar IO (Gast)]

Japp, das sollte funktionieren, aber anscheinend machst Du was falsch...

Datenkeimling schrieb:
> in eine externe Datei.h legen und dieses XY in zwei Dateien code1.c und
> code2.c verwenden?

Datei.h ist noch so extern oder intern wie code1.c und code2.c. Das Wort 
"extern" passt in diesem Fall nicht.

Datenkeimling schrieb:
> Ich habe es mit include Datei.h in beiden .c Dateien
> versucht. Wurde trotzdem nicht geunden.

Include muss ganz am Anfang stehen und ansonsten könnte man noch weiter 
raten was da los ist, da keinen Code und so...

Vllt. hilft dir auch dieses Kapitel weiter, da sollte ein Beispiel mit 
drinnen sein

http://openbook.galileocomputing.de/c_von_a_bis_z/010_c_praeprozessor_002.htm#mjead11e16556a0e94647592cad9e3c437

[Hans Ulli K. (Gast)]

Datenkeimling schrieb:
> Kann ich
>
> #define XY
>
> in eine externe Datei.h legen und dieses XY in zwei Dateien code1.c und
> code2.c verwenden? Ich habe es mit include Datei.h in beiden .c Dateien
> versucht. Wurde trotzdem nicht geunden.

Hoffentlich kennst du den Unterschied zwischen

#incude "extern.h"
und
#include <extern.h>

[Datenkeimling (Gast)]

Schon in dieser Konstellation wird das #define nicht gefunden (alle 
Dateien sind im Projekt eingebunden).


additional_adc_adc_example1.h:

//#ifndef ADDITIONAL_ADC_ADC_EXAMPLE1
//#define ADDITIONAL_ADC_ADC_EXAMPLE1


#define MENU_HEADER "\n\r" \
"===================================================\n\r" \
"Menu: press a key to change the configuration.\n\r" \
"---------------------------------------------------------\n\r"

//Prototypes
static void display_menu(void);

//#endif

additional_adc_adc_example1.c:

#include <string.h>
#include "asf.h"
#include "conf_board.h"
#include "additional_adc_adc_example1.h"

static void display_menu(void)
{
  uint8_t uc_char;

  puts(MENU_HEADER);
  //...
  
}

ergibt:
Error  1  'MENU_HEADER' undeclared (first use in this function)

[Jörg W. (dl8dtl) (Moderator)]

Datenkeimling schrieb:
> Error  1  'MENU_HEADER' undeclared (first use in this function)

Hier nicht:

$ cat > additional_adc_adc_example1.h
//#ifndef ADDITIONAL_ADC_ADC_EXAMPLE1
//#define ADDITIONAL_ADC_ADC_EXAMPLE1


#define MENU_HEADER "\n\r" \
"===================================================\n\r" \
"Menu: press a key to change the configuration.\n\r" \
"---------------------------------------------------------\n\r"

//Prototypes
static void display_menu(void);

//#endif
^D
$ touch asf.h
$ echo "#include <stdint.h>" > conf_board.h
$ cat > additional_adc_adc_example1.c
#include <string.h>
#include "asf.h"
#include "conf_board.h"
#include "additional_adc_adc_example1.h"

static void display_menu(void)
{
  uint8_t uc_char;

  puts(MENU_HEADER);
  //...
  
}
^D
$ avr-gcc -Wall -Wextra -mmcu=atmega1281 -Os -c additional_adc_adc_example1.c
additional_adc_adc_example1.c: In function ‘display_menu’:
additional_adc_adc_example1.c:10:3: warning: implicit declaration of function ‘puts’
additional_adc_adc_example1.c:8:11: warning: unused variable ‘uc_char’
additional_adc_adc_example1.c: At top level:
additional_adc_adc_example1.c:6:13: warning: ‘display_menu’ defined but not used

[Datenkeimling (Gast)]

Der uC hier ist n SAM3X8E mit gcc.

Keine Ahnung warum das bei dir funktioniert. Bei mir geht's immer noch 
net. Trotzdem, Danke.

73

[mar IO (Gast)]

Datenkeimling schrieb:
> ergibt:
> Error  1  'MENU_HEADER' undeclared (first use in this function)

Du hast vorhin von mehreren Dateien geschrieben... Wird wo anders das 
#define aufgelöst? - Wenn nicht, dann poste bitte alle Fehler.

[Datenkeimling (Gast)]

Nee, wird nirgends aufgelöst. Mach ich morgen mit weiter.
Aber im Moment hab ichn schlimmeres Problem: scanf verursacht böse 
Speicherfehler.

#include <string.h>
#include "asf.h"
#include "conf_board.h"

#define NUM_CHANNELS    (4)

/** ADC convention done mask. */
#define ADC_DONE_MASK   ( (1<<NUM_CHANNELS) - 1 )

/** Size of the receive buffer and transmit buffer. */
#define BUFFER_SIZE     NUM_CHANNELS

/** Reference voltage for ADC, in mv. */
#define VOLT_REF        (3300)


/** The maximal digital value */
#define MAX_DIGITAL     (4095)

#define STRING_EOL    "\r"
#define STRING_HEADER "\r\n\r\n-- ADC Example --\r\n" \
    "-- Compiled: "__DATE__" "__TIME__" --"STRING_EOL
    
#define MENU_HEADER "\n\r" \
    "===================================================\n\r" \
    "Menu: press a key to change the configuration.\n\r" \
    "---------------------------------------------------------\n\r"

/** ADC channel for potentiometer */
int g_i_selected_channel;



/** Global timestamp in milliseconds since start of application */
static volatile uint32_t gs_ul_ms_ticks = 0;



static void display_menu2(void)
{  
  puts(MENU_HEADER);
  printf ("Select channel: ");
  scanf("%d", &g_i_selected_channel); //  <-PROBLEM
  printf ("\rg_i_selected_channel = %d\r", g_i_selected_channel);
}




/**
 * \brief Display ADC configuration menu.
 */
static void display_menu(void)
{
  uint8_t uc_char;

  puts(MENU_HEADER);
  uc_char = (g_adc_test_mode.uc_trigger_mode == TRIGGER_MODE_SOFTWARE) ? 'X' : ' ';
  printf("[%c] 0: Set ADC trigger mode: Software.\r", uc_char);
...
...
...
  uc_char = (g_adc_test_mode.uc_offset_en) ? 'E' : 'D';
  printf("[%c] O: Enable/Disable offset for potentiometer channel.\r",
      uc_char);

  puts("    Q: Quit configuration and start ADC.\r");
  puts("=========================================================\r");
}

mit Aufruf in main:

...
...
  display_menu2();
  display_menu(); //show only
  start_adc();
...
...






vrursacht willkürlichen Speichermüll in der Ausgabe:




g_i_selected_channel = 0

===================================================

Menu: press a key to change the configuration.
---------------------------------------------------------

[X] 0: Set ADC trigger mode: Software.
[ ] 1: Set ADC trigger mode: ADTRG.
[ ] 2: Set ADC trigger mode: Timer TIOA.
[ ] 3: Set ADC trigger mode: PWMEvent Line.
[ ] 4: Set ADC trigger mode: Free run mode.
[E] T: Enable/Disable to transfer with PDC.
[D] S: Enable/D…‰±•to use user sequence mode.
[D] P: Enable/Disable ADC power save mode.
[D] G: Enable/Disable to set gain = 2x for potentiometer channel.
[D] O: Enable/Disable offset for potentiometer channel.
    Q: Quit configuration and start ADC.
================êêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêêj
Press any key to display configuration menu.


-- ADC Example --
-- Compiled: Jun 27 2013 21:36:39 --
Configure system tick to get 1ms tick period.


===================================================

Menu: press a key to change the configuration.

---------------------------------------------------------

Select channel:


uC istn SAM3X8E. Ich hab jetzt mehrere Datentypen für 
g_i_selected_channel und für den Spezifizierer für scanf ausprobiert 
weil ich eigentlich dachte der Speicher reicht nicht... hilft aber nicht 
weiter.

[Mark B. (markbrandis)]

Also mit gcc auf'm PC (MinGW) habe ich keine Probleme beim Kompilieren 
und zur Laufzeit, wenn ich:

-diverse Sachen rausschmeiße die mir nicht vorliegen:
#include "asf.h"
#include "conf_board.h"
"Pünktchen Pünktchen Pünktchen" Code

-die richtigen Header inkludiere:
#include <stdint.h>
#include <stdio.h>

Datenkeimling schrieb:
> uC istn SAM3X8E.

Die Information nützt uns eher wenig. Viel interessanter wäre es zu 
wissen, wie kompiliert wird. Von Hand an der Kommandozeile? Per 
makefile? Mit einer IDE (welche?)

Außerdem kann auch und gerade so ein Menü-Code problemlos auf dem PC 
übersetzt und getestet werden, wenn die µC-Anteile vernünftig gekapselt 
sind. Dafür haben die C-Götter das #ifdef erschaffen.

Ach ja und das hier:

> uint8_t uc_char;

schreibt man als:
char uc_char;
weil genau das gemeint ist. (char = character)