Incorrect number of objects getting added to mutable array

送分小仙女□ 提交于 2019-11-30 09:23:45

问题


I have a view controller with table view that contains 8 cells(sections).I have created a textField and added that text field as a subview to 6 cells,remaining 2 cells of which one cell contains a text view and the other with a button.Because I am using the same text field,I have assigned tag values and then created a mutable array,added the textfield to that array.Here is the implementation code:

Taking in regard the suggestion given by Mr.Lanc,I have changed the implementation code

EDIT

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

{

 NSString *identifier = @"UITableViewCell";

UITableViewCell *cell = (UITableViewCell *)[atableView dequeueReusableCellWithIdentifier:identifier];

    atableView.backgroundColor = [UIColor clearColor];

    textField = [[[UITextField alloc]initWithFrame:CGRectMake(15, 12, 300, 24)]autorelease];
    textField.textColor = [UIColor whiteColor];
    textField.delegate = self;
    tagValues = textField.tag;


switch (indexPath.section) 
{
    case 0:
    {
        NSString *cellIdentifierA = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierA];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        textField.placeholder = @"Enter name";
        textField.tag = 101;
        textField.text = reminderInstance.Name;
        textField.autocorrectionType = UITextAutocorrectionTypeNo;
        [cell.contentView addSubview:textField];
    }
        break;

    case 1:
    {
        NSString *cellIdentifierB = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierB];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        textField.tag = 102;
        textField.text = reminderInstance.Event;
        [cell.contentView addSubview:textField];
    }
        break;

    case 2:
    {
        NSString *cellIdentifierC = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierC];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        textField.placeholder = @"Click here to set date and time";
        textField.inputView = self.datePicker;
        textField.text = reminderInstance.Date;
        textField.tag = 103;
        [cell.contentView addSubview:textField];
    }   
        break;

    case 3:
    {
        NSString *cellIdentifierD = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierD];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        textField.tag = 105;
        textField.text = reminderInstance.numDays;
        textField.inputView = self.reminderPicker;
        [cell.contentView addSubview:textField];
    }
        break;

    case 4:
    {   
        NSString *cellIdentifierE = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierE];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        checkboxButton = [[[UIButton alloc] initWithFrame:CGRectMake(16,1,120, 44)]autorelease];
        [checkboxButton setImage:[UIImage imageNamed:@"ewee.png"] forState:UIControlStateNormal];
        [checkboxButton addTarget:self action:@selector(toggleButton:) forControlEvents:UIControlEventTouchUpInside];

        NSString *one = reminderInstance.selString;
        NSNumber* i = [NSNumber numberWithInt:[one intValue]];
        BOOL isOn = [i boolValue];

        if(isOn)
        {
            [checkboxButton setImage:[UIImage imageNamed:@"checkarrow.png"] forState:UIControlStateNormal];
        }
        else
        {
            [checkboxButton setImage:[UIImage imageNamed:@"ewee.png"] forState:UIControlStateNormal];
        }            

        [checkboxButton setContentHorizontalAlignment:UIControlContentHorizontalAlignmentLeft];
        [checkboxButton setImageEdgeInsets:UIEdgeInsetsMake(0.0, 0.0, 0.0, 0.0)];
        [cell addSubview:checkboxButton];
        UILabel *label = [[UILabel alloc]initWithFrame:CGRectMake(55, 10, 225, 24)];
        label.text = @"Every Year";
        label.textColor = [UIColor whiteColor];
        label.backgroundColor = [UIColor clearColor];
        [cell addSubview:label];
        cell.textLabel.textColor = [UIColor whiteColor];
        [label release];
    }
        break;

    case 5:
    {
        NSString *cellIdentifierF = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierF];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        textField.placeholder = @"Enter the number here";
        textField.text = num;
        textField.text = reminderInstance.number;
        textField.tag = 106; 
        textField.userInteractionEnabled = YES;
        textField.keyboardType = UIKeyboardTypeNumberPad;
        [cell.contentView addSubview:textField];
    }
        break;

    case 6:
    {
        NSString *cellIdentifierG = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierG];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        cell.backgroundColor = [UIColor clearColor];
        textView.clipsToBounds = YES;
        textView = [[UITextView alloc]initWithFrame: CGRectMake(-2, -3, 307, 154)];
        UIImageView *imgView = [[[UIImageView alloc]initWithFrame: textView.frame]autorelease];
        imgView.image = [UIImage imageNamed: @"reminderbuttonxl.png"];
        [textView addSubview: imgView];
        [textView sendSubviewToBack: imgView];
        textView.backgroundColor = [UIColor clearColor];
        textView.delegate = self;
        textView.tag = 11;
        textView.text = reminderInstance.msgBody;
        tagValues = textView.tag;
        textView.textColor = [UIColor whiteColor];
        [cell.contentView addSubview:textView];
        [textView release];
    }
        break;

    case 7:
    {
        NSString *cellIdentifierH = [NSString stringWithFormat:@"S%1dR%1d",indexPath.section,indexPath.row];
        if (cell == nil) 
        {
            cell = [[UITableViewCell alloc]initWithStyle:UITableViewCellStyleDefault reuseIdentifier:cellIdentifierH];
            cell.backgroundColor = [[UIColor alloc]initWithPatternImage:[UIImage imageNamed:@"reminderbutton.png"]];
        }
        textField.tag = 107;
        textField.inputView = self.groupPicker;
        tagValues = textField.tag;
        textField.text = reminderInstance.remGroup;
        [cell.contentView addSubview:textField];
    }
        break;

    default:
        break;
}

    [self.fields addObject:textField];

