ArrayList Size Comes Back 0

[Adam Wentz]

I have a ListView connected to an Activity by a custom Adapter. In the Adapter, I collect the names of all non-system applications and display them in the list. Each list item also includes a Switch. When a list item's switch is clicked, I have the relevant package name being written to an ArrayList<String> in the Activity.

This is the relevant part of the Adapter code where that is occurring:

public View getView(final int position, View convertView, ViewGroup parent) {
        final ViewHolder holder;
        LayoutInflater inflater = context.getLayoutInflater();

        if (convertView == null) {
            convertView = inflater.inflate(R.layout.apps_row, parent, false);
            holder = new ViewHolder();
            holder.appIcon = (ImageView) convertView.findViewById(R.id.icon_apps_row);
            holder.appName = (TextView) convertView.findViewById(R.id.title_apps_row);
            holder.appPackage = (TextView) convertView.findViewById(R.id.package_apps_row);
            holder.appSwitch = (Switch) convertView.findViewById(R.id.switch_apps_row);
            convertView.setTag(holder);
        } else {
            holder = (ViewHolder) convertView.getTag();
        }

        final PackageInfo packageInfo = (PackageInfo) getItem(position);
        Drawable appIcon = packageManager.getApplicationIcon(packageInfo.applicationInfo);
        String appName = packageManager.getApplicationLabel(packageInfo.applicationInfo).toString();
        String appPackage = packageInfo.packageName;
        appIcon.setBounds(0, 0, 125, 125);
        //holder.appName.setCompoundDrawables(appIcon, null, null, null);
        //holder.appName.setCompoundDrawablePadding(0);
        holder.appIcon.setImageDrawable(appIcon);
        holder.appName.setText(appName);
        holder.appPackage.setText(appPackage);
        holder.appSwitch.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                if (holder.appSwitch.isChecked()){

                    applicationsActivity.selectedApps.add(packageInfo.packageName);
                    if (applicationsActivity.selectedApps.contains(packageInfo.packageName)) {
                        Toast.makeText(view.getContext(), packageInfo.packageName + " added", Toast.LENGTH_SHORT).show();
                    }
                } else {

                    applicationsActivity.selectedApps.remove(packageInfo.packageName);
                    Toast.makeText(view.getContext(), packageInfo.packageName + " removed", Toast.LENGTH_SHORT).show();
                }
            }
        });
        return convertView;
    }
}

Then in my Activity, on a button click, I send the ArrayList<String> as an extra with an Intent to another Activity that triggers an Intent as startActivityForResult(). Here is the relevant code for that:

Here's where the Intent starts...

Intent intent = new Intent(AddNewActivity.this, ApplicationsActivity.class);
                    startActivityForResult(intent, 1);

This is where the ArrayList<String> is being passed back to the parent Activity...

//This is the listener for the save button.
        saveAppsButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                //This is what happens when the save button is clicked.
                Intent intent = new Intent();
                intent.putStringArrayListExtra("selectedApps", selectedApps);
                setResult (1, intent);
                finish();

            }
        });

On the receiving end of the Intent, I grab the extra and then attempt to set a TextView to display the number of items in the ArrayList<String>. However, when I test the app in the emulator, the TextView displays 0...

Here is the code for the triggering Activity, where I get the extras and set the TextView:

@Override
    protected void onActivityResult (int requestCode, int resultCode, Intent data) {

        super.onActivityResult(requestCode, resultCode, data);

        if (requestCode == 1) {


           
                selectedApps = data.getStringArrayListExtra("selectedApps");
                if (selectedApps != null) {

                    int itemCount = selectedApps.size();
                    View view0 = listView.getChildAt(0);
                    TextView toolTip0 = view0.findViewById(R.id.addnew_tooltip);
                    toolTip0.setText(itemCount + " applications/notifications");
                }

            } else {
                    Toast.makeText(this, "NULL!", Toast.LENGTH_SHORT).show();
                }

            }

I have also attached an image from the emulator demonstrating that the TextView is coming back as 0. For the life of me, I cannot figure out why it isn't working right.

[Adam Wentz]

(Forgive me if this is bad forum manners) ::Bump::

I still haven't been able to figure out what is happening here.

[Stephan van Hulst]

Could you create an SSCCE for us? Something we can easily run on an emulator without all the code and data that is not relevant to the problem?

[Norm Radder]

Try doing some debugging by adding print statements to show the values of different variables being used.
What is in the Intent: data?

