Question

I have a peculiar problem. I created a normal Objective C class called SampleTable. I then extended UITableView instead of NSObject. Then in the initWithFrame constructor I initialized the table. I also made a NSMutableArray object for the datasource. I also conformed UITableViewDelegate and UITableViewDatasource. I have overridden the necessary methods also.

Now I made an object of this class in another class, and added the object as a subview. The tableView is getting drawn according to the CGRectMake() coordinates I gave to the initWithFrame constructor. But it is not getting populated with the data. I don’t know what the problem is.

SampleTable.h

 #import <Foundation/Foundation.h>

 @interface SampleTable : UITableView { 

    NSMutableArray *ItemArray; 

 } 

@property (nonatomic,retain) NSMutableArray *ItemArray;

 -(NSMutableArray *) displayItemArray; 

@end

SampleTable.m

 #import "SampleTable.h"

 @implementation SampleTable

 @synthesize ItemArray;

 -(id)initWithFrame:(CGRect)frm {

     [super initWithFrame:frm];

     self.delegate=self;

     self.dataSource=self;

     [self reloadData];

     return self;

}

 -(NSMutableArray *) displayItemArray { 

    if(ItemArray==nil) { 

       ItemArray=[[NSMutableArray alloc] initWithObjects:@"1",@"2",@"3",@"4",@"5",nil];

    }

  return ItemArray;

 }

 - (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView
 { 

     return 1; 

 }

   -(NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section { 

    return [ItemArray count];

}

 -(UITableViewCell *)tableView: (UITableView *)tableView cellForRowAtIndexPath: (NSIndexPath *)indexPath {

    static NSString *CellIdentifier = @"Cell"; 

    UITableViewCell *cell= [tableView dequeueReusableCellWithIdentifier:CellIdentifier]; 

   if (cell==nil) {

   cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleSubtitle    reuseIdentifier:CellIdentifier]; 

    [cell autorelease]; 

   } 

   NSString *temp=[self.ItemArray objectAtIndex:indexPath.row]; 

   cell.textLabel.text = temp; 

  return cell; 

}

 - (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath { 

   NSLog(@"didselect");

}

 -(void) dealloc { 

     [ItemArray release];

     [super dealloc]; 

 }

  @end

Answer 1

You did never initialize the ItemArray within your code snippet. Remove the displayItemData method and change the initializer towards:

-(id)initWithFrame:(CGRect)frm 
{
    if ((self = [super initWithFrame:frm])) != nil)
    {
        self.delegate=self;
        self.dataSource=self;
        ItemArray=[[NSMutableArray alloc] initWithObjects:@"1",@"2",@"3",@"4",@"5",nil];
        [self reloadData];
    }
    return self;
}

You could also simply call that displayItemArray method within the initializer. I feel that method makes no sense as it stands and hence my recommendation to remove it altogether.

Without trying it myself, I am still pretty confident that you can also get rid of that [self reloadData] within the initializer.