Det mest OOP'ske?

Erhm... Det er den der der er mest åpske... Det du har gjort i eksempel 2 er blot omskrivning af koden til noget mere let-læseligt...

Hvis du skal noget der udnytter bare lidt af den objekt-orienteret tankegang.

Og så skal du da lige tænke navnekonventioner ind i dine metodenavne. Kamelnavne (camelcasing) og underscores skal ikke mikses, med mindre der er en logisk idé med det. Har omdøbt dine metoder til noget der er mere logisk.

Eks:

class MyObj {
	// tidl.: addBlocked_ip()
	function addBlockedIP($input) {
		$ip_input = explode(";", $ip_input);
		
		$this->blocked = array_merge($this->blocked,$ip_input);
		$this->blocked = array_unique($this->blocked);
	}
	// tidl.: print_Blocked_ip()
	function printCommands() {
		foreach($this->blocked as $ip) {
			print "iptables -A INPUT -s ".$ip." -j DROP\\n";
		}
	}
}

$this->blocked skal vel oprettes som var i starten som

var $this->blocked;

eller? Jeg har endnu ikke været så meget inde i OOP, men er det denne måde det gøre på eller referer $this->blocked automatisk til $blocked?

EDIT: Ved ikke lige om jeg beholder printCommands(), da jeg laver en Generator til IPtables og derfor laver en speciel struktur for hvad der kommer, men det kan være det skal implamenteres direkte i samme funktion.

Eks:

$global = 'global';

class MyClass {
	var $a = 'public';
	private $b = 'private';
	protected $c = 'protected';

	function testA() {
		print $this->a;
	}

	function testB() {
		print $this->b;
	}
	
	function testC() {
		print $this->c;
	}
	
	function testGlobal() {
		global $global;
		
		print $global;
	}
}

$mc = new MyClass();
print $mc->a; // OK
print $mc->b; // Fejl
print $mc->c; // Fejl

$mc->testA(); // OK
$mc->testB(); // OK
$mc->testB(); // OK

$mc->testGlobal(); // OK

Dette skulle gerne forklare tilgængeligheden for variabler.

Ok, det siger jeg mange tak for. Så har jeg lidt og arbejde videre med.