Point in ArrayList packen

[Abbadon]

Hallo habe da ein Problem mit einer aufgabe. Sie ist fürs Studium nur mein Tutor hält es iwie nicht für nötig mir auch nur im geringsten mal auf meine Fragen zu antworten.

Mein Problem ist: Ich habe ein MosaikSpiel erstellt mit einer Verschiedene Rastereinstellung und farbeinstellung. Ich soll nun ein StatistikModel erstellen das mir im CMD die Zeilen und Spalten anzahl wiedergibt welche ich angeklickt habe.
Ich bekomme es soweit hin das mir mein CMD die letzte spalte und zeile anzeigt die geklickt wurde, allerdings nicht die die ich davor angeklickt habe.
Jetzt habe ich gedacht die Zeilen und Spalten in eine ArrayList zu packen nur bekomme ich es einfach nicht hin das es läuft.#

import java.awt.Point;



public class StatistikModel implements ModelIF
{
	private int zeile, spalte;
	private Point klick;


	public StatistikModel()
	{
	}
	public void action(int zeile, int spalte)
	{
		klick = new Point(zeile, spalte);
		this.zeile=zeile;
		this.spalte=spalte;
	}

	public String getStatistikInfo()
	{

		String returnString = "Anzahl Versuche: ";
		returnString += "zeile "+zeile + " / spalte " +spalte;
		returnString += "\n";
		return returnString;
	}


	public void putObj(int zeile, int spalte, MosaikIF mObj)
	{
		// in diesem Model kein Implementierungsbedarf
	}
}

soweit habe ich den Code nur wie bekomme ich das alles ins Array gepackt?

Gruß Mike

 

[SlaterB]

was alles? einen Point oder ein StatistikModel?
als allererstes brauchst du einen Platz für die Liste oder das Array und musst dort die Variable deklarieren, ein Objekt erzeugen

irgendwelche Kenntnisse zu Array oder Listen vorhanden?

 

[Abbadon]

Also habe nur ein paar kenntnisse zu dem umgang mitz Arrays und Lists. Ich soll die Points in nem Array packen damit jeder klick registriert wird und in der cmd Protokolliert sobald ich das applet schlíesse über den appletviewer. Ich habe die ausgabe bereits in der Destroy methode geschrieben.
Nur soll das point objeckt jeden klick in einem array speichern und das soll mir alles in der CMD ausgegeben werden.

 

[SlaterB]

wie gesagt:
1. irgendwo List list = .. oder Array array = ..
2. an der richtigen Stelle, anscheinend action(), list.add(point) oder was auch immer
3. am Ende ausgeben

da ist doch an sich nix dabei

 

[Abbadon]

Lol

ok das war wirklich nicht schwer muss ich nun gestehen.

Aber eine frage habe ich da jetzt doch noch. Wie kann ich die Koordinaten x und y umschreiben das mir die CMD nicht immer java.awt.Point[x=00, y=01] ausgibt sondern nur zeile=00, spalte=01

Gruß Mike

 

[SlaterB]

String x ="zeile="+point.x
usw.

 

[Abbadon]

Ah der mist funkt immer noch nicht wie es soll.

import java.awt.Point;
import java.util.*;


public class StatistikModel implements ModelIF
{
	private int x, y;
	private Point point;


	private ArrayList list = new ArrayList();
	public StatistikModel()
	{
	}
	public void action(int zeile, int spalte)
	{

		point = new Point(x, y);
		this.x= zeile;
		this.y=spalte;
		list.add(point);
		System.out.println("Zeile: " +point.getX().list());

	}



	public String getStatistikInfo()
	{

		String returnString = "Anzahl Versuche: ";
		returnString = "zeile="+point.getX()+ " / spalte " +point.getY();
		returnString += "\n";

		return returnString;

	}


	public void putObj(int zeile, int spalte, MosaikIF mObj)
	{
		// in diesem Model kein Implementierungsbedarf
	}
}

soweit so gut. Alles läuft nur die ausgabe erfolgt immer noch [java.awt.Point[x=00, y=00]]´
ich habe schon versucht mit System.out.println("Zeile: " +list.getX().getX()); läuft nicht.
Ich weiss nicht mehr was ich noch machen solol derweil.
Kann mir da mal jemand einen konkreten hinweis zu geben wie genau ich das hinbekomme das mein CMD mir Zeile: 00 / Spalte00 Zeile:01/Spalte:04 etc.

 

