php - PHP OOP fejl... - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

<< < 1 2 > >>

Bruger #11328 @ 16.05.08 23:57

Hej allesammen!
Jeg er gået lidt ned i OOP når det gælder PHP.
Jeg er dog rendt ind i et problem: min kode virker ikke..

Kan virkelig ikke spotte fejlen, men håber en af jer måske kan..
Kode:

Kode

<?php
    class Garage
    {
        var $carNames = array();
        var $carYears = array();
        var $carEngines = array();
        var $errors = array();

        // Konstruktør
        function Garage($newName = "", $newYear = "", $newEngine = "")
        {
            if(empty($newName) || empty($newYear) || empty($newEngine))
                return;
            else
            {
                $this->carNames[] = $newName;
                $this->carYears[] = $newYear;
                $this->carEngines[] = $newEngine;
            }
        }

        function addCar($carName, $carYear, $carEngine)
        {
            $this->carNames[] = $carName;
            $this->carYears[] = $carYear;
            $this->carEngines[] = $carEngine;
        }

        function getCars($options = 1)
        {
            switch($options)
            {
                case 1:
                    return array("allNames" => $this->carNames, "allYears" => $this->carYears, "allEngines" => $this->carEngines);
                break;
                case 2:
                    return $this->carNames;
                break;
                case 3:
                    return $this->carYears;
                break;
                case 4:
                    return $this->carEngines;
                break;
                default:
                    $this->errors[] = "options can't be: " . $options;
                    return array("allNames" => $this->carNames, "allYears" => $this->carYears, "allEngines" => $this->carEngines);
                break;
            }
        }

        function getErrors($lastError = false)
        {
            if($lastError == false)
                return $this->errors[];
            elseif($lastError == true)
            {
                $errorCount = count($this->errors);
                return $this->errors[$errorCount-1];
            }
            else
            {
                $this->errors[] = "lastError is a boolean, i can't be: " . $lastError;
                return $this->errors[];
            }
        }

        function deleteCar($carNumber, $emptyGarage = false)
        {
            if($emptyGarage == false)
            {
                if(empty($carNumber))
                {
                    $this->errors[] = "No identifier was given";
                    return;
                }
                else
                {
                    if(empty($this->carNames[$carNumber]) || empty($this->carEngines[$carNumber]) || empty($this->carYears[$carNumber]))
                    {
                        $this->errors[] = "No car with such data is in the garage";
                        return;
                    }
                    else
                    {
                        unset($this->carNames[$carNumber], $this->carEngines[$carNumber], $this->carYears[$carNumber]);
                        return;
                    }
                }
            }
            elseif($emptyGarage == true)
            {
                unset($this->carNames, $this->carEngines, $this->carYears);
                return;
            }
            else
            {
                $this->errors[] = "Second parameter is a boolean, it can't hold: " . $emptyGarage;
                return;
            }
        }
    }

    $garage = new Garage("BMW Z Serie Z4","2005","231");
    $allcars = $garage->getCars();
    echo count($allcars["allNames"]);
?>

Ved godt at det er en større samling, men det er fordi jeg virkelig har prøvet at skrive den flotteste syntax jeg har prøvet...

Vil egentlig også gerne have noget feedback på min opbygning af klassen, og lidt om min syntax.
Håber i kan hjælpe!
~Jakob

Indlæg senest redigeret d. 17.05.2008 00:01 af Bruger #11328

Bruger #6559 @ 17.05.08 00:16

843

Jeg har ændret din konstruktør fra function Garage til
public void __construct

Her er den ændrede kode

Kode

