Wertetabelle, Werte nachschlagen

[babuschka]

Hallo,

ich bin neu hier im Forum, daher ganz kurz zu mir. Ich habe vor etwa 5 Jahren angefangen mich überhaupt mit Java zu beschäftigen. Ich hatte für ein Jahr einen Nebenjob an der Uni und habe etwas Java programmiert (auf niedrigem Niveau). Da ich die Sprache recht intuitiv finde, habe ich vor einigen Monaten wieder angefangen mich mit der Java Programmierung zu beschäftigen, mit dem Ziel eigene Tools in Richtung Verfahrenstechnik/Maschinenbau zu entwickeln. Dazu habe ich das Buch "Java ist auch eine Insel" erstmal quergelesen und dann losgelegt! Also habe ich etwas programmier Erfahrung, finde aber oft die richtigen Wege nicht.

Google und auch eure Suche brachte leider keinen Erfolg. Vielleicht habt ihr auch einfach einen guten Link zu dem Thema.

Ich möchte mittels zwei vorgegebenen Werten einen dritten Wert aus einer Tabelle ablesen. In Excel habe ich das leicht hinbekommen (mit "lookup")! :lol: Die Tabelle hat folgendes Schema (ich zeige es mal mit Trennzeichen Semikolon):

Diameter [Inch]; Schedule; inner Diameter [mm]
1/2“; 10; 17,12
1/2“; 40; 15,8
1/2“; 80; 13,87
1“; 10; 27,86
1“; 40; 26,64
1“; 80; 24,31
2“; 10; 54,79
2“; 40; 52,5
2“; 80; 49,25
20“; 5; 498
20“; 10; 497

Problem Teil 1: Also Diameter und Schedule gibt der Benutzer (per Combobox) vor und ich möchte den inner Diamter als Rückgabewert. Welchen Datentyp nehme ich am Besten? Da in der Tabelle Strings (Diameter), Intergers (Schedule) und Double (inner Diameter) Typen stehen, scheiden Arrays nach meinem Wissen aus. Ich hätte dort die Suche relativ einfach mit Schleifen lösen können (suche erst nach vorgegebenem Diameter in Spalte 1 und suche dann nach vorgegebenem Schedule in Spalte 2, gebe dann Wert aus Spalte 3 aus). Mit "Maps" geht es auch nicht, da es immer nur ein Schlüssel-Wert-Paar gibt. Ich denke eher, dass ich einen eigenen Typ brauche, der in eine Collection passt. Wenn ich für diesen Typ noch die Sortierbarkeit implementiere, dann könnte man leicht den gesuchten Wert bestimmen. (Typ Pipe oder so) Was ist eure Meinung oder habt ihr einen Hinweise? Bestimmt ist es ganz einfach, da eine solche Problemstellung häufiger Vorkommen muss (Wertetabellen).

Problem Teil 2: Wie bekomme ich die Tabelle einfach in mein Programm? Jedes mal aus einer Datei lesen oder "hart" im Programm einkodieren? Gegen die zweite Methode spricht eigentlich nichts, da sich die Tabelle nicht ändert.Dann stellt sich aber die Frage wie ich eine übersichtliche Zuweisung im Code mache ohne jede Zahl einzelnen meinem Objekt zuzuweisen.

Problem Teil 3: Ich möchte schon wärend der Eingabe der Werte dem Benutzer unter die Arme greifen. Wenn also der Diameter schon ausgewählt ist, dann sind nicht mehr alle Schedules erlaubt. Z.B. gibt es bei 20" kein Schedule 20! Um diese "intelligente" Eingabe umzusetzen sehe ich nur die Möglichkeit "intelligente" ComboBoxen zu machen. Daran finde ich allerdings unschön, dass diese Einschränkung ja schon eine Tabelleneigenschaft ist und ich diese eigentlich nicht mit der grafischen Oberfläche vermischen möchte. Wie seht ihr das?

