PHP IF statement not taking variable into account!

Question

I have a tabled view in a while loop, where a user can view information on books.

For example, book ISBN, book name, read status...

Basically, when the user sets their 'readstatus' to 'complete' I want that specific table row to become grey! The logic is very straight forward, however I can't get my IF statement to recognise this:

if ($readstatus == 'complete') {
                        echo '<tr class="completed">';
              }

                    else if  ($readstatus != 'complete') {
                        echo '<tr class="reading">';
              }

I'm obviously doing something wrong here, table content to change if the value of 'readstatus' = 'complete', if not, then output is the default

This is not answering the question, but you're equally well doing if( a ) { ... } else { ... }, without the test which should obviously return true. — xtofl
– xtofl, Commented Mar 19, 2010 at 12:54
Do you have your if sdtatement inside of the loop or, as it stated above, right under $readstatus = $_GET['readstatus']; ? — Your Common Sense
– Your Common Sense, Commented Mar 19, 2010 at 12:54
The if statement should at least enter one of the clauses; which one does it enter? — xtofl
– xtofl, Commented Mar 19, 2010 at 12:55

Richard JP Le Guen · Accepted Answer · 2010-03-19 12:55:52Z

1

Why are you using $_GET? Does this information come from an HTML form or a URL etc... ?

I suspect you meant to change $readstatus = $_GET['readstatus']; to $readstatus = $row['readstatus'];.

$_GET is an aray of GET parameters which come from the query string.

$row is a row in your database, so if the information is in the database - which I suspect it is - you want to use $row instead of $_GET.

answered Mar 19, 2010 at 12:55

Richard JP Le Guen

28.8k8 gold badges93 silver badges121 bronze badges

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

Comments

Michał Tatarynowicz · Accepted Answer · 2010-03-19 12:54:02Z

1

Try changing $readstatus = $_GET['readstatus']; to $readstatus = $row['readstatus'];

answered Mar 19, 2010 at 12:54

Michał Tatarynowicz

1,3743 gold badges16 silver badges34 bronze badges

Comments

Scott Arrington · Accepted Answer · 2010-03-19 12:56:13Z

0

The $_GET function relies on the value being contained in the query string of the URL, and it has nothing to do with the database. I have a hunch you're trying to get the value from the database here and you're using the wrong function to do it.

answered Mar 19, 2010 at 12:56

Scott Arrington

12.6k3 gold badges45 silver badges56 bronze badges

1 Comment

Richard JP Le Guen Over a year ago

$_GET isn't a function though ;)

dnagirl · Accepted Answer · 2010-03-19 12:58:06Z

0

$_GET['readstatus'] says the value is coming from the browser. $row['readstatus'] says the value is coming from the database.

You need to decide which should take precedence-- probably the $_GET['readstatus']` because it's what the user wants to change. If that's the case, you need to update your database with the new readstatus before you requery the db for the dataset.

answered Mar 19, 2010 at 12:58

dnagirl

20.5k13 gold badges86 silver badges127 bronze badges

Collectives™ on Stack Overflow

PHP IF statement not taking variable into account!

4 Answers 4

Comments

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related