NSLog(@"Fields = %@",fields);

return cell;

}

Now when I am trying to print the array using the following statement:

NSLog(@"Fields = %@",fields); I could observe the following in console window:

Fields = (
    "<UITextField: 0x9d043a0; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 101; layer = <CALayer: 0x9d044e0>>"
)
Fields = (
    "<UITextField: 0x9d043a0; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 101; layer = <CALayer: 0x9d044e0>>",
    "<UITextField: 0x9d07d50; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 102; layer = <CALayer: 0x9d07ca0>>"
)
Fields = (
    "<UITextField: 0x9d043a0; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 101; layer = <CALayer: 0x9d044e0>>",
    "<UITextField: 0x9d07d50; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 102; layer = <CALayer: 0x9d07ca0>>",
    "<UITextField: 0x7108650; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 103; layer = <CALayer: 0x7108620>>"
)
Fields = (
    "<UITextField: 0x9d043a0; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 101; layer = <CALayer: 0x9d044e0>>",
    "<UITextField: 0x9d07d50; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 102; layer = <CALayer: 0x9d07ca0>>",
    "<UITextField: 0x7108650; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 103; layer = <CALayer: 0x7108620>>",
    "<UITextField: 0x5c7bf90; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 105; layer = <CALayer: 0x5c7c920>>"
)
Fields = (
    "<UITextField: 0x9d043a0; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 101; layer = <CALayer: 0x9d044e0>>",
    "<UITextField: 0x9d07d50; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 102; layer = <CALayer: 0x9d07ca0>>",
    "<UITextField: 0x7108650; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 103; layer = <CALayer: 0x7108620>>",
    "<UITextField: 0x5c7bf90; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; tag = 105; layer = <CALayer: 0x5c7c920>>",
    "<UITextField: 0x9d08b20; frame = (15 12; 300 24); clipsToBounds = YES; opaque = NO; layer = <CALayer: 0x9d08af0>>"
)

Now as we can observe from the above o/p that number of fields that are getting printed is 5 i.e the 4 textfields and the cell with button,But as per our coding in -CellForRowAtIndexPath,6 cells contains textfield,hence the number of fields must be 6,but we are getting wrong result i.e. 5 (4 textfields+button) which is a blunder.

This is also leading me to saving the data in table view issue,i.e. for right navigation bar button item titled "Save" in my case,I have implemented:

-(IBAction)save:(id)sender
{
//Code for saving entered data using sqlite

    UITextField *fieldOne = [self.fields objectAtIndex:0];
    UITextField *fieldTwo = [self.fields objectAtIndex:1];
    UITextField *fieldThree = [self.fields objectAtIndex:2]; 
    UITextField *fieldFour = [self.fields objectAtIndex:3];
    UITextField *fieldFive = [self.fields objectAtIndex:5];
    UITextField *fieldSix = [self.fields objectAtIndex:7];
sqlite3_stmt *statement = nil;
    const char *dbpath = [databasePath UTF8String];

    if (sqlite3_open(dbpath, &remindersDB) == SQLITE_OK && textField.text != nil)
    {
if (self.navigationItem.rightBarButtonItem.title == @"Save")
        {
            NSLog(@"am in the save loop");
            NSString *insertSQL = [NSString stringWithFormat:@"INSERT INTO reminders(name,event,date,bfr,val,num,bod,grp) VALUES (\"%@\", \"%@\", \"%@\", \"%@\", \"%@\", \"%@\", \"%@\", \"%@\")", fieldOne.text, fieldTwo.text,fieldThree.text,fieldFour.text,isSelected,fieldFive.text,textView.text,fieldSix.text]; 
            const char *insert_stmt = [insertSQL UTF8String];
            sqlite3_prepare_v2(remindersDB, insert_stmt, -1, &statement, NULL);

            if (sqlite3_step(statement) == SQLITE_DONE)
            {
                UIAlertView *alert = [[UIAlertView alloc]initWithTitle:@"\nReminder Saved" message:nil delegate:nil cancelButtonTitle:nil otherButtonTitles:nil,nil];
                [alert show];
                [alert dismissWithClickedButtonIndex:0 animated:YES];
                [alert release];
            }

            else 
            {
                UIAlertView *alert = [[UIAlertView alloc]initWithTitle:@"Alert" message:@"Reminder not saved" delegate:self cancelButtonTitle:@"Ok" otherButtonTitles:nil];
                [alert show];
                [alert release];
            }
        }
//Check the last inserted row id
        rowID =  sqlite3_last_insert_rowid(remindersDB);
        NSLog(@"last inserted rowId = %d",rowID);

        sqlite3_finalize(statement);
        sqlite3_close(remindersDB);

        [self.tableView reloadData];
 }
}

Now the problem is with the index location values,since the number of values(fields) that "fields" array is holding is 5,the index values would be 0-4,but If I give the values accordingly,i.e. while saving,I would get a field less while inserting data in to sqlite database,Also I am getting null values in 5th and 7th locations of table view when printing in console.In the above implementation snippet,we can observe that I have inserted using the index location to that of table view section values(indices).

When I save according to section index values of table view,the values are getting printed properly.But I am getting crash problem when I don't touch the 5th and 7th locations of table view.i.e. '*** -[NSMutableArray objectAtIndex:]: index 5 beyond bounds [0 .. 4]'

I don't understand the reason why only 5 field values are getting inserted in to the array instead of 6.Can any one please guide me in right direction

Sorry for this huge post,I wanted to be informative,hence I have made it look detailed so that the issue can be answered with ease.

EDIT

The code for number of sections and rows for table view:

#pragma mark -
#pragma mark TableView Datasource

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

    return 8;

}

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

    int arr = 1;

    return arr;
}

Implementation of height for row at index path :

- (CGFloat)tableView:(UITableView *)atableView heightForRowAtIndexPath:(NSIndexPath *)indexPath 
{
    float f;
    if (indexPath.section==6&&indexPath.row==0) 
    {
        f=140.0f;
        return f;
    }
    else
    {
        f = 44.0f;
    return f;
    }
}

All the table view cells are not loading before scrolling.But when I scroll then the control is reaching 5th,6th and 7th cells.I found out that's the reason why the textfield in 5th and 7th cells are not getting added to "fields" array.

I searched for solution mostly the posts of this type

Thanks all in advance :)


回答1:


Did you try scrolling the tableview? I

If you have added this code inside if( cell == nil ), then it may not create all the text fields, as the table cells are reused. Better you have different cell identifiers for all the 8 cells. That would solve your problem.




回答2:


I found out the solution to the question here

Thanks every one who viewed the question.



来源:https://stackoverflow.com/questions/9091647/incorrect-number-of-objects-getting-added-to-mutable-array

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!