NULL pointer assignment..

..med på en lytter

Har stregkoderne ikke en bestemt længde? I så fald hvad med at definere dit array som:
char kode_db[10000][20];

Du kopierer jo en streng til en pointer der ikke er initialiseret!!!
(du har kun et array af pointere, ikke noget memory...)

pointer array er altid meget komplex i disse string tider. Men kunne du ikke nøjes med et char [] istedet for så er der garenterede ikke noget problem

Men nu har du et pointer array så prøv et af de to metoder.

først & tegnet.

strcpy(kode_db[i],&ny_streg_kode);

eller

strcpy(*kode_db[i],&ny_streg_kode);

Brug strdup istedet. Den allokere selv memmory for strengen.

kode_db[i] = strdup(ny_streg_kode);

Det som mickni33 gør er farligt. Hvis ny_streg_kode er temporer eller på anden måde bliver deallokeret senere bliver strengen i kode_db illegal.

Husk at deallokere det memory strdup allokerede med free()

<<farligt>> lyder lidt voldsomt :) det er jo bare kode :)

Jeg kigger lidt på dem og vender tilbage...

hvorfor bruger du ikke en dynamisk struktur som fx en vector?

tror ikke man kan bruge en vector fra STL da der arbejdes med c struct. Har selv haft problemer med at arbejde med structure når man blander med c++ kode.

=>morten2
Du skriver \"Hvis ny_streg_kode er temporer eller på anden måde bliver deallokeret senere bliver strengen i kode_db illegal\".
Jeg tror ikke mickni33\'s mening er (håber ikke) at kopiere ny_streg_kode\'s adresse ind i arrayet, men blot forestiller sig at ny_streg_kode er defineret som en char-array og ikke en pointer.
Måske mickni33 vil kommentere?

Kan ikke lide(eller ikk finde ud af det) at arbejde med char*[], kan man egentlig ikke lave

kode_db[i]=ny_streg_kode; 

istedet

Man kan godt, men det kræver jo netop, at ny_streg_kode ikke går ud af scope (fjernes fra stakken) og det er absolut ikke anbefalelsesværdigt!

Er det en C eller C++ compiler du har?

Null pointer assignment, får du fordi dit char* array ikke har fået tildelt nogen memory endnu. Hvis du ikke kan (eller vil !) tildele fast plads til de 10000 stregkoder fra starten af, skal du sørge for at alle pointere har værdien NULL til at begynde med, og så allokere hukommelse til dem efterhånden som du får brug for det:

#include <string.h>
const int TRUE  = 1;
const int FALSE = 0;
const int maxBarcodes = 10000;
int  nbrBarcodes;
char *barCodes[maxBarcodes];

for (nbrBarcodes 0; nbrBarcodes < maxBarCodes; idx++)
  barCodes[idx] = NULL;
nbrBarCodes = 0;
:
:
int storeBarCode(char *newBarCode)
{
  if (++nbrBarcodes > maxBarcodes)
  {
    // Skriv en fejl og hop ud.
    return FALSE;
  };
  barCodes[nbrBarcodes] = (char *)malloc(strlen(newBarCode)+1);
  strncpy(barCodes[nbrBarcodes], newBarcode, strlen(newBarcode)+1);
  return TRUE;
} //
:
:
// Når programmet er færdigt.
for (idx = nbrBarcodes - 1; idx >= 0; idx--)
  if (barCodes[idx] != NULL)
    free(barCodes[idx]);
:

Skift
for (nbrBarcodes 0; nbrBarcodes < maxBarCodes; idx++)
  barCodes[idx] = NULL;

ud med

ZeroMemory(barCodes, maxBarcodes*sizeof(char*));

eller

memset(barCodes, 0, sizeof(barCodes);

men for-løkken er nemmere at forstå.

soepro:
Hvofor allokere mem i forvejen når man blot kan bruge strdup?
I gør det langt mere besværligt end nødvendigt.
strdup allokere memory samt kopiere strengen.
Mere skal der ikke til.

Morten2 >> Det er vist religion. malloc anvendes generelt, strdup \"duer\" kun med char-arrays.

soepro: Ja strdup retunere en char* men det er jo netop hvad er streng er? Jeg forstår ikke hvad du mener?

#define STREGKODE_LEN  20;

LPSTR lpszKode[10000];
LPSTR lpszBuffer;

//Allokér en stor mem chunck
lpszBuffer = (LPSTR)LocalAlloc(LPTR, 10000*STREGKODE_LEN);

//Fordel mem ud på de 10000 pointers
for (int x=0; x<10000; x++)
  lpszKode[x] = lpszBuffer + STREGKODE_LEN*x;

//Brug dem
lstrcpy(lpszKode[8], \"2378428\");
//osv.

//Frigør mem igen
LocalFree((HLOCAL)lpszBuffer);