Solved

Transitioning to PDO/MySQL.  Need help  with SELECT statement.

Posted on 2014-12-17
8
102 Views
Last Modified: 2014-12-19
How do I make this script work with PDO?

$conn = new PDO("mysql:host=$host;dbname=$db;charset=$charset", $user, $pass);

$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

$conn->$sql = "SELECT * FROM `manufacturers` WHERE name = '$data[8]'";
foreach($pdo->query($sql) as $row1){

$statement = $conn->prepare("INSERT INTO manufacturers_sku (sku,manufacturer)
    VALUES(:sku, :manufacturer)");
$statement->execute(array(
    "sku" => $row1['sku'];
    "manufacturer" => $row1['sku'];
));
}

Open in new window

0
Comment
Question by:lawrence_dev
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 3
  • 3
  • 2
8 Comments
 
LVL 58

Accepted Solution

by:
Gary earned 500 total points
ID: 40505672
$conn = new PDO("mysql:host=$host;dbname=$db;charset=$charset", $user, $pass);

$conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

$results = $conn->prepare("SELECT * FROM `manufacturers` WHERE name = :data")
$results->execute(array(
	":data" => $data[8]
));
$rows = $results->fetchAll(); // Just one way to do it.


$myinsert = $conn->prepare("INSERT INTO manufacturers_sku (sku,manufacturer)
	VALUES(:sku, :manufacturer)");

foreach($rows as $row){
	$myinsert->execute(array(
		":sku" => $row['sku'],
		":manufacturer" => $row['sku'] // Shouldn't this be something else?
	));
}

Open in new window

0
 
LVL 110

Expert Comment

by:Ray Paseur
ID: 40506005
A complete explanation of the transition process, including code examples, is available here.
http://www.experts-exchange.com/Web_Development/Web_Languages-Standards/PHP/PHP_Databases/A_11177-PHP-MySQL-Deprecated-as-of-PHP-5-5-0.html
0
 

Author Comment

by:lawrence_dev
ID: 40506210
OK got an error...

Parse error: syntax error, unexpected '$results' (T_VARIABLE)

$results->execute(array(
      ":data" => $data[8]      //ERROR THIS LINE
));

How do I fix this error?
0
What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

 
LVL 58

Expert Comment

by:Gary
ID: 40506215
Theres a missing semi colon after this line

$results = $conn->prepare("SELECT * FROM `manufacturers` WHERE name = :data"); // add the semi colon like I have here

Open in new window

0
 

Author Comment

by:lawrence_dev
ID: 40506246
Thank You!
0
 
LVL 58

Expert Comment

by:Gary
ID: 40506327
Why accept a comment that never answered the question or corrected your code, if you want a PDO tutorial there are far better places to go than some generic here's a one for all blah blah
0
 

Author Comment

by:lawrence_dev
ID: 40507921
I agree Gary!  I will take that into consideration next time.  I just wanted to reward the time and  effort to help.  You did get 450 points! The comments that just say "read this" and provide a link don't really help anyone.  If I could understand all that overly technical stuff with very little explanation,  I would not be paying to be on EE!!!  Thanks for helping me with my code!  Much appreciated!
0
 
LVL 110

Expert Comment

by:Ray Paseur
ID: 40508429
There are some things about PHP which are kind of "sine-qua-non."  For example we have to assume that anyone asking about conversion to PDO and specifically asking about a SELECT query statement would understand that a PHP  "parse error" containing the word "unexpected" indicates an up-stream error in the script.  But if the real issue was a PHP parse error, why not just tell us that is your problem?  I'd have gladly helped you correct a parse error, too.  I just thought there was a PDO-related issue (a very common thing today, since MySQL is going to be killed off soon).

@Gary: If you have something to offer the E-E community that is better than my article, please post it - perhaps write your own article or add your comments to show what my article is missing.  I have a great appreciation of constructive criticism and welcome anything you want to add to the conversation!
0

Featured Post

Why Off-Site Backups Are The Only Way To Go

You are probably backing up your data—but how and where? Ransomware is on the rise and there are variants that specifically target backups. Read on to discover why off-site is the way to go.

Question has a verified solution.

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

Things That Drive Us Nuts Have you noticed the use of the reCaptcha feature at EE and other web sites?  It wants you to read and retype something that looks like this. Insanity!  It's not EE's fault - that's just the way reCaptcha works.  But it i…
Password hashing is better than message digests or encryption, and you should be using it instead of message digests or encryption.  Find out why and how in this article, which supplements the original article on PHP Client Registration, Login, Logo…
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…
This tutorial will teach you the core code needed to finalize the addition of a watermark to your image. The viewer will use a small PHP class to learn and create a watermark.

630 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