Ich hoffe ihr habt ein paar Hinweise!
Viele Grüße
Sven

 

[Andreas29]

Hi,

also ich würde hier mit mehreren Maps arbeiten.

Jetzt eins nach dem anderen:
1. Für das erste Problem, den richtigen dritten Wert anhand der zwei eingegebenen Werte zu bestimmen, würde ich mir erstmal eine Key-Klasseanlegen, die die zwei Werte Diameter und Schedule speichert. Diese Klasse müsste dann die Methoden equals() und hashCode() überschreiben, um in der Map richtig zu funktionieren. Anschließend kannst du diese Klasse als Key und den long-Wert als Value der Map nehmen (okay, genau geschrieben muss ich Long groß schreiben, damit klar wird, das die Wrapper-Klasse gemeint ist, das Java nur Objekte und keine primitiven Datentypen in Maps und Collections verwalten kann).

2. Als für dieses Problem gibt es mehrere Möglichkeiten:
Man könnte zum Beispiel (so wie du in deinem Posting oben) die Tabelle / Map in eine platte Text-Datei serialisieren. Oder eine XML-Datei (vielleicht ein kleiner overhead,ja,...) oder einfach dieDaten mittels Java Serialisierung wegschreiben und lesen. Ich persönlich würde, denke ich, entweder eine Text- oder XML-Datei nehmen. Dann kann man im Notfall die Daten auch mal mit einem simplen Texteditor kontrollieren.

3. Für die Intelligenten Comboboxen würde ich wieder eine Map nehmen. Diesmal eine von Diameter auf Collection<Schedule>. Dann kannst du jedes mal, wenn der Benutzer die erste Combobox ändert einfach die Map nach den Werten für die zweite Box fragen. Eventuell würde ich mir auch noch eine zweite Map von Schedule auf Collection<Diameter> halten. Je nachdem, ob beide Comboboxen gleichzeitig editierbar sein sollen oder nicht.

Hoffe dir ein wenig geholfen zu haben.

Grüße,
Andreas

 

[till123]

Hallo,

ich würde jeden Diameter als eine eigene Tabelle ablegen.
In den jeweiligen Tabellen stehen dann die Werte von Schedule und inner Diameter.
Zum Schluss dann noch eine MetaTabelle, in denen alle Tabelle aufgelistet werden.

Zu 1:
Ich würde alles in Strings speichern/ablegen - als im StringArray. Die Umwandlung in den entsprechenden Datentyp ist dann immer noch möglich.

Zu 2:
Ich würde die Daten in einer separaten Datei ablegen - schön im xml-Format
(Dann könntest diese Tabellen auch eventuell anderweitig benutzen...)

Zu 3:
Für die erste Anzeige aller Tabellen, list du einfach die MetaTabelle aus. So bald du einen Diameter ausgewählt hast, dann lädst du die entsprechende Tabelle und liest dann die weiteren Daten aus.

 

[Andreas29]

Hi remmar,

also, ich war jetzt mal fleißig und habe Dir ein kleines DataModel geschrieben, das deine Bedürfnisse zufrieden stellen müsste. Schau einfach mal rein, was fehlt, ist eine Klasse, die das ganze mit Inhalt füllt. Dazu würde ich dir raten, dir mal ein zwei Tutorials zum Thema XML & Java durchzulesen (sprich, wie man XML liest und schreibt). Danach sollte dies nicht mehr allzuschwer sein...

Hier der Code:
Klasse DataModel:

package com.javaforum.datamodel;

import java.util.Collection;
import java.util.HashMap;
import java.util.LinkedList;
import java.util.Map;

public class DataModel {

	private final Map<DiameterSchedulerKey, Long> diameterSchedulerInnerDiameter = new HashMap<DiameterSchedulerKey, Long>();

	private final Map<Integer, Collection<Integer>> diameterToSchedule = new HashMap<Integer, Collection<Integer>>();

