Preisberechnung über Array

[Sunny81]

Hi Leute, will bei meinem Programm zusätzlich eine Preisberechnung über einen Array ausgeben. Komm aber nicht weiter.

Wo muss ich den Array und wo die Berechnuzng einbauen damit der Array sich die eingegebenen Daten holt?

Danke

[JAVA=42]

public class Verwaltung



{

private String prozessor;
private int ram;
private int festplatte;
static int anzahl;
String []Artikel = {"PC", "Laptop"};
double [] prozessorpreis = {249.99, 499.99};
String [] prozessortyp = {"AMD", "INTEL"};







public void setProzessor(String prozessor)
{this.prozessor=prozessor;}

public String getProzessor()
{return prozessor;}

public void setRam(int ram)
{this.ram = ram;}

public int getRam()
{return ram;}

public void setFestplatte(int festplatte)
{
this.festplatte = festplatte;
}

public int getFestplatte()
{
return festplatte;
}




public Verwaltung(String pprozessor, int pram, int pfestplatte) //Konstruktor
{
prozessor=pprozessor;
ram=pram;
festplatte=pfestplatte;
anzahl++;

}


public void print(int i)
{
IO.writeln("Ihr" +Artikel+ "PC wurde erfolgreich zusammengestellt.\nProzessor: " + prozessor + "\nGbRAM: " + ram + "\nFestplatte: " + festplatte + "\nAnzahl: " + anzahl + "\nPreis: " + cpupreis);
}

public void bestimmeProzessorPreis()
{
if(prozessor.equals(prozessortyp[0]))
{
cpupreis=prozessorpreis[0];
}

else
{
cpupreis=prozessorpreis[1];
}

}




}


[/code]

Kann ich die rot markierten intervalle so angeben?


[JAVA=42]
public class Verwaltung



{

private String prozessor;
private int ram;
private int festplatte;
static int anzahl;

String []Artikel = {"PC", "Laptop"};
double [] prozessorpreis = {249.99, 499.99};
String [] prozessortyp = {"AMD", "INTEL"};
int [] festplattengroesse = {160 - 250, 250-400, 400-500};
double [] festplattenpreis = {59.99, 149.99, 249.99};
int [] ramSpeicher = {4-8, 8-16};
double [] ramSpeicherPreis = {249.99 , 499.99};








public void setProzessor(String prozessor)
{this.prozessor=prozessor;}

public String getProzessor()
{return prozessor;}

public void setRam(int ram)
{this.ram = ram;}

public int getRam()
{return ram;}

public void setFestplatte(int festplatte)
{
this.festplatte = festplatte;
}

public int getFestplatte()
{
return festplatte;
}




public Verwaltung(String pprozessor, int pram, int pfestplatte) //Konstruktor
{
prozessor=pprozessor;
ram=pram;
festplatte=pfestplatte;
anzahl++;

}


public void print(int i)
{
IO.writeln("Ihr" +Artikel+ "PC wurde erfolgreich zusammengestellt.\nProzessor: " );
}






}


[/code]

 

[javimka]

Ich kann dir leider nicht ganz folgen. Wieso hast du zwei Klassen mit dem gleichen Namen? Was für eine Preisberechnung möchtest du machen? Über einem Array das die Preise enthält?
Was genau meinst du mit "Das Array holt sich die Daten"? Du musst die Daten in das Array speichern mit array = ...;

 

[Sunny81]

Oh sorry, die 1. Klasse heisst Verwaltung und die 2. Bestellung.

Also ch frage ja den User in Bestellung ab welche Komponenten er will.
Und ich möchte nach dem bestellen ihm dann einen Preis ausgeben der von seinen Angaben abhängt.
Wie es in den Arrays gespeichert ist.

Sprich in den Intervallen vom Ram Speicher, prozessorabhängig und Festplattengroesse in Intervallen abhängig.

 

[javimka]

Dann hast du also zu jedem Ram, Prozessor usw einen Preis und willst den Gesamtpreis ausrechnen?

Du könntest eine HashMap verwenden und die Namen der RAM/CPU etc. als Schlüssel verwenden. Mit put("Intel",300) kannst du dann festlegen, dass ein Intel 300 Euro kostet. Mit get("Intel") kriegst du den Preis für einen Intel-Prozessor wieder zurück und kannst die Werte zusammenrechnen.

 

[Sunny81]

Wie würde das gehen?
Hab das noch nie gehört.
Und wie mach ich das mit den Intervallen bei ram und festplatten?
Danke

 

[javimka]

Nein, deine Intervalle funktionieren nicht. Du meinst sicher das da:
int [] festplattengroesse = {160 - 250, 250-400, 400-500};
Für den Compilierer sind das ganz normale Subtraktionen, du erhälst das Array {-90,-150,-100}.
Du könntest sie aus String abspeichern, aber dann kannst du sie natürlich nicht mehr als Zahlen verwenden, sie sind dann halt Strings.

Mti der HashMap könnte das etwa so aussehen:

HashMap<String, Integer> hashMap = new HashMap<String, Integer>();
hashMap.put("Intel", 300);
hashMap.get("Intel"); // 300 kommt zurück

 

[eRaaaa]

	Map<String, Double> artikel = new HashMap<String, Double>();
	artikel.put("PC", 249.99);
	artikel.put("Laptop", 499.99);
	artikel.put("Festplatte160", 59.99);
	artikel.put("Festplatte250", 149.99);
	//usw.
	artikel.put("RAM8", 249.99);
	artikel.put("RAM16", 499.99);
	
	System.out.println("kaufe 1 PC (160 Festplatte, 8er RAM");
	System.out.println("Das kostet: "
		+ (artikel.get("PC") + artikel.get("Festplatte160") + artikel.get("RAM8")) + " €");

:bae:

/edit: too late ;/

 

[Sunny81]

Gibts eine Möglichkeit die Intervalle unterzubringen?
Und kann man das auch irgendwie über Arrays lösen?
Danke

 

[javimka]

Du könntest eine Klasse Intervall machen mit einer unteren und oberen Schranke, die eine Methode "boolean contains(int v)" bietet, die für einen int v berechnet, ob er im Intervall liegt oder nicht.
Mehrere solche Objekte könntest du dann natürlich in ein Array legen.

 

[Sunny81]

Du könntest eine Klasse Intervall machen mit einer unteren und oberen Schranke, die eine Methode "boolean contains(int v)" bietet, die für einen int v berechnet, ob er im Intervall liegt oder nicht.
Mehrere solche Objekte könntest du dann natürlich in ein Array legen.

Vielleicht bisschen genauer...
Bin noch wirklich sehr unbelesen...
Danke Danke Danke

 

[javimka]

Die Klasse Intervall:

public class Intervall {
  private int lower;
  private int upper;
  public Intervall(int lower, int upper) {
    this.lower = lower;
    this.upper = upper;
  }

