Discussion :

[Budy123]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
  }

[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.

[papajoker]

Bonjour,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$dbup->exec()

quoi ca ? exec() ... ne pas faire de $dbup prepare() avant la boucle oui pas vraiment bon

[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

[Budy123]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();
 
}

[ABCIWEB]

Et pourquoi ne veux-tu pas faire une requête préparée pour l'insertion ?

[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..

[ABCIWEB]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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']));
    }

[Budy123]

Je pensais que tu parlais de préparer le sélect....

[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