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 #10216 @ 17.05.08 00:56

4.283

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

Kode
print "Objektet er initialiseret";

Syntaksfejl resulterer i at scriptet ikke afvikles. Derfor hvis display_errors i php.ini på et eller andet tidspunkt er slået fra vil det resultere i en blank side.

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

@param, @return og en masse andre navne er med til at definere en masse ting for funktionen. @param er en indikator for at man tale om en parameter til funktion. @return er indikator for hvad funktionen returnerer.
Alt dette er til hvis man en dag beslutter sig for at udskrive en dokumentation for sin kode. Eksempelvis med phpdoc.
Se mere på: http://manual.phpdoc.org/HTMLSmartyConverter/HandS/phpDocumentor/tutorial_phpDocumentor.pkg.html

Jeg bruger det i forbindelse med Zend Studio. Dette hjælper programmet til at give brugbare oplysninger igennem intellisense.

Bruger #8845 @ 17.05.08 01:07

Hmm... havde ikke lige set næste side

så svarede lige på det samme som GNU

Indlæg senest redigeret d. 17.05.2008 01:09 af Bruger #8845

Bruger #13033 @ 17.05.08 09:34

Hvis der ikke kommer noget output overhovedet skyldes det nok at du har slået error reporting fra. Du kan slå det til med error_reporting(E_ALL); i starten af dit script.
Så får du både errors og notices hvilket er rart. Desuden kan det også give nogle gode hints omkring din kode hvis du så E_STRICT til også - error_reporting(E_ALL | E_STRICT);

Bruger #1151 @ 17.05.08 15:43

421

Vær da lidt hardcore og brug noget assert til at debugge med. Tager lidt tid at sætte op, men koden du ender op med under debugningen, bliver meget overskueligt, http://dk2.php.net/manual/en/function.assert.php

Bruger #11328 @ 17.05.08 16:57

1.323

Jeg bruger en host hvor jeg selv har meget lidt kontrol over selve PHP's opbygning... Kan fakstisk ingengang oprette filer der ikke ender med .txt

Bruger #6788 @ 17.05.08 17:18

118

Men som regel vil der jo være angivet en linje. Og jeg synes efterhånden at jeg rimelig hurtig kan "spore" en syntax fejl ved hjælp af den fejl php giver.

Bruger #11328 @ 18.05.08 13:34

1.323

Så er jeg løbet ind i endnu et problem.
Kan virkelig ikke selv se fejlen:

class.php

Kode

<?php
    error_reporting(E_ALL);


    class Garage
    {
            public $garages = array();
            public $garageNumber = 0;
            public $garageLimit = 20;

        // Konstruktør
        function __construct($garageName = "")
        {
            if(empty($garageName))
            {
                    $this->garages[] = "Garage" . $this->garageNumber;
                    $this->garageNumber++;
                    return;
            }
            else
            {
                $this->garages[] = $garageName;
                return;
            }
        }

        function addGarage($garageName = "")
        {
            if(empty($garageName))
            {
                    $this->garages[] = "Garage" . $this->garageNumber;
                    $this->garageNumber++;
                    return;
            }
            else
            {
                $this->garages[] = $garageName;
                return;
            }
        }

        function getGarageNames($name = "")
        {
            if(empty($name))
                return $this->garages;
            else
            {
                if(in_array($name, $this->garages))
                    return $name;
                else
                    return $this->garages;
            }
        }

        function setGarageLimit($newLimit)
        {
            $this->garageLimit = $newLimit;
            return;
        }

        function getGarageLimit()
        {
            return $this->garageLimit;
        }

        function getCarsInGarage($garageName = "")
        {
            if(empty($garageName))
            {
                return;
            }
            else
            {
                $carsInGarage = array_count_values(Car::getGarage());
                return $carsInGarage[$garageName];
            }
        }

    }

    class Car
    {
        public $carName = array();
        public $carYear = array();
        public $carHK = array();
        public $carGarage = array();

        function __construct($newName = "", $newYear = "", $newHK = "", $newGarage = "")
        {
            if(empty($newName) || empty($newYear) || empty($newHK) || empty($newGarage))
                return;
            else
            {
                if(!is_array(Garage::getGarages($newGarage)))
                {
                $this->carName = $newName;
                $this->carYear = $newYear;
                $this->carHK = $newHK;
                $this->carGarage = $newGarage;
                return;
                }
                else
                    return;
            }
        }

        function getGarage()
        {
            return $this->carGarage;
        }
    }

?>

class_use.php

Kode

<?php
    require_once "class.php";

    $garage = new Garage("Test");
    $car = new Car("Chevrolet Tacuma","2007","150","Test");
    echo $garage->getCarsInGarage("Test");

?>

Men den udskriver ikke noget. Fejlen ligger i getCarsInGarage() men kan ikke selv finde den.

BTW: Egentlig ville jeg gerne have getCarsInGarage til at returnere et array med navne, år HK osv. men kan ikke lige regne ud hvordan det skulle gøres, og har stirret mig blind i manualen... Nogen ideer??

EDIT: er det muligt som i fx C++ at lave en public: så man ikke er nød til at skrive public foran hver variabel..?

Indlæg senest redigeret d. 18.05.2008 13:37 af Bruger #11328

Bruger #8845 @ 20.05.08 14:39

Hey,

Tror ikke din fejl ligger i getCarsInGarage(), men i denne linje: if(!is_array(Garage::getGarages($newGarage)))
Der er nemlig ikke nogen function der hedder getGarages()...

Det med at retunere et array, er vel bare at retunere et...

<< < 1 2 > >>

PHP OOP fejl...

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler