Is this the correct way to assign array value?

[radishcarrot]

unsigned char *m_pVolumeData;
m_pVolumeData = new unsigned char[m_nSizeX * m_nSizeY * m_nSizeZ];

FILE *fp = fopen(filename, "rb");
if(fp) {
fread(m_pVolumeData, 1, m_nSizeX * m_nSizeY * m_nSizeZ, fp);
fclose(fp);
}

unsigned char *pSrcPixels = new unsigned char[m_nSizeX * m_nSizeY];
for(int m=0; m<m_nSizeX * m_nSizeY; m++) { //copy the value to
psrcpixels
//is this the correct way to assign the value to another array??
pSrcPixels[m] = m_pVolumeData[sliceNo * (m_nSizeX * m_nSizeY)+m];
}

Hi, like to check with you all.
Whether within the for loop, assigning the values are correct? I was to
assign the value of m_pVolume to pSrcPixels but it doesnt seem to
assign it correct. Pls help.

 

[Kai-Uwe Bux]

unsigned char *m_pVolumeData;
m_pVolumeData = new unsigned char[m_nSizeX * m_nSizeY * m_nSizeZ];

FILE *fp = fopen(filename, "rb");
if(fp) {
fread(m_pVolumeData, 1, m_nSizeX * m_nSizeY * m_nSizeZ, fp);
fclose(fp);
}

unsigned char *pSrcPixels = new unsigned char[m_nSizeX * m_nSizeY];
for(int m=0; m<m_nSizeX * m_nSizeY; m++) { //copy the value to
psrcpixels
//is this the correct way to assign the value to another array??
pSrcPixels[m] = m_pVolumeData[sliceNo * (m_nSizeX * m_nSizeY)+m];

Shouldn't this be:

pSrcPixels[m] = m_pVolumeData[sliceNo * m_nSizeZ + m];

}

Hi, like to check with you all.
Whether within the for loop, assigning the values are correct? I was to
assign the value of m_pVolume to pSrcPixels but it doesnt seem to
assign it correct. Pls help.

Best

Kai-Uwe Bux

 

[george.priv]

unsigned char *m_pVolumeData;
m_pVolumeData = new unsigned char[m_nSizeX * m_nSizeY * m_nSizeZ];

FILE *fp = fopen(filename, "rb");
if(fp) {
fread(m_pVolumeData, 1, m_nSizeX * m_nSizeY * m_nSizeZ, fp);
fclose(fp);
}

unsigned char *pSrcPixels = new unsigned char[m_nSizeX * m_nSizeY];
for(int m=0; m<m_nSizeX * m_nSizeY; m++) { //copy the value to
psrcpixels
//is this the correct way to assign the value to another array??
pSrcPixels[m] = m_pVolumeData[sliceNo * (m_nSizeX * m_nSizeY)+m];

Shouldn't this be:

pSrcPixels[m] = m_pVolumeData[sliceNo * m_nSizeZ + m];

This is incorrect, assuming the data is sequence of frames size
m_nSizeX x m_nSizeY the original poster was correct. It may be that
your assumption about file format is wrong. Look what you see as an
image and it may give you some clues. In particular check if image is
simply distorted or it has no sense at all.

Regards,

George Privalov