How to make recipients in MFMessageComposeViewController using a variable - ios

Question

I'm trying to use the MFMessageComposer to pre-fill sms for the user. I am trying use a variable that is stored in a mutable array and use that number to pre-fill the phone number. However it doesn't work with the code below at the recipient part. I've tried changing it to a string, adding it to the array. The phone just doesn't do anything.
However if I do a manual number using #[#"123132131235"]; for the recipients it would work.
- (IBAction)smsSend:(id)sender {
{
MFMessageComposeViewController *controller = [[MFMessageComposeViewController alloc] init];
[controller setMessageComposeDelegate:self];
if([MFMessageComposeViewController canSendText])
{
controller.recipients = [[NSArray alloc] initWithObjects:[Detail valueForKey:#"Phone"], nil];
NSLog(#"%#",[Detail valueForKey:#"Phone"]);
controller.body = #"Hello testing";
[self presentViewController:controller animated:YES completion:NULL];
} else {
NSLog(#"cannot send");
}
}
}
Please help me.
Cheers,
Tony.

Answer 1

Try this:
- (IBAction)smsSend:(id)sender
{
{
MFMessageComposeViewController *controller = [[MFMessageComposeViewController alloc] init];
[controller setMessageComposeDelegate:self];
if([MFMessageComposeViewController canSendText])
{
controller.recipients = [NSArray arrayWithObjects:[Detail valueForKey:#"Phone"], nil];
controller.body = #"Hello testing";
[self presentViewController:controller animated:YES completion:NULL];
}
else
{
NSLog(#"cannot send");
}
}
}
The only change I have made is recipient array allocation. Recipient array is not the array that you own so instead of [NSArray alloc] you should directly initialise array with objects.
initWithObject returns an array with a retain count of 1, i.e. you own the array and you must release it at some point to prevent memory leaks.
arrayWithObject returns an autoreleased array so you do not have to worry about releasing it when you don't need it anymore (but if you store it in an instance variable, you should retain it to prevent the autorelease pool from freeing it)