UILongPressGestureRecognizer 执行两次的问题

Question

I am detecting if the user has pressed down for 2 seconds:

UILongPressGestureRecognizer *longPress = [[UILongPressGestureRecognizer alloc]
                                             initWithTarget:self 
                                             action:@selector(handleLongPress:)];
        longPress.minimumPressDuration = 2.0;
        [self addGestureRecognizer:longPress];
        [longPress release];

This is how I handle the long press:

-(void)handleLongPress:(UILongPressGestureRecognizer*)recognizer{
    NSLog(@"double oo");
}

The text "double oo" gets printed twice when I press down for longer than 2 seconds. Why is this? How can I fix?

OdNairy · Accepted Answer · 2012-01-28 18:04:37Z

UILongPressGestureRecognizer is a continuous event recognizer. You have to look at the state to see if this is the start, middle or end of the event and act accordingly. i.e. you can through away all events after the start, or only look at movement as you need. From the Class Reference:

Long-press gestures are continuous. The gesture begins (UIGestureRecognizerStateBegan) when the number of allowable fingers (numberOfTouchesRequired) have been pressed for the specified period (minimumPressDuration) and the touches do not move beyond the allowable range of movement (allowableMovement). The gesture recognizer transitions to the Change state whenever a finger moves, and it ends (UIGestureRecognizerStateEnded) when any of the fingers are lifted.

The next answer down shows how to do it, but I gave this answer the +1 because of its detailed explaination and link to documentation.
Could be more helpful with a code example, rather than linking just to the documentation. I have posted my code snippet below. check the UIGestureRecognizerStateBegan state.

Josh Brown · Answer 2 · 2011-04-11 14:18:26Z

up vote 62 down vote

To check the state of the UILongPressGestureRecognizer just add an if statement on the selector method:

- (void)handleLongPress:(UILongPressGestureRecognizer*)sender { 
    if (sender.state == UIGestureRecognizerStateEnded) {
        NSLog(@"Long press Ended");
    }
    else {
        NSLog(@"Long press detected.");
    }
}

edited Apr 11 '11 at 14:18

Josh Brown
6,577 4 18 37

answered Nov 11 '10 at 15:43

Dan
645 1 4 3

8

You don't want that if/else block, since there are more states than Ended. "Long press detected" will print multiple times when the state changes. Check the UIGestureRecognizerStateBegan state instead. – Paul Solt Aug 18 '11 at 20:59

Exactly. Just got bitten by the same if/else code. – Stephen Eilert Mar 14 at 0:48

Someone should really edit that answer to fit with what the top comment says. As it stands the code provided does not work. – Deco Sep 11 at 22:40

feedback

Paul Solt · Answer 3 · 2012-03-19 00:24:54Z

You need to check the correct state, since there are different behaviors for each state. Most likely you're going to need the UIGestureRecognizerStateBegan state with the UILongPressGestureRecognizer.

UILongPressGestureRecognizer *longPress = [[UILongPressGestureRecognizer alloc]
                                             initWithTarget:self 
                                             action:@selector(handleLongPress:)];
        longPress.minimumPressDuration = 1.0;
        [myView addGestureRecognizer:longPress];
        [longPress release];

...

- (void)handleLongPress:(UILongPressGestureRecognizer *)gesture {
    if(UIGestureRecognizerStateBegan == gesture.state) {
        // Called on start of gesture, do work here
    }

    if(UIGestureRecognizerStateChanged == gesture.state) {
        // Do repeated work here (repeats continuously) while finger is down
    }

    if(UIGestureRecognizerStateEnded == gesture.state) {
        // Do end work here when finger is lifted
    }
}

Seems like you have to move your fingers to get the state to change; is that correct?
It might trigger StateChanged when moving fingers around, that sounds similar to what I was doing in my test code.
UIGestureRecognizerStateBegan seems to only be called once which is perfect for my situation trying to show a dialog on detecting a long press on a button. The other states get called multiple times. Thanks!

score -2 · Answer 4

-(IBAction)longPressHandle:(id)sender{
    if (accessedLongPressHandlerCount == 0) {
        accessedLongPressHandlerCount = 1;
        //do something

    } else if (accessedLongPressHandlerCount == 1) {
        accessedLongPressHandlerCount = 0; 
        //this is whats called after the second time it gets to this method
    } else {
        NSLog(@"Accessed Long Pressed some how made it here %@ times", accessedLongPressHandlerCount);
    }
}

Is what worked for me.

UILongPressGestureRecognizer 执行两次的问题

4 Answers