Note: I'd use a more unique id as request code than 1 to identify the Intent.

[Campbell Ritchie]

You were right to bump the thread after a week. Anything significantly more than a day could justify a bump.

I can't seem to see where you are adding anything to the List.

Last week, you wrote:Each list item also includes a Switch.

That isn't clear. Do you mean that each item added to the List includes a Switch?

[Adam Wentz]

@Campbell Ritchie - Yes, there is a switch in each list item. I have not included the full source code, so no, you do not see where I am adding items to the list. However, there are things being added from PackageManager.

@Norm Radder - I am using Toasts to help with debugging. When a switch in the ListView is clicked, a Toast prints the relevant String being added to my ArrayList. Then another toast prints the ArrayList's new size after the item has been added. Insofar as I can see, the ArrayList is properly populating. It seems that there is something happening with the Intent. After clicking on 'Save' in the Activity where the Intent extras originate, I have a Toast in onActivityResult() that looks like this:

@Override
    protected void onActivityResult (int requestCode, int resultCode, Intent data) {

        super.onActivityResult(requestCode, resultCode, data);

        if (requestCode == 11) {


                variableOne = data.getBooleanExtra("varOne", false);
                variableTwo = data.getBooleanExtra("varTwo", false);
                variableThree = data.getBooleanExtra("varThree", false);
                selectedStuff = data.getStringArrayListExtra("selectedStuff");
                if (selectedApps.size() >= 1) {

                    int itemCount = selectedStuff.size();
                    View view0 = listView.getChildAt(0);
                    TextView toolTip0 = view0.findViewById(R.id.addnew_tooltip);
                    toolTip0.setText(itemCount);
                    Toast.makeText(AddNewActivity.this, Integer.toString(itemCount), Toast.LENGTH_SHORT).show();
                }

            } else {
                    Toast.makeText(this, "selectedStuff NULL!", Toast.LENGTH_SHORT).show();
                }

            }

That Toast shows the itemCount as being 0. However, I have a conditional that only shows the Toast if selectedStuff isn't empty. AND, I can go into the connected database and see that variableOne, variableTwo, and variableThree are appropriately coming through. However, in the field for selectedStuff it simply shows []. PS--I had changed my code to 11 since my OP.

@Stephen van Hulst - Due to the nature of the project I am working on, there is a lot in terms of limitation. I cannot share all of the code, and the code I do share has to be made generified. It's a lot of extra leg work on my part.

[Norm Radder]

The problem with using Toasts is that the output can't be copied from the logcat and pasted it here so we can see what is happening.

Another possible problem is the use of literal Strings in the put and get methods.  There should be one final String defined that is used in  both places to make sure the same value is used in both places.
Same for the request code: it should be a final int, not hardcode literals in different places in the code.

[Adam Wentz]

Norm Radder wrote:The problem with using Toasts is that the output can't be copied from the logcat and pasted it here so we can see what is happening.

Another possible problem is the use of literal Strings in the put and get methods.  There should be one final String defined that is used in  both places to make sure the same value is used in both places.
Same for the request code: it should be a final int, not hardcode literals in different places in the code.

Using final Strings is probably a better "best practice" option but it is not the problem for me. I have checked and re-checked both put and get statements and they are identical. Furthermore, as I have said, my other extras are working just fine. It is only the selectedStuff ArrayList that is giving me problems.

[Norm Radder]

Seeing these bits and pieces of code that don't go together makes it hard to suggest what the problem is.
These have different id Strings:

selectedStuff = data.getStringArrayListExtra("selectedStuff");
intent.putStringArrayListExtra("selectedApps", selectedApps);

[Pete Letkeman]

A few items if I may.

Point 1:
Instead if using Toast, you logcat as Norm has already suggested. Android Studio (and possibly other Android development environments) make this easy to do.
You can do this with

Log.v("Array Size: ", selectedApps.size().toString());

as noted here https://developer.android.com/studio/debug/am-logcat.html and here https://developer.android.com/reference/android/util/Log.html.
Within Android Studio, there is even a short cut for this, just type logd then press the tab key.

Point 2:
There is an limit to the amount of data that can be sent using the putExtraXXX methods as noted here https://www.neotechsoftware.com/blog/android-intent-size-limit.
It is therefore recommended that you put small objects when using putExtraXXX methods.
Instead of placing an array/arraylist in a bundle and sharing that, create a temporary file or database entry and place the path to that in the bundle.
Do you know the approx size of the array/arrayList?

