();
for (int i = 0; i < bottles.length; i++) {
bottles[i] = loadImage("water-bottle-"+i+".png");
}
for (int i = 0; i < 5; i++) {
int index = (int(random(0, bottles.length)));
trash.add(new Trash(bottles[index], 120+i*100, 645, random (32, 72)));
}
}
```
What i want is to click in-top of any of the images and remove it, so to start I wrote this:
```auto
void removebottle() {
for (int i = bottles.length-1; i>=0; i--) {
bottles[i] = bottles[i].get();
if (mouseX > x && mouseX < x + diamW * randomSize && mouseY > y + yay
&& mouseY < y + yay + diamH * randomSize) {
trash.remove(i);
}
}
}
```
This removes the bottle array, all 3 images at once, leaving the other 2 images (made by the trash array) and when hovered over it shows an ‘out of bound’ error or a CurrentModificationException error.
---
### Author:  [@josephh](https://discourse.processing.org/u/josephh)
#### Post date: [September 10, 2018, 3:16pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/4 "2018-09-10T15:16:14Z")
Are the bottles images overlaping each other?
What do you want to do by saying that ? :
```auto
bottles[i] = bottles[i].get();
```
Maybe a solution is to add a `break` after `trash.remove(i);` so you only remove the first bottle.
[https://processing.org/reference/break.html](https://processing.org/reference/break.html)
---
### Author:  [@neilcsmith](https://discourse.processing.org/u/neilcsmith)
#### Post date: [September 10, 2018, 3:20pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/5 "2018-09-10T15:20:45Z")
> [@tefa](#):
>
> CurrentModificationException
Or _Concurrent_ ModificationException? 😉 If you see that, something else is going on here too, because that means two things are trying to change the list at the same time (concurrently). What is calling `removebottle()`?
---
### Author:  [@tefa](https://discourse.processing.org/u/tefa)
#### Post date: [September 10, 2018, 3:33pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/6 "2018-09-10T15:33:50Z")
When i say :
```auto
bottles[i] = bottles[i].get();
```
I want to capture the index of the array, but then I am only capturing the index of 3 images when in truth there are 5 appearing
---
### Author:  [@tefa](https://discourse.processing.org/u/tefa)
#### Post date: [September 10, 2018, 3:39pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/7 "2018-09-10T15:39:08Z")
Hehe yes concurrent 😅
This is calling removebottle()
```auto
void draw(){
for (Trash part : trash) {
part.removebottle();
}
}
```
and when the _Concurrent_ ModificationException appears it highlights this ’ for (Trash part : trash) { ’ part
---
### Author:  [@neilcsmith](https://discourse.processing.org/u/neilcsmith)
#### Post date: [September 10, 2018, 4:00pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/8 "2018-09-10T16:00:38Z")
> [@tefa](#):
>
> When i say :
>
> ```auto
> bottles[i] = bottles[i].get();
>
> ```
>
> I want to capture the index of the array, but then I am only capturing the index of 3 images when in truth there are 5 appearing
What you’re actually doing is just making a needless copy of the image each time!
> [@tefa](#):
>
> This is calling removebottle()
>
> ```auto
> void draw(){
> for (Trash part : trash) {
> part.removebottle();
> }
> }
>
> ```
>
> and when the _Concurrent_ ModificationException appears it highlights this ’ for (Trash part : trash) { ’ part
You cannot do this! You’re attempting to modify the list by removing items, while using an iterator over it in the loop. You could theoretically use a backwards for loop like you have in `removebottle()`, but I _really_ don’t understand from your code what you’re trying to do. In particular why are you using the index in the bottle array to remove stuff from the trash list?!
---
### Author:  [@tefa](https://discourse.processing.org/u/tefa)
#### Post date: [September 10, 2018, 4:17pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/9 "2018-09-10T16:17:33Z")
how do I do it? I tried this
```auto
for (int i = trash.size()-1; i>=0; i--) {
if (mouseX > x && mouseX < x + diamW * randomSize && mouseY > y + yay
&& mouseY < y + yay + diamH * randomSize) {
trash.remove(i);
}
}
```
and it throws a Concurrent ModificationException on the
```auto
void draw(){
for (Trash part : trash) {
part.removebottle();
}
}
```
---
### Author:  [@neilcsmith](https://discourse.processing.org/u/neilcsmith)
#### Post date: [September 10, 2018, 4:43pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/10 "2018-09-10T16:43:46Z")
I meant
```auto
void draw() {
for (int i = trash.size()-1; i>=0; i--) {
trash.get(i).removebottle();
}
}
```
although that should stop the CME, it will likely crash if you remove more than one item from trash inside removebottle(). This code doesn’t actually make a lot of sense, unless there’s something key missing - you might need to go back to the drawing board with this one.
---
### Author:  [@quark](https://discourse.processing.org/u/quark)
#### Post date: [September 10, 2018, 4:54pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/11 "2018-09-10T16:54:48Z")
> [@tefa](#):
>
> please someone teach me how to keep track of an item in an array and then the delete item when clicked, without using an arrayList.
What is wrong with an ArrayList? If you have a collection of items (in your case I assume it is images) and the size of the collection can change either through deletion or adding at run-time then an ArrayList (or other list) is much better than using an array.
It is not clear what you are doing with the Trash class can you show the class here.
I am going to assume that you have 2 ArrayLists `bottles` to hold the current images to shows, and `trash` to hold the the bottles to be deleted.
So we have the lists
```auto
ArrayList bottles = new ArrayList();
ArrayList trash = new ArrayList();
```
Now we want to update the list `bottles`
```auto
for (PImage bottle : bottles){
if( some_condition_to_see_if_the_bottle_should_be_deleted ){
trash.add(bottle; // if condition true - add bottle to be deleted
}
}
bottles.removeAll(trash); // remove unwanted bottles
trash.clear(); // clear trash can before next update
```
---
### Author:  [@tefa](https://discourse.processing.org/u/tefa)
#### Post date: [September 10, 2018, 4:57pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/12 "2018-09-10T16:57:04Z")
> [@neilcsmith](#):
>
> for (int i = trash.size()-1; i\>=0; i–) { trash.get(i).removebottle(); }
I have three images but I want 5 random images from those 3 images to appear.
---
### Author:  [@tefa](https://discourse.processing.org/u/tefa)
#### Post date: [September 10, 2018, 4:58pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/13 "2018-09-10T16:58:14Z")
I edited the post because I imagine I overwhelmed you, i made this change
```auto
ArrayList trash = new ArrayList();
ArrayList bottles = new ArrayList();
void setup(){
for (int i = 0; i < bottles.size(); i++) {
bottles.add(loadImage("water-bottle-"+i+".png"));
}
for (int i = 0; i < 5; i++) {
int index = (int(random(0, bottles.size())));
trash.add(new Trash(bottles, 120+i*100, 645, random (32, 72)));
}
}
}
```
It is throwing this error here `trash.add(new Trash(bottles, 120+i*100, 645, random (32, 72)));` in the coma after index
This is my class
```auto
class Trash {
float x;
float y;
float speedY = 0.5;
float randomSize;
float offSetThis = 10;
float scalarThis = 3;
float yay;
float diamW = int(900/400), diamH = int(800/400);
PImage bottle;
float upDown = 0.0;
float speedTrash =.04;
//rectangel
int rectW = width;
int rectH = height/2;
int rectX = 0, rectY = height/2;
Trash(PImage tempImg, float tempX, float tempY, float tempD) {
x = tempX;
y = tempY;
randomSize = tempD;
bottle = tempImg;
}
void move() {
if (y + diamH > height) {
speedY *= 1;
}
y--;
if (y + diamH < height/2 + 40) {
y = y + y;
speedY *= -1;
}
}
void removebottle() {
for (int i = trash.size()-1; i>=0; i--) {
if (mouseX > x && mouseX < x + diamW * randomSize && mouseY > y + yay
&& mouseY < y + yay + diamH * randomSize) {
trash.remove(i);
}
}
}
void display() {
imageMode(CENTER);
yay = offSetThis + sin(angle) * scalarThis;
noTint();
image(bottle, x, y+yay, diamW * randomSize, diamH * randomSize);
}
}
```
---
### Author:  [@TfGuy44](https://discourse.processing.org/u/TfGuy44)
#### Post date: [September 10, 2018, 7:09pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/15 "2018-09-10T19:09:11Z")
The problem I see, honestly, is that you have your PImage variables inside your class.
You don’t need them in there!
Have one array for your images - ALL your images - and then your class only needs an index into that array from the image it wants to use. This also makes it so you don;t have duplicated images all over the place. You only need one copy of each image loaded!
```auto
PImages[] all_images = new PImage[6];
void setup(){
size(400,400);
all_images[0] = loadImage("bottle1.png");
all_images[1] = loadImage("bottle2.png");
all_images[2] = loadImage("bottle3.png");
all_images[3] = loadImage("plasticbag.png");
// etc
}
class Trash{
int image_index;
int x,y;
Trash( int in_image_index ){
image_index = in_image_index;
}
void draw(){
image( all_images[image_index], x, y);
}
}
```
---
### Author:  [@tefa](https://discourse.processing.org/u/tefa)
#### Post date: [September 10, 2018, 7:27pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/16 "2018-09-10T19:27:56Z")
I want to have duplicated images. In total I want 5 images to appear out of 3 images i got. So, randomly I am making that happen
```auto
for (int i = 0; i < 5; i++) {
int index = (int(random(0, bottles.length)));
trash.add(new Trash(bottles[index], 120+i*100, 645, random (32, 72)));
}
```
---
### Author:  [@jb4x](https://discourse.processing.org/u/jb4x)
#### Post date: [September 10, 2018, 7:33pm UTC](https://discourse.processing.org/t/how-to-sort-an-array-and-keep-track-of-the-index/3427/17 "2018-09-10T19:33:39Z")
> [@tefa](#):
>
> I want to have duplicated images
You want to draw several time the same image at the same time but you don’t want to have duplicated image in your memory.
You should only have 5 images in memory.
The way you are building your class create a new image in memory, every time you create a new object of that class.
Instead, you should do as @TfGuy44 suggested and use an index that refers to the image that you want.