Wo liegt der Fehler? *verzweifel*

[Pluta]

Hallo Leute,

bin erst seit einigen Wochen dabei Java zu lernen. Bin dabei meine zweites Miniprojekt für die Uni zu bearbeiten. Dabie gibt es fünf Fussballmannschaften, welche jeweils einmal gegeneinander spielen. Die Ergebnisse stehen schon fest und werden in folgender Form gespeichert : Im Array tore [hier Mannschaft 0-4] [hier Gegner 0-4] = Wieviele Tore der erstgenannte Verein gegen den Anderen geschossen hat. Der Sieger wird zuallererst über die meisten Siege errechnet. Falls dort einen Unentschieden auftrifft zählt die Tordifferenz. Danach die Anzahl der Tore. Anonsten gibt es keinen Sieger.

Bin meiner Meinung eigentlich auch schon fertig. Aber um unser Ergebnis zu überprüfen, gibt es einen Jack Server. Dieser findet im dynamischen Test jedoch noch einen Fehler. Und zwar wenn angeblich Mannschaft2 aufgrund der Tordifferenz Sieger wird; dann würde jedoch trotzdem einen 0 zurückgegeben werden. Richtig wäre hier dann aber die 2 (1+1, da ja die Arrays mit 0 anfangen).. Habe das mit mehreren Beispielen durchgespielt, aber ich kann dort keinen Fehler entdecken. Vielleicht kann mir ja jemand von euch helfen. Hier mein Code :




public class Miniprojekt2 {
public int findeSieger (int[] [] ergebnisse) {

int Z1, Z2 = 0, Z;
int U1= 0, U2 = 0, U3 = 0;
int TS1 = 0, TS2 = 0, TS3 = 0;
int GT = 0, sieger = 0;
int Daten [] [] = new int [5][5];

for (Z1 = 0; Z1 < ergebnisse.length; Z1++) // Array Daten füllen mit Anzahl Siege / Tore / Tordiffererenz
{GT = 0;
for (Z2 = 0 ; Z2 < ergebnisse[0].length; Z2++)
{if (ergebnisse [Z1] [Z2] > ergebnisse [Z2] [Z1]) Daten [Z1] [0] += 1;
Daten [Z1] [1] += ergebnisse [Z1] [Z2];
GT += ergebnisse [Z2] [Z1];
Daten [Z1] [2] = Daten [Z1] [1] - GT;
;}}

for (Z = 0; Z < Daten.length; Z++) // Überprüfung auf Sieger durch Anzahl Siege
{if (Daten [Z] [0] == TS1) U1 = 1;
if (Daten [Z] [0] > TS1 ) TS1 = Daten [Z] [0];}
if (U1 == 0)
{for (Z = 0; Z <Daten[0].length; Z++)
{if (Daten [Z] [0] == TS1) return sieger = Z+1 ;}}

for (Z = 0; Z < Daten.length; Z++) // Überprüfung Sieger aufgrund von Tordifferenz
{if (Daten [Z] [2] == TS2) U2 = 1;
if (Daten [Z] [2] > TS2 && Daten [Z] [0] == TS1 ) TS2 = Daten [Z] [2];}
if (U2 == 0)
{for (Z = 0; Z <Daten[0].length; Z++)
{if (Daten [Z] [2] == TS2) sieger = Z+1 ;}
return sieger;}

for (Z = 0; Z < Daten.length; Z++) // Überprüfung auf Sieger durch Anzahl Tore
{if (Daten [Z] [1] == TS3) U3 = 1;
if (Daten [Z] [1] > TS3 && Daten [Z] [0] == TS1 && Daten [Z] [2] == TS2 ) TS3 = Daten [Z] [1];}
if (U3 == 0)
{for (Z = 0; Z <Daten[0].length; Z++)
{if (Daten [Z] [1] == TS3) sieger = Z+1 ;}}

return sieger;} // Falls kein Ergebnis müßte hier 0 zurückgeben werden


public static void main (String[] args) {
int tore [] [] = new int [5][5];


tore [0][0] = 0;
tore [0][1] = 1;
tore [0][2] = 1;
tore [0][3] = 9;
tore [0][4] = 3;

tore [1][0] = 3;
tore [1][1] = 0;
tore [1][2] = 1;
tore [1][3] = 6;
tore [1][4] = 0;

tore [2][0] = 3;
tore [2][1] = 3;
tore [2][2] = 0;
tore [2][3] = 0;
tore [2][4] = 0;

tore [3][0] = 1;
tore [3][1] = 1;
tore [3][2] = 0;
tore [3][3] = 0;
tore [3][4] = 2;

tore [4][0] = 1;
tore [4][1] = 0;
tore [4][2] = 3;
tore [4][3] = 0;
tore [4][4] = 0;



Miniprojekt2 projekt = new Miniprojekt2 ();
int sieger=projekt.findeSieger(tore);
if (sieger > 0)
System.out.println("Der Sieger ist Mannschaft " + sieger);
else System.out.println("Es gibt keinen Sieger");
}}

 

[EOB]

nimm ma code tags, sonst liest sich das ..... :roll:

grüße
eob

 

[SlaterB]

sehe ich das richtig, dass dein Programm korrekt arbeitet aber andere Ergebnisse liefert als ein anderes korrektes Programm?
was soll man dann bitte dazu sagen?

und formuliere deine Beispiel-Beschreibung genauer:

> Und zwar wenn angeblich Mannschaft2 aufgrund der Tordifferenz Sieger wird; dann würde jedoch trotzdem einen 0 zurückgegeben werden. Richtig wäre hier dann aber die 2 (1+1, da ja die Arrays mit 0 anfangen)..

häh? wenn Mannschaft 2 der Gesamtsieger ist, dann ist die Ausgabe 2 oder meinetwegen ArrayIndex 1
(da bei 0 angefangen wird)

wer gibt 0 zurück, dein Programm oder das andere und wieso?
und was hat das ganze mit 1+1 zu tun?

??

 

[Joker]

ohne code Tag ist das nur sehr schwer zu lesen (+ Codeconventions )

Hab so was damals im erstem Semester auch mal programmieren müssen, war aber wohl etwas komplexer, da die Spiele automatisiert per Zufallsgenerator abliefen. Dafür ist die Anzahl der Mannschaften beliebig, einfach in der main Methode weitere hinzufügen. Etwas komplizierter ist nur der SpielplanGenerator. Vielleicht kannst du es ja gebrauchen.

import java.util.*;


class LigaSimulator {
	private static ArrayList<Spiel> spiele;
	private static ArrayList<Mannschaft> teams;

	private static void buildTable() {
		Collections.sort(teams, new Comparer()); //Nach Punkten sortieren
		
		Mannschaft t;
		System.out.println("Team       Spiele   Punkte  Tore");
		for(Iterator<Mannschaft> i = teams.iterator();i.hasNext();) {
			t = i.next();
			System.out.println(t.getName()+"   "+t.getAnzSpiele()+"   "+t.getPunkte()+"  "+t.getGeschosseneTore()+":" +t.getKassierteTore());
		}
	}
	
	private static void playSeason(SpielplanGenerator spielplan) {
		ArrayList<ArrayList<String>> plan = spielplan.getPlan();
		String[] partie;
		int spieltagCounter=1;;
		
		for (Iterator<ArrayList<String>> i = plan.iterator(); i.hasNext();) {
			ArrayList<String> spieltag =  i.next();
			for (Iterator<String> k = spieltag.iterator(); k.hasNext();) {
				partie = k.next().split("-");
				spiele.add(new Spiel(teams.get(Integer.parseInt(partie[0])-1),teams.get(Integer.parseInt(partie[1])-1),spieltagCounter));			
			}
			spieltagCounter++;
		}
		buildTable();
	}
	
	
	public static void main(String[] args) {
		spiele= new ArrayList<Spiel>();
		teams = new ArrayList<Mannschaft>(5);
		
		teams.add(new Mannschaft("Mannschaft1"));
		teams.add(new Mannschaft("Mannschaft2"));
		teams.add(new Mannschaft("Mannschaft3"));
		//teams.add(new Mannschaft("Mannschaft4"));

		playSeason(new SpielplanGenerator(teams.size(),false)); //zweites Argument für hinzuschalten einer Rückrunde.
	}
}

class Mannschaft {
	private String name;
	private int geschosseneTore;
	private int kassierteTore;
	private int punkte;
	private int anzSpiele;
	public Mannschaft(String name) {
		this.name=name;
		geschosseneTore=0;
		kassierteTore=0;
		punkte=0;
		anzSpiele=0;
	}

	public int getGeschosseneTore() {
		return geschosseneTore;
	}

	public void addGeschosseneTore(int geschosseneTore) {
		this.geschosseneTore += geschosseneTore;
	}

	public int getKassierteTore() {
		return kassierteTore;
	}

	public void addKassierteTore(int kassierteTore) {
		this.kassierteTore += kassierteTore;
	}

	public int getPunkte() {
		return punkte;
	}

	public void addPunkte(int p) {
		this.punkte += p;
	}
	
	public String getName() {
		return name;
	}

	public int getAnzSpiele() {
		return anzSpiele;
	}

	public void addAnzSpiele() {
		this.anzSpiele++;
	}
	
}


class Spiel {
	private int spieltag;
	private Mannschaft heimMannschaft;
	private Mannschaft gaesteMannschaft;
	private int heimTore;
	private int gastTore;
	private static Random random=new Random();
	
	public Spiel(Mannschaft heimMannschaft, Mannschaft gaesteMannschaft, int spieltag) {
		this.heimMannschaft=heimMannschaft;
		this.gaesteMannschaft=gaesteMannschaft;
		this.spieltag=spieltag;
		this.heimTore=random.nextInt(10);
		this.gastTore=random.nextInt(10);
		
		//System.out.println(heimMannschaft + " - "+ gaesteMannschaft + "   "+heimTore+":"+gastTore);
		saveResult();
	}
	
	private void saveResult() {
		heimMannschaft.addPunkte(getHomePoints());
		heimMannschaft.addGeschosseneTore(heimTore);
		heimMannschaft.addKassierteTore(gastTore);
		heimMannschaft.addAnzSpiele();
		
		gaesteMannschaft.addPunkte(getGuestPoints());
		gaesteMannschaft.addGeschosseneTore(gastTore);
		gaesteMannschaft.addKassierteTore(heimTore);
		gaesteMannschaft.addAnzSpiele();
		//System.out.println(getOutput());
	}
	
	private Mannschaft getWinnerTeam() {
		if(heimTore > gastTore) {
			return heimMannschaft;
		} else if (heimTore < gastTore) {
			return gaesteMannschaft;
		} else {
			return null;
		}
	}
	
	private int getHomePoints(){
		if(getWinnerTeam() == null) {
			return 0;
		}
		
		if(getWinnerTeam().equals(heimMannschaft)) {
			return 3;
		} else if (getWinnerTeam().equals("Unendschieden")){
			return 1;
		}
		
		return 0;
	}
	
	private int getGuestPoints(){	
		if(getWinnerTeam() == null) {
			return 0;
		}
		
		if(getWinnerTeam().equals(gaesteMannschaft)) {
			return 3;
		} else if (getWinnerTeam().equals("Unendschieden")){
			return 1;
		}
		return 0;			
	}
	
	public String getOutput() {
		return heimMannschaft.getName()+"  -  "+ gaesteMannschaft.getName()+"   "+heimTore+":"+gastTore;
	}
}


/**
 * Generiert einen Spielplan für die Übergebene Anzahl an Teams.
 * @param anzTeams , die Anzahl der Mannschaften der Liga.
 * @param rueckRunde , true wenn die Rückrunde gespielt werden soll, false sonst.
 */
class SpielplanGenerator {
	private int anzTeams;
	private boolean rueckRunde;
	
	private boolean dummyTeam;
	private int anzSpieltage;
	private Random random;
	private ArrayList<ArrayList<String>> plan;
	
	public SpielplanGenerator (int anzTeams, boolean rueckRunde) {
		this.anzTeams = anzTeams;
		this.rueckRunde = rueckRunde;
		random=new Random();
		
		if(anzTeams % 2 == 1) {
			dummyTeam = true;
			anzSpieltage = anzTeams;
		} else {
			dummyTeam = false;
			anzSpieltage = (anzTeams - 1);
		}
		
		if(rueckRunde) {
			anzSpieltage*=2;
		}
		
		plan= new ArrayList<ArrayList<String>>(anzSpieltage);
		
		buildPlan();
	}
	
	public ArrayList<ArrayList<String>> getPlan() {
		return plan;
	}
	
	private void buildPlan() {
		LinkedList<Integer> list = new LinkedList<Integer>();
		int tmp;
		if(!dummyTeam) {
			for(int i=1;i<=anzTeams;i++) {
				list.add(i);
			}
		} else {
			for(int i=1;i<=anzTeams+1;i++) {
				list.add(i);
			}
		}

		ArrayList<String> games = new ArrayList<String>();
		for(int i=0;i<anzSpieltage;i++) {
			
			for(int k=0;k<list.size();k+=2) {
				if(rueckRunde) { //sorge für Auswärtsspiele von Team 1 ;)
					if(random.nextBoolean()) {
						games.add(list.get(k) +"-"+ list.get(k+1));
					} else {
						games.add(list.get(k+1) +"-"+ list.get(k));
					}
				} else {
					games.add(list.get(k) +"-"+ list.get(k+1));
				}
			}

			//Shift list
			tmp=list.get(1);
			list.remove(1);
			list.addLast(tmp);
			
			if(dummyTeam) {
				String[] params;
				for(int d=0;d<games.size();d++) {
					params = games.get(d).split("-");
					if((Integer.parseInt(params[0])==(anzTeams+1)) || (Integer.parseInt(params[1])==(anzTeams+1))) {
						games.remove(d);
					}
				}
			}
			//Save
			plan.add(games);
			games = new ArrayList<String>();
		}	
	}
	
	private void printPlan() {
		
		for(int i=0;i<plan.size();i++) {
			for(int k=0;k<plan.get(i).size();k++) {
				
				System.out.println(plan.get(i).get(k));
			}
		}
	}
	
	
}


class Comparer implements Comparator<Mannschaft> {
    public int compare(Mannschaft team1, Mannschaft team2)
    {
    	int tmp = team2.getPunkte() - team1.getPunkte();
    	if(tmp==0) {
    		return (team2.getGeschosseneTore()-team2.getKassierteTore()) - (team1.getGeschosseneTore() - team2.getKassierteTore());
    	} else {
    		return tmp;
    	}
    	    
    }
}

 

[DP]

oder

public class Miniprojekt2 { public int findeSieger (int[] [] ergebnisse) { int Z1, Z2 = 0, Z; int U1= 0, U2 = 0, U3 = 0; int TS1 = 0, TS2 = 0, TS3 = 0; int GT = 0, sieger = 0; int Daten [] [] = new int [5][5]; for (Z1 = 0; Z1 < ergebnisse.length; Z1++) // Array Daten füllen mit Anzahl Siege / Tore / Tordiffererenz {GT = 0; for (Z2 = 0 ; Z2 < ergebnisse[0].length; Z2++) {if (ergebnisse [Z1] [Z2] > ergebnisse [Z2] [Z1]) Daten [Z1] [0] += 1; Daten [Z1] [1] += ergebnisse [Z1] [Z2]; GT += ergebnisse [Z2] [Z1]; Daten [Z1] [2] = Daten [Z1] [1] - GT; ;}} for (Z = 0; Z < Daten.length; Z++) {if (Daten [Z] [0] == TS1) U1 = 1; if (Daten [Z] [0] > TS1 ) TS1 = Daten [Z] [0];} if (U1 == 0) {for (Z = 0; Z <Daten[0].length; Z++) {if (Daten [Z] [0] == TS1) return sieger = Z+1 ;}} for (Z = 0; Z < Daten.length; Z++) {if (Daten [Z] [2] == TS2) U2 = 1; if (Daten [Z] [2] > TS2 && Daten [Z] [0] == TS1 ) TS2 = Daten [Z] [2];} if (U2 == 0) {for (Z = 0; Z <Daten[0].length; Z++) {if (Daten [Z] [2] == TS2) sieger = Z+1 ;} return sieger;} for (Z = 0; Z < Daten.length; Z++) {if (Daten [Z] [1] == TS3) U3 = 1; if (Daten [Z] [1] > TS3 && Daten [Z] [0] == TS1 && Daten [Z] [2] == TS2 ) TS3 = Daten [Z] [1];} if (U3 == 0) {for (Z = 0; Z <Daten[0].length; Z++) {if (Daten [Z] [1] == TS3) sieger = Z+1 ;}} return sieger;} public static void main (String[] args) { int tore [] [] = new int [5][5]; tore [0][0] = 0; tore [0][1] = 1; tore [0][2] = 1; tore [0][3] = 9; tore [0][4] = 3; tore [1][0] = 3; tore [1][1] = 0; tore [1][2] = 1; tore [1][3] = 6; tore [1][4] = 0; tore [2][0] = 3; tore [2][1] = 3; tore [2][2] = 0; tore [2][3] = 0; tore [2][4] = 0; tore [3][0] = 1; tore [3][1] = 1; tore [3][2] = 0; tore [3][3] = 0; tore [3][4] = 2; tore [4][0] = 1; tore [4][1] = 0; tore [4][2] = 3; tore [4][3] = 0; tore [4][4] = 0; Miniprojekt2 projekt = new Miniprojekt2 (); int sieger=projekt.findeSieger(tore); if (sieger > 0) System.out.println("Der Sieger ist Mannschaft " + sieger); else System.out.println("Es gibt keinen Sieger"); }}

 

[sliwalker]

obfuscated contest?

 

[DP]

nein, sondern

System.out.println("Es gibt keinen Sieger");

 

[Pluta]

Habs selber nochmal ganz neu geschrieben. Danke für die Hilfe, falls jemand interessiert, so funktioniert es public



class Miniprojekt2 {
public int findeSieger (int[] [] ergebnisse) {

int Z1, Z2 = 0, Z, Test1 = 0, Test2 = -20, Test3 = 0;
int sieger = 0, GT = 0, Abbruch1 = 0, Abbruch2 = 0, Abbruch3 = 0;
int Siege [] = new int [5];
int Tordifferenz [] = new int [5];
int AnzahlTore [] = new int [5];

for (Z1 = 0; Z1 < ergebnisse.length; Z1++)
{GT = 0;
for (Z2 = 0 ; Z2 < ergebnisse[0].length; Z2++)
{if (ergebnisse [Z1] [Z2] > ergebnisse [Z2] [Z1]) Siege [Z1] += 1;
AnzahlTore [Z1] += ergebnisse [Z1] [Z2];
GT += ergebnisse [Z2] [Z1];
Tordifferenz [Z1] = AnzahlTore [Z1] - GT;
;}}

for (Z = 0; Z < Siege.length; Z++)
{if (Siege [Z] > 0) Test1 = Siege [Z];}
if (Test1 == 0) Abbruch1 = 0;
for (Z = 0; Z < Siege.length; Z++)
{if (Siege [Z] == Test1) Abbruch1 +=1;}
if (Abbruch1 == 1)
{for (Z = 0; Z < Siege.length; Z++)
{if (Siege [Z] == Test1) return sieger = Z +1;}}



for (Z = 0; Z < Tordifferenz.length; Z++)
{if (Siege [Z] == Test1 && Tordifferenz [Z] > Test2) Test2 = Tordifferenz [Z];}
for (Z = 0; Z < Tordifferenz.length; Z++)
{if (Siege [Z] == Test1 && Tordifferenz [Z] == Test2) Abbruch2 +=1;}
if (Abbruch2 == 1)
{for (Z = 0; Z < Tordifferenz.length; Z++)
{if (Siege [Z] == Test1 && Tordifferenz [Z] == Test2) return sieger = Z +1;}}

for (Z = 0; Z < AnzahlTore.length; Z++)
{if (Siege [Z] == Test1 && Tordifferenz [Z] == Test2 && AnzahlTore [Z] > Test3) Test3 = AnzahlTore [Z];}
for (Z = 0; Z < AnzahlTore.length; Z++)
{if (Siege [Z] == Test1 && Tordifferenz [Z] == Test2 && AnzahlTore [Z] == Test3) Abbruch3 +=1;}
if (Abbruch3 == 1)
{for (Z = 0; Z < AnzahlTore.length; Z++)
{if (Siege [Z] == Test1 && Tordifferenz [Z] == Test2 && AnzahlTore [Z] == Test3) return sieger = Z +1;}}

return sieger;}

public static void main (String[] args) {
int tore [] [] = new int [5][5];

tore [0][0] = 0;
tore [0][1] = 2;
tore [0][2] = 0;
tore [0][3] = 3;
tore [0][4] = 1;

tore [1][0] = 0;
tore [1][1] = 0;
tore [1][2] = 1;
tore [1][3] = 0;
tore [1][4] = 4;

tore [2][0] = 5;
tore [2][1] = 0;
tore [2][2] = 0;
tore [2][3] = 0;
tore [2][4] = 1;

tore [3][0] = 0;
tore [3][1] = 3;
tore [3][2] = 2;
tore [3][3] = 0;
tore [3][4] = 0;

tore [4][0] = 1;
tore [4][1] = 0;
tore [4][2] = 3;
tore [4][3] = 2;
tore [4][4] = 0;

Miniprojekt2 projekt = new Miniprojekt2 ();
int sieger=projekt.findeSieger(tore);
if (sieger > 0)
System.out.println("Der Sieger ist Mannschaft " + sieger );
else System.out.println("Es gibt keinen Sieger");
}}

 

[DP]

wieso benutzt du keine code-tags um code zu posten?

 

[Saxony]

Irgendwie hat es den ganzen Thread formatierungstechnisch zerschossen.

 

[SlaterB]

was heißt hier 'irgendwie'?
DP hat es mir unverständlicherweise in einem Post mit Absicht gemacht,

oder aus Versehen, was von Unfähigkeit zeugt