Point 3:
I think that Norm mentioned this, but it can be hard to keep track of numbers like

if (requestCode == 11) {} 

. It is usually easier to do something like this:

public static final String REQUEST_ACTION_NAME_CODE = 1;
public static final String ANOTHER_REQUEST_ACTION_NAME_CODE = 2;
public static final String ONE_MORE_ACTION_NAME_CODE = 3;
//...
if (requestCode == ONE_MORE_ACTION_NAME_CODE) {}

Doing something like that you can not only share the codes if you need to, but you can quickly change the values without do that many find and replaces.

[Adam Wentz]

Norm Radder wrote:Seeing these bits and pieces of code that don't go together makes it hard to suggest what the problem is.
These have different id Strings:

selectedStuff = data.getStringArrayListExtra("selectedStuff");
intent.putStringArrayListExtra("selectedApps", selectedApps);

😩 They are actually one and the same. As I had said before, I have to generify the code when I share it. It looks like I forgot to generify a line or two.

[Norm Radder]

Different names etc  makes the code very hard to understand.
Did you add lots of print statements to record the states as the code executes to show what is happening?

Please copy the print out and paste it here.

[Adam Wentz]

Pete Letkeman wrote:A few items if I may.

Point 1:
Instead if using Toast, you logcat as Norm has already suggested. Android Studio (and possibly other Android development environments) make this easy to do.
You can do this with

Log.v("Array Size: ", selectedApps.size().toString());

as noted here https://developer.android.com/studio/debug/am-logcat.html and here https://developer.android.com/reference/android/util/Log.html.
Within Android Studio, there is even a short cut for this, just type logd then press the tab key.

Point 2:
There is an limit to the amount of data that can be sent using the putExtraXXX methods as noted here https://www.neotechsoftware.com/blog/android-intent-size-limit.
It is therefore recommended that you put small objects when using putExtraXXX methods.
Instead of placing an array/arraylist in a bundle and sharing that, create a temporary file or database entry and place the path to that in the bundle.
Do you know the approx size of the array/arrayList?

Point 3:
I think that Norm mentioned this, but it can be hard to keep track of numbers like

if (requestCode == 11) {} 

. It is usually easier to do something like this:

public static final String REQUEST_ACTION_NAME_CODE = 1;
public static final String ANOTHER_REQUEST_ACTION_NAME_CODE = 2;
public static final String ONE_MORE_ACTION_NAME_CODE = 3;
//...
if (requestCode == ONE_MORE_ACTION_NAME_CODE) {}

Doing something like that you can not only share the codes if you need to, but you can quickly change the values without do that many find and replaces.

Thank you for taking the time type that all out!

Regarding point #1: Your code wouldn't work in Android Studio as is. It should be...

Log.v("Array Size: ", String.valueOf(selectedApps.size()));

Regarding point #2: The array I am testing with is very small and contains only one short String in it. So, size is not the issue here. However, it might create an issue in future, so I thank you for the references.

Regarding point #3: I do acknowledge that using a final string is probably a better practice. However, not using such practices is not at the root of my problem. I have verified as much multiple, multiple times.

[Pete Letkeman]

Sorry I forgot that you must used String.valueOf when dealing with the Android logger.

What if, simply for testing you did

ArrayList<String> myList = data.getStringArrayListExtra("selectedApps");

Instead of

selectedApps = data.getStringArrayListExtra("selectedApps");

If this change gives you the data you are expecting then you have a problem somewhere else with how selectedApps is being used/created.

Over here https://stackoverflow.com/questions/6543811/intent-putextra-list they mention that one option could be to use getSerializable. However they also say that what you have should work.

[Adam Wentz]

Pete Letkeman wrote:Sorry I forgot that you must used String.valueOf when dealing with the Android logger.

What if, simply for testing you did

ArrayList<String> myList = data.getStringArrayListExtra("selectedApps");

Instead of

selectedApps = data.getStringArrayListExtra("selectedApps");

If this change gives you the data you are expecting then you have a problem somewhere else with how selectedApps is being used/created.

Over here https://stackoverflow.com/questions/6543811/intent-putextra-list they mention that one option could be to use getSerializable. However they also say that what you have should work.

I did as you've suggested but to no avail. Here's where things currently are:

