udskriver ikke status

du mangler vist også en løkke?


{
        if(is_string($status))
        {
            $this->status = $status;
            return true;
        }
        else
            return false;
        } // (DENNE LØKKE ER INDSAT)
    }

Den løkke har jeg inde:

public function setStatus($status)
    {
        if(is_string($status))
        {
            $this->status = $status;
            return true;
        }
        else
            return false;
    }

nej ? .. du afslutter ikke din "ELSE LØKKE" .. eller rettere - du afslutter ikke din "public function"-løkke!
Prøv lige at sætte det jeg skrev ind, og se om det ikk virker, :)

Den klasse er god nok det er cronaldo som lige skal tælle { og } efter igen.

Men din kode når du kalder din klasse er der fejl i:

$mit_hold = new Fodboldhold("Liverpool");
echo $mit_hold->getNavn();
$mit_hold->LogOnline();
if($mit_hold->status === true)
    echo $mit_hold->getStatus;
else
    echo $mit_hold->getStatus;

Du havde glemt en _ i din variabel.

Dkfire ->

Det har jeg rettet nu, men der udskriver stadig intet.

$this->status = 1
til:
$this->status == 1

Ahh, når jeg kigger nærmere, så er den vist helt gal flere steder.
Tilbage på tegnebrættet.

Ingen der kan hjælpe? :)

Der er flere ting som er helt galt med din kode.
Når du sammenligner en variabel med en værdi så skal du bruge ==.
= er kun til at tildele en værdi.

Så der din værdi i status, først sætter du den til "Online", dernæst tjekker du om værdien er 1 og på andre steder true, hvilket den bestem ikke er. Du bør nok gøre som fant0mas skriver, start forfra og tænk din kode rigtig godt igennem.

Ellers kunne det være:

<?php
class Fodboldhold
{
    public $navn;
    private $status = "Offline";
 
    public function __construct($navn = null, $status = "Ofline")
    {
        $this->setNavn($navn);
        $this->setStatus($status);
     
    }
 
    public function LogOnline()
    {
        $this->status = true;
    }
 
    public function LogOffline()
    {
        $this->status = false;
    }
 
    public function TjekLog()
    {
        if ($this->status == "Online" ) {
            return true;
        }
        else
            return false;
     
    }
    public function setNavn($navn)
    {
        if(is_string($navn))
        {
            $this->navn = $navn;
            return true;
        }
        else
            return false;
    }

    public function setStatus($status)
    {
        if(is_string($status))
        {
            $this->status = $status;
            return true;
        }
        else
            return false;
    }

    public function getStatus()
    {
        return $this->status;
    }
    public function getNavn()
    {
        return $this->navn;
    }
}
?>

<?php
$mit_hold = new Fodboldhold("Liverpool");
echo $mit_hold->getNavn();
$mit_hold->LogOnline();
if($mit_hold->getStatus === "Online")
    echo "Bruger Online";
else
    echo "Bruger Offline";

?>

dkfire ->

Nu udskriver den "bruger offline" - men passer vel ikke når man først sætter LogOnline();

Prøv lige at rette:
if($mit_hold->getStatus === "Online")
til
if($mit_hold->getStatus() == "Online")

Dkfire det virker nu... Så koden burde være ok nu? - eller har den sine fejl?

Lig et svar.

Der er nok et par ting som kan rettes til, bl.a. vil jeg personlig ikke bruge Online/Offline, men i stedet bruge true/false eller 1/0.
Dernæst bør du ikke have funktionen setStatus, da du så ingen mulighed har for at være sikker på at Status enten er Online eller Offline.

Men arbejd du bare videre fra det du har nu, så vil du sikkert senere selv finde ud af hvad der er smartest.