Cannot read from listbox item array

[Acemaker]

I can't find a solution to read one Item from CDesCArray *itemList.

Here is one method what i've tried, but result is KERN-EXEC 3 in the if- statement.

Is there anyone there who can help me find a better working solution.


My Code:


void CDialogQueryAppView::SetDialogTest(){

TInt index( 0 );
CAknListQueryDialog* dlg = new( ELeave ) CAknListQueryDialog( &index );

// construct listbox item array
CDesCArray* itemList = new (ELeave) CDesCArrayFlat(3);

_LIT( KFirst, "1st Item" );
_LIT( KSecond, "2nd Item" );
_LIT( KThird, "3rd Item" );

HBufC* item = HBufC::NewL(20);

*item = KFirst;
itemList->AppendL(*item);
*item = KSecond;
itemList->AppendL(*item);
*item = KThird;
itemList->AppendL(*item);

dlg->PrepareLC( R_AVKON_LIST_QUERY );
dlg->SetItemTextArray( itemList );

if( dlg->RunLD()){
// ok pressed, index is the selected item index.

*item = (*itemList)[index];
iEikonEnv->InfoMsg( *item );
}
delete item;
}

-Desperately seeking

[cdavies]

It isn't a pointer array, the CDesCArray in to which it'll copy the strings you store. If you want a copy on the heap of the selected item after the dialog has completed, you code should look something like this:

CDesCArray* itemList = new (ELeave) CDesCArrayFlat(3);

_LIT( KFirst, "1st Item" );
_LIT( KSecond, "2nd Item" );
_LIT( KThird, "3rd Item" );

itemList->AppendL(KFirst);
itemList->AppendL(KSecond);
itemList->AppendL(KThird);

dlg->PrepareLC( R_AVKON_LIST_QUERY );
dlg->SetItemTextArray( itemList );

if( dlg->RunLD()){
// ok pressed, index is the selected item index.

HBufC* selectedItem = (*itemList)[index].AllocL();
iEikonEnv->InfoMsg( *selectedItem );

You could also do:
iEikonEnv->InfoMsg((*itemList)[index]);

To get the same result.

[wizard_hu_]

You can try if SetOwnershipType(ELbmDoesNotOwnItemArray) helps. KERN-EXEC 3 may happen because the dialog simply resets and deletes your descriptor array.

[Acemaker]

Thanks for your helpiing answers.

Now it works with "SetOwnershipType(ELbmDoesNotOwnItemArray)", but i still got App.Closed Alloc:xxxxxxx message, when I use OK-softkey ( if- statement in code), and then close the application.


So this code works and reads from CDesCArray:


void CDialogQueryAppView::SetDialogTest(){

TInt index( 0 );
CAknListQueryDialog* dlg = new( ELeave ) CAknListQueryDialog( &index );

// construct listbox item array
CDesCArray* itemList = new (ELeave) CDesCArrayFlat(3);

_LIT( KFirst, "1st Item" );
_LIT( KSecond, "2nd Item" );
_LIT( KThird, "3rd Item" );

itemList->AppendL(KFirst);
itemList->AppendL(KSecond);
itemList->AppendL(KThird);

dlg->PrepareLC( R_AVKON_LIST_QUERY );
dlg->SetItemTextArray( itemList );
dlg->SetOwnershipType( ELbmDoesNotOwnItemArray );

if( dlg->RunLD()){
// ok pressed, index is the selected item index.
HBufC *selectedItem = HBufC::NewL(20);
selectedItem = (*itemList)[index].AllocL();
iEikonEnv->InfoMsg( *selectedItem );
delete selectedItem;
}
}

[cdavies]

With your new code, you're leaking the array and the redundant HBufC you create. Your code should go something like this:

CDesCArray* itemList = new (ELeave) CDesCArrayFlat(3);
CleanupStack::PushL(itemList);

_LIT( KFirst, "1st Item" );
_LIT( KSecond, "2nd Item" );
_LIT( KThird, "3rd Item" );

itemList->AppendL(KFirst);
itemList->AppendL(KSecond);
itemList->AppendL(KThird);
dlg->PrepareLC( R_AVKON_LIST_QUERY );
dlg->SetItemTextArray( itemList );
dlg->SetOwnershipType( ELbmDoesNotOwnItemArray );

if( dlg->RunLD()){
// ok pressed, index is the selected item index.
HBufC* selectedItem = (*itemList)[index].AllocL(); /* This line creates a new HBufC, no need to create one first. */
iEikonEnv->InfoMsg( *selectedItem );
delete selectedItem;
}

CleanupStack::PopAndDestroy(itemList);

[Acemaker]

Thanks.

SolvedAndClosed();

Too much buffers today.

[wizard_hu_]

A slight addition: if you have the array containing descriptors, it is not that important to allocate one of them separately on the heap.

if(dlg->RunLD()){
    iEikonEnv->InfoMsg((*itemList)[index]);
}

should work.