Solved

Transitioning to PDO/MySQL.  Need assistance with if/else statement

Posted on 2014-12-21
5
279 Views
Last Modified: 2014-12-21
Are changes necessary for if/else statements in PHP/PDO?

I am getting the following error:
Parse error: syntax error, unexpected 'catidexp1' (T_STRING)

(Script works fine without if/else statement)



$query3 = $conn->query('SELECT DISTINCT cat1, catidexp1 FROM download ORDER BY catidexp1);

while ($row3 = $query3->fetch(PDO::FETCH_ASSOC))
{

 if $row3['catidexp1']==0 {      //error here
continue; 
} else {



$statement = $conn->prepare('INSERT INTO category (category_id) VALUES(:catid)');
$params = array(':catid'=>$row3['catidexp1']);
$statement->execute($params); 

}

Open in new window

0
Comment
Question by:lawrence_dev
5 Comments
 
LVL 58

Accepted Solution

by:
Gary earned 450 total points
ID: 40512197
Is that your real code?
You are missing the closing apostrophe
$query3 = $conn->query('SELECT DISTINCT cat1, catidexp1 FROM download ORDER BY catidexp1');

Open in new window


Also don't put prepared statements inside a loop, put them before you loop - their purpose is to tell MySQL what to expect, not keep telling it what is coming...

$statement = $conn->prepare('INSERT INTO category (category_id) VALUES(:catid)');
0
 
LVL 17

Assisted Solution

by:Chris Millard
Chris Millard earned 50 total points
ID: 40512199
Change line 6 to:-

 if ($row3['catidexp1']==0) {
0
 
LVL 108

Expert Comment

by:Ray Paseur
ID: 40512208
Parse error: syntax error, unexpected ...
always means something is wrong "upstream" from the line with the error.  It may be the line above or a line many lines above.  Typically this is due to missing or unmatched parentheses, quotes, etc.

This has nothing to do with PDO, per se -- it's a PHP syntax error that may have been introduced by a conversion from MySQL to a current SQL extension, but looking to PDO for an answer is a red herring.  Instead, go back to the basic PHP syntax and you'll find the error through that process.
0
 

Author Closing Comment

by:lawrence_dev
ID: 40512231
Agreed Gary!  Thanks for your help!  Thanks Chris!
0
 

Author Comment

by:lawrence_dev
ID: 40512264
Take 30 seconds to look at the code to see the mistake and point it out. ---  100% Agree!  Thanks again for your help!!
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Introduction HTML checkboxes provide the perfect way for a web developer to receive client input when the client's options might be none, one or many.  But the PHP code for processing the checkboxes can be confusing at first.  What if a checkbox is…
Generating table dynamically is the most common issue faced by php developers.... So it seems there is a need of an article that explains the basic concept of generating tables dynamically. It just requires a basic knowledge of html and little maths…
Explain concepts important to validation of email addresses with regular expressions. Applies to most languages/tools that uses regular expressions. Consider email address RFCs: Look at HTML5 form input element (with type=email) regex pattern: T…
The viewer will learn how to dynamically set the form action using jQuery.

864 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

19 Experts available now in Live!

Get 1:1 Help Now