[SlaterB]

worum gehts denn genau,
in deinem Programm sind zwei Stellen, die etwas damit zu tun haben könnten:
> System.out.println("Zeile: " +point.getX().list());

point.getX() ist gut, aber das ist ein int-Wert, was soll danach noch .list()?

> returnString = "zeile="+point.getX()+ " / spalte " +point.getY();

sieht sogar ganz ok aus,
der String "Anzahl Versuche:" wird natürlich sofort überschrieben wenn du returnString einen neuen String zuweist,

als dritte Variante dann aus deinem Test
> nur die ausgabe erfolgt immer noch [java.awt.Point[x=00, y=00]]´
das passt zu nichts aus dem Programm

und noch eine vierte Variante:
> System.out.println("Zeile: " +list.getX().getX());
list ist wahrscheinlich die ArrayList, da gibt es keine Methode getX(), insofern wieder seltsamer Code

-----

den Rückgabewert von getStatistikInfo() ausgeben, das wär doch eine gute Richtung

Fehlermeldungen immer komplett posten, nicht 'das geht nicht'..

 

[Abbadon]

Also das problem das ich damit habe ist das. In der ArrayList packe ich die ganzen Point kooridnaten. Also ich lasse jden Klick registrieren.
In dem String möchte ich jeden registrierten klick ausgeben lassen. Wenn ich dann aber schreiben returnString +="Zeile: " +list); bekomme ich alle point koordniaten ausgegeben und das führt dann zu einer endlos langen liste von [java.awt.Point[x=01, y=01]] [java.awt.Point[x=02, y=04]] und so weiter.
was ich möchte ist das der mir die Liste ausgibt und zwar untereinander aber eben in diese form:
zeile 01/ spalte01
zeile 04/splate 02

und da fängt das problem bei mir an. Ich weiss nicht wie ich aus der ArrayList die punkte so ausgegeben bekomme.

Also fehlermeldungen habe ich immer beseitigt.
> System.out.println("Zeile: " +list.getX().getX());
weglässt klappt der code auch also die zeiel sollte nimmer dabei sein. Soweit Läuft der code nur die ArrayList ausgabe läuft nicht wie sie soll.

Gruß Mike

 

[SlaterB]

übe erstmal die Ausgabe eines Punktes, und danach ist es das immergleiche Konzept für viele Elemente:
eine Schleife

for ... {
string += beschreibung für einen Punkt
}

 

[faetzminator]

Du kannst [c]public String toString()[/c] überschreiben, dann wird dies mit [c]obj.toString()[/c] in [c]System.out.println(Object obj)[/c] ausgegeben.

 

[Abbadon]

So havbe es nun endlich hinbekommen mit zwei Listen.
Habe aber noch ne frage ob ihr es so lassen würdet oder ob noch iwo verbesserungs bedarf eurer Seite besteht.
Die ausgabe schaut nun so aus
[zeile = 01 / spalte =03
, zeile = 04 / spalte=01
etc]
meint ihr man kann es so lassen oder gibt es noch ein trick die eckigen klammern weg zu bekommen.

import java.awt.Point;
import java.util.*;

public class StatistikModel implements ModelIF
{
	private Point point;

	private String returnString;
	private List list = new Vector();
	private List list1 = new Vector();
	public StatistikModel()
	{

	}
	public void action(int zeile, int spalte)
	{
		list.add(new Point(zeile, spalte));
		returnString ="Zeile: " +zeile+ "/ Spalte: " +spalte+ "\n";
		list1.add(returnString);
	}

	public String getStatistikInfo()
	{
			String versuchString;
			versuchString = "Anzahl Versuche: "+ "\n" +list1;

			return versuchString;
	}

