• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Próba lepszej parametryzacji metod w celu uniknięcia licznych powtórzeń kodu.

VPS Starter Arubacloud
0 głosów
103 wizyt
pytanie zadane 11 marca 2017 w Java przez sh1nen Obywatel (1,570 p.)

Witam, piszę trochę z nietypowym problem, niestety dość mocno "kłującym" mnie w oczy, przez co nie mogę przejść obok niego obojętnie. Otóż mam do napisania program liczący szereg parametrów dla danego zbioru którym jest zbiór irysów. Wyliczeń mam dokonać dla poszczególnych cech (długość, szerokość itd) dla wszystkich gatunków oraz dla każdego z osobna.  Oto przykładowy kod, który obecnie z bólem serca umieszczam, bo strasznie mi za niego wstyd, w poniższej liście przechowuje wszystkie obiekty irysów wczytane z pliku

ArrayList<Iris> irisData;

a to deklaracja klasy Iris:

public class Iris {
	
	private double sepalLength;
	private double sepalWidth;
	private double petalLength;
	private double petalWidth;
	private String irisClass;
	
	public Iris(double sepL, double sepW, double petL, double petW, String iriC) {
		this.sepalLength = sepL;
		this.sepalWidth = sepW;
		this.petalLength = petL;
		this.petalWidth = petW;
		this.irisClass = iriC;
	}
}

Poniżej umieszczam kod przykładowej metody liczącej średnią arytmetyczną, jak widać jest tu powtarzanie kodu na praktycznie każdym miejscu:

    public ArrayList<Double> getSepalWidthsOfSpiece(String spiece) {
    	ArrayList<Double> result= new ArrayList<>();
    	for(Iris iris : irisData){
    		if(spiece.equals(iris.getIrisClass())){
    			result.add(iris.getSepalWidth());
    		}
    	}
    	return result;
    }
    
    public ArrayList<Double> getSepalLengthsOfSpiece(String spiece) {
    	ArrayList<Double> result= new ArrayList<>();
    	for(Iris iris : irisData){
    		if(spiece.equals(iris.getIrisClass())){
    			result.add(iris.getSepalLength());
    		}
    	}
    	return result;
    }
    
    public ArrayList<Double> getPetalLengthsOfSpiece(String spiece) {
    	ArrayList<Double> result= new ArrayList<>();
    	for(Iris iris : irisData){
    		if(spiece.equals(iris.getIrisClass())){
    			result.add(iris.getPetalLength());
    		}
    	}
    	return result;
    }
    
    public ArrayList<Double> getPetalWidthsOfSpiece(String spiece) {
    	ArrayList<Double> result= new ArrayList<>();
    	for(Iris iris : irisData){
    		if(spiece.equals(iris.getIrisClass())){
    			result.add(iris.getPetalWidth());
    		}
    	}
    	return result;
    }
    public void arithmeticAvarageForAll() {
    	double resultSepalL = 0;
    	double resultSepalW = 0;
    	double resulPetalL = 0;
    	double resultPetalW = 0;
    	for(Iris iris : irisData){
    		resultSepalL += iris.getSepalLength();
    		resultSepalW += iris.getSepalWidth();
    		resulPetalL += iris.getPetalLength();
    		resultPetalW += iris.getPetalWidth();
    	}
    	System.out.println("Arithmetic avarage for each feature:" + "\nsepal length: " + resultSepalL/irisData.size() +
    			"\nsepal width: " + resultSepalW/irisData.size() + "\npetal length: " + resulPetalL/irisData.size() + "\npetal width: " + 
    			resultPetalW/irisData.size() + "\n");
    	
		arithmeticAvarageForSpieces("Iris-setosa");
		arithmeticAvarageForSpieces("Iris-versicolor");
		arithmeticAvarageForSpieces("Iris-virginica");
    }
    
    public void arithmeticAvarageForSpieces(String spiece) {
    	double[] resultSepalL = getSepalLengthsOfSpiece(spiece).stream().mapToDouble(Double::doubleValue).toArray();
    	double[] resultSepalW = getSepalWidthsOfSpiece(spiece).stream().mapToDouble(Double::doubleValue).toArray();
    	double[] resulPetalL = getPetalLengthsOfSpiece(spiece).stream().mapToDouble(Double::doubleValue).toArray();
    	double[] resultPetalW = getPetalWidthsOfSpiece(spiece).stream().mapToDouble(Double::doubleValue).toArray();

    	System.out.println(spiece + "\nArithmetic avarage for each spieces:" + "\nsepal length: " + DoubleStream.of(resultSepalL).sum()/resultSepalL.length +
    			"\nsepal width: " + DoubleStream.of(resultSepalW).sum()/resultSepalW.length + "\npetal length: " + 
    			DoubleStream.of(resulPetalL).sum()/resulPetalL.length  + "\npetal width: " + 
    			DoubleStream.of(resultPetalW).sum()/resultPetalW.length+ "\n");
    }

 

Będę bardzo wdzięczny jeśli ktoś mógł podrzucić jakiś pomysł jak zminimalizować takowe operacje do minimum, gdyż nie ukrywam że cały program liczący 12 różnych wartości zajął mi prawie 700 linijek co wydaję mi się że jest okropnie złą praktyką. Pozdrawiam i dziękuję z góry za wskazówki.

1 odpowiedź

+2 głosów
odpowiedź 11 marca 2017 przez mokrowski Mędrzec (155,460 p.)
Zapoznaj się z pojęciem lambdy i jej przekazywaniem do metody.

http://www.oracle.com/webfolder/technetwork/tutorials/obe/java/Lambda-QuickStart/index.html

Podobne pytania

0 głosów
2 odpowiedzi 175 wizyt
0 głosów
1 odpowiedź 108 wizyt
pytanie zadane 3 maja 2017 w C# przez Apacz700 Nowicjusz (150 p.)
0 głosów
2 odpowiedzi 543 wizyt
pytanie zadane 29 grudnia 2018 w Java przez studenciak Nowicjusz (230 p.)

92,416 zapytań

141,222 odpowiedzi

318,985 komentarzy

61,831 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...