schlechter objektorientierter stil

[holde maid]

hallo,

ich muss morgen ne algo übung abgeben. und da ist die Frage:
Die methode multiply (int x, int y) realisiert die Aufgabe zwei zahlen zu multiplizieren, in einem schlechten objektorientierten Stil. inwiefern?

private int multiply (intx, int y) {
    int res;

    if (x == 0) return 0;
    else {
        res = y + multiply (x-1, y) ;
        return res;
    }
}

kann mir da irgendjemand helfen????????
edit deathbyaclown: code tags !!!

 

[bygones]

weil es rekursiv gelöst ist und somit mehr an funktionelle Programmierung erinner als an OOP... (Rekursion allein ist nicht schlecht)...

war das die Frage ?

 

[holde maid]

keine ahnung. rekursion an sich ist ja nicht schlecht. ich hab da auch schon mehr leute gefragt. und was anderes ist da auch nicht rausgekommen. also vielen dank. vielleicht bekomm ich wenigstens ein paar punkte :wink:

 

[Bleiglanz]

das mit der rekursion ist natürlich totaler Quatch (weil an dieser Stelle unnötig und es mit grossen zahlen nicht funktioniert)

ausserdem bricht das zeug bei übergabe eines x<0 nicht ab

aber das ist nur schlecht programmiert und hat nichts mit Objektorientierung zu tun?!

meine Idee: die Methode benötigt keinerlei Infos aus der umgebenden Klasse, sollte also "static" sein!

Übrigens ist das so windig (und eigentlich rein prozedural), dass man von einem "objektorientierten Stil" gar nicht reden kann

 

[Sky]

Also, wenn Du es OO lösen wolltest, dann mußtest Du eine Klasse haben, die eine Zahl repräsentiert und eine Methode multiply mit einem Parameter. Der Aufruf von mulitply würde dann den Startwert in deiner Klasse mit dem Parameter muliplizieren.

Das ganze könnte ungefähr so aussehen:

public class myClass {
private int value;
  public myClass( int i) {
    this.value = i;
  }
  
  public void multiply( int x ) {
    this.value *= x;
  }

  public int getCurrentValue() {
    return value;
  }
}

 

[mr1st]

wieso nicht

public static int multiply(int x, int y)
{ return (x*y);
}

 

[Benutzername]

wieso nicht

public static int multiply(int x, int y)
{ return (x*y);
}

genau, so hätt ichs auch gemacht, vermutlich sollte man sich sowas in der art überlegen
public & static !!