es gibt eine Funktion GetData(int &len, unsigned char &data).
wie muss ich der Funktion mein Array übergeben?

Mein erster Versuch erzeugt einen leeren buffer:
int len=255;
unsigned char buf[256];
memset(buf,0,256);
GetData(len,buf[0]);

Mein zweiter Versuch ebenfalls:
int len=255;
unsigned char buf = new unsigned char[256];
memset(buf,0,256);
GetData(len,*buf);

Was mache ich falsch?

Allerdings wird das Array nicht kopiert, sondern es wird nur ein Zeiger übergeben. Wenn du eine Kopier automatisch willst, kannst du einen Standardcontainer nehmen, wie z.B std::vector, welcher auch dynamisch ist.

Die Schreibweisen buf[0] und *buf sind übrigens gleichwertig. Was geht denn an dem nicht?

GetData(len, buf[0])

Aber es ist merkwürdig, dass der unsigned char als Referenz übergeben wird. Bei Arrays ist es meistens ein Zeiger, oder dann eine Referenz auf das gesamte Array.

Für Array-Container würde ich std::tr1::array empfehlen.

...GetData(int &len, unsigned char &data)...

Hi,

bist Du sicher, dass diese Funktion für die Array-Übergabe geeignet ist?
Wie Nexus empfinde ich das als hochgradig unüblich...
Außerdem: Vielleicht ist es gar nicht verkehrt, wenn Du 0-en zurückbekommst.

drakon schrieb:

const unsigned int len=256;
unsigned char buf[len];
//füllen
GetData(len,buf);

...

Für die Schnittstelle kann len allerdings nicht const sein...

Gruß,

Simon2.

Für die Schnittstelle kann len allerdings nicht const sein...

Stimmt, das kommt noch dazu.

Was ist das für eine Funktion, die die Länge möglicherweise verändert? Falls das wirklich für Arrays gedacht ist und die Länge geändert werden soll, geht das sowieso nur mit dynamischen Arrays. Aber selbst dann ist die Signatur komisch...

Simon2 schrieb:

Für die Schnittstelle kann len allerdings nicht const sein...

Stimmt, das kommt noch dazu.

Was ist das für eine Funktion, die die Länge möglicherweise verändert? Falls das wirklich für Arrays gedacht ist und die Länge geändert werden soll, geht das sowieso nur mit dynamischen Arrays. Aber selbst dann ist die Signatur komisch...

Es geht darum Daten einzulesen und in len steht dann die tatsächlich gelesene Anzahl.
Da das mit buf[0] und *buf nicht funktioniert scheint wohl die Funktion Macken zu haben...

Fordert die Funktion selber genügend Speicher an oder musst du sicher sein, dass du ein genügend grosses Array hast? Solche Dinge sind eben gefährlich und können zu Pufferüberläufen führen.

Für die Schnittstelle kann len allerdings nicht const sein...

Jop. War nicht ganz dabei. Bin gerade sowieso ziemlich genervt..

GetData(size_t len, unsigned char *data);

lauten und nicht

GetData(size_t len, unsigned char &data);

unsigned int& kann nicht unsigned int* referenzieren...

...Was ist das für eine Funktion, die die Länge möglicherweise verändert? Falls das wirklich für Arrays gedacht ist und die Länge geändert werden soll, geht das sowieso nur mit dynamischen Arrays. Aber selbst dann ist die Signatur komisch...

Also aus den reinen C-Umfeld ist mir das sattsam bekannt: len ist dabei ein "update-Parameter":
- Input: Länge des bereitgestellten Buffers
- Output: Länge der tatsächlich genutzen Einträge (natürlich <= Input)
Hat diverse Nachteile (z.B. weiß man bei output=input nicht, ob das nun tatsächlich alle Daten waren), ist aber trotzdem eine durchaus gängige Methode.
Das heißt: Die Länge des Buffers wird gar nicht verändert.

Das kann ich natürlich nicht 100%ig auch für diesen Fall sagen, aber ich halte es für sehr wahrscheinlich.

Gruß,

Simon2.

Die Schnittstelle stimmt nicht. Sie muss

GetData(size_t len, unsigned char *data);

lauten ...

Ich vermute ja auch, dass das eigentlich gemeint ist (allerding mit len als & oder ... aber andererseits fällt mir gerade kein Argument ein es so zu machen:

GetData(size_t& len, unsigned char &data) {
   for(size_t i=0; i<len && i<23 ; ++i) {
      (&data)[i] = 0x12;
   }
   len = i;   
}

Wäre das verboten/falsch/unmöglich?

Gruß,

Simon2.

Simon2 schrieb:

Für die Schnittstelle kann len allerdings nicht const sein...

Jop. War nicht ganz dabei. Bin gerade sowieso ziemlich genervt..

Mein herzliches Beileid!
Solche Sachen sind extrem ätzend...

*daumen-drück*

Gruß,

Simon2.

Die Schnittstelle stimmt nicht. Sie muss

GetData(size_t len, unsigned char *data);

lauten und nicht

GetData(size_t len, unsigned char &data);

unsigned int& kann nicht unsigned int* referenzieren...

Sie kann aber die Adresse des ersten Wertes eines Arrays übergeben.

Da ich die Funktion aus einer Dll aufrufe, dessen Quellcode mir nicht vorliegt, werd ich mal den Entwickler kontaktieren.

unsigned int& kann nicht unsigned int* referenzieren...

Aber unsigned int und dann indirekt den Zeiger über dessen Adresse. Wie kommst du eigentlich auf size_t ?

Simon2 schrieb:

Ich vermute ja auch, dass das eigentlich gemeint ist (allerding mit len als & oder ... aber andererseits fällt mir gerade kein Argument ein es so zu machen:

Daran habe ich eben auch gedacht, also dass intern die Adresse genommen wird und so die nächsten Elemente des Arrays gelesen werden können.

MastaEda schrieb:

Da ich die Funktion aus einer Dll aufrufe, dessen Quellcode mir nicht vorliegt, werd ich mal den Entwickler kontaktieren.

Hast du denn keine Dokumentation dazu? Falls doch, was sagt die zu den erwarteten Parametern?

Es hat sich jetzt herausgestellt dass ... das mit dem buf[0] bzw *buf funktionieren würde.

Das entspricht ja auch der Signatur der Funktion. Aber so kann man kein Array übergeben.

MastaEda schrieb:

Es hat sich jetzt herausgestellt dass ... das mit dem buf[0] bzw *buf funktionieren würde.

Das entspricht ja auch der Signatur der Funktion. Aber so kann man kein Array übergeben.

Doch wie schon erwähnt wird die Adresse, des ersten Arrayelements übergeben. Mit Hilfe der Längen-Variablen weiß die Funktion dann wieviel Speicher beschrieben werden darf.

Naja, normalerweise hat man das Problem eh nicht, da man mit Containern arbeitet. Da kann man sich auch veränderliche Dimensionsparameter sparen...
Bei einer DLL würde ich aber wenigstens eine C-Style-Funktion erwarten.