();
> int n = 600, min = 0, max = 600, it = 0;
> void setup() {
> size(600,600);
> for(int i = 0; i < n; i++) values.add(random(min,max));
> }
> void draw() {
> background(0);
> stroke(255);
> for(int i = 0; i < n; i++) {
> line(i,height,i,height-values.get(i));
> }
> }
> void keyPressed() { for(int i = 0; i < n; i++) sortArray(); }
> void sortArray() {
> int mx = it;
> for(int i = it; i < n; i++) {
> if(mx < n && values.get(i) > values.get(mx)) {
> swap(mx,i);
> mx = i;
> }
> }
> it++;
> }
>
> void swap(int a, int b) {
> float temp = values.get(a);
> values.set(a,values.get(b));
> values.set(b,temp);
> }
>
> ```
what did I do wrong?
---
### Author:  [@SNICKRS](https://discourse.processing.org/u/SNICKRS)
#### Post date: [January 18, 2021, 5:34pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/2 "2021-01-18T17:34:45Z")
Can I have more details? I don’t have an idea of what to help you with.
---
### Author:  [@Chrisir](https://discourse.processing.org/u/Chrisir)
#### Post date: [January 18, 2021, 5:36pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/3 "2021-01-18T17:36:11Z")
you can enlarge his code with the arrow
the graphic shows that the sorting is not complete
that’s his issue
---
### Author:  [@Chrisir](https://discourse.processing.org/u/Chrisir)
#### Post date: [January 18, 2021, 5:36pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/4 "2021-01-18T17:36:59Z")
> [@CodeMasterX](#):
>
> `for(int i = 0; i < n; i++)`
What happens when you press key like 6 times?
for example this works
```auto
ArrayList values = new ArrayList();
int n = 600, min = 0, max = 600, it = 0;
void setup() {
size(600, 600);
for (int i = 0; i < n; i++)
values.add(random(min, max));
}
void draw() {
background(0);
stroke(255);
for (int i = 0; i < n; i++) {
line(i, height, i, height-values.get(i));
}
}
void keyPressed() {
for (int i2 = 0; i2 < 7; i2++) {
it=0;
for (int i = 0; i < n; i++)
sortArray();
}
}
void sortArray() {
int mx = it;
for (int i = it; i < n; i++) {
if (mx < n && values.get(i) > values.get(mx)) {
swap(mx, i);
mx = i;
}
}
it++;
}
void swap(int a, int b) {
float tempA = values.get(a);
float tempB = values.get(b);
values.set(a, tempB);
values.set(b, tempA);
}
```
maybe you have to repeat not 7 but until there is no swapping occured
---
### Author:  [@SNICKRS](https://discourse.processing.org/u/SNICKRS)
#### Post date: [January 18, 2021, 5:46pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/5 "2021-01-18T17:46:08Z")
Hey @CodeMasterX, maybe you can use the sort() method in processing.
This is an implementation:
```auto
void setup() {
int[] array = new int[10];
for(int i = 0; i < 100; i+= 10) {
array[i] = i+((int)random(1000);
}
printArray(array);
array = sort(array);
printArray(array);
}
```
---
### Author:  [@Chrisir](https://discourse.processing.org/u/Chrisir)
#### Post date: [January 18, 2021, 5:52pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/6 "2021-01-18T17:52:04Z")
similar:
```auto
import java.util.Collections; // Import the Collections class
ArrayList list = new ArrayList();
// --------------------------------------------------------------------------------
list.add (17.1);
list.add (7.7);
list.add (12.9);
// -----------------------------------------------
// Use
println(list);
Collections.sort(list);
println(list);
```
---
### Author:  [@CodeMasterX](https://discourse.processing.org/u/CodeMasterX)
#### Post date: [January 18, 2021, 6:19pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/7 "2021-01-18T18:19:56Z")
The whole purpose of the program is to create a sorting algorithm : P
---
### Author:  [@CodeMasterX](https://discourse.processing.org/u/CodeMasterX)
#### Post date: [January 18, 2021, 6:28pm UTC](https://discourse.processing.org/t/issue-with-a-sorting-program/27163/8 "2021-01-18T18:28:50Z")
I found a problem : P
```auto
ArrayList values = new ArrayList();
int n = 600, min = 0, max = 600, it = 0;
void setup() {
size(600,600);
for(int i = 0; i < n; i++) values.add((float)i);//random(min,max));
}
void draw() {
background(0);
stroke(255);
for(int i = 0; i < n; i++) {
line(i,height,i,height-values.get(i));
}
}
void keyPressed() { while( sortArray()); }
boolean sortArray() {
int mx = it;
boolean aaa = true;
for(int i = 0; i < n; i++) {
if(mx < n && i > mx && values.get(i) > values.get(mx)) {
swap(i,mx);
mx = i;
aaa = false;
}
}
it++;
return !aaa;
}
void swap(int a, int b) {
float temp = values.get(a);
values.set(a,values.get(b));
values.set(b,temp);
}
```
it returns this:
