Problema ejecucion de programa de ordenamiento

[zucho2007]

Que tal , soy nuevo en esto de java y tengo este problema con un codigo que lo que hace es pedir la cantidad de valores dentro de un vector, luego ingresas los valores, y al final te deberia salir ordenados los valores dentro del vector, tengo este error al ejecutar y no se cual es el problema
Les dejo el codigo:
package menor;
import java.util.Arrays;
import java.util.Scanner;

public class Menor {
int n=0;
private int[] numeros;
private Scanner teclado;


public void cargar() {
int length = 0;

teclado=new Scanner(System.in);

System.out.print ("El tamaño de la Serie: ");



length = Integer.parseInt(teclado.nextLine());
numeros=new int[length];
long[] numeros = new long[length];

for(int f=0;f<numeros.length;f++) {
System.out.print("Ingrese el sueldo:");
numeros[f]=teclado.nextInt();
}
}

public void ordenar() {
for(int k=0;k<4;k++) {
for(int f=0;f<4-k;f++) {
48 if (numeros[f]>numeros[f+1]) {
int aux;
aux=numeros[f];
numeros[f]=numeros[f+1];
numeros[f+1]=aux;
}
}
}}

public void imprimir() {
System.out.println("ordenados de menor a mayor.");
for(int f=0;f<numeros.length;f++) {
System.out.println(numeros[f]);
System.out.print(Arrays.toString(numeros));
}
}

public static void main(String[] args) {
Menor pv=new Menor();
pv.cargar();
68 pv.ordenar();
pv.imprimir();
}}

Y ahora el Error

run:
El tamaño de la Serie: 3
Ingrese el sueldo:5
Ingrese el sueldo:9
Ingrese el sueldo:8
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 3
at menor.Menor.ordenar(Menor.java:48)
at menor.Menor.main(Menor.java:68)
Java Result: 1

Las lineas a las que se refiere las señalo arriba.
Si me pudieran ayudar se los agradeceria

[Xerelo]

Si estás metiendo los valores en un array de tres posiciones, ¿por qué pides en el bucle hasta la quinta posición?

[zucho2007]

ya veo , no es de 3 valores , sino n valores .. entonces solo tendria que modificarl y cambiar 4 por n?

[Tipdar]

Hablas de un vector, pero no usas ningún vector. Por cierto, usar vectores a estas alturas es posible, pero te recomendaría otra estructura de datos, digamos un ArrayList o un LinkedList.

No obstante, tu algoritmo de ordenación por burbuja está mal. El algoritmo correcto es de esta forma:

Código PHP:

  for (int k = 0; k < numeros.length - 1; k++) 
    for (int f = k+1; f < numeros.length; f++)  
        if (numeros[k]>numeros[f]){ 
            int aux = numeros[k]; 
            numeros[k] = numeros[f]; 
            numeros[f] = aux; 
        }  
   

He usado tus mismas variables, pero igual hay otra manera más sencilla de ordenar, teniendo en cuenta que estás hablando de un arreglo de números enteros (los int son primitivos) y no de objetos (en cuyo caso deberías redefinir el método compareTo de la interfaz Comparable) puedes usar directamente el método sort de la clase Arrays. Sería esto:

Código PHP:

  Arrays.sort(numeros);  
   

Este método retorna void, por lo que no tienes que hacer nada más que eso.