[Erledigt] Unittests mit private Methoden

coola · 31.01.2011, 11:17

Hallo

Ich kämpfe mal wieder damit herum, wie man in OOP etwas richtig programmiert.

Ich habe folgende Klasse:

PHP-Code:

  class X

{

   private function _algorithmus()

   {

      // hier passiert irgendetwas kompliziertes

   }

   public function m()

   {

      // irgendwelche Befehle

 
      // dann kommt ein komplizierter Algorithmus:

      $this->_algorithmus(); // in einer eigenen Methode ausgelagert

 
      // dann kommen wieder irgendwelche Befehle

   }

}

Hierin gibt es also die public Methode 'm()'. In dieser Methode stünde ein komplizierter Algorithmus. Um diesen jedoch extra gründlich mittels UnitTests prüfen zu können, ist er in einer eigenen Methode '_algorithmus()' ausgelagert. Diese darf dann nur private sein.

Allerdings kann ich von außen nicht auf eine private Methode zugreifen. Ich kann also keine Unittests durchführen.

Also müsste ich die private Methode entweder public machen oder eine zusätzliche public-Methode definieren, z. B.:

PHP-Code:

  class X

{

   private function _algorithmus()

   {

      // hier passiert irgendetwas kompliziertes

   }

   public function testAlgorithmus() // diese Methode ist dann nur für die Unit-Tests

   {

      // macht nur eines: es ruft die private Methode _algorithmus auf:

      $this->_algorithmus();

   }

   public function m()

   {

      // irgendwelche Befehle

 
      // dann kommt ein komplizierter Algorithmus:

      $this->_algorithmus();

 
      // dann kommen wieder irgendwelche Befehle

   }

}

Allerdings finde ich das auch nicht so toll. Gibt es vielleicht noch eine andere Lösung??

Flor1an · 31.01.2011, 12:59

Eigentlich sollten private Methoden ja "versteckt" sein. Und deshalb sollten auch nur öffentliche Methoden getestet werden. Wenn du nämlich mal an deiner Implementierung was verändern willst bleibt vlt. das Verhalten nach außen gleich, alle Unittests würden also weiter durchlaufen, aber die internen Funktionen funktionieren möglicherweise anders. In deinem Fall würde dann doch der Unittest brechen.

Falls du es trotzdem brauchst, was bei komplizierten Methoden vlt. doch sinnvoll sein könnte, dann probiers doch mit Reflection.

PHP-Code:

  protected static function getMethod($name) {
  $class = new ReflectionClass('MyClass');
  $method = $class->getMethod($name);
  $method->setAccessible(true);
  return $method;
}

public function testFoo() {
  $foo = self::getMethod('foo');
  $obj = new MyClass();
  $foo->invokeArgs($obj, array(...));
  ...
}

coola · 31.01.2011, 13:19

Bin eigentlich kein Fan von Reflection, ehrlich gesagt. Reflection sollte man meiner Meinung nach eher nur zu Dokumentationszwecken verwenden.

Flor1an · 31.01.2011, 13:25

Reflections sind genau für sowas gut. Unittests gehören ja nicht zur Anwendung selber. Und zum testen sind Reflections doch wie geschaffen dafür.

coola · 31.01.2011, 13:39

OHHHH!!!!! Jetzt wird mir erst bewusst, was du da eigentlich machst. Du greifst dann von außen auf eine private-deklarierte Methode zu? Ich kann das leider nicht ausprobieren, weil ich nur 5.3.1 habe. Aber das ist ja echt arg! Da könnte ich ja rein theoretisch via Reflection auf jede beliebige private-Methode zugreifen. AU WEH! Was haben sich die PHP-Programmierer denn dabei gedacht.?.?.?

Flor1an · 31.01.2011, 13:44

Das hat nichts mit PHP zutun. Das gibts auch in anderen Sprachen (Java zum Beispiel). Reflections sind ein mächtiges Werkzeug grade um etwas zu debuggen oder zu testen. In der Anwendung selber solltest du eigentlich drauf verzichten und eben den normalen OOP Paradigmen gerecht werden. Aber eben genau so Dinge die nicht zur Anwendung gehören und komplett von außen (einmalig, also Debugging, Unittesting) getan werden ist das nen super Werkzeug.

