Pointer Problem

[Andi79 (Gast)]

Hallo habe ein kleines Problem was mein leider recht dürftiges 
Verständnis von Pointern überschreitet.Ich habe folgenden Code 
geschrieben der auch soweit funktioniert.

int WritePICData(Uint8 *pData, Uint8 len, Uint8 SlaveAddr )
{

   Uint8 Res =0;
   Uint8 i = 0;

   struct I2CMSG SerMes;  
 
   GI2CMsgStatus = I2C_MSGSTAT_SEND_NOSTOP;


   SerMes.SlaveAddress = SlaveAddr<<1;
   SerMes.MasterAddress = I2C_MASTER_ADDR;
   SerMes.Offset= 0x80;
   SerMes.NumOfRcvBytes = 2;
   SerMes.NumOfSendBytes = 0x0B;
   
   for (i =0; i <len ;i++)
   {
      SerMes.MsgSendBuffer[i] = *(pData++);
   }
   
   SerMes.MsgSendBuffer[len]  = Checksum( (Uint8*)&SerMes, 15 );

   Res = I2CMTransceive( SlaveAddr, SerMes.NumOfSendBytes+DEFAULT_MESS_BYTES, (Uint8*)&SerMes.MasterAddress,
         SerMes.NumOfRcvBytes, (Uint8*)&SerMes.MsgRcvBuffer );  
           
   if (Res == I2C_NO_SLAVE)
   {
      return PCK_WRITE_DATA_ERROR;
   }
          
   return PCK_WRITE_DATA_OK;      
}

So jetzt soll ich aber meinen Pointer nicht dirket im Funktionsaufruf 
übergeben. Sondern der Pointer soll erst später übergeben werden.Dies 
soll über einen funktion geschehen. Weis nur nicht wie ich das nun 
realisieren soll?
Soetwas habe ich noch nicht gemacht. Hab die Pointer bisher immer direkt 
übergeben.

[Albert .. (albert-k)]

Andi79 schrieb:
> So jetzt soll ich aber meinen Pointer nicht dirket im Funktionsaufruf
> übergeben. Sondern der Pointer soll erst später übergeben werden.Dies
> soll über einen funktion geschehen. Weis nur nicht wie ich das nun
> realisieren soll?
Was heißt "später"? Meinst du das du WritePICData() aufrufst und die 
Funktion dann solange mit dem Daten versenden wartet bis über eine 
andere Funktion der Pointer geladen wurde?

Oder soll es die ganze Zeit senden und du aktualisierst halt irgendwann 
mal den Pointer?

Oder soll die Funktion selbst nach einem pointer fragen?

Es ist aus deiner Fragestellung nicht ganz klar was du willst.

[Andi79 (Gast)]

Sorry wenn ich micht ungeschickt ausgedrückt habe. Die Funktion selbst 
soll nach dem Pointer fragen.

[Albert .. (albert-k)]

Ok, nur damit ich es richtig verstanden habe:
- Du rufst WritePICData() auf, aber ohne einen pointer zu übergeben.
- Innerhalb von WritePICData() rufst du dann eine Funktion (ich nenne 
sie mal GetPointer) auf, die dir einen Pointer zurückliefert.
- Nach erhalt dieses Pointers läuft WritePICData weiter

Habe ich deinen Plan so richtig verstanden? Dann wäre der einfachste Weg 
den Pointer in einer static/globalen Variable zu speichern die nur 
GetPointer() kennt. Zusätzlich noch ein Flag dazu um festzustellen ob 
der Pointer den GetPointer() kennt auch gültig ist.
Durch irgendeine andere Funktion wird dann dieser Pointer geladen. Oder 
GetPointer selbst frägt woanders nach einem gültigen Pointer und dient 
somit nur als Interface.

[Andi79 (Gast)]

Ja ich denke das trifft es ganz genau nur wie würde meine Funktion 
GetPointer() den dann aussehen?

[Albert .. (albert-k)]

Könnte bsw. so aussehen:

MyCode.h

#ifndef _MY_CODE_H
#define _MY_CODE_H

extern Uint8* GetPointer(void);
extern void SetPointer(Uint8* pPointer);

#endif

MyCode.c

#include "MyCode.h"

Uint8* pData=NULL;  //pointer for Data

//Funktion die Pointer auf Daten ausliest
Uint8* GetPointer(void){
   
  return pData;
}

void SetPointer(Uint8* pPointer){
    pData = pPointer;
}

Das wäre die eifnachste Variante für das ganze. In WritePICData() musst 
du dann nur noch prüfen ob der zurückgegebene pointer ungleich NULL ist.

[Andi79 (Gast)]

Würde das dann für meine code z.B. so aussehen?

for (i =0; i <len ;i++)
   {
      SerMes.MsgSendBuffer[i] = Uint8* GetPointer(void){

                                                        return pData++;
                                                        }

  }

[Mark B. (markbrandis)]

Eher so in der Art:

uint8_t* ptr = NULL;
ptr = GetPointer();
if(ptr != NULL)
{
  //mach was mit dem Pointer
}
else
{
  //eventuell hier Fehlerbehandlung
}

Und nein, Du hast den Code im ersten Posting oben nicht selbst 
geschrieben, stimmt's? ;-)