C: extern Funktionen

Hallo zusammen,
ich bin beim C-programmieren ein anfänger. Ich habe folgendes Ziel:
Ich will eine Funktion programmieren, die ein Array aus nullen und einser erzeugt.
so ist das Prinzip der Funktion:
name: GenerateBitStream(int *bitfolge, 13)
also die funktionen erzeugt einen Array "Bitfolge" der länge 13. Inhalt vom Array ist zufällige Folge von Null und eins.
Ich bin folgender Massen gegangen:
1: Ich habe folgende Header Datei geschrieben (BitStream.h):

---------------------------
#ifndef BitStream_h
#define BitStream_h

#include<stdio.h>
#include<stdlib.h>

extern int *GenerateBitStream(int*, int );

#endif
--------------------------

2: dann habe ich die funktion GenerateBitStream.c geschrieben:

---------------------------
#include"BitStream.h"
#include<stdio.h>
#include<stdlib.h>
#include<time.h>

int *GenerateBitStream(int *bits, int length){

srand48(time(NULL));

for (int i = 0; i <= length-1; i++) {
if (drand48() < 0.5)
*(bits+i) = 1;
else
*(bits+i) = 0;
}
return bits;
}
----------------------

zum Testen der Funktion habe ich folgendes Testprogramm (test.c)geschrieben:
----------------------

#include<stdio.h>
#include"BitStream.h"

int main(){

int *BitSequence;
int i=0;

GenerateBitStream(BitSequence,10);

for (i=0; i<10; i++){
printf("%d ",*(BitSequence+i));
}
}
-----------------------------

Das Programm test.c habe ich so compiliert: "gcc test.c -o test" leider kommt folgende Fehlermeldung:

Undefined first referenced
symbol in file
GenerateBitStream /usr/tmp/ccp19vfe.o
ld: fatal: Symbol referencing errors. No output written to test
collect2: ld returned 1 exit status

Meine Frage wäre: gibts einen Fehler im Code? ist das zusammen binden der drei dateien test.c BitStream.h und GenerateBitStream so wie ich es gemacht habe richtig?

viele Grüsse
Dreisson

 

Jede Menge:

- Kein Speicherplatz fuer dein Array reserviert
- Array Elemente falsch addressiert, ein Integer ist 32 bit also
musst du schon 4 zur Basis mit jeder Iteration addieren
- Warum gibst du einen Pointer zurueck den du der Funktion als
Parameter uebergeben hast?
- Falsch kompiliert, wenn du GenerateBitStream.c nicht kompilierst
kann der Linker die Funktion auch nicht finden
.
.
.

Hier mal die fix korrigierte Version, sollte das tun was du willst.
4Byte fuer jedes Bit ist aber eine ziemliche Verschwendung

Code:

/* BitStream.h */
#ifndef BitStream_h
#define BitStream_h

#include<stdio.h>
#include<stdlib.h>

extern void GenerateBitStream(int*, int);

#endif

Code:

/* GenerateBitStream.c*/
#include "BitStream.h"
#include <stdio.h>
#include <stdlib.h>
#include <time.h>

void GenerateBitStream(int *bits, int length) {

srand48(time(NULL));

int i;

for (i = 0; i < length; i++) {
	bits[i] = (drand48() < 0.5)? 1 : 0;
	}
}

Code:

/* test.c */
#include <stdio.h>
#include "BitStream.h"

int main(){

int *BitSequence;
int i = 0;
int length = 10;

BitSequence = malloc(sizeof(int) * length);

GenerateBitStream(BitSequence, length);

for (i = 0; i < 10; i++) {
	printf("%d ", BitSequence[i]);
	}
}

(bei malloc sollte man noch aufpassen das man keinen
Nullpointer bekommt aber das hab ich jetzt mal weggelassen)

Und so kompilieren:

Code:

$ gcc -o tollesProgramm GenerateBitStream.c test.c

 

Hey! Danke für die ausführliche Antwort. Ich habe das Programm so implimentiert wie du es beschrieben hast. Allerdings Es kommen folgende Fehlermeldungen:

GenerateBitStream.c: In function `GenerateBitStream':
GenerateBitStream.c:13: parse error before `int'
GenerateBitStream.c:15: `i' undeclared (first use in this function)
GenerateBitStream.c:15: (Each undeclared identifier is reported only once
GenerateBitStream.c:15: for each function it appears in.)

ich kappiere nicht warum meckert er wegen dem i ?? Das i habe ich doch deklariert.