	private final Map<Integer, Collection<Integer>> scheduleToDiameter = new HashMap<Integer, Collection<Integer>>();

	public DataModel() {
		// do nothing
	}

	/**
	 * This method returns the innerDiameter value for the given diameter and
	 * schedule combination.
	 * 
	 * @param diameter
	 *            the diameter value
	 * @param schedule
	 *            the schedule value
	 * @return the innerDiameter value for the given diameter and schedule
	 *         combination
	 */
	public long getInnerDiameter(int diameter, int schedule) {
		return diameterSchedulerInnerDiameter.get(
				new DiameterSchedulerKey(diameter, schedule)).longValue();
	}

	/**
	 * This method returns the allowed schedule values for the given diameter
	 * value
	 * 
	 * @param diameter
	 *            the diameter value, which allowed schedule values are searched
	 * @return the allowed schedule values for the given diameter
	 */
	public Collection<Integer> getSchedule(int diameter) {
		return diameterToSchedule.get(Integer.valueOf(diameter));
	}

	/**
	 * This method returns the allowed diameter values for the given schedule
	 * 
	 * @param schedule
	 *            the schedule, which allowed diameter values are returned
	 * @return the allowed diameter values for the given schedule
	 */
	public Collection<Integer> getDiameter(int schedule) {
		return scheduleToDiameter.get(Integer.valueOf(schedule));
	}

	public void putDiameterScheduleInnerDiameter(int diameter, int scheduler,
			long innerDiameter) {
		diameterSchedulerInnerDiameter.put(new DiameterSchedulerKey(diameter,
				scheduler), Long.valueOf(innerDiameter));
	}

	public void putDiameterSchedule(int diameter, int schedule) {
		putValuesToMap(diameter, schedule, diameterToSchedule);
		putValuesToMap(schedule, diameter, scheduleToDiameter);
	}

	private void putValuesToMap(int key, int value,
			Map<Integer, Collection<Integer>> map) {
		// get the value for the given key out of the given map
		Integer intKey = Integer.valueOf(key);
		Integer intValue = Integer.valueOf(value);
		Collection<Integer> values = map.get(intKey);

		// if the value is null, create anew collection (linked list)
		if (values == null) {
			values = new LinkedList<Integer>();
		}

		// add the value to the collection
		values.add(intValue);

		// put key andvalue back into the map
		map.put(intKey, values);
	}
}

und die Klasse DiameterScheduleKey:

package com.javaforum.datamodel;

public class DiameterSchedulerKey {
	
	private final int diameter;
	
	private final int schedule;

	public DiameterSchedulerKey(int diameter, int schedule) {
		super();
		this.diameter = diameter;
		this.schedule = schedule;
	}

	public int getDiameter() {
		return diameter;
	}

	public int getSchedule() {
		return schedule;
	}

	@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime * result + diameter;
		result = prime * result + schedule;
		return result;
	}

	@Override
	public boolean equals(Object obj) {
		if (this == obj)
			return true;
		if (obj == null)
			return false;
		if (getClass() != obj.getClass())
			return false;
		DiameterSchedulerKey other = (DiameterSchedulerKey) obj;
		if (diameter != other.diameter)
			return false;
		if (schedule != other.schedule)
			return false;
		return true;
	}
	
	
}

Wie du siehst, speichere ich im Gegensatz zur Meinung von till nichts als String ab, denn ich bin der Meinung, wenn ich eine Zahl habe will ich sieauch als Zahl speichern, alles andere macht wenig sinn. Denn stell dir mal vor, du willst mitden Zahlen nicht nur irgendwelche Comboboxen füllen sondern auch berechnungen anstellen... Mit Strings wird das kompliziert. Also immer alles schön brav in dem Datentyp speichern, den der Wert hat und du hast ja selbst gesagt, das es Zahlen sind. Also ist int / Integer der richtige Datentyp.

