Using a struct of pointers in C

Question

As above, I'm trying to edit a bit of code I wrote last week, the old code:

    char *pixel_b = NULL;
    char *pixel_g = NULL;
    char *pixel_r = NULL;

    for (i=0;i<416;i++)
    {
        for (j=0;j<576;j++)
        {
        pixel_b = &framebuff[GET_PIXEL_B(j,i)];
        pixel_g = &framebuff[GET_PIXEL_G(j,i)];
        pixel_r = &framebuff[GET_PIXEL_R(j,i)];

        *pixel_b = 255-*pixel_b;
        *pixel_g = 255-*pixel_g;
        *pixel_r = 255-*pixel_r;
        }
    }

This successfully accessed the bytes in the array and changed the values (used to invert an image).

I wanted to create a structure containing the three pixel values, like so:

struct Pixel {
    char *pixel_b;
    char *pixel_g;
    char *pixel_r;
};

Then change the first bit of code to:

struct Pixel pixel;

    for (i=0;i<416;i++)
    {
        for (j=0;j<576;j++)
        {
        pixel.pixel_b = &framebuff[GET_PIXEL_B(j,i)];
        pixel.pixel_g = &framebuff[GET_PIXEL_G(j,i)];
        pixel.pixel_r = &framebuff[GET_PIXEL_R(j,i)];

        pixel.*pixel_b = 255-pixel.*pixel_b;
        pixel.*pixel_g = 255-pixel.*pixel_g;
        pixel.*pixel_r = 255-pixel.*pixel_r;
        }
    }

However it seems you can't just do this :P So after some more looking around I thought it may be best to change pixel to *pixel, and don't have pointers within it, however that didn't seem to work either. Is there a nice way to do what I'm trying to do? I haven't used structs in C in quite a while so I'm partially expecting I'm forgetting something very basic.

Any help would be greatly appreciated.

caveman · Accepted Answer · 2015-02-28 16:39:49Z

2

You have to dereference the struct.field, not just the field. The precedence for the . operator is higher than the * dereference operator, so no parenthesis are needed.

struct Pixel pixel;

for (i=0;i<416;i++)
{
    for (j=0;j<576;j++)
    {
    pixel.pixel_b = &framebuff[GET_PIXEL_B(j,i)];
    pixel.pixel_g = &framebuff[GET_PIXEL_G(j,i)];
    pixel.pixel_r = &framebuff[GET_PIXEL_R(j,i)];

    *pixel.pixel_b = 255 - *pixel.pixel_b;
    *pixel.pixel_g = 255 - *pixel.pixel_g;
    *pixel.pixel_r = 255 - *pixel.pixel_r;
    }
}

answered Feb 28, 2015 at 16:39

caveman

1,7551 gold badge14 silver badges19 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

GeorgeStorm Over a year ago

Can't believe I didn't try that, thought I had. Thanks a lot :) From running it first time it feels a little slower, although that could just be imaginary, would it in theory be any slower than the original method?

caveman Over a year ago

There is no reason in principle that it should be slower. The . dereference is handled at compile time. Of course, compilers can be silly sometimes.

sczizzo · Accepted Answer · 2015-02-28 18:38:21Z

0

It is the syntax. Switch the pixel.*pixel_b for *(pixel.pixel_b).

edited Feb 28, 2015 at 18:38

sczizzo

3,20622 silver badges28 bronze badges

answered Feb 28, 2015 at 16:52

Nilo Araujo

8158 silver badges18 bronze badges

Collectives™ on Stack Overflow

Using a struct of pointers in C

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related