  public boolean contains(int v) {
    return lower <= v && v <= upper;
  {
}

Und dann kannst du ein Array aus Intervallen machen:

private Intervall[] festplattengroesse = new Intervall[] {
    new Intervall(160,250),
    new Intervall(250,400),
    new Intervall(400,500) };

Wenn du dann den Preis für eine Festplatte willst:

// Das kommt in den Konstruktor:
for (Intervall i:festplattengroesse) {
  hashMap.put(i,300); // Festplatte i kostet 300 Euro
}

//Das ist die Methode, die den Preis für eine Festplatte findet:
public int getPreisFor(int v) { // sucht den Preis für eine Festplatte mit Grösse v
  for (Intervall i:festplattengroesse) {
    if (i.contains(v)) {
      return hasMap.get(i);
    }
  }
  return 0; // Falls keine gefunden wurde, was natürlich nicht passieren darf
}

 

[Sunny81]

Mein Eclipse erkennt HashMap nicht???

 

[javimka]

Sollte im package java.util.* sein

 

[Sunny81]

was muss ich dann machen?

 

[javimka]

oben ins File schreiben: [c]import java.util.*;[/c]

 

[Sunny81]

ah ok...

Was stimmt mit meinem COde hier grad nicht?

[JAVA=42]

import java.util.*;




public class Verwaltung



{

private String prozessor;
private int ram;
private int festplatte;
static int anzahl;
String []Artikel = {"PC", "Laptop"};

HashMap<String, Integer> hashMap = new HashMap<String, Integer>();
hashMap.put("Intel", 300);
hashMap.put("AMD", 400);

HashMap<Integer, Integer> hashMap = new HashMap<Integer, Integer>();
hashMap.put(4, 100);
hashMap.put(8, 200);
hashMap.put(16, 400);

HashMap<Integer, Integer> hashMap = new HashMap<Integer, Integer>();
hashMap.put(160, 100);
hashMap.put(320, 250);
hashMap.put(500, 400);









public void setProzessor(String prozessor)
{this.prozessor=prozessor;}

public String getProzessor()
{return prozessor;}

public void setRam(int ram)
{this.ram = ram;}

public int getRam()
{return ram;}

public void setFestplatte(int festplatte)
{
this.festplatte = festplatte;
}

public int getFestplatte()
{
return festplatte;
}




public Verwaltung(String pprozessor, int pram, int pfestplatte) //Konstruktor
{
prozessor=pprozessor;
ram=pram;
festplatte=pfestplatte;
anzahl++;

}


public void print(int i)
{
IO.writeln("Ihr" +Artikel+ "PC wurde erfolgreich zusammengestellt.\nProzessor: " + prozessor + "\nGbRAM: " + ram + "\nFestplatte: " + festplatte + "\nAnzahl: " + anzahl);
}





}
[/code]

 

[javimka]

Du darfst [c]HashMap<String, Integer> hashMap[/c] nur einmal deklarieren. Du machst das in den Zeilen 60,64 und 69 aber insgesamt 3 Mal. Ich verstehe nicht, was du mit der zweiten und dritten bezwecken möchtest. Du speicherst darin ja die Artikel (auch wenn sie aus verschiedenen Kategorien bzw. Array kommen) als Schlüssel und den zugehörigen Wert.
Ausserdem hast du das ja alles einfach so in die Klasse geschrieben, die put-Anweisungen gehören natürlich in den Konstruktor

Vielleicht deklarierst du deine HashMap dann besser mit <Object,Integer>, dann kannst du auch int und Intervall als Schlüssel brauchen.

Also streiche Zeilen 64 und 69 ersatzlos und dann ändere die Deklarierung auf Zeile 60 in:
HashMap<Object,Integer> hasMap = new HashMap<Object,Integer>();

Die drei Blöcke mit den put-Befehlen kommen in den Konstruktor.
Und dann musst du aus den Intervallen natürlich Instanzen der Klasse Intervall machen. Diese dann als Schlüssel verwenden.

 

[Sunny81]

Ich brauche doch noch ein HAsh Map für die Variante Integer,Integer?
Da ich bei der Abfrage nach der Festplatte doch einen Integer und keinen String kriege?
Das mit den Intervallen lass ich mal.
Ist mir zu kompliziert.

 

[eRaaaa]

du kannst der aber auf garkeinen fall dann den gleichen namen geben, wie der anderen hashmap

String sunny = "Sunny81";

und

String sunny = "eRaaaa";


geht ja schließlioch auch nicht ;D

 

[Sunny81]

Wie soll ich das dann machen?

 

[eRaaaa]

Du darfst [c]HashMap<String, Integer> hashMap[/c] nur einmal deklarieren. Du machst das in den Zeilen 60,64 und 69 aber insgesamt 3 Mal. Ich verstehe nicht, was du mit der zweiten und dritten bezwecken möchtest. Du speicherst darin ja die Artikel (auch wenn sie aus verschiedenen Kategorien bzw. Array kommen) als Schlüssel und den zugehörigen Wert.
Ausserdem hast du das ja alles einfach so in die Klasse geschrieben, die put-Anweisungen gehören natürlich in den Konstruktor

Vielleicht deklarierst du deine HashMap dann besser mit <Object,Integer>, dann kannst du auch int und Intervall als Schlüssel brauchen.

Also streiche Zeilen 64 und 69 ersatzlos und dann ändere die Deklarierung auf Zeile 60 in:
HashMap<Object,Integer> hasMap = new HashMap<Object,Integer>();

Die drei Blöcke mit den put-Befehlen kommen in den Konstruktor.
Und dann musst du aus den Intervallen natürlich Instanzen der Klasse Intervall machen. Diese dann als Schlüssel verwenden.

so z.b.

oder einfach anders benennen:

mapCPU,
mapHDD
mapRAM

wobei eine map vllt. für dich einfacher wäre, bzw ich auch garnicht genau weiss, was du damit dann noch so vorhast

 

[Sunny81]

Wie soll ich das dann machen?

 

[javimka]

Kennst du das Spiel, wo das erste Kind etwas sagt und das zweite dann immer fragt "warum?" und dann gibt das erste wieder eine Antwort und das zweite fragt wieder "warum?", "warum?", "warum?", "warum?" und so weiter? So kommt mir das hier etwa vor :noe:

was GENAU meinst du denn mit "wie macht man DAS"?

 

[Sunny81]

Leute bitte helft mir noch einmal.
Was muss ich jetzt in meine print Methode eingeben damit der Preis nach den Angaben vom User berechnet wird.
[JAVA=42]import java.util.*;




public class Verwaltung



{

private String prozessor;
private int ram;
private int festplatte;
static int anzahl;
String []Artikel = {"PC", "Laptop"};

HashMap<String, Integer> cpu = new HashMap<String, Integer>();
HashMap<Integer, Integer> Ram = new HashMap<Integer, Integer>();
HashMap<Integer, Integer> platte = new HashMap<Integer, Integer>();










public void setProzessor(String prozessor)
{this.prozessor=prozessor;}

public String getProzessor()
{return prozessor;}

public void setRam(int ram)
{this.ram = ram;}

public int getRam()
{return ram;}

public void setFestplatte(int festplatte)
{
this.festplatte = festplatte;
}

public int getFestplatte()
{
return festplatte;
}




public Verwaltung(String pprozessor, int pram, int pfestplatte) //Konstruktor
{
prozessor=pprozessor;
ram=pram;
festplatte=pfestplatte;
anzahl++;

cpu.put("Intel", 300);
cpu.put("AMD", 400);

Ram.put(4, 100);
Ram.put(8, 200);
Ram.put(16, 400);

platte.put(160, 100);
platte.put(320, 250);
platte.put(500, 400);


}


public void print(int i)
{
IO.writeln("Ihr" +Artikel+ "PC wurde erfolgreich zusammengestellt.\nProzessor: " + prozessor + "\nGbRAM: " + ram + "\nFestplatte: " + festplatte + "\nAnzahl: " + anzahl + "\nIhr Preis: " + HIER);
}





}[/code]

 

[eRaaaa]

also, erstmal solltest du den code mal formatieren, kann ja kein mensch lesen ;D
dann würde ich mir eine neue methode schreiben:

    private double calc() {
	return cpu.get(this.prozessor) + Ram.get(this.ram)
		+ platte.get(this.festplatte);
    }

und diese halt in deiner print methode aufrufen:

    public void print(int i) {
	System.out.print("Ihr " + Artikel[i]
		+ " PC wurde erfolgreich zusammengestellt.\nProzessor: "
		+ prozessor + "\nGbRAM: " + ram + "\nFestplatte: " + festplatte
		+ "\nAnzahl: " + anzahl + "\nIhr Preis: " + calc());
    }

wobei ich persönlich finde, dass die klasse irgendwie durcheinander ist, also irgendwie ist da verwaltung, mit bestellung + warenkorb gemischt ;D
aber funktionieren sollte es jetzt zumindestens, nur halt nicht optimal

import java.util.HashMap;

public class Verwaltung

{

    private String prozessor;
    private int ram;
    private int festplatte;
    static int anzahl;
    String[] Artikel = { "PC", "Laptop" };

    HashMap<String, Integer> cpu = new HashMap<String, Integer>();
    HashMap<Integer, Integer> Ram = new HashMap<Integer, Integer>();
    HashMap<Integer, Integer> platte = new HashMap<Integer, Integer>();

    public void setProzessor(String prozessor) {
	this.prozessor = prozessor;
    }

    public String getProzessor() {
	return prozessor;
    }

    public void setRam(int ram) {
	this.ram = ram;
    }

    public int getRam() {
	return ram;
    }

    public void setFestplatte(int festplatte) {
	this.festplatte = festplatte;
    }

    public int getFestplatte() {
	return festplatte;
    }

    public Verwaltung(String pprozessor, int pram, int pfestplatte) {

	prozessor = pprozessor;
	ram = pram;
	festplatte = pfestplatte;
	anzahl++;

	cpu.put("Intel", 300);
	cpu.put("AMD", 400);

	Ram.put(4, 100);
	Ram.put(8, 200);
	Ram.put(16, 400);

	platte.put(160, 100);
	platte.put(320, 250);
	platte.put(500, 400);

    }

    public void print(int i) {
	System.out.print("Ihr " + Artikel[i]
		+ " wurde erfolgreich zusammengestellt.\nProzessor: "
		+ prozessor + "\nGbRAM: " + ram + "\nFestplatte: " + festplatte
		+ "\nAnzahl: " + anzahl + "\nIhr Preis: " + calc());
    }

    private double calc() {
	return cpu.get(this.prozessor) + Ram.get(this.ram)
		+ platte.get(this.festplatte);
    }

    public static void main(String[] args) {
	Verwaltung verwaltung = new Verwaltung("Intel", 16, 500);
	verwaltung.print(0);
    }
}

 

[Sunny81]

Wieso gibst du im Konstruktor schon Werte vor?
Ich will doch dass der User die eingibt über die IO Abfrage?
Danke


[JAVA=42]public class Bestellung


{
static String prozessor=null;
static int ram=0;
static int festplatte=0;

public static void main(String args[])
{



for(;
{

int select= IO.promptAndReadInt("Wählen Sie 1 um einen PC zu erstellen!\nWählen Sie 2 um einen Laptop zu erstellen!\nWählen Sie 3 um zu beenden!\n");

switch(select)

{




case 1:


auswahl();







Verwaltung o1 = new Verwaltung(prozessor, ram, festplatte);

o1.print(0);
break;



case 2:

auswahl();

Verwaltung o2 = new Verwaltung(prozessor, ram, festplatte);

o2.print(1);
break;


case 3: IO.writeln("Vielen Dank für Ihren besuch - Auf Wiedersehen!");
break;

default: IO.writeln("Falsche Taste - Bitte erneut wählen.");
break;

}






}
}

public static void auswahl()
{
while(prozessor==null)
{
prozessor=IO.promptAndReadString("Bitte wählen Sie den Prozessor aus:");
if(!"INTEL".equals(prozessor) && !"AMD".equals(prozessor))
{
prozessor=null;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(ram==0)
{
ram=IO.promptAndReadInt("Bitte wählen Sie den Arbeitsspeicher aus:");
if(ram<4 || ram>16)
{
ram=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(festplatte==0)
{
festplatte=IO.promptAndReadInt("Bitte wählen Sie die Festplatte aus:");
if(festplatte<160 || festplatte>500)
{
festplatte=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}
}

}
[/code]





[JAVA=42]import java.util.*;




public class Verwaltung



{

private String prozessor;
private int ram;
private int festplatte;
static int anzahl;
String []Artikel = {"PC", "Laptop"};

HashMap<String, Integer> cpu = new HashMap<String, Integer>();
HashMap<Integer, Integer> Ram = new HashMap<Integer, Integer>();
HashMap<Integer, Integer> platte = new HashMap<Integer, Integer>();










public void setProzessor(String prozessor)
{this.prozessor=prozessor;}

public String getProzessor()
{return prozessor;}

public void setRam(int ram)
{this.ram = ram;}

public int getRam()
{return ram;}

public void setFestplatte(int festplatte)
{
this.festplatte = festplatte;
}

public int getFestplatte()
{
return festplatte;
}




public Verwaltung(String pprozessor, int pram, int pfestplatte) //Konstruktor
{
prozessor=pprozessor;
ram=pram;
festplatte=pfestplatte;
anzahl++;

cpu.put("Intel", 300);
cpu.put("AMD", 400);

Ram.put(4, 100);
Ram.put(8, 200);
Ram.put(16, 400);

platte.put(160, 100);
platte.put(320, 250);
platte.put(500, 400);


}


public void print(int i)
{
IO.writeln("Ihr" +Artikel
+ "PC wurde erfolgreich zusammengestellt.\n" +
"Prozessor: " + prozessor + "\n" +
"GbRAM: " + ram + "\n" +
"Festplatte: " + festplatte + "\n" +
"Anzahl: " + anzahl + "\n" +
"Ihr Preis: " + calc() );
}

private double calc() {
return cpu.get(this.prozessor) + Ram.get(this.ram)
+ platte.get(this.festplatte);
}




}
[/code]

 

[eRaaaa]

ja dann machs doch so ???:L

ich hab die main methode doch nur aus testzwecken, bzw vorführungszwecken wenn mans so nennen mag, eingebaut

 

[Sunny81]

Hi Leute, soweit klappts fast schon.

Kann mir vllt jmd erklären wieso wenn ich bei der Abfrage "INTEL" eingebe es nicht klappt?

bei AMD funktioniert es.

Danke






[JAVA=42]public class Bestellung


{
static String prozessor=null;
static int ram=0;
static int festplatte=0;

public static void main(String args[])
{



for(;
{

int select= IO.promptAndReadInt("Wählen Sie 1 um einen PC zu erstellen!\nWählen Sie 2 um einen Laptop zu erstellen!\nWählen Sie 3 um zu beenden!\n");

switch(select)

{




case 1:


auswahl();







Verwaltung o1 = new Verwaltung(prozessor, ram, festplatte);

o1.print(0);
break;



case 2:

auswahl();

Verwaltung o2 = new Verwaltung(prozessor, ram, festplatte);

o2.print(1);
break;


case 3: IO.writeln("Vielen Dank für Ihren besuch - Auf Wiedersehen!");
break;

default: IO.writeln("Falsche Taste - Bitte erneut wählen.");
break;

}






}
}

public static void auswahl()
{
while(prozessor==null)
{
prozessor=IO.promptAndReadString("Bitte wählen Sie den Prozessor aus:");
if(!"INTEL".equals(prozessor) && !"AMD".equals(prozessor))
{
prozessor=null;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(ram==0)
{
ram=IO.promptAndReadInt("Bitte wählen Sie den Arbeitsspeicher aus:");
if(ram<4 || ram>16)
{
ram=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}




while(festplatte==0)
{
festplatte=IO.promptAndReadInt("Bitte wählen Sie die Festplatte aus:");
if(festplatte<160 || festplatte>500)
{
festplatte=0;
IO.writeln("Ungültig - Bitte erneut wählen!");
}

else
{
IO.writeln("Erfolgreich hinzugefügt!");
}
}
}

}

[/code]








[JAVA=42]

{

private String prozessor;
private int ram;
private int festplatte;
static int anzahl;
String []Artikel = {"PC", "Laptop"};

HashMap<String, Integer> cpu = new HashMap<String, Integer>();
HashMap<Integer, Integer> Ram = new HashMap<Integer, Integer>();
HashMap<Integer, Integer> platte = new HashMap<Integer, Integer>();










public void setProzessor(String prozessor)
{this.prozessor=prozessor;}

public String getProzessor()
{return prozessor;}

public void setRam(int ram)
{this.ram = ram;}

public int getRam()
{return ram;}

public void setFestplatte(int festplatte)
{
this.festplatte = festplatte;
}

public int getFestplatte()
{
return festplatte;
}




public Verwaltung(String pprozessor, int pram, int pfestplatte) //Konstruktor
{
prozessor=pprozessor;
ram=pram;
festplatte=pfestplatte;
anzahl++;

cpu.put("Intel", 300);
cpu.put("AMD", 400);

Ram.put(4, 100);
Ram.put(8, 200);
Ram.put(16, 400);

platte.put(160, 100);
platte.put(320, 250);
platte.put(500, 400);


}


public void print(int i)
{
IO.writeln("Ihr" +Artikel
+ "PC wurde erfolgreich zusammengestellt.\n" +
"Prozessor: " + prozessor + "\n" +
"GbRAM: " + ram + "\n" +
"Festplatte: " + festplatte + "\n" +
"Anzahl: " + anzahl + "\n" +
"Ihr Preis: " + calc() );
}

private double calc() {
return cpu.get(this.prozessor) + Ram.get(this.ram)
+ platte.get(this.festplatte);
}




}
[/code]

 

[javimka]

Wahrscheinlich hast du es einmal klein und einmal gross geschrieben.

Wieso machst du deinen Code eigentlich partout nicht leserlich? Z.B. Zeile 54 bis 63 sind leer, Zeile 64 bis 84 haben dann 8-fache Einrückungen ???:L
Schau mal, wie schön das eRaaaa gemacht hat.

 

[Sunny81]

Was meinst du mit 8fache Einrückungen?

 

[eRaaaa]

Kann mir vllt jmd erklären wieso wenn ich bei der Abfrage "INTEL" eingebe es nicht klappt?

ok, vllt sind da auch wir ein wenig dran schuld
also es wäre vllt besser, in der map alle strings immer als lower-case zu halten.
also anstatt map.put("Intel",..); und map.put("AMD",..) lieber map.put("intel",..) und map.put("amd"..)

und wenn du dann mit map.get(...) die abfrage machst, rufst du z.b. lieber map.get(this.prozessor.toLowerCase()) usw. auf.

 

[javimka]

Einfache Einrückung (leserlich):

    public void setProzessor(String prozessor) {
        this.prozessor=prozessor;
    }

8-facht Einrückung (Schrott):

                                public void setProzessor(String prozessor)
                                {this.prozessor=prozessor;}

Benutze etweder, wie eRaaaa sagt toLowerCase() oder schreib die Strings korrekt. Und frag jetzt nicht "wie macht man das?"

 

[Sunny81]

Danke Leute!
Danke für euere Geduld!