Konstruktor, Objekte

[Secretdeadlyallo]

Hey,
Ich will folgendes:
Der Konstruktor soll als Parameter einen Scanner haben, der eine File einliest die folgendermassen aufgebaut ist:
Name Double Double
Und diese erzeugten Objekte will ich dann in einer Liste speichern.

import java.io.File;
import java.io.FileNotFoundException;
import java.util.*;


public class Früchte {

 String name;
 double einkaufspreis;
 double verkaufspreis;
 List<Früchte> sortiment = new ArrayList<>();


 public Früchte(Scanner scanner) {
  while (scanner.hasNext()) {
   this.name = scanner.next();
   this.einkaufspreis = scanner.nextDouble();
   this.verkaufspreis = scanner.nextDouble();
   sortiment.add(this);
  }
 }

 public static List<Früchte> Gewinn(double win) throws FileNotFoundException {
  Scanner sc = new Scanner(new File("fruits.txt"));
  List<Früchte> ret = new ArrayList<Früchte>();

  Früchte fruits = new Früchte(sc);
 
  for (int i = 0; i < fruits.sortiment.size(); i++) {
   if (fruits.verkaufspreis - fruits.einkaufspreis > win) {
    ret.add(fruits);
   }
  }
  return ret;
 }
 public static void main(String[] args) throws FileNotFoundException {
   System.out.println(Gewinn(1.0));
 }
 public String toString() {
  return name + " " + einkaufspreis + " " + verkaufspreis;
 }
}

Aber irgendwie funktioniert das nicht so ganz wie ich will. Wäre froh um Hilfe um den Konstruktor erstmal richtig zu machen

 

[JCODA]

Eine Frucht hat kein Sortiment. Deine Klasse sollte eben "Frucht" und nicht "Früchte" heißen, denn ein solches Objekt stellt nur eine spezielle Frucht dar.
Wenn du eine Liste willst, die aus mehreren Frucht-Objekten besteht, musst du, wie du schon angedeutet hast, diese in der main-Methode erstellen, befüllen und generell damit hantieren.

Desweiteren sollte der Konstruktor nur eine Frucht lesen. und in der main-Methode innerhalb einer Schleife mehrere Frucht-Objekte erstellt werden.

 

[Secretdeadlyallo]

Hey, danke erstmals für deine Anregungen. Habs nun folgendermassen geändert.

import java.io.File;
import java.io.FileNotFoundException;
import java.util.*;
public class Frucht {

 String name;
 double einkaufspreis;
 double Verkaufspreis;
 static List<Frucht> sortiment = new ArrayList<>();

 public Frucht(Scanner scanner) {

  this.name = scanner.next();
  this.einkaufspreis = scanner.nextDouble();
  this.verkaufspreis = scanner.nextDouble();

 }


 public static List<Frucht> Gewinn(double win, File file) throws FileNotFoundException {
  Scanner sc = new Scanner(file);
  List<Frucht> ret = new ArrayList<Frucht>();

  while (sc.hasNext()) {
   Frucht fruits = new Frucht(sc);
   sortiment.add(fruits);
  }

  for (int i = 0; i < sortiment.size(); i++) {
   if (sortiment.get(i).verkaufspreis - sortiment.get(i).einkaufspreis > win) {
    ret.add(sortiment.get(i));
   }
  }
  return ret;
 }

 public static void main(String[] args) throws FileNotFoundException {
  System.out.print(Gewinn(1.0, new File("fruits.txt")));
 }

 public String toString() {
  return name + " " + einkaufspreis + " " + verkaufspreis;
 }
}

// Input File:
Apfel 5.0 7.0
Birne 3.5 5.2
Banane 6.0 7.0
Salat 1.0 1.4
Erdbeere 4.2 5.3

//Output : [Apfel 5.0 7.0, Birne 3.5 5.2, Erdbeere 4.2 5.3]

Ist dies etwa so, wie du es machen würdest bzw. gedacht hast ?

 

[JCODA]

Ja das kommt dem schon ziemlich nahe, hätte noch ein bisschen umsortiert und dann sowas gemacht:

import java.io.File;
import java.io.FileNotFoundException;
import java.util.*;

public class Frucht {

    private String name;
    private double einkaufspreis;
    private double verkaufspreis;

    public Frucht(Scanner scanner) {
        this.name = scanner.next();
        this.einkaufspreis = scanner.nextDouble();
        this.verkaufspreis = scanner.nextDouble();
    }

    public double gewinn() {       
        return verkaufspreis-einkaufspreis;
    }
   
    public String toString() {
        return name + " " + einkaufspreis + " " + verkaufspreis;
    }
   
    public static List<Frucht> filterMindestGewinn(double mindGewinn, List<Frucht> sortiment){
        List<Frucht> gefiltertesSortiment = new ArrayList<>();
        for (Frucht frucht : sortiment) {
            if (frucht.gewinn() > mindGewinn) {
                gefiltertesSortiment.add(frucht);
            }
        }
        return gefiltertesSortiment;
    }
   
    public static void main(String[] args) throws FileNotFoundException {
       
        Scanner sc = new Scanner(new File("fruits.txt"));
        List<Frucht> sortiment = new ArrayList<Frucht>();

        while (sc.hasNext()) {
            Frucht fruits = new Frucht(sc);
            sortiment.add(fruits);
        }

        List<Frucht> gefiltertesSortiment = filterMindestGewinn(1.0,sortiment);       
        System.out.print(gefiltertesSortiment);
    }

   
}

 

[Secretdeadlyallo]

alles klar. besten dank für deine hilfe