Drukowana wersja tematu

Kliknij tu, aby zobaczyć temat w orginalnym formacie

Forum PHP.pl _ Object-oriented programming _ Ustawienie argumentu metody jako WYMAGANY.

Napisany przez: Grzesiek1 1.11.2016, 13:02:50

Co prawda rozwiązanie mojego problemu znalazłem ale tylko w PHP 7. Chciałbym jednak żeby kod działał także na wcześniejszych wersjach.
W php7 wykonałem to tak:

[PHP] pobierz, plaintext 
<?php
class test_class
{
	function arg(int $var, string $var2, string $var3)
	{
		if($var && $var2 && $var3)
		{
			return 'OK, function execute correct';
		}
		else
		{
			return 'ERROR, if variable is empty or not exist I not want execute method!';
		}	 
	}
}
$object = new test_class();
$var = 50;
$var2 = 50;	
//$var3 = 50;
 
http://www.php.net/echo $object->arg($var,$var2,$var3);
?>
[PHP] pobierz, plaintext

Po prostu jeśli $var3 jest pusta to nie jest ona zmienną typu string i kod się nie wykonuje:

[PHP] pobierz, plaintext 
Fatal error: Uncaught TypeError: Argument 3 passed to test_class::arg() must be of the type string, null given, called in D:\xampp\htdocs\argument.php on line 26 and http://www.php.net/defined in D:\xampp\htdocs\argument.php:5 Stack trace: #0 D:\xampp\htdocs\argument.php(26): test_class->arg(50, '50', NULL) #1 {main} thrown in D:\xampp\htdocs\argument.php on line 5
[PHP] pobierz, plaintext

Ewentualnie wymyśliłem rozwiązanie mojego problemu dla wcześniejszej wersji php ale bardzo nie dobrze to wygląda:

[PHP] pobierz, plaintext 
<?php
class test_class
{
	function arg($var, $var2, $var3)
	{
		// My temporary solution
		$list = http://www.php.net/func_get_args();
		$count = 0;
		foreach ($list as $value) {
			if($value)
			$count++;
		}
		if($count != http://www.php.net/func_num_args()){
			return false;
		}
		// My temporary solution end
 
		if($var && $var2 && $var3)
		{
			return 'OK, function execute correct';
		}
		else
		{
			return 'ERROR, if variable is empty or not exist I not want execute method!';
		}	 
	}
}
$object = new test_class();
$var = 50;
$var2 = 50;
//$var3 = 50;
 
http://www.php.net/echo $object->arg($var,$var2,$var3);
?>
[PHP] pobierz, plaintext

Tworzona jest po prostu lista podanych argumentów z podanymi wartościami, w pętli sprawdzane są wartości czy nie są NULL, zliczane są wszystkie i porównywane z ilością wymaganych argumentów ( func_num_args() ). Jeśli liczba się nie zgadza przerywana jest metoda ( return false ).
Jednak rozwiązanie ma taki minus, że trzeba to wklejać na początku każdej funkcji no i nie wygląda zbyt dobrze z mojego punktu widzenia takie rozwiązanie.

Ktoś ma jakieś pomysły na lepsze rozwiązanie tego problemu?
Nie mogę nawet tego kodu wrzucić do osobnej funkcji i tylko w każdej wywoływać, bo wtedy func_*_args() nie spełnia swojego zadania.

Napisany przez: KsaR 1.11.2016, 13:09:24

Ustaw domyślną wartość na null

[PHP] pobierz, plaintext 
<?php
 
function arg($arg = null, $arg2 = null)
{
    if (!http://www.php.net/isset($arg, $arg2)) {
        return 'error';
    }
}
 
http://www.php.net/echo arg(0, 0)."\n";
http://www.php.net/echo arg(0)."\n";
http://www.php.net/echo arg();
[PHP] pobierz, plaintext

Demo: https://3v4l.org/tu0Fh

Wtedy też możesz rzucać http://php.net/manual/en/language.exceptions.php zamiast "error".

Napisany przez: Pyton_000 1.11.2016, 13:12:47

E...

[PHP] pobierz, plaintext 
<?php
class test_class
{
	function arg($var, $var2, $var3)
	{
		if(!http://www.php.net/isset($var, $var2, $var3)) {
		return 'NOT';
		}
		return 'OK';
	}
}
$object = new test_class();
$var = 50;
$var2 = 50;
//$var3 = 50;
 
http://www.php.net/echo $object->arg($var,$var2,$var3);
[PHP] pobierz, plaintext

Napisany przez: Grzesiek1 1.11.2016, 13:22:17

Cytat(KsaR @ 1.11.2016, 14:09:24 )

Ustaw domyślną wartość na null

O to chodzi że argument nie może być NULL, jeśli jest NULL chcę aby metoda się właśnie nie wykonywała.

Cytat(Pyton_000 @ 1.11.2016, 14:12:47 )

E...

[PHP] pobierz, plaintext 
<?php
class test_class
{
	function arg($var, $var2, $var3)
	{
		if(!http://www.php.net/isset($var, $var2, $var3)) {
		return 'NOT';
		}
		return 'OK';
	}
}
$object = new test_class();
$var = 50;
$var2 = 50;
//$var3 = 50;
 
http://www.php.net/echo $object->arg($var,$var2,$var3);
[PHP] pobierz, plaintext

No tak to też się sprawdzi, ale to rozwiązanie jest jeszcze mniej wygodne od mojego. Teraz jak mam 10 funkcji i w każdej po 10 argumentów robi się kłopot.

Myślałem bardziej o zastosowaniu może jakiegoś wyjątku w przypadku, gdy któryś z argumentów jest pusty.

EDIT:
Ewentualnie jeszcze w ten sposób można by to rozwiązać ale to jest strasznie zawiłe rozwiązanie

[PHP] pobierz, plaintext 
<?php
 
class test_class
{
    function check($list)
    {
        // My temporary solution
        foreach ($list as $value) {
            if (!$value)
                return true;
        }
        // My temporary solution end
    }
 
    function arg($var, $var2, $var3)
    {
        if ( $this->check( http://www.php.net/func_get_args() ) )
            return false;
 
        if ($var && $var2 && $var3) {
            return 'OK, function execute correct';
        } else {
            return 'ERROR, if variable is empty or not exist I not want execute method!';
        }
    }
}
 
$object = new test_class();
 
$var = 50;
$var2 = 50;
//$var3 = 50;
 
http://www.php.net/echo $object->arg($var, $var2, $var3);
 
?>
[PHP] pobierz, plaintext

Napisany przez: Pyton_000 1.11.2016, 14:30:11

http://php.net/manual/en/reflectionfunctionabstract.getnumberofparameters.php