Home/ Questions/Q 103765
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-11T01:10:32+00:00

When retrieving objects from an NSMutableArray in cocoa-touch is the below code ok? Should

  • 0

When retrieving objects from an NSMutableArray in cocoa-touch is the below code ok? Should I be allocating([alloc]) new Page objects each time or is just pointing to it alright? Do I need to do anything to the Page *pageObj after, such as set it to nil?

const char *sql = 'insert into Page(Book_ID, Page_Num, Page_Text) Values(?, ?, ?)'; for (i = 0; i < ([[self pagesArray] count] - 1); i++) {     if(addStmt == nil) {         if(sqlite3_prepare_v2(database, sql, -1, &addStmt, NULL) != SQLITE_OK) {             NSAssert1(0, @'Error while creating add statement. '%s'', sqlite3_errmsg(database));         }     }     Page *pageObj = [[self pagesArray] objectAtIndex:i];     if(pageObj.isNew) {         sqlite3_bind_int(addStmt, 1, self.book_ID);          sqlite3_bind_int(addStmt, 2, pageObj.page_Number);           sqlite3_bind_text(addStmt, 3, [[pageObj page_Text] UTF8String], -1, SQLITE_TRANSIENT);         if(SQLITE_DONE != sqlite3_step(addStmt)) {             NSAssert1(0, @'Error while inserting data. '%s'', sqlite3_errmsg(database));         }         NSLog(@'Inserted Page: %i into DB. Page text: %@', pageObj.page_Number, pageObj.page_Text);     }     //Reset the add statement.     sqlite3_reset(addStmt);                      }

Thanks. I also understand this should probably be in a transaction but I didn’t quite get that working just yet.

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. The way you’re declaring a pointer is correct. You don’t need alloc, since that creates a new object, when you want to refer to an existing object in the array. You would want to retain it if you were going to keep the reference outside of that method, but since you’re only using it temporarily it’s fine not to.

    The actual pointer variable will be destroyed and recreated every trip to the loop, so there’s no need to set it to nil. Even if you declared the variable outside the loop, simply assigning it to a new object is fine. The only time you’d set it to nil is when you’re releasing the object stored in the pointer (or the object may be released elsewhere). If you didn’t set it to nil in that case, the pointer would refer to an invalid memory location after the object is dealloced, usually causing a crash.

    One bug I can see though, you’re skipping the last element in your array in your for loop by subtracting 1 from the count.

    • 0