macOS Bitmask Setters and Getters

[North Bronson]

This is a little new for me. What's the best way to implement this? I'm not sure if there's any trouble with this approach. It seems to work so far.

enum
{
    MyObjectChecked = 1 << 0
};

@interface MyObject : NSObject
{
    uint8_t myFlags;
}

- (BOOL)isChecked;
- (void)setChecked:(BOOL)checked;

@end

@implementation MyObject

- (BOOL)isChecked
{
    BOOL checked = (myFlags & MyObjectChecked);
    
    return checked;
}

- (void)setChecked:(BOOL)checked
{
    if (checked)
    {
        myFlags |= MyObjectChecked;
    }
    else
    {
        myFlags &= ~MyObjectChecked;
    }
}

@end

 

[lee1210]

Your getter isn't KVC compliant. Not terrible, but might be good to be in the habit. Is this to demonstrate a concept that you will extend? If not, a BOOL seems clearer. Any reason to set your flags to 0xFE when the object is unchecked? Why not 0? If you check first, the flag will be 0x1, if you uncheck then check again it will be 0xFF. This seems confusing to me.

I guess I'd say it looks workable, just seems a bit odd to me.

-Lee

 

[North Bronson]

Why would the flags necessarily be set to 0xFE? I could understand if the flags were already set to 0xFF and I and-not'd 0x1, but if my flags are currently 0x0, when would the left seven bits turn to one? Am I mixing things up?

I guess the idea was that I could have an enum that looks like:

enum
{
    MyObjectChecked = 1 << 0
    MyObject. . .   = 1 << 1,
    MyObject. . .   = 1 << 2,
    . . .
    MyObject. . .   = 1 << n
};

just for some simple on-and-off values and use the same style of getters and setters for each.