Using an if statement on a php function

Question

Im trying to make a function that tests whether a triangle has equal sides and then print the answer but my function doesnt work. Any ideas ?

 public function typeOfTriangle()
 {

    if ($this->lengthSideOne == $this->lengthSideTwo == $this->lengthBase)
    {echo 'the triangle is equal'}
 );
 }

soapergem · Accepted Answer · 2014-03-12 15:25:28Z

3

You can't string together == operations. You need to use AND (aka &&).

Like this:

public function typeOfTriangle()
{
    if ( $this->lengthSideOne == $this->lengthSideTwo && $this->lengthSideTwo == $this->lengthBase ) {
        echo 'the triangle is equal';
    }
}

answered Mar 12, 2014 at 15:25

soapergem

10.2k21 gold badges115 silver badges167 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Mazeltov · Accepted Answer · 2014-03-12 15:25:44Z

0

public function typeOfTriangle() {

if ($this->lengthSideOne == $this->lengthSideTwo && $this->lengthSideOne == $this->lengthBase)
{echo 'the triangle is equal';}

); }

answered Mar 12, 2014 at 15:25

Mazeltov

5515 silver badges18 bronze badges

Comments

martskins · Accepted Answer · 2014-03-12 15:26:27Z

0

Try this..

public function typeOfTriangle()
 {

    if ($this->lengthSideOne == $this->lengthSideTwo && $this->lengthSideTwo == $this->lengthBase)
    {echo 'the triangle is equal'}
 );
 }

answered Mar 12, 2014 at 15:26

martskins

2,9604 gold badges30 silver badges53 bronze badges

Comments

Beau · Accepted Answer · 2014-03-12 15:28:42Z

0

public function typeOfTriangle()
 {

    if ( $this->lengthSideOne == $this->lengthSideTwo == $this->lengthBase )
    { echo 'the triangle is equal'; }
    // remove this );
 }

answered Mar 12, 2014 at 15:28

Beau

735 bronze badges

Comments

xero · Accepted Answer · 2014-03-12 15:34:34Z

0

the error is your parenthetical notation.

public function typeOfTriangle() {
 if($this->lengthSideOne == $this->lengthSideTwo && $this->lengthSideTwo == $this->lengthBase) {
     echo 'the triangle is equal';
  }
}

if your using brances the syntax is:

if( ...condition... ) {
   ...do stuff...
}

brace-less conditionals work like this

if(...condition...)
   ...do stuff...

more info here: http://www.php.net/manual/en/control-structures.if.php

edited Mar 12, 2014 at 15:34

answered Mar 12, 2014 at 15:26

xero

4,3391 gold badge27 silver badges40 bronze badges

Comments

Matt The Ninja · Accepted Answer · 2014-03-12 16:58:07Z

-2

You need to pass the variable to the function.

When you call it do this. (each number is a side)

typeOfTriangle(2,2,4)

Then change the start of your function to retrieve this data and assign it to $this like below.

    public function typeOfTriangle($side1, $side2, $side3)
 {

    if ($side1 == $side2 && $side2 == $side3) //this check side 1,2,3 are equal with 2 statements. 
    {echo 'the triangle is equal';}
 }

edited Mar 12, 2014 at 16:58

answered Mar 12, 2014 at 15:26

Matt The Ninja

2,7315 gold badges33 silver badges62 bronze badges

2 Comments

xero Over a year ago

i didn't down vote, but i can say your getting them because you dont' always "need to" pass params to a function. i see the word public in the function declaration, that means this is probably part of a larger class where $this->sideX is being set. in that context you dont need to pass variables to the function to use them.

Paul Blundell Over a year ago

I also wasn't the one to down vote, but as well what xero said your original answer still included the incorrect if statement (which I see you have changed now) and you have a syntax error with ');'

Collectives™ on Stack Overflow

Using an if statement on a php function

6 Answers 6

Comments

Comments

Comments

Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

Comments

Comments

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related