C18 - Array als Rückgabeparameter

[Christian H. (jhetfield)]

Hallo,

ich möchte ein Array als Rückgabeparameter einer Funktion nutzen, 
bekomme aber einen syntax Error beim compilieren. Viellicht kann jemand 
helfen. Ich hänge mal meinen Code unten an:

void main (void)
{
  char dummy_SPI_data;
  char *aqfb_bytes;
  
  //...
  
  aqfb_bytes = poll_Single();

  

  while(1){} 
}


char * poll_Single(void)
{
  char i=0;
  char poll_response[16];
    
  green_LED = 0; //clear green LED
  red_LED = 0; //clear red LED
  dummy_SPI_data=write_SPI_data(poll_Single);  
  dummy_SPI_data=write_SPI_data(0x00);  //AFI code for all Type B cards
   dummy_SPI_data=write_SPI_data(0x00);  //wake command, using 1 slot for polling
  while(rfid_iSTAT==0){} //waiting for Reader to handshake
  
  //reading response to poll single command
  while(rfid_iSTAT)
  {
    poll_response[i] = read_SPI_data();
    i++;
  }
  
  //clearing the remaining of response data field 
  while(i<16)
  {
    poll_response[i] = 0;
    i++;
  }
  return poll_response;
}

Der Prototyp:

char * poll_Single(void);

Danke
Gruß
Christian

[Rufus Τ. F. (rufus)]

Was Du da machst, funktioniert nicht. Du gibst die Adresses einer 
lokalen Variablen (das Array "poll_response") zurück. Lokale Variablen 
aber sind nur solange gültig, wie der Kontext existiert, in dem sie 
definiert sind. Bei Verlassen dieses Kontextes, hier also Deiner 
Funktion "poll_single", verlieren automatische Variablen ihre 
Gültigkeit. Der Speicher, den die Variable belegt hat, kann und wird 
fortan für andere Dinge verwendet.



Mit dem Syntaxfehler allerdings hat das nichts zu tun.

Üblicherweise geben Compiler noch nähere Hinweise aus; in welcher Zeile 
z.B. so ein Fehler auftritt.

Beim Überfliegen Deines unvollständigen Codeschnipsels sehe ich nicht 
sofort einen Syntaxfehler.

Also: Vollständigen Code posten, und vollständige Fehlermeldung angeben.

[Christian H. (jhetfield)]

Hallo und danke zunächst mal für deine Antwort.

Rufus Τ. Firefly schrieb:
> Was Du da machst, funktioniert nicht. Du gibst die Adresses einer
> lokalen Variablen (das Array "poll_response") zurück. Lokale Variablen
> aber sind nur solange gültig, wie der Kontext existiert, in dem sie
> definiert sind.

Ist mir auch schon aufgefallen. Mit "static" funktioniert es aber auch 
nicht.

Der Compiler verweisst auf die Funktionsdeklaration in der Header Datei.

F:\Eigene Dateien\Microchip 
Projects\RandomAccess\RandomAccess.h:29:Error: syntax error


Hier mal der komplette Code:

// config-settings siehe Help-->Topics-->PIC18 config settings

#pragma config OSC = HS //clock source is high speed crystal
#pragma config PWRT = ON  //power up timer enabled
#pragma config BOR = OFF //Brown out Reset disabled
#pragma config WDT = OFF  //watchdog timer off
#pragma config LVP = OFF  //Single-Supply ICSP disabled 
//#pragma config DEBUG = ON
//#pragma config PBADEN = OFF  //turns ADC on port B off

#include <p18f458.h>
#include <stdio.h>
#include <delays.h>

#include "RandomAccess.h"


void main (void)
{
  char dummy_SPI_data;
  char *aqfb_bytes;

  //setting status LEDs
  TRISAbits.TRISA1 = 0; //making RA1 an output pin
  TRISAbits.TRISA0 = 0; //making RA0 an output pin
        

  //configure all neccessary pins ('1' means input, '0' makes the     pin an output pin)
    TRISCbits.TRISC1 = 1; //Reader Status bit
  TRISCbits.TRISC1 = 0; //Reader Reset bit
  TRISCbits.TRISC2 = 0; //SPI CS
    TRISCbits.TRISC3 = 0; //SPI Clock
  TRISCbits.TRISC4 = 1; //SPI Data in
    TRISCbits.TRISC5 = 0; //SPI Data out

  //init MSSP for SPI communication
    SSPSTATbits.SMP = 0;  //input data sampled at middle of data output
  SSPSTATbits.CKE = 1;  //transmission occurs from idle to active clock
  SSPCON1bits.CKP = 0; //Idle state of clock is a low level

  //spi clock is fosc/64
  SSPCON1bits.SSPM3 = 0;
  SSPCON1bits.SSPM2 = 0;
  SSPCON1bits.SSPM1 = 1;
  SSPCON1bits.SSPM0 = 0;

  SSPCON1bits.SSPEN = 1;  //enables serial port function on  dedicated i/o pins
  
  INTCONbits.GIEH = 0;  //disable all interrupts

  
  rfid_reset = 1; //turning reset for reader off
  turn_RX_off();   
  turn_RX_on();
  aqfb_bytes = poll_Single();

  

  while(1){} 
}