Grüße,
Andreas

 

[babuschka]

Das ging ja flott! Ich danke euch schonmal und schaue mir das mal in Ruhe an. Bzgl. XML muss ich wirklich erstmal etwas lesen. Aber wenn man mit etwas neuem Anfängt, muss man sich halt erstmal in viele Dinge einlesen.

Bis später
Sven

 

[babuschka]

Ich denke mal - oh wunder- das ich deinen Code prinzipiell verstanden habe. Es bietet mir ja schon alles was ich brauche. Nur die Map mit dem selbstgebauten Key und die Maps für die ComboBoxes muss ich noch füllen muss. Dann lese ich jetzt erstmal ein bisschen über XML.

Grüße
Sven

 

[babuschka]

soooo. Ich habe deine Sache etwas umgestrickt, da ich gerne den "Diameter" als String speichern möchte. Er kann ja Werte wie "1 1/2" haben und ich möchte das nicht in Zahlen mit Nachkommastellen umrechen. Den innerDiameter habe ich in "double" geändert. Wieso hattest du "long" genommen?

Da nun "Diameter" und "Schedule" nicht mehr vom gleichen Typ sind brauchte ich noch eine zweite Funktion für die Collectionrückgabe. Denn einmal stehen in der Collection Strings und einmal Integer Werte:

private void putValuesToDiameterToScheduleMap(String key, Integer value,
         Map<String, Collection<Integer>> map) {
      // get the value for the given key out of the given map
      String strKey = key;
      Integer intValue = Integer.valueOf(value);
      Collection<Integer> values = map.get(strKey);

      // if the value is null, create anew collection (linked list)
      if (values == null) {
         values = new LinkedList<Integer>();
      }

      // add the value to the collection
      values.add(intValue);

      // put key andvalue back into the map
      map.put(strKey, values);
   }
   private void putValuesToScheduleToDiameterMap(Integer key, String value,
             Map<Integer, Collection<String>> map) {
          // get the value for the given key out of the given map
          Integer intKey = key;
          String intValue = value;
          Collection<String> values = map.get(intKey);

          // if the value is null, create anew collection (linked list)
          if (values == null) {
             values = new LinkedList<String>();
          }

          // add the value to the collection
          values.add(intValue);

          // put key andvalue back into the map
          map.put(intKey, values);
       }

Daher musste ich auch die Klasse PipeDiameterScheduleKey etwas anpassen:

public class PipeDiameterScheduleKey {
   
   private final String diameter;
   
   private final int schedule;

   public PipeDiameterScheduleKey(String diameter, int schedule) {
      super();
      this.diameter = diameter;
      this.schedule = schedule;
   }

   public String getDiameter() {
      return diameter;
   }

   public int getSchedule() {
      return schedule;
   }

   @Override
   public int hashCode() {
      final int prime = 31;
      int result = 1;
      result = prime * result + diameter.hashCode();
      result = prime * result + schedule;
      return result;
   }

   @Override
   public boolean equals(Object obj) {
      if (this == obj)
         return true;
      if (obj == null)
         return false;
      if (getClass() != obj.getClass())
         return false;
      PipeDiameterScheduleKey other = (PipeDiameterScheduleKey) obj;
      if (diameter != other.diameter)
         return false;
      if (schedule != other.schedule)
         return false;
      return true;
   }
   
   
}

Die XML Datei sieht wie folgt aus:

<?xml version ="1.0" ?>
<PipeSched source="Internet Page">
	<pipe name="1&_5" diameter="1" schedule="5" innerDiameter="30.1">
	</pipe>
	<pipe name="1&_10" diameter="1" schedule="10" innerDiameter="27.86">
	</pipe>
	<pipe name="2&_10" diameter="2" schedule="10" innerDiameter="54.79">
	</pipe>
</PipeSched>

Ob das nun vollkommen der XML Idee entspricht bin ich überfragt, aber es funktioniert erstmal.

