Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

PHP Object-Oriented PHP Basics (Retired) Properties and Methods Mid-Course Challenge

Posted by William J. Terrell
William J. Terrell
17,403 Points

Objects and Properties Code Challenge - Part 5 of 5

I seem to be a bit "caught up" on something...

The question is as follows:

Create a method on Fish named getInfo that takes no parameters and returns a string that includes the common_name, flavor, and record_weight for the fish. When called on $bass, getInfo might return "A Largemouth Bass is an Excellent flavored fish. The world record weight is 22 pounds 5 ounces."

Would anyone mind having a look?

Thanks!

fish.php 
<?php

class Fish {
  public $common_name;
  public $flavor;
  public $record_weight;

  function __construct($name, $flavor, $record) {

    $this->common_name = $name;
    $this->flavor = $flavor;
    $this->record_weight = $record;

    public function getInfo() {
     return "A " .
            $this->name  .
            " is a(n)" .
            $this->flavor . 
            " flavored fish. " .
            "The world record weight is " .
            $this->record  .
            ".";
    }

  }

}

$bass = new Fish("Largemouth Bass", "Excellent", "22 pounds 5 ounces");
echo $bass->getInfo();

?>

1 Answer

Jaro Schwab
Jaro Schwab
8,957 Points

Hey.

You need to return $this->common_name and $this->record_weight instead of just $this->name and $this->record. So your return statement should look like this:

            return "A " .
            $this->common_name  .
            " is a(n)" .
            $this->flavor . 
            " flavored fish. " .
            "The world record weight is " .
            $this->record_weight .
            ".";

EDIT: And you need to close your Construtor properly. So your Code looks like this:

<?php

class Fish {

  public $common_name;
  public $flavor;
  public $record_weight;

   function __construct($name, $flavor, $record) {

    $this->common_name = $name;
    $this->flavor = $flavor;
    $this->record_weight = $record;

   } // Close the constructor!

   public function getInfo() {
     return "A " .
            $this->common_name  .
            " is a(n)" .
            $this->flavor . 
            " flavored fish. " .
            "The world record weight is " .
            $this->record_weight  .
            ".";
    }
}

$bass = new Fish("Largemouth Bass", "Excellent", "22 pounds 5 ounces");

?>
J J
J J
15,775 Points

It keeps saying try again.

William J. Terrell
William J. Terrell
17,403 Points

Thanks! This code got it!

<?php

class Fish {
  public $common_name;
  public $flavor;
  public $record_weight;

  function __construct($name, $flavor, $record) {

    $this->common_name = $name;
    $this->flavor = $flavor;
    $this->record_weight = $record;

  }

  public function getInfo() {
     return "A " .
            $this->common_name  .
            " is a(n)" .
            $this->flavor . 
            " flavored fish. " .
            "The world record weight is " .
            $this->record_weight  .
            ".";
    }

}

$bass = new Fish("Largemouth Bass", "Excellent", "22 pounds 5 ounces");
echo $bass->getInfo();

?>

Having the two different names ($name and $common_name) tripped me up a bit.

Why would we not want the names to be the same, though? That constructor function is specific to the class "Fish", so it's not like we would use that constructor function for anything else...

Thanks!