In my adapter class, I am adding/removing items to/from the Array list...

 if (holder.appSwitch.isChecked()){

                    selectedApps.add(packageInfo.packageName);
                    Log.v("Array Size: ", String.valueOf(selectedApps.size()));
                }

The log reflects that the item(s) are successfully added:

V/Array Size:: 1

Then I create and send the Intent when the save button is clicked, as such (the ... is a placeholder for redacted code):

Intent intent = new Intent();
                ...
                Log.v("Array Size: ", String.valueOf(selectedApps.size()));
                intent.putStringArrayListExtra("selectedApps", selectedApps);
                setResult (11, intent);
                finish();

At this point, the log changes:

V/Array Size:: 1

Subsequently, one can easily guess what the log will reflect on the get side of things:

if (requestCode == 11) {


                ...
                ArrayList<String> selectedApps = data.getStringArrayListExtra("selectedApps");
                Log.v("Array Size: ", String.valueOf(selectedApps.size()));
}

V/Array Size:: 1

This is all quite bizarre.

[Adam Wentz]

As you were...

I was copying/pasting code but forgot to update certain things.

The first log is:

V/Array Size:: 1

The second log is:

V/Array Size:: 0

The third log is:

V/Array Size:: 0

[Norm Radder]

The message text's should be unique so you can tell which statement generated which output.
These look all the same.

[Adam Wentz]

Norm Radder wrote:The message text's should be unique so you can tell which statement generated which output.
These look all the same.

I know the placement of the log lines in relation to how my code is being executed, as I communicated clearly in my previous post where
I actually lay out the various steps.

The first log is written right after a switch is clicked on the list. The second log is written just prior to the put statement for my ArrayList.
Finally, the third log is written just after the get statement is executed.

[Norm Radder]

It is clearer if all the log contents are unique so everyone can see which statement generated which output.

[Adam Wentz]

Though it is nothing more than a "best practice," I went ahead and made the logs unique.

V/Array Size - Switch on: 1
V/Array Size - Before put: 0
V/Array Size - After get: 0

That said, nothing has changed with regards to the functionality of my application.

[Adam Wentz]

It is evident that something is happening to the ArrayList between the time the switch is clicked and when it is being added to the Intent. Hmmmm....

[Norm Radder]

nothing more than a "best practice,"

No, I think it is more than that. It  is one of the ways to find bugs in code.

How can the value go from 1 at Switch to 0 at put?  Is a new ArrayList created?  Is there a Log statement for when the ArrayList is created?

[Adam Wentz]

How can the value go from 1 at Switch to 0 at put?  Is a new ArrayList created?  Is there a Log statement for when the ArrayList is created?

I stripped everything out, in terms of the ArrayList and am re-working it step-by-step. Here is what I have so far:

At the top of my adapter class, I initialize a new ArrayList...

public class MyAdapter extends BaseAdapter {

    ...
    ArrayList<String> selectedApps = new ArrayList<>();

Then, in the onClick method for my switch, I add or remove strings to/from the ArrayList accordingly...

@Override
            public void onClick(View view) {

                if (holder.appSwitch.isChecked()){

                    selectedApps.add(packageInfo.packageName);
                    Log.v("Array Size - Switch on", String.valueOf(selectedApps.size()));
                } else {

                    selectedApps.remove(packageInfo.packageName);
                    Log.v("Array Size - Switch off", String.valueOf(selectedApps.size()));                }


            }

Now, I must put the ArrayList with the Intent I trigger in the Activity to which the Adapter connects. That's where I seem to be running into problems.

[Adam Wentz]

So, I attempted to clone the ArrayList in the Adapter (which the Switch adds to) to an ArrayList in the Activity:

@Override
            public void onClick(View view) {

                selectedApps = new ArrayList<>();
                for (int i = 0; i < applicationsAdapter.getSelectedApps().size(); i++) {

                    selectedApps.add(applicationsAdapter.getSelectedApps().get(i));
                }

                Intent intent = new Intent();
                ...
                Log.v("Array Size - Before put", String.valueOf(selectedApps.size()));
                //intent.putStringArrayListExtra("selectedApps", selectedApps);
                setResult (11, intent);
                finish();

            }

I am still getting:

V/Array Size - Before put: 0

[Norm Radder]

I am still getting:

V/Array Size - Before put: 0

Is that all that is printed by the Log statements?

Why are you filtering out any of the print outs?
Does this ever get printed: Array Size - Switch off

The debug print outs should have a full history of the state of the ArrayList:
When it is created
when any change is made to it: adds and removes
When its contents are accessed

[Adam Wentz]

I have a bit of a digression I want to explore before we continue on this track.

Is there a way for me to get an instance of the switch in my Adapter, in my Activity? My thought is that if I can setup the onClickListener in the Activity, I can cut a lot of the "middleman" stuff out, thus fixing my problem.

I created an instance of my Adapter's ViewHolder in my Activity:

ApplicationsAdapter.ViewHolder viewHolder = new ApplicationsAdapter.ViewHolder();

Then I set up an onClickListener for the switch in my Activity's onCreate method:

viewHolder.appSwitch.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                if (viewHolder.appSwitch.isChecked()) {
                    //do stuff
                } else {
                    //do other stuff
                }
            }
        });