Dann habe ich in den Konstruktor noch das Einlesen der XML Datei eingefügt. Bei dem Code zum Einlesen der XML Datei habe ich mich an dem einfachsten Beispiel aus "Java ist auch eine Insel" orientiert. Hier noch mal der Gesamte Code von der Klasse PipeSizeDataModel:

import java.io.FileInputStream;
import java.io.IOException;
import java.util.Collection;
import java.util.HashMap;
import java.util.LinkedList;
import java.util.Map;

import javax.xml.stream.XMLInputFactory;
import javax.xml.stream.XMLStreamConstants;
import javax.xml.stream.XMLStreamException;
import javax.xml.stream.XMLStreamReader;


public class PipeSizeDataModel {

   private final Map<PipeDiameterScheduleKey, Double> diameterScheduleInnerDiameter = new HashMap<PipeDiameterScheduleKey, Double>();

   private final Map<String, Collection<Integer>> diameterToSchedule = new HashMap<String, Collection<Integer>>();

   private final Map<Integer, Collection<String>> scheduleToDiameter = new HashMap<Integer, Collection<String>>();

   public PipeSizeDataModel() {
	   {
		    try
		    {
		      XMLInputFactory factory = XMLInputFactory.newInstance();
		      XMLStreamReader parser = factory.createXMLStreamReader( new FileInputStream( "/home/sven/workspace/FreeFlowTools/GeneralFiles/PipeSched.xml" ) );
		      
		      StringBuilder spacer = new StringBuilder();
		      
		      while ( parser.hasNext() )
		      {
		     //   System.out.println( "Event: " + parser.getEventType() );
		        switch ( parser.getEventType() )
		        {
		     
		          case XMLStreamConstants.START_ELEMENT:
		           spacer.append( "  " );
		            System.out.println( spacer + "START_ELEMENT: " + parser.getLocalName() );
		      
		      //       Der Event XMLStreamConstants.ATTRIBUTE wird nicht geliefert!
		            if (parser.getLocalName().equalsIgnoreCase("pipe")){
		            for ( int i = 0; i < parser.getAttributeCount(); i++ )
		              System.out.println( spacer + "  Attribut: "
		                                  + parser.getAttributeLocalName( i )
		                                 + " Wert: " + parser.getAttributeValue( i ) );
		            putDiameterScheduleInnerDiameter(parser.getAttributeValue(1),Integer.valueOf(parser.getAttributeValue(2)),Double.valueOf(parser.getAttributeValue(3)));
		            putDiameterSchedule(parser.getAttributeValue(1),Integer.valueOf(parser.getAttributeValue(2)));
		            }
		          break;
		          default:
		          break;
		        }
		        parser.next();
		      }
		    }
		    catch ( IOException e )
		    {
		      e.printStackTrace();
		    }
		    catch ( XMLStreamException e )
		    {
		      e.printStackTrace();
		    }
		  }
   }

   /**
    * This method returns the innerDiameter value for the given diameter and
    * schedule combination.
    *
    * @param diameter
    *            the diameter value
    * @param schedule
    *            the schedule value
    * @return the innerDiameter value for the given diameter and schedule
    *         combination
    */
   public double getInnerDiameter(String diameter, int schedule) {
      return diameterScheduleInnerDiameter.get(
            new PipeDiameterScheduleKey(diameter, schedule)).doubleValue();
   }

   /**
    * This method returns the allowed schedule values for the given diameter
    * value
    *
    * @param diameter
    *            the diameter value, which allowed schedule values are searched
    * @return the allowed schedule values for the given diameter
    */
   public Collection<Integer> getSchedule(String diameter) {
      return diameterToSchedule.get(diameter);
   }

   /**
    * This method returns the allowed diameter values for the given schedule
    *
    * @param schedule
    *            the schedule, which allowed diameter values are returned
    * @return the allowed diameter values for the given schedule
    */
   public Collection<String> getDiameter(int schedule) {
      return scheduleToDiameter.get(Integer.valueOf(schedule));
   }