Und funktionieren Reflections nicht ab PHP 5 ?

nikosch · 31.01.2011, 14:01

Ansonsten kann man die Klasse auch einfach noch einmal ableiten. Deshalb solltest DU wo möglich auch immer protected statt private verwenden.

Zitat:

AU WEH! Was haben sich die PHP-Programmierer denn dabei gedacht.?.?.?

Member visibility dient keinen Sicherheitsaspekten. Von daher ist da nichts schlimmes dran.

coola · 31.01.2011, 14:56

setAccessible gibt es anscheinend erst ab 5.3.2

Na OK. Also mit Reflections. Danke

Flor1an · 31.01.2011, 15:02

Ab 5.3.0 steht doch im Manual oder?

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
[Erledigt] Objekt mit statischen Methoden	hts	JavaScript, Ajax und mehr	3	18.12.2010 21:07
VisualSVN Server und PHP5	Frederic_S	Server, Hosting und Workstations	12	18.11.2010 17:50
Klassendesign	SilentSight	PHP Tipps 2010	13	24.09.2010 08:55
PHP Unittesting	Flor1an	PHP-Fortgeschrittene	42	09.06.2010 16:34
[Erledigt] Werte aus private function werden in public function nciht übernommen	litterauspirna	PHP Tipps 2010	3	03.02.2010 11:31
PEAR::Auth - MVC Implementierung	dergestalter	PHP-Fortgeschrittene	6	06.06.2009 11:49
Kindklasse: Methoden über _call() ergänzen?	ciss	PHP Tipps 2008	2	29.12.2008 18:08
[Erledigt] Vererbung - Zugriff auf Elternmethoden nicht möglich	unimow	PHP Tipps 2008	5	03.11.2008 18:53
[Erledigt] Methoden auf Objekt aufrufen (und fangen), die nicht existieren	tsisson	PHP Tipps 2008	2	07.10.2008 18:19
Referenzen Private Methoden.	notyyy	PHP-Fortgeschrittene	5	29.05.2008 09:40
UML - Abstrakte und vererbte Methoden zwingend anzugeben?	MaMo-Net	Off-Topic Diskussionen	3	17.03.2008 06:04
public, protected, private Funktionen	ecomeback	PHP Tipps 2007	10	21.08.2007 02:21
Code Syntax Highlighting	imported_Lumio	PHP-Fortgeschrittene	9	16.08.2005 11:06
PHPBB - Private	Stümper	PHP Tipps 2005-2	7	22.06.2005 14:49

