On 4 Nov 2008, at 10:13 pm, Mahaboob wrote:
content = [[[[[tableView
tableColumnWithIdentifier:@"MediaType"] dataCellForRow:i]stringValue]
stringByAppendingString:@","]
stringByAppendingString:[[[tableView
tableColumnWithIdentifier:@"Category"]
dataCellForRow:i]stringValue]] ;
Ouch.
Why don't you split this out into separate lines so you can see what's
going on? There is no advantage to cramming everything into one line -
it's merely less readable and no faster. You'll then be able to debug
it line by line and see where things are not as you expect.
I suspect something's returning nil in the middle of this lot and so
breaking the gradual accumulation of strings. This code assumes it
will never do that which isn't a good assumption. Since -
stringByAppendingString: returns a new string each time, you are also
making a lot of redundant temporary objects. That might not be a
problem, but why do it when you don't have to?
for(i=0;i<[tableView numberOfRows];i++)
From the docs:
"Typically you should not ask the table view how many rows it has;
instead you should interrogate the table view's data source."
How can I access all selected row values?
It's easy to iterate over the selection, just get -selectedRowIndexes
and enumerate the index set returned.
hth, Graham
_______________________________________________
Cocoa-dev mailing list (Cocoa-dev@lists.apple.com)
Please do not post admin requests or moderator comments to the list.
Contact the moderators at cocoa-dev-admins(at)lists.apple.com
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/cocoa-dev/archive%40mail-archive.com
This email sent to [EMAIL PROTECTED]
