Changing SKSpriteNode property via touchesBegan

Question

I was wondering if there was a more efficient way to changing the property of a SKSpriteNode in touchesBegan than my current method. The following is my method:

- (void) touchesBegan:(NSSet *)touches withEvent:(UIEvent *)event {
    UITouch *touch = [touches anyObject];
    CGPoint location = [touch locationInNode:self];
    SKNode *node = [self nodeAtPoint:location];

    if ([node.name isEqualToString:@"Start"]){
        for (SKSpriteNode *node2 in [self children]){
            if ([node2.name isEqualToString:@"Start"]){
                node2.texture = [SKTexture textureWithImageNamed:@"button_beige_pressed"];
                }
            }
    }
        ...
}

Why do you need to have if ([node.name isEqualToString:@"Start"]){ and then if ([node2.name isEqualToString:@"Start"]){ ? — El Tomato
On a separate note, you must reuse textures as much as possible. The method textureWithImageNamed loads the textures which is an expensive operation. Maybe you can pre-load the texture and just apply it later. — mohkhan
Can you create the SKTexture outside of the for loop (and then assign the node2.texture to that texture? Would that be a better way? — Gliderman
I can't access/change the properties of the node with the name "Start" when I try it in the first for loop; that's because its a spritenode. That is why I must make another for loop that accesses the sprite nodes. I have made an edit: It was supposed to be SKNode *node = [self nodeAtPoint: location]; — Exprosul

sangony sangony · Accepted Answer · 2015-11-20T14:09:21

Your current logic suggest that you have multiple nodes with the name 'start'. If this is actually the case I suggest creating an array and storing all 'start' nodes in said array.

If however you only have a single node with the name 'start', then no loop is needed as you already have a reference to the node with the touch function.

Changing SKSpriteNode property via touchesBegan

2 Answers