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

Object-Oriented PHP Basics - Mid-course challenge

Where did I go wrong? This is Challenge task 5 of 5.

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."

<?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 an " . $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();

?>

3 Answers

Andrew Shook
Andrew Shook
31,709 Points

I'm pretty sure you messed up the concatenation near the end of the getInfo method.

    "record weight is ." $this->record_weight;

you put the period inside the quotation mark instead of outside of it. Also, I think you need to add on the period to the end of the sentence.

 "record weight is " . $this->record_weight . "." ;

This worked for me

<?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 an " . $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();
?>

Yup that worked.

Thanks

jamesdei
jamesdei
6,738 Points

There was an other mistake in your code. In your getInfo method, you wrote "$this->$flavor" instead of "$this->flavor".