void delay(int _10ms)
{
  int i;
  for (i=0;i<_10ms;i++)
    Delay10KTCYx(10);
}

char write_SPI_data(char data)
{
  rfid_cs = 0;  //chip select for RFID Reader pulled low
  SSPBUF = data;  //write to SPI buffer register. data is sent when register is completely written (8 bits)
  while (!SSPSTATbits.BF){}  //wait for buffer full status bit is set
  rfid_cs = 1;  //de-select RFID Reader by releasing CS signal line 
  return SSPBUF;  //SSPBUF must be read before next read/write operation can be started
}

char read_SPI_data(void)
{
  TRISCbits.TRISC5 = 1; //while in "receive only mode" the SPI Data out pin should be disabled (programmed as input) 
  rfid_cs = 0;  //chip select for RFID Reader pulled low
  SSPBUF = 0x00; //dummy data written to SSPBUF register to trigger clock generation for reading data from slave 
  while (!SSPSTATbits.BF){}  //wait for buffer full status bit is set
  rfid_cs = 1;  //release CS signal line
  TRISCbits.TRISC5 = 0; //reconfigure SPI data out pin
  return SSPBUF;
}


void turn_RX_off(void)
{
  green_LED = 0; //clear green LED
  red_LED = 0; //clear red LED
  dummy_SPI_data = write_SPI_data(rf_off);  //sending SPI Command to RFID Reader
  while(rfid_iSTAT==0){} //waiting for Reader to handshake
  if (read_SPI_data() && 0b00000001 == 1)  //checking for acknowledge bit
     green_LED = 1;
  else
    red_LED = 1;
}


void turn_RX_on(void)
{
  green_LED = 0; //clear green LED
  red_LED = 0; //clear red LED
  dummy_SPI_data = write_SPI_data(rf_on);  //sending SPI Command to RFID Reader
  while(rfid_iSTAT==0){} //waiting for Reader to handshake
  if (read_SPI_data() && 0b00000001 == 1)  //reading ACK/NACK bit
     green_LED = 1;
  else
    red_LED = 1;
}

void abort_RFID(void)
{
  green_LED = 0; //clear green LED
  red_LED = 0; //clear red LED
  dummy_SPI_data = write_SPI_data(abort);  //sending SPI Command to RFID Reader
  while(rfid_iSTAT==0){} //waiting for Reader to handshake
  if (read_SPI_data() && 0b00000001 == 1)  //reading ACK/NACK bit
     green_LED = 1;
  else
    red_LED = 1;
}

char * poll_Single(void)
{
  char i=0;
  static char poll_response[16];
    
  green_LED = 0; //clear green LED
  red_LED = 0; //clear red LED
  dummy_SPI_data=write_SPI_data(poll_Single);  
  dummy_SPI_data=write_SPI_data(0x00);  //AFI code for all Type B cards
   dummy_SPI_data=write_SPI_data(0x00);  //wake command, using 1 slot for polling
  while(rfid_iSTAT==0){} //waiting for Reader to handshake
  
  //reading response to poll single command
  while(rfid_iSTAT)
  {
    poll_response[i] = read_SPI_data();
    i++;
  }
  
  //clearing the remaining of response data field 
  while(i<16)
  {
    poll_response[i] = 0;
    i++;
  }
  return poll_response;
}

und der Header

#ifndef PICKIT_H
#define PICKIT_H

//defining inport pins
#define rfid_iSTAT PORTCbits.RC0 
#define rfid_data_in PORTCbits.RC4

//defining outport pins
#define rfid_reset LATCbits.LATC1
#define rfid_cs LATCbits.LATC2
#define rfid_clock LATCbits.LATC3
#define rfid_data_out LATCbits.LATC5
#define red_LED LATAbits.LATA1
#define green_LED LATAbits.LATA0

//define SPI RFID Reader commands
#define abort 0x0D
#define rf_on 0x0A
#define rf_off 0x0B
#define poll_Single 0x01


//** P R O T O T Y P E S ******************************
char write_SPI_data(char data);
char read_SPI_data(void);
void delay(int _1ms);
void turn_RX_off(void);
void turn_RX_off(void);
char * poll_Single(void);


#endif

[Rufus Τ. F. (rufus)]

> Der Compiler verweisst auf die Funktionsdeklaration in der Header Datei.
>
> F:\Eigene Dateien\Microchip
> Projects\RandomAccess\RandomAccess.h:29:Error: syntax error

Kein Wunder:


> #define poll_Single 0x01
>
> char * poll_Single(void);

[Christian H. (jhetfield)]

Oh ja, sorry. Ich hab grad erst wieder angefangen nach etlichen Jahren 
Abstinenz.

Ich danke dir vielmals!