31.01.2011, 11:17
coola Gesperrt Registriert seit: 03.12.2010 Beiträge: 860 PHP-Kenntnisse: Anfänger	[Erledigt] Unittests mit private Methoden Hallo Ich kämpfe mal wieder damit herum, wie man in OOP etwas richtig programmiert. Ich habe folgende Klasse: PHP-Code: `class X { private function _algorithmus() { // hier passiert irgendetwas kompliziertes } public function m() { // irgendwelche Befehle // dann kommt ein komplizierter Algorithmus: $this->_algorithmus(); // in einer eigenen Methode ausgelagert // dann kommen wieder irgendwelche Befehle } }` Hierin gibt es also die public Methode 'm()'. In dieser Methode stünde ein komplizierter Algorithmus. Um diesen jedoch extra gründlich mittels UnitTests prüfen zu können, ist er in einer eigenen Methode '_algorithmus()' ausgelagert. Diese darf dann nur private sein. Allerdings kann ich von außen nicht auf eine private Methode zugreifen. Ich kann also keine Unittests durchführen. Also müsste ich die private Methode entweder public machen oder eine zusätzliche public-Methode definieren, z. B.: PHP-Code: class X { private function _algorithmus() { // hier passiert irgendetwas kompliziertes } public function testAlgorithmus() // diese Methode ist dann nur für die Unit-Tests { // macht nur eines: es ruft die private Methode _algorithmus auf: $this->_algorithmus(); } public function m() { // irgendwelche Befehle // dann kommt ein komplizierter Algorithmus: $this->_algorithmus(); // dann kommen wieder irgendwelche Befehle } } Allerdings finde ich das auch nicht so toll. Gibt es vielleicht noch eine andere Lösung??


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

31.01.2011, 12:59
Flor1an da schreibt der ElePHPant Registriert seit: 18.06.2008 Beiträge: 8.903 PHP-Kenntnisse: Fortgeschritten	Eigentlich sollten private Methoden ja "versteckt" sein. Und deshalb sollten auch nur öffentliche Methoden getestet werden. Wenn du nämlich mal an deiner Implementierung was verändern willst bleibt vlt. das Verhalten nach außen gleich, alle Unittests würden also weiter durchlaufen, aber die internen Funktionen funktionieren möglicherweise anders. In deinem Fall würde dann doch der Unittest brechen. Falls du es trotzdem brauchst, was bei komplizierten Methoden vlt. doch sinnvoll sein könnte, dann probiers doch mit Reflection. PHP-Code: `protected static function getMethod($name) { $class = new ReflectionClass('MyClass'); $method = $class->getMethod($name); $method->setAccessible(true); return $method; } public function testFoo() { $foo = self::getMethod('foo'); $obj = new MyClass(); $foo->invokeArgs($obj, array(...)); ... }`

31.01.2011, 13:19
coola Gesperrt Registriert seit: 03.12.2010 Beiträge: 860 PHP-Kenntnisse: Anfänger	Bin eigentlich kein Fan von Reflection, ehrlich gesagt. Reflection sollte man meiner Meinung nach eher nur zu Dokumentationszwecken verwenden.

31.01.2011, 13:25
Flor1an da schreibt der ElePHPant Registriert seit: 18.06.2008 Beiträge: 8.903 PHP-Kenntnisse: Fortgeschritten	Reflections sind genau für sowas gut. Unittests gehören ja nicht zur Anwendung selber. Und zum testen sind Reflections doch wie geschaffen dafür.

31.01.2011, 13:39
coola Gesperrt Registriert seit: 03.12.2010 Beiträge: 860 PHP-Kenntnisse: Anfänger	OHHHH!!!!! Jetzt wird mir erst bewusst, was du da eigentlich machst. Du greifst dann von außen auf eine private-deklarierte Methode zu? Ich kann das leider nicht ausprobieren, weil ich nur 5.3.1 habe. Aber das ist ja echt arg! Da könnte ich ja rein theoretisch via Reflection auf jede beliebige private-Methode zugreifen. AU WEH! Was haben sich die PHP-Programmierer denn dabei gedacht.?.?.?

31.01.2011, 13:44
Flor1an da schreibt der ElePHPant Registriert seit: 18.06.2008 Beiträge: 8.903 PHP-Kenntnisse: Fortgeschritten	Das hat nichts mit PHP zutun. Das gibts auch in anderen Sprachen (Java zum Beispiel). Reflections sind ein mächtiges Werkzeug grade um etwas zu debuggen oder zu testen. In der Anwendung selber solltest du eigentlich drauf verzichten und eben den normalen OOP Paradigmen gerecht werden. Aber eben genau so Dinge die nicht zur Anwendung gehören und komplett von außen (einmalig, also Debugging, Unittesting) getan werden ist das nen super Werkzeug. Und funktionieren Reflections nicht ab PHP 5 ?

31.01.2011, 14:56
coola Gesperrt Registriert seit: 03.12.2010 Beiträge: 860 PHP-Kenntnisse: Anfänger	setAccessible gibt es anscheinend erst ab 5.3.2 Na OK. Also mit Reflections. Danke

31.01.2011, 15:02
Flor1an da schreibt der ElePHPant Registriert seit: 18.06.2008 Beiträge: 8.903 PHP-Kenntnisse: Fortgeschritten	Ab 5.3.0 steht doch im Manual oder?