<?php
    class Garage
    {
        var $carNames = array();
        var $carYears = array();
        var $carEngines = array();
        var $errors = array();

        // Konstruktør
        public function __construct($newName = "", $newYear = "", $newEngine = "")
        {
            if(empty($newName) || empty($newYear) || empty($newEngine))
                return;
            else
            {                
				array_push($this->carNames, $newName);
				array_push($this->carYears, $newYear);
                array_push($this->carEngines, $newEngine);                
            }
        }

        function addCar($carName, $carYear, $carEngine)
        {
            $this->carNames[] = $carName;
            $this->carYears[] = $carYear;
            $this->carEngines[] = $carEngine;
        }

        function getCars($options = 1)
        {
            switch($options)
            {
                case 1:
                    return array("allNames" => $this->carNames, "allYears" => $this->carYears, "allEngines" => $this->carEngines);
                break;
                case 2:
                    return $this->carNames;
                break;
                case 3:
                    return $this->carYears;
                break;
                case 4:
                    return $this->carEngines;
                break;
                default:
                    $this->errors[] = "options can't be: " . $options;
                    return array("allNames" => $this->carNames, "allYears" => $this->carYears, "allEngines" => $this->carEngines);
                break;
            }
        }

        function getErrors($lastError = false)
        {
            if($lastError == false)
                return $this->errors[];
            elseif($lastError == true)
            {
                $errorCount = count($this->errors);
                return $this->errors[$errorCount-1];
            }
            else
            {
                $this->errors[] = "lastError is a boolean, i can't be: " . $lastError;
                return $this->errors[];
            }
        }

        function deleteCar($carNumber, $emptyGarage = false)
        {
            if($emptyGarage == false)
            {
                if(empty($carNumber))
                {
                    $this->errors[] = "No identifier was given";
                    return;
                }
                else
                {
                    if(empty($this->carNames[$carNumber]) || empty($this->carEngines[$carNumber]) || empty($this->carYears[$carNumber]))
                    {
                        $this->errors[] = "No car with such data is in the garage";
                        return;
                    }
                    else
                    {
                        unset($this->carNames[$carNumber], $this->carEngines[$carNumber], $this->carYears[$carNumber]);
                        return;
                    }
                }
            }
            elseif($emptyGarage == true)
            {
                unset($this->carNames, $this->carEngines, $this->carYears);
                return;
            }
            else
            {
                $this->errors[] = "Second parameter is a boolean, it can't hold: " . $emptyGarage;
                return;
            }
        }
    }

    $garage = new Garage("BMW Z Serie Z4","2005","231");
    $allcars = $garage->getCars();
    echo count($allcars["allNames"]);
?>

Edit: Har lavet en lille ændring mere.

Indlæg senest redigeret d. 17.05.2008 00:24 af Bruger #6559

Bruger #11328 @ 17.05.08 00:27

1.323

Koden virker bare desværre ikke..

(Og hvorfor den avancerede funktion? og hvorfor bruger du array_push??)

Bruger #6559 @ 17.05.08 00:28

843

Du skriver koden virker ikke? Hvad virker ikke? Hvis du lige ville udspecificere hvad der ikke virker, så er det også nemmere at hhjælpe

Bruger #11328 @ 17.05.08 00:31

1.323

Jeg kan desværre ikke sige hvad der ikke virker da en PHP side jo (som bekendt) går fuldstændigt white-out hvis sntaxen er forkert...

Bruger #10216 @ 17.05.08 00:31

4.283

Der er fejl i Garage::getErrors(). Du kan ikke returnere "$errors[]". Det er det samme som at du kan heller ikke skrive "$t = $errors[];" hvilket heller ikke giver nogen mening og derfor ikke muligt.
Kan ikke lure andre fejl, uddyb.

Garage::Garage() er egentlig ikke særlig pænt for min smag og det kræver at man ændrer to steder, hvis man pludselig ændrer navnet på klassen.

function __construct() {} er en "magic function" og kaldes på samme vis som Garage::Garage()

"var" er udgået og erstattet med "public", "private" og "static".

Koden er sådan set ganske pæn og overskuelig, alt hvad du mangler er kommentarer.

Kode

<?php
    class Garage
    {
        public $carNames = array();
        
        ...

        /**
         * Constructor
         *
         * @param string $newName
         * @param string $newYear
         * @param string $newEngine
         */
        function __construct($newName = "", $newYear = "", $newEngine = "")
        {
        }

        /**
         * Adds new car data to object
         *
         * @param string $carName
         * @param string $carYear
         * @param string $carEngine
         * @return void
         */
        function addCar($carName, $carYear, $carEngine)
        {
        }

        /**
         * Returns ...
         *
         * @param int $options What to specify?
         * @return array|string
         */
        function getCars($options = 1)
        {
        }
        
        ...
    }

    $garage = new Garage("BMW Z Serie Z4","2005","231");
    $allcars = $garage->getCars();
    echo count($allcars["allNames"]);
?>

For at være avanceret kan du jo lave "Cars" om til klasser også.
så kan lege med ting som:

Kode

$car = new Car("...");
$car->isBroken = true;
$garage->addCar($car);

Bruger #6559 @ 17.05.08 00:37