	public void putObj(int zeile, int spalte, MosaikIF mObj)
	{
		// in diesem Model kein Implementierungsbedarf
	}

 

[SlaterB]

> meint ihr man kann es so lassen

keine Meinung zum Sylistischen,
wenn es eine Aufgabe ist, dann könnte aber die Verwendung von List.toString() als besonders schlau oder zu einfach angesehen werden, eine Gratwanderung

> oder gibt es noch ein trick die eckigen klammern weg zu bekommen.
..

na die nun ewig besprochene normale Ausgabe,

übe erstmal die Ausgabe eines Punktes, und danach ist es das immergleiche Konzept für viele Elemente:
eine Schleife

for ... {
string += beschreibung für einen Punkt
}

denkbar ist auch, den String mit der List-Beschreibung nachzubearteiten,
string.replace(),
aber das ist dann bald kompizierter als eine simple Schleife

 

[ARadauer]

meint ihr man kann es so lassen oder gibt es noch ein trick die eckigen klammern weg zu bekommen.

nein so kann man es nicht lassen! Was willst du konkret machen? Klicks sammeln?
list1? list? wieso? Du hast doch die Info bereits in der Liste mit den Punkten, warum brauchst du dann noch eine Liste mit Strings? Und warum nennst du das Liste? Bitte sprechende Variablen verwenden...

versuchString = "Anzahl Versuche: "+ "\n" +list1;

ist list1 die Anzahl? nein! Warum gibst du das dann aus?

wozu brauchst du die Member returnString und Point, die werden eh nur lokal verwendet....

was hälst du von dem?

import java.awt.Point;
import java.util.ArrayList;


public class StatistikModel {


    private ArrayList<Point> pointList = new ArrayList<Point>();

    public StatistikModel()
    {
 
    }
    public void action(int zeile, int spalte)
    {
    	pointList.add(new Point(zeile, spalte));       
    }
 
    public String getStatistikInfo()
    {
            StringBuffer versuchString = new StringBuffer();
            versuchString.append("Anzahl Versuche: "+ pointList.size()+"\n");
            for(Point p: pointList)
            	versuchString.append("Zeile: " +p.x+ "/ Spalte: " +p.y+ "\n"); 
            return versuchString.toString();
    }
    
    public static void main(String[] args) {
		//zum Testen
    	
    	StatistikModel model = new StatistikModel();
    	model.action(2, 3);
    	model.action(13, 12);
    	model.action(5, 1);
    	
    	System.out.println(model.getStatistikInfo());
    	
	}
 
}

 

[Abbadon]

Also bei deinem Code bekomme ich nur fehler meldungen. Und die zeilen und spalten sind veränderbar deswegen kann ich keine exakten zeilen und spalten mitgeben.

Ich speicher in der Liste die klicks von einem Applet. Bei der Destroy methode sollen die klicks aufgelistet werden.
Gebe ich dem String versuchString die list gibt der mir zwar jeden klick aus aber in dieser form [java.awt.Point[x=01, y=01]] etc
Das aber will ich nicht haben.

Also gebe ich der ersten Liste die Points zeile spalte und lasse mir diese in dem ersten String returnString ausgeben. Dann bekomme ich diese Ausgabe Zeile: 01 / Spalte: 02 etc

	list.add(new Point(zeile, spalte));
		returnString ="Zeile: " +zeile+ "/ Spalte: " +spalte+ "\n";

Gebe ich den String jetzt dierekt an den versuchString weiter also so:

 versuchString = "" + returnString;

Bekomme ich nur den Letzten klick ausgegeben.
Also habe ich den String returnString in einen zweite liste gepack

list1.add(returnString);

mit dieser liste klappt dann auch die ausgabe das jeder Klick in getStatistikInfo ausgegeben wird.
In der getStatistikInfo gebe ich diese Liste an den versuchString aus.

			String versuchString;
			versuchString = "Anzahl Versuche: "+ "\n" +returnString;

So funktioniert alles wie es soll. Nur die Ausgabe mit den eckicken klammern stört mich daran. Deshalb die frage ob ich die klammern weg bekomme also die [] diese hier?

 

[ARadauer]

Also bei deinem Code bekomme ich nur fehler meldungen.

welche? kennt er keine generics? neuere java version verwenden...

Und die zeilen und spalten sind veränderbar deswegen kann ich keine exakten zeilen und spalten mitgeben.

wo mitgeben?

Gebe ich den String jetzt dierekt an den versuchString weiter also so:

du gibst da nix weiter... du weißt zu... was soll den drin in der variable drin stehen, wenn du a, b, c und d zuweist? d natürlich, du überschreibst ja die werte...

Deshalb die frage ob ich die klammern weg bekomme also die [] diese hier?

nicht irgendwelche dinge zusammen hängen (arrays, listen, punkte) sondern strings und zahlen...