However, I am getting a NullPointerException. What am I missing?

[Norm Radder]

Please start a new thread for the new problem.

[Pete Letkeman]

I would be curious to know what is the result of the following line if placed right before the for loop?

Log.v("Adapter Size: ", String.valueOf(applicationsAdapter.getSelectedApps().size()));

With a value of 0 for the ArrayList selectedApps I would suspect that the reported size of applicationsAdapter to be 0 as well.

[Adam Wentz]

Pete Letkeman wrote:I would be curious to know what is the result of the following line if placed right before the for loop?

Log.v("Adapter Size: ", String.valueOf(applicationsAdapter.getSelectedApps().size()));

With a value of 0 for the ArrayList selectedApps I would suspect that the reported size of applicationsAdapter to be 0 as well.

Yes, it come s back 0 too.

[Pete Letkeman]

In that case I am suspect that there is something not exactly correct with applicationsAdapter.getSelectedApps().
Effectively the for loop which you have as

for (int i = 0; i < applicationsAdapter.getSelectedApps().size(); i++) {
    selectedApps.add(applicationsAdapter.getSelectedApps().get(i));
}

is actually

for (int i = 0; i < 0; i++) {
    selectedApps.add(applicationsAdapter.getSelectedApps().get(i));
}

[Adam Wentz]

Pete Letkeman wrote:In that case I am suspect that there is something not exactly correct with applicationsAdapter.getSelectedApps().
Effectively the for loop which you have as

for (int i = 0; i < applicationsAdapter.getSelectedApps().size(); i++) {
    selectedApps.add(applicationsAdapter.getSelectedApps().get(i));
}

is actually

for (int i = 0; i < 0; i++) {
    selectedApps.add(applicationsAdapter.getSelectedApps().get(i));
}

The Strings are added in the Adapter class, where I presently have the Switch's onClickListener. I suspect there is something happening to the data between the Adapter and the Activity it's attached to (where I add the ArrayList to an Intent).
That's why I am now exploring moving the onClickListener over to the Activity from the Adapter. I just don't know how to properly get an instance of the Switch with which to do that. I am getting a NullPointerException on the switch. I've started a new thread to try to tackle that issue.

[Adam Wentz]

Norm Radder wrote:

I am still getting:

V/Array Size - Before put: 0

Is that all that is printed by the Log statements?

Why are you filtering out any of the print outs?
Does this ever get printed: Array Size - Switch off

The debug print outs should have a full history of the state of the ArrayList:
When it is created
when any change is made to it: adds and removes
When its contents are accessed

This is (redacted) verbose from the Logcat:

01-29 22:50:46.240 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.250 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.279 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.292 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.315 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.328 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.333 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:46.333 10797-10856/... E/Surface: getSlotFromBufferLocked: unknown buffer: 0xa2263b40
01-29 22:50:46.335 10797-10856/... D/OpenGLRenderer: endAllStagingAnimators on 0xae4dd600 (ListView) with handle 0xb2ebe210
01-29 22:50:47.796 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:51.192 10797-10797/... V/Array Size - Switch on: 1
01-29 22:50:51.252 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:51.254 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:51.260 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:51.269 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:53.194 10797-10856/... E/Surface: getSlotFromBufferLocked: unknown buffer: 0xa22642c0
01-29 22:50:58.735 10797-10797/... V/Array Size - Before put: 0
01-29 22:50:58.782 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:58.792 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:58.809 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:58.823 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:58.844 10797-10856/... D/EGL_emulation: eglMakeCurrent: 0xae5d4c00: ver 2 0 (tinfo 0xb2ebe0e0)
01-29 22:50:58.860 10797-10856/... E/Surface: getSlotFromBufferLocked: unknown buffer: 0xa22640e0
01-29 22:50:58.861 10797-10856/... D/OpenGLRenderer: endAllStagingAnimators on 0xa12ba100 (RippleDrawable) with handle 0xae4ef510

It doesn't appear to include anything of great importance, in addition to the logs I've already shared.

[Norm Radder]

What other accesses to the ArrayList happen before line 11 and between line 11 and line 17?

[Adam Wentz]

Norm Radder wrote:What other accesses to the ArrayList happen before line 11 and between line 11 and line 17?

Before 11:

ArrayList<String> selectedApps = new ArrayList<>();

Between 11 & 17:

saveButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                Intent intent = new Intent();
                ...
                Log.v("Array Size - Before put", String.valueOf(selectedApps.size()));
                //intent.putStringArrayListExtra("selectedApps", selectedApps);
                setResult (11, intent);
                finish();

            }
        });

[Norm Radder]

When is the statement that prints this: V/Array Size - Switch on: 1 executed?

[Adam Wentz]

Norm Radder wrote:When is the statement that prints this: V/Array Size - Switch on: 1 executed?

That occurs in the ViewHolder for the Adapter class:

holder.appSwitch.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                if (holder.appSwitch.isChecked()){

                    selectedApps.add(packageInfo.packageName);
                    Toast.makeText(context, selectedApps.get(0) + " - " +
                            String.valueOf(selectedApps.size()), Toast.LENGTH_SHORT).show();
                    Log.v("Array Size - Switch on", String.valueOf(selectedApps.size()));
                } else {

                    selectedApps.remove(packageInfo.packageName);
                    Log.v("Array Size - Switch off", String.valueOf(selectedApps.size()));                }


            }
        });

[Norm Radder]

Ok, I'm giving up.  This is too frustrating. Trying to get the order of events is too hard without seeing the code.  The print outs don't show enough information.

Maybe someone with more patience can help.

[Pete Letkeman]

Adam, can we start at the beginning? Maybe I can sum up what is happen here, please correct me if I'm wrong.
1) You suspected that there was a problem with an ArrayList loosing it's data.
2) Recently we discovered that the actual problem was not with the ArrayList, but the Adapter feeding the ArrayList.
3) You are using a ListView?
4) You are working on creating some sort of Launcher for Android

What kind of adapter are you using? It could be a RecycleViewAdapter, but it could be a different adapter such as an ArrayAdapter or some other kind?
Here is a sample/tutorial on the ArrayAdapter http://abhiandroid.com/ui/arrayadapter-tutorial-example.html.

I would recommend that you use the RecycleView which does support at least Android 4.4 (KitKat).
Eveything that a ListView  can do a RecycleView can do.
RecycleView also has features that are not found in ListView like the built in Holder class/pattern.

Currently you are trying to implement an onClick event in the holder which you have attached to the adapter. Is this correct?

Is all of what I've stated above in this post correct?

------------ Unpaid Book Promo ------------
I can tell you from experience that in the book Android Programming: The Big Nerd Ranch Guide (3rd Edition) found here
https://www.bignerdranch.com/books/android-programming/ which you can see on Amazon.com here
https://www.amazon.com/Android-Programming-Ranch-Guide-Guides/dp/0134706056/
They walk though a tutorial in one of the chapters giving you the code create your own custom Android launcher.
Not only that they provide you with a primer on using SQLite, Fragments, Activities and more.
You could get the PDF version of this book and be on you way very quickly with not only the project that you are currently working on, but other projects.
-----------------------------------------------

[Adam Wentz]

Pete Letkeman wrote:Adam, can we start at the beginning? Maybe I can sum up what is happen here, please correct me if I'm wrong.
1) You suspected that there was a problem with an ArrayList loosing it's data.
2) Recently we discovered that the actual problem was not with the ArrayList, but the Adapter feeding the ArrayList.
3) You are using a ListView?
4) You are working on creating some sort of Launcher for Android

What kind of adapter are you using? It could be a RecycleViewAdapter, but it could be a different adapter such as an ArrayAdapter or some other kind?
Here is a sample/tutorial on the ArrayAdapter http://abhiandroid.com/ui/arrayadapter-tutorial-example.html.

I would recommend that you use the RecycleView which does support at least Android 4.4 (KitKat).
Eveything that a ListView  can do a RecycleView can do.
RecycleView also has features that are not found in ListView like the built in Holder class/pattern.