843

Jeg kan desværre ikke sige hvad der ikke virker da en PHP side jo (som bekendt) går fuldstændigt white-out hvis sntaxen er forkert...

Ah, det passer ikke helt. Når du laver debug kunne du lave følgende til sidst i scriptet:

Kode

print "Objektet er initialiseret";

Bruger #11328 @ 17.05.08 00:39

1.323

Super!
Det virker:

Kode

<?php
    class Garage
    {
        public $carNames = array();
        public $carYears = array();
        public $carEngines = array();
        public $errors = array();

        // Konstruktør
        function __construct($newName = "", $newYear = "", $newEngine = "")
        {
            if(empty($newName) || empty($newYear) || empty($newEngine))
                return;
            else
            {
                $this->carNames[] = $newName;
                $this->carYears[] = $newYear;
                $this->carEngines[] = $newEngine;
            }
        }

        function addCar($carName, $carYear, $carEngine)
        {
            $this->carNames[] = $carName;
            $this->carYears[] = $carYear;
            $this->carEngines[] = $carEngine;
        }

        function getCars($options = 1)
        {
            switch($options)
            {
                case 1:
                    return array("allNames" => $this->carNames, "allYears" => $this->carYears, "allEngines" => $this->carEngines);
                break;
                case 2:
                    return $this->carNames;
                break;
                case 3:
                    return $this->carYears;
                break;
                case 4:
                    return $this->carEngines;
                break;
                default:
                    $this->errors[] = "options can't be: " . $options;
                    return array("allNames" => $this->carNames, "allYears" => $this->carYears, "allEngines" => $this->carEngines);
                break;
            }
        }

        function getErrors($lastError = false)
        {
            if($lastError == false)
                return $this->errors;
            elseif($lastError == true)
            {
                $errorCount = count($this->errors);
                return $this->errors[$errorCount-1];
            }
            else
            {
                $this->errors[] = "lastError is a boolean, i can't be: " . $lastError;
                return $this->errors;
            }
        }

        function deleteCar($carNumber, $emptyGarage = false)
        {
            if($emptyGarage == false)
            {
                if(empty($carNumber))
                {
                    $this->errors[] = "No identifier was given";
                    return;
                }
                else
                {
                    if(empty($this->carNames[$carNumber]) || empty($this->carEngines[$carNumber]) || empty($this->carYears[$carNumber]))
                    {
                        $this->errors[] = "No car with such data is in the garage";
                        return;
                    }
                    else
                    {
                        unset($this->carNames[$carNumber], $this->carEngines[$carNumber], $this->carYears[$carNumber]);
                        return;
                    }
                }
            }
            elseif($emptyGarage == true)
            {
                unset($this->carNames, $this->carEngines, $this->carYears);
                return;
            }
            else
            {
                $this->errors[] = "Second parameter is a boolean, it can't hold: " . $emptyGarage;
                return;
            }
        }
    }

    $garage = new Garage("BMW Z Serie Z4","2005","231");
    $allcars = $garage->getCars();
    echo $allcars["allNames"][0];
?>

Nogle syntaxfejl?

(og tak for ideen med det med at lave cars om til en klasse... Det vil jeg prøve!

(BTW: Hvorfor skriver du: @PARAM før du beskriver en variabel i kommentarene?)

Indlæg senest redigeret d. 17.05.2008 00:41 af Bruger #11328

Bruger #11328 @ 17.05.08 00:42

1.323

Jeg kan desværre ikke sige hvad der ikke virker da en PHP side jo (som bekendt) går fuldstændigt white-out hvis sntaxen er forkert...

Ah, det passer ikke helt. Når du laver debug kunne du lave følgende til sidst i scriptet:

Kode
print "Objektet er initialiseret";

Debug? Hvad mener du med at "lave debug"... Er det et PHP-mode? kender self. godt ordet, men lyder da lidt spøjst..

Bruger #6559 @ 17.05.08 00:48

843

Når du debugger så kigger du jo efter fejl. Så kan man altid lave et print "noget tekst"; for at se om funktionen udskriver det forventede

Bruger #11328 @ 17.05.08 00:54

1.323

Nej... Hvis der bare er én fejl i en PHP kode virker HELE scriptet ikke..

Nu taler jeg syntax fejl, ikke fx regnefejl

<< < 1 2 > >>

PHP OOP fejl...

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler