Contacts Framework - fetching Email Field in Objective C - objective-c

Question

I am trying to fetch needed fields from the contacts, using the Contact Framework. All needed fields are properly fetched, EXCEPT the email. If there is one contact without an email recorded, then the entire app crashes. If all contacts have an email defined, all works well. I have a class called ContacList in which a method
fetchContactsFromContactsFrameWork is defined as follows:
-(void)fetchContactsFromContactsFrameWork{
contactStore = [[CNContactStore alloc] init];
NSArray *keyToFetch = #[CNContactEmailAddressesKey,CNContactFamilyNameKey,CNContactGivenNameKey,CNContactPhoneNumbersKey,CNContactPostalAddressesKey,CNContactThumbnailImageDataKey,CNContactOrganizationNameKey];
CNContactFetchRequest *fetchRequest = [[CNContactFetchRequest alloc] initWithKeysToFetch:keyToFetch];
[contactStore enumerateContactsWithFetchRequest:fetchRequest error:nil usingBlock:^(CNContact * _Nonnull contact, BOOL * _Nonnull stop) {
[groupsOfContact addObject:contact];
}];
phoneNumberArray = [#[] mutableCopy];
NSDictionary *peopleDic;
for (CNContact *contact in groupsOfContact) {
NSArray *thisOne = [[contact.phoneNumbers valueForKey:#"value"] valueForKey:#"digits"];
peopleDic = #{#"name":contact.givenName,
#"familyName":contact.familyName,
#"company":contact.organizationName,
#"image":contact.thumbnailImageData != nil ? contact.thumbnailImageData:#"",
#"email":contact.emailAddresses.firstObject != nil ? contact.emailAddresses.firstObject:#"",
#"phone":thisOne,
#"selected":#"NO"
};
[phoneNumberArray addObject:peopleDic];
}
totalPhoneNumberArray = [phoneNumberArray mutableCopy];
}
I am then using the array totalPhoneNumberArray in my PeopleTableViewController where I want to have access to the fetched information. In this I have created a method fetchedContacts, that loos like:
- (void)fetchContacts{
[[ContactList sharedContacts] fetchAllContacts]; //fetch all contacts by calling single method thanks to class 'ContactList"
NSArray *contactsArray = [[NSArray alloc]init];
_arrayOfContacts = [[NSMutableArray alloc]init]; //This will be the edited array to pass to the next ViewController
contactsArray = [[ContactList sharedContacts]totalPhoneNumberArray];//fetched original array now has local name 'contactsArray'
if (contactsArray.count !=0) {
for (int i=0; i<contactsArray.count; i++) {
NSMutableArray *person = [[NSMutableArray alloc]init];//person array will hold 6 properties...
UIImage *contactPic = [UIImage imageWithData:[[contactsArray objectAtIndex:i] valueForKey:#"image"]] ?: [UIImage imageNamed:#"noPerson.png"];
[person addObject:contactPic];
[person addObject:[[contactsArray objectAtIndex:i] valueForKey:#"name"]];
[person addObject:[[contactsArray objectAtIndex:i] valueForKey:#"familyName"]];
[person addObject:[[contactsArray objectAtIndex:i] valueForKey:#"company"]];
[person addObject:[[contactsArray objectAtIndex:i] valueForKey:#"phone"]];
CNLabeledValue *emailValue = [[contactsArray objectAtIndex:i]valueForKey:#"email"] ?:[[CNLabeledValue alloc]initWithLabel:#"email" value:#""];
//[person addObject:#"no email"];
[person addObject:emailValue.value];
//person array now has the form:{image,name,familyName,company,phone,email}
[_arrayOfContacts addObject:person];//arrayOfContacts holds all 'persons' fetched from main contactsArray...
}
}
}
As I explained already, this works perfectly, UNLESS one person has an empty email field, in which case it crashes...I have tried so many things: Turning the CNLabeledValue *emailValue to a NSString, testing for the length of that string, to no avail. Any help will be so much appreciated...

Answer 1

You have code that says:
CNLabeledValue *emailValue = [[contactsArray objectAtIndex:i]valueForKey:#"email"] ?: [[CNLabeledValue alloc]initWithLabel:#"email" value:#""];
[person addObject:emailValue.value];
But, when you populated that email value in the dictionary, you said that if there was no email address, use #"". So, the code above will then proceed to call value on #"", which will not work.
Rather than populating the dictionary with #"" when there is no email address, I'd suggest not setting that dictionary key at all. Or, if you're going to use a "special value" to indicate that there was no email address, you'd generally store [NSNull null]. Frankly, you should be doing the same with the image, too.
For example, I might do:
peopleDic = #{#"name" : contact.givenName ?: #"",
#"familyName" : contact.familyName ?: #"",
#"company" : contact.organizationName ?: #"",
#"image" : contact.thumbnailImageData ?: [NSNull null],
#"email" : contact.emailAddresses.firstObject ?: [NSNull null],
#"phone" : thisOne,
#"selected" : #"NO"
};
Of course, that means that when you retrieve the email value, you check to see if it is NSNull, and if so, set the label accordingly:
for (NSDictionary *contact in contactsArray) {
NSMutableArray *person = [[NSMutableArray alloc]init];//person array will hold 6 properties...
UIImage *contactPic = [contact[#"image"] isKindOfClass:[NSData class]] ? [UIImage imageWithData:contact[#"image"]] : [UIImage imageNamed:#"noPerson.png"];
[person addObject:contactPic];
[person addObject:contact[#"name"]];
[person addObject:contact[#"familyName"]];
[person addObject:contact[#"company"]];
[person addObject:contact[#"phone"]];
CNLabeledValue *email = contact[#"email"];
[person addObject:[email isKindOfClass:[CNLabeledValue class]] ? email.value : #""];
//person array now has the form:{image,name,familyName,company,phone,email}
[_arrayOfContacts addObject:person];//arrayOfContacts holds all 'persons' fetched from main contactsArray...
}
If you're interested in showing phone numbers as a string, there are a few options. Here are a few alternatives:
for (CNContact *contact in groupsOfContact) {
// if you want all phone numbers, you can get as an array, or build a joined string
NSArray *allPhoneNumberAsArray = [[contact.phoneNumbers valueForKey:#"value"] valueForKey:#"stringValue"];
NSString *allPhoneNumberAsSingleString = [allPhoneNumberAsArray count] > 0 ?[allPhoneNumberAsArray componentsJoinedByString:#"; "] : #"No phones";
// if you want only home phone numbers, filter those with `label` of `CNLabelHome`
NSArray *allHomePhoneNumberAsArray = [[[contact.phoneNumbers filteredArrayUsingPredicate:[NSPredicate predicateWithFormat:#"label == %#", CNLabelHome]] valueForKey:#"value"] valueForKey:#"stringValue"];
NSString *allHomePhoneNumberAsSingleString = [allHomePhoneNumberAsArray count] > 0 ? [allHomePhoneNumberAsArray componentsJoinedByString:#"; "] : #"No home phones";
// if you only want the first home number, it's even easier
NSString *firstHomePhoneNumberAsSingleString = [allHomePhoneNumberAsArray firstObject] ?: #"No home phones";
...
}

Answer 2

NSMutableArray *emailArray;
NSString* email;
for (CNLabeledValue <NSString *>*label in contact.emailAddresses)
{
email = label.value;
NSLog(#"email : %#",email);
if ([email length] > 0) {
[emailArray addObject:email];
}
}