Currently you are trying to implement an onClick event in the holder which you have attached to the adapter. Is this correct?

Is all of what I've stated above in this post correct?

------------ Unpaid Book Promo ------------
I can tell you from experience that in the book Android Programming: The Big Nerd Ranch Guide (3rd Edition) found here
https://www.bignerdranch.com/books/android-programming/ which you can see on Amazon.com here
https://www.amazon.com/Android-Programming-Ranch-Guide-Guides/dp/0134706056/
They walk though a tutorial in one of the chapters giving you the code create your own custom Android launcher.
Not only that they provide you with a primer on using SQLite, Fragments, Activities and more.
You could get the PDF version of this book and be on you way very quickly with not only the project that you are currently working on, but other projects.
-----------------------------------------------

Although I have suspicions as to what is happening, it is only speculation because I cannot collect enough debug info.

That said, let me summarize what is happening and, in the process, hopefully answer your prior questions.
For the record, after reading your previous post, I did decide to change my ListView to a RecyclerView. Between the time I
read your response and now, I have been working on writing a new adapter class and implementing it. So, now I have a
RecyclerView in my Activity that uses a custom RecyclerView Adapter. In my override of onBindViewHolder in the Adapter,
I have a Switch view. When the Switch isChecked, it adds a String, which is packageInfo.PackageName, to an ArrayList
that lives in the Activity. When the switch is clicked, I write the size of the ArrayList in the Activity to the log, as such:

holder.appSwitch.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                if (holder.appSwitch.isChecked()) {

                    applicationsActivity.selectedApps.add(appPackage);
                    Log.v("Array size - Switch on", String.valueOf(applicationsActivity.selectedApps.size()));
                } else {

                    applicationsActivity.selectedApps.remove(appPackage);
                    Log.v("Array size - Switch off", String.valueOf(applicationsActivity.selectedApps.size()));
                }
            }
        });

Then, in the Activity I create an Intent to which I attach some Extras, including the ArrayList. At that point, I have
another log entry being made, which is the size of the ArrayList right prior to the putExtra statement for the Intent (... denotes redacted code):

//This is the listener for the save button.
        saveAppsButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                Intent intent = new Intent();
                ...
                Log.v("Array Size - Before put", String.valueOf(selectedApps.size()));
                intent.putStringArrayListExtra("selectedApps", selectedApps);
                setResult (11, intent);
                finish();

            }
        });

Then, in the Activity prior to this, the one waiting for this Activity's result, I use a standard conditional with get statements.
There are 4 Extras altogether, 3 Booleans and 1 StringArray. These values are later written to a database.

Now, the problem...I can click the aforementioned and see in the log that the ArrayList in the Activity has a size of 1...
or however many switches have been clicked. However, my next log entry comes back 0:

Log.v("Array Size - Before put", String.valueOf(selectedApps.size()));

Implementation:

In the Activity using the Adapter, I initialize a new ArrayLise just before the onCreate method:

ArrayList<String> selectedApps = new ArrayList<>();

Then, in the Adapter, just before the Constructor, I grab an instance of the Activity:

ApplicationsActivity applicationsActivity = new ApplicationsActivity();

Then, in the ViewHolder for the adapter, I grab the Switch:

public static class ApplicationsAdapterViewHolder extends RecyclerView.ViewHolder {

        ...
        Switch appSwitch;
        int position;

        public ApplicationsAdapterViewHolder(View itemView) {
            super(itemView);

           ...
            appSwitch = (Switch) itemView.findViewById(R.id.switch_apps_row);
        }
    }

Then, in the onBindViewHolder method within my Adapter, I set a new onClickListener for the Switch.
In the onClick method, I add a conditional that adds the String to the ArrayList in the Activity if the Switch isChecked:

holder.appSwitch.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                if (holder.appSwitch.isChecked()) {

                    applicationsActivity.selectedApps.add(appPackage);
                    Log.v("Array size - Switch on", String.valueOf(applicationsActivity.selectedApps.size()));
                } else {

                    applicationsActivity.selectedApps.remove(appPackage);
                    Log.v("Array size - Switch off", String.valueOf(applicationsActivity.selectedApps.size()));
                }
            }
        });

[Adam Wentz]

I finally took the time to create an SSCCE (Gradle build in Android Studio)...


Download a zip archive of the project. (It is about 20 MB)