Java-Prog. abkürzen ?

[Dennis9]

Hallo, habe mal eine Fage und zwar ob ma folgenden JAVA-Code irgendwie kürzen kann ? kommt mir ziemlich lang vor bin neu in der JAVA-programmierung und auch in dem Forum hier .. also es geht um ein Programm, welches drei eingegebene Zahlen nach Größe sortiert wiedergeben soll .. bin froh über jeden Hinweis wie man es abkürzen könnte mfg

import java.io.*;
public class Aufgabe_5 {



	/**
	 * @param args
	 */
	public static void main(String[] args)
	throws IOException 
	{
		// TODO Auto-generated method stub

		int x = 0;
		int y = 0;
		int z = 0;
		
		System.out.println("Bitte geben Sie die 1. Zahl ein:");
		x = io.read_int();
		
		System.out.println("Bitte geben Sie die 2. Zahl ein:");
		y = io.read_int();
		
		System.out.println("Bitte geben Sie die 3. Zahl ein:");
		z = io.read_int();
		
		
// Fälle für z am größten
		
		if (x < y & y < z) {		// für z y x
			System.out.println(z);
			System.out.println(y);
			System.out.println(x);
		}
		if (y < x & x < z) {		// für z x y
			System.out.println(z);
			System.out.println(x);
			System.out.println(y);
		}
//Fälle für x am größten
		if (z < y & y < x) {		// für x y z
			System.out.println(x);
			System.out.println(y);
			System.out.println(z);
		}
		if (y < z & z < x) {		// für x z y
			System.out.println(x);
			System.out.println(z);
			System.out.println(y);
		}
//Fälle für y am größten
		if (z < x & x < y) {		// für y x z
			System.out.println(y);
			System.out.println(x);
			System.out.println(z);
		}
		if (x < z & z < y) {		// für y z x
			System.out.println(y);
			System.out.println(z);
			System.out.println(x);
		}
		
		
	}	
}

 

[hdi]

Schau dir mal Math.max() bzw. Math.min() an

 

[javimka]

Funktioniert mit wenigen Anpassungen auch für mehr als 3 Variabeln:

		int array[] = {x,y,z};
		Arrays.sort(array);
		System.out.println(array[0]+" "+array[1]+" "+array[2]);

 

[faetzminator]

auch das Einlesen und Ausgeben kann man in einer Schleife machen:

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.Arrays;

public class Aufgabe5 {

	public static void main(String... args) throws IOException {
		BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
		int[] ints = new int[3];
		for (int i = 0; i < ints.length; i++) {
			System.out
					.println("Bitte geben Sie die " + (i + 1) + ". Zahl ein:");
			ints[i] = Integer.parseInt(br.readLine());
		}
		Arrays.sort(ints);
		for (int i : ints) {
			System.out.println(i);
		}
	}
}

 

[eRaaaa]

er will es aber absteigend sortiert haben

List<Integer> list = Arrays.asList(x, y, z);
		Collections.sort(list, new Comparator<Integer>() {
			@Override
			public int compare(Integer o1, Integer o2) {
				return o2.compareTo(o1);
			}
		});
		System.out.println(list);

mein vorschlag

/edit: oder siehe javimka (+ andersherum ausgeben ;D)

int array[] = { x, y, z };
		Arrays.sort(array);
		for (int i = array.length-1; i >= 0; i--) {
			System.out.println(array[i]);
		}

 

[hdi]

Also ich finde ihr erschlagt ihn damit total. Er ist eindeutig ein absoluter Anfänger, und ich weiß nicht ob es so gut ist ihn mit einer Kombination aus einer anonymen Klasse, einem Comparator und Listen zuzuschütten :autsch: Der fängt ja heute abend noch an mit C# wenn ihr so weiter macht

 

[javimka]

so sieht man wenigstens, dass seine Frage ernst genommenw wird :bae:

 

[Dennis9]

hmmm also thx für die hilfe schonmal aber davon versteh ich leider noch nicht soviel :S hatte jetzt meine 3. Vorlesung auf der Uni in java und bin demnach auch auf diesem Niveau wie das Prog. geschrieben ist .. viel mehr kann ich noch nicht
sowas wie
import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.Arrays;
hab ich nicht
nur:
import java.io.IOException;

es muss auch nicht soooooooooo kurz sein wie ihr das hier hin zaubert aber wenigstens irgendwie so abkürzen dass man nicht alle 6 kombinationen einzeln da aufschreiben muss weil das kanns ja net sein sobald man das programm mit 6 zahlen oder mehr machen will wirds ja mit dieser methode etwas aufwendig

 

