Question

I’ve created a mapview which has a button to switch between an ‘edit’ mode and a ‘drag’ mode based on the project requirements. I realize that it’s easy enough to have your annotations draggable from creation by setting them draggable in the viewForAnnotation, but the behavior required won’t allow this. I’ve tried a couple different ways of changing the annotations to draggable without success. The first thought was to loop through the existing annotations and set each one to ‘draggable’ and ‘selected’, but I get an unrecognized selector sent to instance error (I did try instantiating a new annotation to pass in the object and re-plot while in the loop, but I get the same error as well):

NSLog(@"Array Size: %@", [NSString stringWithFormat:@"%i", [mapView.annotations count]]);

    for(int index = 0; index < [mapView.annotations count]; index++) {

        if([[mapView.annotations objectAtIndex:index]isKindOfClass:[locAnno class]]){
            NSLog(@"** Location Annotation at Index: %@", [NSString stringWithFormat:@"%i", index]);
            NSLog(@"* Location Marker: %@", [mapView.annotations objectAtIndex:index]);
        }

        if([[mapView.annotations objectAtIndex:index]isKindOfClass:[hydAnno class]]) {
            NSLog(@"** Hydrant Annotation at Index: %@", [NSString stringWithFormat:@"%i", index]);
            NSLog(@"* Hydrant Marker: %@", [mapView.annotations objectAtIndex:index]);

            [[mapView.annotations objectAtIndex:index]setSelected:YES];
            [[mapView.annotations objectAtIndex:index]setDraggable:YES];
        }
    }

The second thought was to use ‘didSelectAnnotationView’, and set selected and draggable on the annotation when it’s selected, and reset the properties when the mode switches back again. This works, but very poorly as the event doesn’t always fire and your left to tap the annotation one or more times before it will change the properties:

- (void)mapView:(MKMapView *)mapView didSelectAnnotationView:(MKAnnotationView *)view {
NSLog(@"Annotation Selected!");
if(!editMode) {
    view.selected = YES;
    view.draggable = YES;
}

}

The first attempt seems the most simple solution if I can get it to work. Using the didSelect method on the other hand is cumbersome and hack-licious. I’m quite new to iOS development, so I apologize if I’ve overlooked something novice while hammering away at this. I appreciate any insight the community can offer. Thanks much.

Answer 1

The first method is better than using the didSelectAnnotationView delegate method.

The problem with the code which causes the “unrecognized selector” error is that it is calling setSelected: and setDraggable: on the annotation objects (type id<MKAnnotation>) instead of their corresponding MKAnnotationView objects. The id<MKAnnotation> objects don’t have such methods so you get that “unrecognized selector” error.

The map view’s annotations array contains references to the id<MKAnnotation> (data model) objects — not the MKAnnotationView objects for those annotations.

So you need to change this:

[[mapView.annotations objectAtIndex:index]setSelected:YES];
[[mapView.annotations objectAtIndex:index]setDraggable:YES];

to something like this:

//Declare a short-named local var to refer to the current annotation...
id<MKAnnotation> ann = [mapView.annotations objectAtIndex:index];

//MKAnnotationView has a "selected" property but the docs say not to set
//it directly.  Instead, call deselectAnnotation on the annotation...
[mapView deselectAnnotation:ann animated:NO];

//To update the draggable property on the annotation view, get the 
//annotation's current view using the viewForAnnotation method...
MKAnnotationView *av = [mapView viewForAnnotation:ann];
av.draggable = editMode;

You must also update the code in the viewForAnnotation delegate method so that it also sets draggable to editMode instead of a hard-coded YES or NO so that if the map view needs to re-create the view for the annotation after you’ve already updated it in the for-loop, the annotation view will have the right value for draggable.