Gros problème géolocalisation
Adam
Membre
Bonjour,
Cela fait au moins une semaine que je suis bloqué sur mon programme de géolocalisation.
Mon but est de faire un programme assez simple avec des points d'intérêts sur une carte, qui renvoient à des sites web.
Chaque annotation (point d'intérêt), est associé à un identifiant précis.
Puis chaque annotation est liée à un bouton (dans (MKAnnotationView*)mapView:(MKMapView *)mapView viewForAnnotation:(id<MKAnnotation>)annotation) grâce à mon identifiant.
j'ai alors : monannotation.identifiant=bouton.tag...
Le tout me semble correct, mais lorsque dans mon selector de mon bouton, je veux trier mes résultats (pour associer la bonne annotation au bon site web), il mélange tout les identifiants. Et donc l'association se fait pas correctement.
Jai une classe monAnnotation (pour initialiser l'annotation avec mapKit).
Et le reste du code se trouve dans RootViewController.h :
Je vois pas ou est mon erreur, si quelqu'un pourrait me venir en aide.
Merci.
Cela fait au moins une semaine que je suis bloqué sur mon programme de géolocalisation.
Mon but est de faire un programme assez simple avec des points d'intérêts sur une carte, qui renvoient à des sites web.
Chaque annotation (point d'intérêt), est associé à un identifiant précis.
Puis chaque annotation est liée à un bouton (dans (MKAnnotationView*)mapView:(MKMapView *)mapView viewForAnnotation:(id<MKAnnotation>)annotation) grâce à mon identifiant.
j'ai alors : monannotation.identifiant=bouton.tag...
Le tout me semble correct, mais lorsque dans mon selector de mon bouton, je veux trier mes résultats (pour associer la bonne annotation au bon site web), il mélange tout les identifiants. Et donc l'association se fait pas correctement.
Jai une classe monAnnotation (pour initialiser l'annotation avec mapKit).
Et le reste du code se trouve dans RootViewController.h :
<br />@implementation RootViewController<br /><br /><br />- (void)viewDidLoad<br />{<br /> maMapView=[[MKMapView alloc] initWithFrame:CGRectMake(0, 0, 320, 360)];<br /> maMapView.showsUserLocation=YES;<br /> maMapView.mapType=MKMapTypeHybrid;<br /> maMapView.delegate=self;<br /> [self.view addSubview:maMapView];<br /> <br /> //Declaration du Label affichant l'adresse actuelle de l'utilisateur<br /> labelReverseGeocoder=[[UILabel alloc] initWithFrame:CGRectMake(20, 380, 280, 40)];<br /> [self.view addSubview:labelReverseGeocoder];<br /> <br /> //Declaration du bouton qui va lancer le reverse geo coder<br /> buttonLaunchReverseGeocode=[UIButton buttonWithType:UIButtonTypeRoundedRect];<br /> buttonLaunchReverseGeocode.frame=CGRectMake(20, 420, 150, 30);<br /> [buttonLaunchReverseGeocode setTitle:@"Mon adresse" forState:UIControlStateNormal];<br /> [buttonLaunchReverseGeocode addTarget:self action:@selector(reverseGeocodeCurrentLocation:) forControlEvents:UIControlEventTouchUpInside];<br /> [self.view addSubview:buttonLaunchReverseGeocode];<br /> <br /> //Declaration de la roue qui indique une activité<br /> activityReverseGeoCod =[[UIActivityIndicatorView alloc] initWithActivityIndicatorStyle:UIActivityIndicatorViewStyleGray];<br /> activityReverseGeoCod.hidesWhenStopped=YES;<br /> activityReverseGeoCod.center=CGPointMake(260, 430);<br /> [self.view addSubview:activityReverseGeoCod];<br /> <br /> <br /> <br /> <br /> <br /> //[b]Création du tableau d'annotations[/b] !!!!!<br /> arrayOfAnnotations=[[NSMutableArray alloc] initWithCapacity:4];<br /> <br /> MonAnnotation * annotationIut1=[[MonAnnotation alloc] init];<br /> annotationIut1.title=@"Lieu1";<br /> annotationIut1.subtitle=@"aaaaa";<br /> annotationIut1.longitude=[NSNumber numberWithDouble:6.0+0.15226];<br /> annotationIut1.latitude=[NSNumber numberWithDouble:48.0+0.66064];<br /> annotationIut1.identifiant=[NSNumber numberWithInt:99999]; //Identifiant<br /> [arrayOfAnnotations addObject:annotationIut1];<br /> [annotationIut1 release];<br /> <br /> <br /> MonAnnotation * uneAnnotation1=[[MonAnnotation alloc] init];<br /> uneAnnotation1.title=@"Lieu2 ";<br /> uneAnnotation1.subtitle=@"bbbbbb";<br /> uneAnnotation1.longitude=[NSNumber numberWithDouble:4.0+0.15226];<br /> uneAnnotation1.latitude=[NSNumber numberWithDouble:12.0+0.66064];<br /> uneAnnotation1.identifiant=[NSNumber numberWithInt:1]; //Identifiant<br /> [arrayOfAnnotations addObject:uneAnnotation1];<br /> [uneAnnotation1 release];<br /> <br /> MonAnnotation *uneAnnotation2=[[MonAnnotation alloc] init];<br /> uneAnnotation2.title=@"Lieu3";<br /> uneAnnotation2.subtitle=@"ccccc";<br /> uneAnnotation2.longitude=[NSNumber numberWithDouble:5.0+22.0/60.0];<br /> uneAnnotation2.latitude=[NSNumber numberWithDouble:43.0+17.0/60.0];<br /> uneAnnotation2.identifiant=[NSNumber numberWithInt:2]; //Identifiant<br /> [arrayOfAnnotations addObject:uneAnnotation2];<br /> [uneAnnotation2 release];<br /> <br /> <br /> MonAnnotation *uneAnnotation3=[[MonAnnotation alloc] init];<br /> uneAnnotation3.title=@"lieu4";<br /> uneAnnotation3.subtitle=@"dddd";<br /> uneAnnotation3.longitude=[NSNumber numberWithDouble:8.0+22.0/60.0];<br /> uneAnnotation3.latitude=[NSNumber numberWithDouble:40.0+17.0/60.0];<br /> uneAnnotation3.identifiant=[NSNumber numberWithInt:88]; //Identifiant<br /> [arrayOfAnnotations addObject:uneAnnotation3];<br /> [uneAnnotation3 release];<br /> <br /> <br /> [maMapView addAnnotations:arrayOfAnnotations];<br /> <br /> [super viewDidLoad];<br />}<br /><br /><br /><br />//Méthode position utilisateur, OK<br />-(void) reverseGeocodeCurrentLocation: (id) sender {.....}<br /><br /><br /><br />//zoom sur l'utilisateur, OK <br />-(void) mapView:(MKMapView *)mapView didUpdateUserLocation:(MKUserLocation *)userLocation {...}<br /><br /><br /><br /><br />//Affiche l'erreur en cas de probleme, OK<br />-(void) reverseGeocoder:(MKReverseGeocoder *)geocoder didFailWithError:(NSError *)error {....}<br /><br /><br />//OK<br />-(void) reverseGeocoder:(MKReverseGeocoder *)geocoder didFindPlacemark:(MKPlacemark *)placemark {... }<br /><br /> <br /><br /><br />//METHODE POUR PLACER UNE PUNAISE SUR LA CARTE, peut être ici le problème ?<br />-(MKAnnotationView*)mapView:(MKMapView *)mapView viewForAnnotation:(id<MKAnnotation>)annotation {<br /> //Si c'est la position de l'utilisateur, on laisse tel quel<br /> if ([annotation isKindOfClass:[MKUserLocation class]]) {<br /> return nil;<br /> } <br /> <br /> //On teste nos types d'annotations<br /> if ([annotation isKindOfClass:[MonAnnotation class]]) {<br /> <br /> //On essaye de deplier une annotation view en premier<br /> static NSString *MonAnnotationIdentifier=@"monAnnotationIdentifier";<br /> MKPinAnnotationView* pinView=(MKPinAnnotationView*)[maMapView dequeueReusableAnnotationViewWithIdentifier:MonAnnotationIdentifier];<br /> <br /> if (!pinView) {<br /> <br /> MonAnnotation*myAnn=(MonAnnotation*)annotation; //Crée une annotation pour se servir de l'identifiant<br /> <br /> <br /> //Si il ,'y en avait aucune de disponible, on en crée une tout simplement, on peut mm mettre une image (MonAnnotaionView)<br /> MKPinAnnotationView* customPinView=[[[MKPinAnnotationView alloc] initWithAnnotation:myAnn reuseIdentifier:MonAnnotationIdentifier] autorelease];<br /> customPinView.pinColor=MKPinAnnotationColorGreen;<br /> customPinView.animatesDrop=YES;<br /> customPinView.canShowCallout=YES;<br /> <br /> <br /> <br /> //On ajoute une fleche pour ajouter plus d'informations<br /> UIButton* rightButton=[UIButton buttonWithType:UIButtonTypeDetailDisclosure];<br /> <br /> <br /> rightButton.tag=[myAnn.identifiant integerValue];<br /> <br /> <br /> customPinView.rightCalloutAccessoryView=rightButton;<br /> <br /> NSLog(@"nom: %@ avec id : %@ avec un tag : %d ",myAnn.title, myAnn.identifiant, rightButton.tag);<br /><br /> <br /> <br /> [rightButton addTarget:self action:@selector(afficherLesDetails:) forControlEvents:UIControlEventTouchUpInside];<br /> <br /> <br /> //Ajout d'une image a gauche<br /> UIImageView *imageView=[[UIImageView alloc] initWithFrame:CGRectMake(0, 0, 20, 20)];<br /> imageView.image=[UIImage imageNamed:@"annotationImage.png"];<br /> customPinView.leftCalloutAccessoryView=imageView;<br /> [imageView release];<br /> <br /> return customPinView;<br /> }<br /> else {<br /> pinView.annotation=annotation;<br /> }<br /> return pinView;<br /> }<br /> <br /> <br /> return nil;<br />} <br /><br /><br />//SELECTOR DU BOUTON<br />-(void) afficherLesDetails: (id) sender {<br /> UIButton*theButton=(UIButton*)sender;<br /> NSLog(@"La valeur de l'ID est %@", theButton.tag);<br /> <br />}<br /><br /><br /><br /><br />- (void)dealloc<br />{<br /> [arrayOfAnnotations release];<br /> if(reverseGeocoder)<br /> [reverseGeocoder release];<br /> [maMapView release];<br /> [activityReverseGeoCod release];<br /> [labelReverseGeocoder release];<br /> [buttonLaunchReverseGeocode release];<br /> [super dealloc];<br />}<br /><br />@end<br /><br />
Je vois pas ou est mon erreur, si quelqu'un pourrait me venir en aide.
Merci.
Connectez-vous ou Inscrivez-vous pour répondre.
Réponses
Les MKAnnotations, tout comme les UITableViewCells dans une TableView, sont recyclées, pour éviter d'avoir trop d'allocations mémoires inutiles et prenant du temps. Du coup une MKAnnotation qui est utilisé pour un des POI peut être réutilisée plus tard pour un autre POI (surtout, typiquement, si tu scroll sur ta carte et qu'un POI qui était visible devient invisible / sort de la carte, et du coup la MKAnnotation associé sera réutilisée pour un POI qui lui vient d'apparaà®tre, etc)
Sauf que si tu ne prends pas en compte ce mécanisme, tu vas créer une MKAnnotation, avec un tag, et puis plus tard elle va être réutilisée, mais si tu la réutilises telle quel sans changer le tag, c'est le tag de l'ancienne qui va être utilisé, etc...
Si je comprend bien, mon problème pourrait provenir de cette ligne :
C'est MonAnnotationIdentifier qui pose problème ? mais je ne vois pas par quoi le remplacer, vu que l'id doit être un NSString.
Ainsi, imagine que tu as une carte sur laquelle tu veux positionner 10 POI en tout.
Si ta carte est initialement un peu zoomée, au point que sur les 10 POI il n'y en a que disons 3 d'affichés, les 7 autres étant positionnés hors de la zone visible de la carte donc n'étant pas affichés à l'écran.
- Quand l'utilisateur va "scroller" pour déplacer la carte, un des 3 POI visibles va finir par disparaà®tre de la zone visible de la carte pour n'en laisser que deux. Au lieu de le détruire (et de le supprimer de la mémoire), iOS va alors le mettre de côté dans un "bac de recyclage".
- Si l'utilisateur continue à scroller, ou qu'il dézoome, et qu'un nouveau POI devient alors visible sur la carte, le code "dequeueReusableAnnotationViewWithIdentifier" va essayer de récupérer un MKAnnotationView du "bac de recyclage", plutôt que d'en créer un neuf avec un alloc+init. C'est ce que veux dire "dequeueReusable", il en récupère un existant (mais qui ne sert plus) pour le réutiliser. Cela permet d'éviter d'avoir à en créer un neuf avec alloc+init et de le configurer (couleur du texte, vue personnalisée, bouton et son action dans la vue, etc... (car tout ça est un peu coûteux, pas tant que ça mais dans un contexte de "temps réel" où on scroll la carte rapidement et qu'on veut pas que le scroll "accroche" parce que le CPU va mettre 1/4s à créer une nouvelle MKAnnotation, c'est important)
- Or dans ton cas vu qu'on avait en effet une MKAnnotationView dans le "bac de recyclage", il va justement donc réutiliser cette MKAnnotation qui était avant utilisée par le POI qui a depuis disparu de la carte, cela lui évite d'en créer un nouveau. Donc "dequeueReusable..." va te retourner effectivement un MKAnnotation existant (non nil) qui avait été créé (et configuré) précédemment mais ne sert plus et donc va être recyclé
- Si tu continues à scroller ou dézoomer et qu'un 4e POI apparaà®t sur la carte, là encore il va essayer de récupérer un MKAnnotationView du "bac de recyclage". Sauf que cette fois-ci il n'y en a plus, ils ont tous été recyclés et il n'y en a aucun qui traà®ne qui avait été créé avant mais ne sert plus. donc "dequeueReusable..." va te retourner "nil" car il ne va pas réussir à en récupérer un "reusable", et du coup dans ce contexte il faut en créer un nouveau avec alloc+init+autorelease et le configurer.
D'où la construction "if/else" qu'on doit respecter pour ce mécanisme de recyclage :
Maintenant cela me donne :
Mais j'ai toujours le même problème, peut-être ai-je mal compris une notion.
Bah oui, réfléchit, tout ce qui est commun à tes MKAnnotationView genre le rajout du bouton ou l'image à gauche, si c'est recyclé pour être utilisé pour une autre MKAnnotation, ça changera pas d'une MKAnnotation à l'autre donc pas de souci tu mets ça dans le "if"...
Mais vu que tes MKAnnotationViews sont recyclées pour être réutilisées pour d'autres MKAnnotations (POI), par contre tout ce qui dépend du POI (de la MKAnnotation) lui va devoir être ajusté.
Ainsi si cette méthode est appellée, pour prévoir une MKAnnotationView pour une MKAnnotation donnée, il va sans doute recycler une MKAnnotationView existante, déjà configurée avec une image à gauche et un bouton à droite, mais il faut quand même que tu réajustes ce qui est propre à la MKAnnotation ! En particulier le tag de ton rightButton, dont tu te sers pour savoir où doit mener le clic !
Il faut donc simplemebnt sortir de ton "if" tout ce qui est en rapport avec "myAnn", du [tt]MonAnnotation*myAnn=(MonAnnotation*)annotation;[/tt] à toutes les lignes comme [tt]rightButton.tag = [myAnn.identifiant integerValue];[/tt] etc.
Donc dans le "if" tu crées ton bouton pour le rightCalloutAccessoryView, car ce bouton n'a a être créé et rajouté qu'une fois, et il sera là pour toutes les MKAnnotationViews
Mais à l'extérieur du "if", donc là où tu mets le code qui de toute façon s'exécute pour toutes les MKAnnotations et donc le code qui dépend de la MKAnnotation et les trucs qui vont changer d'une MKAnnoationView à l'autre, tu mets [tt]pinView.rightCalloutAccessoryView.tag = [myAnn.identifiant integerValue][/tt] comme ça le tag sera bien réajusté que la MKAnnotationView ait été recyclée/récupérée d'une ancienne MKAnnoationView existante mais ne servant plus, ou que tu viennes d'en alloc/init/autorelease une toute nouvelle.
- pourquoi utiliser un NSNumber plutôt qu'utiliser un simple int, pourquoi se compliquer la vie ?
- pourquoi ne pas utiliser [tt]- (void)mapView:(MKMapView *)mapView annotationView:(MKAnnotationView *)view calloutAccessoryControlTapped:(UIControl *)control[/tt] plutôt que le @selector de ton UIButton ?
- dans ton code tu utilises deux variables différentes selon si ton MKAnnotationView a été recyclée/retournée par "dequeueReusable..." (pinView) ou si ce dequeue n'a pas trouvé de cellule à recycler et que tu crées du coup ta MKAnnotation from scratch avec alloc/init/autorelease (et dans ce cas tu utilises une autre variable, customPinView, alors qu'au final à la fin du "if", que tu aies récupéré un objet MKAnnotationView existant que tu as recyclé d'une ancienne utilisation, ou que tu n'aies pas réussi à trouver une MKAnnotationView à recycler et que tu en crées une neuve, cela ne change rien au reste du code, après le "if" dans les 2 cas il faut terminer la customisation de ton MKAnnotationView en ajustant toutes les propriétés qui sont dépendantes de la MKAnnotation et qui changent d'une MKAnnotation à l'autre.
Pour la méthode : - (void)mapView:(MKMapView *)mapView annotationView:(MKAnnotationView *)view calloutAccessoryControlTapped:(UIControl *)control... Je ne connaissais pas, j'essayerai aussi de changer.
Concernant le reste du code, je ne vois pas ce que l'on doit mettre après le if. D'après ce que j'ai pu comprendre et lire, on met : pinView.annotation=annotation;
Cela fonctionne mais le problème c'est que le triage se fait par ordre d'arrivé dans le viewForAnnotation: (ex: lieu3,lieu2,lieu1,lieu4).
Donc j'aurai une inversion entre lieu 3 et lieu 1.
Et je n'arrive pas à remettre cette ordre.
Désolé, je débute en programmation depuis quelques semaines, et j'ai encore quelques notions qui me reste abstraites.
J'avais du mal à comprendre qu'il fallait "recréer" certaines variable pour une pinView déjà initialisée auparavant.
maintenant apres le if, j'ai :
Et les Id sont bien reparties..
Un grand merci, pour cette aide.
Cela me permet enfin d'avancer dans mon programme