Select et Insert a partir de 2 database

Version imprimable

Bonjour,

Je souhaite faire un select d'une base Oracle puis inserer le résultat dans une base MySql en utilisant PDO.
Mon code fonctionne correctement mais je ne pense pas qu'il soit bien optimisé.

Auriez des commentaires sur la méthode la plus performante.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
try
    {
 
    $db = new PDO("odbc:wh", 'wh', 'wh');
    $dbup = new PDO('mysql:host=localhost;dbname=uat', 'root', '');
 
    //select all lines from the Oracle DB
    $result = $db->prepare('SELECT * FROM item_master WHERE ROWNUM <= 5000');
 
    foreach($result as $row)
    {
    //Insert all lines in the MySql DB
      $dbup->exec("INSERT INTO item_master (SKU_ID,SIZE_DESC) VALUES ('" . $row['SKU_ID'] . "','" . $row['SIZE_DESC'] . "')")
	  or die(print_r($dbup->errorInfo(), true));
 
    }
 
 
    // close the database connection
    $db = NULL;
    $dbup = NULL;
  }
  catch(PDOException $e)
  {
    print 'Exception : '.$e->getMessage();
  }

28/11/2013, 17h33
stephanegib2

Salut,

il vaut mieux faire une seule requête INSERT, que plusieurs petites requêtes.
tu peux éventuellement utiliser les transactions.
Dans ta requête SELECT, sélectionne les champs que tu a besoin. Sa sert a rien de récupérer des choses que tu n'utilises pas.
28/11/2013, 23h33
papajoker

Bonjour,

Code:

$dbup->exec()

quoi ca ? exec() ... ne pas faire de $dbup prepare() avant la boucle :( oui pas vraiment bon
29/11/2013, 02h26
ABCIWEB

Oui effectivement il en manque par endroits et même il y en a aussi trop à d'autres endroits : la notion des requêtes préparées est à revoir.
1/ La première requête SELECT ne gagne rien à être préparée car elle n'est pas répétitive et ne contient aucune donnée utilisateur particulièrement sensible à protéger.
2 /Par contre la requête d'insertion risque d'être exécutée de nombreuses fois. C'est donc elle qu'il convient de préparer en mettant la préparation avant la boucle comme suggérer par papajoker

Oui je me suis rendu compte que la requete préparée n'avait pas grand interet.

J'ai éssayer de tenir compte des remarques et avec le code ci-dessous j'obtiens une execution de la requete plus rapide.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
try
{
	$pdo = new PDO('mysql:host=localhost;dbname=uat', 'root', '');
    $db = new PDO("odbc:wh", 'wh', 'wh');
 
    $pdo->beginTransaction();
 
    $result = $db->query('SELECT SKU_ID,SIZE_DESC FROM item_master WHERE ROWNUM <= 5000');
 
	foreach($result as $row)
    {
 
      $pdo->query("INSERT INTO item_master(SKU_ID,SIZE_DESC) VALUES ('" . $row['SKU_ID'] . "','" . $row['SIZE_DESC'] . "')");
 
    }
 
    $pdo->commit();
 
}

29/11/2013, 17h02
ABCIWEB

Et pourquoi ne veux-tu pas faire une requête préparée pour l'insertion ?
29/11/2013, 17h54
Budy123

A vrai dire je ne sais pas, le seconde code est plus rapide à l'éxécution que celle avec la requte préparée.

C'est plus sur la méthode de construction de mon code que je m'interroge, a savoir si la boucle utilisée est bonne ou si je dois m'orienter vers une classe par exemple..

Code:

1
2
3
4
5
6
7
//...
    $insert = $dbup->prepare("INSERT INTO item_master(SKU_ID,SIZE_DESC) VALUES (?,?)");
 
foreach($result as $row)
    {
      $insert->execute(array($row['SKU_ID'],$row['SIZE_DESC']));
    }

29/11/2013, 19h54
Budy123

Je pensais que tu parlais de préparer le sélect....
30/11/2013, 00h24
ABCIWEB

Faudrait être plus attentif. Regardes mes précédents messages ici et là.
Il faudrait apprendre à suivre des conseils sans qu'on te donne le code, parce qu'on ne le fait pas tout le temps, t'as de la chance qu'il n'y avait que deux lignes à écrire ;)