   public void putDiameterScheduleInnerDiameter(String diameter, int scheduler,
         double innerDiameter) {
      diameterScheduleInnerDiameter.put(new PipeDiameterScheduleKey(diameter,
            scheduler), Double.valueOf(innerDiameter));
   }

   public void putDiameterSchedule(String diameter, int schedule) {
	   putValuesToDiameterToScheduleMap(diameter, schedule, diameterToSchedule);
	   putValuesToScheduleToDiameterMap(schedule, diameter, scheduleToDiameter);
   }

   private void putValuesToDiameterToScheduleMap(String key, Integer value,
         Map<String, Collection<Integer>> map) {
      // get the value for the given key out of the given map
      String strKey = key;
      Integer intValue = Integer.valueOf(value);
      Collection<Integer> values = map.get(strKey);

      // if the value is null, create anew collection (linked list)
      if (values == null) {
         values = new LinkedList<Integer>();
      }

      // add the value to the collection
      values.add(intValue);

      // put key andvalue back into the map
      map.put(strKey, values);
   }
   private void putValuesToScheduleToDiameterMap(Integer key, String value,
	         Map<Integer, Collection<String>> map) {
	      // get the value for the given key out of the given map
	      Integer intKey = key;
	      String intValue = value;
	      Collection<String> values = map.get(intKey);

	      // if the value is null, create anew collection (linked list)
	      if (values == null) {
	         values = new LinkedList<String>();
	      }

	      // add the value to the collection
	      values.add(intValue);

	      // put key andvalue back into the map
	      map.put(intKey, values);
	   }
}

Das Einlesen klappt auch, nur ich glaube es gibt Probleme mit dem hashCode. Ich habe zur Generierung ja keine zwei Integers mehr, sondern der "Diameter" liegt als String vor. Ich habe daher einfach die Funktion String.hashCode() verwendet. Bei der zweiten Zeile des folgenden Codes erhalte ich eine NullPointerException:

System.out.println(myPipeSizeDataModel.getSchedule("1"));
System.out.println(myPipeSizeDataModel.getInnerDiameter("2", 10));

Erste Zeile ist kein Problem und liefert eine Collection.

Ich kenne mich mit diesen HashCodes auch noch nicht besonders aus. Ist das ok, wie ich es gelöst habe?

@Override
   public int hashCode() {
      final int prime = 31;
      int result = 1;
      result = prime * result + diameter.hashCode();
      result = prime * result + schedule;
      return result;
   }

Hmm. Alles noch etwas Spagetti-Code-mäßig. Bei dem XML bin ich mir auch noch nicht so sicher, ob es in die richtige Richtung geht. Naja, wenn die NullPointerException weg ist, sollte es zumindest schon mal funktionieren. Learning by doing...

Grüße
Sven

 

[babuschka]

Hallo,

also das Problem lag an der Equals() funktion der PipeDiameterScheduleKey Klasse:

@Override
public boolean equals(Object obj) {
      if (this == obj)
         return true;
      if (obj == null)
         return false;
      if (getClass() != obj.getClass())
         return false;
      PipeDiameterScheduleKey other = (PipeDiameterScheduleKey) obj;
      if (diameter != other.diameter)
         return false;
      if (schedule != other.schedule)
         return false;
      return true;
   }

Das der diameter jetzt kein Integer mehr ist, sondern ein String, funktioniert der einfache Vergleich auf Gleichheit nicht mehr diameter != other.diameter, sondern man muss die equals Funktion des String Objektes verwenden:

 PipeDiameterScheduleKey other = (PipeDiameterScheduleKey) obj;
      if (!diameter.equals(other.diameter))
         return false;

Vielleicht könntet ihr noch meine Umsetzung der Daten in XML kommentieren?