Passing CLGeocoder city name with delegate protocol - objective-c

Question

I´m trying to pass the placemark locality from CLGeocoder to another class UILabel named cityLabel. The class with CLGeocoder named WXManager and the mainViewController named WXController. I have tried every way around to make it work, but I simply can´t get the city name passed to the cityLabel.
WXManager.h
#class WXManager;
#protocol WXManagerDelegate <NSObject>
- (void)WXManagerDidGeocodeCityName:(NSString *)cityName;
#end
#interface WXManager : NSObject <CLLocationManagerDelegate>
- (void)locationManager:(CLLocationManager *)manager didUpdateLocations:(NSArray *)locations;
#end
WXManager.m
- (void)locationManager:(CLLocationManager *)manager didUpdateLocations:(NSArray *)locations {
if (self.isFirstUpdate) {
self.isFirstUpdate = NO;
return;
}
CLLocation *location = [locations lastObject];
if (location.horizontalAccuracy > 0) {
self.currentLocation = location;
[self.locationManager stopUpdatingLocation];}
CLGeocoder *fgeo = [[CLGeocoder alloc] init];
// Reverse Geocode a CLLocation to a CLPlacemark
[fgeo reverseGeocodeLocation:location completionHandler:^(NSArray *placemarks, NSError *error){
if(!error){
for(CLPlacemark *placemark in placemarks){
self.cityName = [placemark locality];
[self.customDelegate WXManagerDidGeocodeCityName:self.cityName];
NSLog(#"city is %#",cityName); }
} else {
NSLog(#"There was a reverse geocoding error\n%#",
[error localizedDescription]);
}
}
];
}
WXController.h
#interface WXController : UIViewController
<WXManagerDelegate>
- (void)WXManagerDidGeocodeCityName:(NSString *)cityName;
#end
And at last in WXController.m where the problem seems to be, my method WXManagerDidGeocodeCityName wont get called. And I don´t know what´s wrong. Does anyone know how to make this method get called?
WXManager *cityName = [[WXManager alloc] init];
cityName.customDelegate = self;
#pragma mark - Custom Delegate Method
- (void)WXManagerDidGeocodeCityName:(NSString *)cityName
{
[self.cityLabel setText:cityName];
NSLog(#"cityName is %#", cityName);
NSLog(#"DidGetCalled");
}
The log doesnt get called so there´s got to be something missing somewhere.

Answer 1

Try this:
WXManager.h
#class WXManager;
#protocol WXManagerDelegate <NSObject>
- (void)WXManagerDidGeocodeCityName:(NSString *)cityName;
#end
#interface WXManager : NSObject <CLLocationManagerDelegate> {
id <WXManagerDelegate> customDelegate;
}
#property (retain)id customDelegate;
#end
WXController.h
#interface WXController : UIViewController <WXManagerDelegate>
//- (void)WXManagerDidGeocodeCityName:(NSString *)cityName;
#end

Answer 2

in WXManager.m you need initialize CLLocationManager:
self.locationManager = [[CLLocationManager alloc] init];
self.locationManager.delegate = self;
[self.locationManager startUpdatingLocation];
and everything will work out fine

Answer 3

I'm going to guess that cityName is weak, or has a custom setter/getter that clears/ignores the correct value.
Edit for clarification:
I was talking about your cityName here:
WXManager *cityName = [[WXManager alloc] init];
cityName.customDelegate = self;
I was assuming it would be assigned some property self.cityName later on.
If you don't, your self.customDelegate will become nil by the time the completion handler runs.

Answer 4

The method is not called? Or variable cityName equal nil?
If first - look for your delegate may be it equal nil.
If second - CLGeocoder work not correct.
Generally stroll debugger. Look at the variables. Quickly understand everything.

Answer 5

in WXManager.m you need initialize CLLocationManager:
self.locationManager = [[CLLocationManager alloc] init];
self.locationManager.delegate = self; [self.locationManager
startUpdatingLocation];
and everything will work out fine