[Schandro]

Dann benutzt ein Array, das werdet ihr wahrscheinlich eh als nächstes drannehmen. Siehe dazu faetzminator's Beitrag. Nur mit einzelnen int's kommt man nicht weit.

 

[javimka]

Was soll das Programm denn ganz genau können? Soll es mit 3 Werten arbeiten oder mit n? Soll es die Zahlen abwärts sortiert nur ausgeben oder auch in einem Array oder wieder auf deine drei Variabeln x,y,z speichern?

An deiner Stelle würde ich den Code von Faetzminator kopieren und einfach die letzte for-Schleife umkehrten, also:

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.Arrays;
 
public class Aufgabe5 {
 
    public static void main(String... args) throws IOException {
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        int[] ints = new int[3];
        for (int i = 0; i < ints.length; i++) {
            System.out.println("Bitte geben Sie die " + (i + 1) + ". Zahl ein:");
            ints[i] = Integer.parseInt(br.readLine());
        }
        Arrays.sort(ints);
        for (int i=ints.length-1;i>=0;i--) {
            System.out.println(ints[i]);
        }
    }
}

// EDIT: Danke eRaaaa, habe es korrigiert

 

[eRaaaa]

for (int i=ints.length-1;i>=0;i--) {
System.out.println(ints);
}

 

[Meldanor]

Wenn du einfach "deinen Quellcode abkürzen" willst, mach das hier:

String ausgabe = "Zahlen sortiert: ";
if ( x > y && x > z){
	ausgabe = ausgabe.append(x+",");
	if (y > z)
		ausgabe = ausgabe.append(y+","+z);
	else
		ausgabe = ausgabe.append(z+","+y);
}
else if ( y > x && y > z){
	ausgabe = ausgabe.append(y+",");
	if (x > z)
		ausgabe = ausgabe.append(x+","+z);
	else
		ausgabe = ausgabe.append(z+","+x);
else{
	ausgabe = ausgabe.append(z+",");
	if ( x > y)
		ausgabe = ausgabe.append(x+","+y);
	else
		ausgabe = ausgabe.append(y+","+x);
	}
}
System.out.println(ausgabe);

Wenn nicht, dann schreibe alle Zahlen in ein Array und sortiere es mittels der Funktion

Arrays.sort(ARRAY_NAME);

und gib es dann aus.

 

[Landei]

Ungetestet:

import java.io.*;
public class Aufgabe_5 {
    private static void swap(int[] data, int i1, int i2) {
        int temp = data[i1];
        data[i1] = data[i2];
        data[i2] = temp; 
    }
    public static void main(String[] args) throws IOException {
        int[] nums = new int[3];
        for(int i = 0; i < 3; i++) {
          System.out.println("Bitte geben Sie die " + (i+1) +". Zahl ein:");
          nums[i] = io.read_int();
        }
        if(nums[0] < nums[1]) swap(nums,0,1);        
        if(nums[0] < nums[2]) swap(nums,0,2);        
        if(nums[1] < nums[2]) swap(nums,1,2);        
        for(int n: nums) System.out.println(n);
    }   
}

 

[Painii]

System.out.println("Bitte geben Sie die 1. Zahl ein:");
x = io.read_int();

System.out.println("Bitte geben Sie die 2. Zahl ein:");
y = io.read_int();

System.out.println("Bitte geben Sie die 3. Zahl ein:");
z = io.read_int();

Du hast 3mal dieselbe Aktion, was sich ändert ist nur ob es die 1. 2. oder 3. Zahl ist.
Daraus kannst du eine Methode bauen die deinen Text mit einer Zahl ausgibt und dann eine einliest.

if (x < y & y < z) { // für z y x
System.out.println(z);
System.out.println(y);
System.out.println(x);
}
if (y < x & x < z) { // für z x y
System.out.println(z);
System.out.println(x);
System.out.println(y);
}
//Fälle für x am größten
[/code]

Hier das gleiche, deine Ausgaben sind immer die gleichen, also fass sie auch in eine Methode zusammen.

Es ist natürlich trivial, du könntest einfach die Argumente in einen String packen und dann ausgeben.
Aber ansich wär es ne gute Übung um Passagen zu finden die mehrfach das gleiche machen, und dann dementsprechend generalisiert werden können.


Achja, für bool'sche Operationen benutz in Java lieber || und &&
| und & sind für bitweise Operationen da.