IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Langage PHP Discussion :

[POO] Méthode privée ou publique [PHP 5.2]


Sujet :

Langage PHP

  1. #21
    Membre actif

    Inscrit en
    Juin 2009
    Messages
    99
    Détails du profil
    Informations forums :
    Inscription : Juin 2009
    Messages : 99
    Points : 228
    Points
    228
    Par défaut
    Bonjour

    j'ai essayé de rassembler ta source (j'ai fais vite, corrigez si je me trompe)

    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
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    <?php
     
    error_reporting(E_ALL);
    class PDO2 extends PDO {
     
    	protected $_dbname = 'my_base' ; // protected car utilisée dans classe fille PDO3
     
    	private static $_instance;
    	private static $_dsn	= 'mysql:host=127.0.0.1;dbname=my_base' ;
    	private static $_dbuser	= 'user' ;
    	private static $_dbpwd	= 'password' ;
     
    	private static $_debug	= false ;
    	private static $_ipadmin= 'XX.XX.XX.XX' ;
     
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct( ) {
    	}
     
    	/* Singleton */
    	public static function getInstance() {
     
    		if (!isset(self::$_instance)) {
     
    			try {
    				self::$_instance = new PDO(self::$_dsn , self::$_dbuser , self::$_dbpwd);
    			}
    			catch (PDOException $e) {
    				if ( self::$_debug === true && ($_SERVER['REMOTE_ADDR'] == self::$_ipadmin) ) 	{
    					echo '<br>/PDO2 Erreur N° : '.$e->getCode();
    					echo '<br/>PDO2 Erreur : '.$e->getMessage().'<br />';
    					die();
    				}
    				else {
    					echo 'Erreur de la classe PDO2 - passer en mode DEBUG';
    					die();
    				}
    			}
    		}
    		return self::$_instance; 
    	}
     
    }
     
    class PDO3 extends PDO2 {
     
    	/* Variables de la classe */
    	private $_debug		= false;
    	protected $_pdo		= NULL;
     
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct($pdo)
    	{
    		$pdo_ext = $this->checkPdoExtension();
     
    		if (isset($pdo) && $pdo_ext) {
     
    			$this->_pdo = $pdo;
     
    		}
    		else {
    			throw new PDOException("No pdo connection");
    		}
    	}
    	/**
    	* Comptage du nbre enregistrements DISTINCTS pour un champ donné avec condition (par requête préparée)
    	* @return	: $count
    	*/
    	private function countDistinct($table_name, $field, $sql_condition) {
     
    		$sql_prep = "SELECT DISTINCT $field FROM $table_name WHERE $sql_condition" ;
    		$query	= $this->_pdo->prepare($sql_prep) ;
    		$query->execute();
    		$count = count($query->fetchColumn());
    		$query->closeCursor();
    		return $count;
    	}
     
    	/**
    	* Comptage du nbre enregistrements DISTINCTS pour un champ donné avec condition (par requete préparée)
    	* @return	: $count
    	*/
     
    	public function isUnique($table_name, $field, $sql_condition) {
    		// méthode alternative : voir http://php.developpez.com/faq/?page=pdo#pdo-count
     
    		$count = $this->countDistinct($table_name, $field, $sql_condition) ;
     
    		$unique = ($count == 1) ? true : false ;
     
    		return $unique;
    	}
    	}
     
     
    try {
    	$connect = PDO2::getInstance();
    	$pdo = new PDO3($connect);
    }
    catch (PDOException $e) {
    	$_SESSION['ERROR'] .= ERR_41 .': ' .$e->getMessage().'<br/>';
    	$securite = false;
    }
     
    $count = $pdo->countDistinct($table_name, $field, $sql_condition);
    $unique = ($pdo->isUnique($table_name, $field, $sql_condition))? "ok":"nok";
    if (DEBUG) echo "<br/>DEBUG: unique= $unique ($count)";
    si cette source est bonne le comportement est très logique je vois pas le problème

    ici

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    $count = $pdo->countDistinct($table_name, $field, $sql_condition);
    n'est même pas exécuté...

  2. #22
    Membre éclairé
    Avatar de hornetbzz
    Homme Profil pro
    Directeur commercial
    Inscrit en
    Octobre 2009
    Messages
    482
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : France

    Informations professionnelles :
    Activité : Directeur commercial

    Informations forums :
    Inscription : Octobre 2009
    Messages : 482
    Points : 773
    Points
    773
    Par défaut
    Bonjour

    Bravo ça a l'air d'être bien rassemblé

    Si je te suis, tu trouves normal que l'appel à une méthode privée depuis le code principal ne produise pas une erreur. Alors j'aurais vraiment besoin que tu m'expliques cela stp.

    Si cela ne s'exécute pas, ce n'est pas mieux. Avec le error_reporting(E_ALL), tu devrais recevoir un Fatal Error je pense.

    Chez moi, non seulement l'appel à la methode privée s'exécute (et génère l'affichage non escompté) et ne génère pas d'erreur.

  3. #23
    Membre éclairé
    Profil pro
    Inscrit en
    Mars 2005
    Messages
    625
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mars 2005
    Messages : 625
    Points : 822
    Points
    822
    Par défaut
    tiens, je viens de faire un petit test idiot...

    Un simple :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    class X extends PDO{
     
    }
    Ne fonctionne pas. Il faut impérativement renseigner le constructeur de PDO pour pouvoir créer une instance.

    Du coup je me dis que ton programme n'est pas bugué... Il a besoin d'être exorcisé

    EDIT: Gros rewind, on oublie tout ce que je viens d'écrire au dessus (je le laisse pour les archives car j'assume mes aneries )
    ceci :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    class B extends PDO{
    	function __construct(){}
    	private function toto( $a ){
    		echo 'fonction privée ' . $a . "\n";
    	}
    }
     
    $u = new B();
    $u->toto( 'test' );
    donne cela :

    Donc cela est lié à PDO... En quoi, no sé...

    RE-EDIT... c'est exactement le même test que toi Hornetbzz, mais là y'a vraiment que ça dans le fichier. Il peut pas y avoir d'erreur en amont

  4. #24
    Membre expert Avatar de RunCodePhp
    Profil pro
    Inscrit en
    Janvier 2010
    Messages
    2 962
    Détails du profil
    Informations personnelles :
    Localisation : Réunion

    Informations forums :
    Inscription : Janvier 2010
    Messages : 2 962
    Points : 3 947
    Points
    3 947
    Par défaut
    Re ...

    J'ai effectuer l'essai du post ci-dessus (class B extends PDO), et j'obtiens bien une erreur :
    Fatal error: Call to private method B::toto()

    Je suis en Php5.3.0 (voir ma signature)
    Et il est précisé Php5.2 pour le topic.

    Serait ce une question de version de Php ? Php5.2 buggé ?

  5. #25
    Membre éclairé
    Profil pro
    Inscrit en
    Mars 2005
    Messages
    625
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mars 2005
    Messages : 625
    Points : 822
    Points
    822
    Par défaut
    Serait ce une question de version de Php ? Php5.2 buggé ?
    Il semblerait bien, j'utilise 5.2 également. Si c'est réparé dans la 5.3 c'est déjà ça

  6. #26
    Membre expert
    Avatar de ThomasR
    Homme Profil pro
    Directeur technique
    Inscrit en
    Décembre 2007
    Messages
    2 230
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 38
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Directeur technique
    Secteur : High Tech - Multimédia et Internet

    Informations forums :
    Inscription : Décembre 2007
    Messages : 2 230
    Points : 3 972
    Points
    3 972
    Par défaut
    Citation Envoyé par RunCodePhp Voir le message
    Re ...

    J'ai effectuer l'essai du post ci-dessus (class B extends PDO), et j'obtiens bien une erreur :
    Fatal error: Call to private method B::toto()

    Je suis en Php5.3.0 (voir ma signature)
    Et il est précisé Php5.2 pour le topic.

    Serait ce une question de version de Php ? Php5.2 buggé ?
    Bonsoir,

    Lorsque tu hérites d'une classe et que tu appelles une méthode privée, tu ne l'appelles pas dans le scope de cette méthode, cela provoque donc une erreur fatale. Ce test est complètement différent du comportement décrit à l'origine de ce post, ce n'est pas un bug au contraire

    scope = portée (contexte d'execution du bloc de code)

  7. #27
    Membre éclairé
    Avatar de hornetbzz
    Homme Profil pro
    Directeur commercial
    Inscrit en
    Octobre 2009
    Messages
    482
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : France

    Informations professionnelles :
    Activité : Directeur commercial

    Informations forums :
    Inscription : Octobre 2009
    Messages : 482
    Points : 773
    Points
    773
    Par défaut
    Merci PtitBidon pour le test idiot, comme ça j'ai pas le monopole

    Bonsoir Thomas,


    J'avoue que je n'ai pas tout compris...

    Quand tu es en phase d'apprentissage sur du sable mouvant, arrrgh ! Au secours ! Vite mon doudou et ma tétine

    Au final, crois-tu que j'ai fait une boulette (et laquelle) ou bien c'est la façon peu "harmonieuse" ou "conventionnelle" (en bref "pourrie") dont j'ai hérité les classes qui en, quelque sorte, tromperait PHP (auquel cas comment dois-je m'y prendre) ?


    Merci d'avance

  8. #28
    Membre actif

    Inscrit en
    Juin 2009
    Messages
    99
    Détails du profil
    Informations forums :
    Inscription : Juin 2009
    Messages : 99
    Points : 228
    Points
    228
    Par défaut
    moi j'ai l'erreur suivante

    Fatal error: Call to private method PDO3::countDistinct() from context

  9. #29
    Membre expert Avatar de RunCodePhp
    Profil pro
    Inscrit en
    Janvier 2010
    Messages
    2 962
    Détails du profil
    Informations personnelles :
    Localisation : Réunion

    Informations forums :
    Inscription : Janvier 2010
    Messages : 2 962
    Points : 3 947
    Points
    3 947
    Par défaut
    J'obtiens aussi la même erreur :
    Fatal error: Call to private method PDO3::countDistinct() from context

    Qu'il y ait une explication au niveau du scope, téléscope ou magnétoscope ... peut être, voir surement (totalement abstrait pour moi, désolé), il me semble qu'il y a quand même quelque chose au niveau des versions.
    Disons qu'il ne faudrait pas l'ignorer.

    Si 2 codes identiques, mais exécutés dans 2 environnements différents (V5.2 / V5.3), donnent 2 résultat différents, il y a bug quelque part, non ?


    Ce test est complètement différent du comportement décrit à l'origine de ce post, ce n'est pas un bug au contraire
    Le test effectué au par avant était peut être différent, mais il démontrait tout de même 2 comportement différents dans 2 environnements différents.
    J'estime donc que l'essai n'était pas si inutile, voir si idiot que ça, même encore.
    Mais j'assume ... même pas peur (je l'avais déjà dis. Me v'là bègue maintenant)


    Que hornetbzz fasse un essai avec Php5.3.0, voir s'il y a un retour d'erreur ou pas en appelant cette méthode PDO3::countDistinct(), serait intéressant à mon sens.

  10. #30
    Membre expert
    Avatar de ThomasR
    Homme Profil pro
    Directeur technique
    Inscrit en
    Décembre 2007
    Messages
    2 230
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 38
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Directeur technique
    Secteur : High Tech - Multimédia et Internet

    Informations forums :
    Inscription : Décembre 2007
    Messages : 2 230
    Points : 3 972
    Points
    3 972
    Par défaut
    Le test effectué au par avant était peut être différent, mais il démontrait tout de même 2 comportement différents dans 2 environnements différents.
    2 tests différents amènent certainement 2 comportements différents.

  11. #31
    Membre expert Avatar de RunCodePhp
    Profil pro
    Inscrit en
    Janvier 2010
    Messages
    2 962
    Détails du profil
    Informations personnelles :
    Localisation : Réunion

    Informations forums :
    Inscription : Janvier 2010
    Messages : 2 962
    Points : 3 947
    Points
    3 947
    Par défaut
    2 tests différents amènent certainement 2 comportements différents.
    2 test différents ?
    Non justement, mon code était strictement identique à celui de Petibidon.
    La logique veut qu'on devrait avoir le même résultat final. Il se trouve que non.
    Ici, c'était pour son code.

    Mais en reprenant le code de hornetbzz (PDO2 et PDO3), du copier/coller, je n'obtiens pas le même résultat.
    Apparemment helmi.dridi aussi.

    Les codes sont les mêmes, donc les test sont les mêmes.
    Je remarque cependant que l'environnement qui est différent : Php5.2 et Php5.30.

    Tu explique ça comment qu'il y ait 2 résultats différent pour un même code ?

  12. #32
    Membre éclairé
    Avatar de hornetbzz
    Homme Profil pro
    Directeur commercial
    Inscrit en
    Octobre 2009
    Messages
    482
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : France

    Informations professionnelles :
    Activité : Directeur commercial

    Informations forums :
    Inscription : Octobre 2009
    Messages : 482
    Points : 773
    Points
    773
    Par défaut
    Oui, alors pour résumer :

    Ce test a pour vocation de reproduire le comportement problématique du code d'origine.

    Donc dans mon, cas ce code a un comportement NORMAL, c'est à dire qu'il génére un Fatal Error quand j'appelle la fonction privée.

    Contrairement au code problématique.


    Donc le problème reste entier.

  13. #33
    Membre expert Avatar de RunCodePhp
    Profil pro
    Inscrit en
    Janvier 2010
    Messages
    2 962
    Détails du profil
    Informations personnelles :
    Localisation : Réunion

    Informations forums :
    Inscription : Janvier 2010
    Messages : 2 962
    Points : 3 947
    Points
    3 947
    Par défaut
    D'accord, en ce qui concerne ton code de teste.

    Mais j'ai l'impression que tu n'as pas fait attention à cette remarque :
    Citation Envoyé par RunCodePhp
    Mais en reprenant le code de hornetbzz (PDO2 et PDO3), du copier/coller, je n'obtiens pas le même résultat.
    Apparemment helmi.dridi aussi.
    Il s'agit d'un essai du code de tes 2 classes PDO2/PDO3 ici (pas celui du test).

    Comment expliquer que je ne parviens pas à reproduire la même erreur que toi, avec ton même code ?

    Pour ma part, 2 choses une :
    - Soit le code que tu mis est incomplet ou erroné, ce qui ne me permet pas de remarquer le même dysfonctionnement (pas d'erreur fatal)
    - Soit ce serait un élément extérieur qui m'empêcherait de voir ce même dysfonctionnement.


    L'autre point qui m'a semblé important, c'est le 2ème test de petitbidon ( class B extends PDO ).
    Même si le test est éloigné de ton cas, on avait lui et moi 2 résultats différents, ce qui théoriquement ne devrait pas être le cas.
    La différence notoire, c'est qu'on a un environnement différent : Php5.2 / Php5.3.
    J'en conclus qu'il ne faudrait pas te faire piéger par environnement qui serait buggé.


    Donc avant de parler de "boulette", voir pire, de fracasser ton code, le vérifier dans un environnement Php5.3 me semble pas si idiot que ça.

  14. #34
    Membre éclairé
    Avatar de hornetbzz
    Homme Profil pro
    Directeur commercial
    Inscrit en
    Octobre 2009
    Messages
    482
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : France

    Informations professionnelles :
    Activité : Directeur commercial

    Informations forums :
    Inscription : Octobre 2009
    Messages : 482
    Points : 773
    Points
    773
    Par défaut
    Bonjour Run

    Pour le code, il y a seulement d'autres fonctions indépendantes des deux que j'ai mises. Je peux mettre tout l'ensemble de PDO3 si tu penses que ça peut servir. Je met ci-dessous l'intégralité de PDO3.

    Si si, j'ai bien vu les commentaires mais je ne savais pas trop quoi en faire.

    Et oui effectivement, je suis d'accord avec toi sur les actions pour tester l'influence de la version de PHP et de son environnement.

    Par contre, j'ai un problème pour passer à PHP5.3, car d'une part, j'ai une machine un peu particulière et d'autre part, j'ai peur de compromettre tout l'existant en changeant de config. C'est qu'elles sont sensibles mes petites lignes de code

    Et justement, en tenant compte des remarques des uns et des autres, mes 2 classes sont mal construites, de toutes façons. Donc je pensais les rebâtir indépendamment du bug potentiel de PHP ou d'un problème d'environnement.

    A voir donc,

    slt

    Ci joint le code PDO3 intégral :
    Code PHP : 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
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    162
    163
    164
    165
    166
    167
    168
    169
    170
    171
    172
    173
    174
    175
    176
    177
    178
    179
    180
    181
    182
    183
    184
    185
    186
    187
    188
    189
    190
    191
    192
    193
    194
    195
    196
    197
    198
    199
    200
    201
    202
    203
    204
    205
    206
    207
    208
    209
    210
    211
    212
    213
    214
    215
    216
    217
    218
    219
    220
    221
    222
    223
    224
    225
    226
    227
    228
    229
    230
    231
    232
    233
    234
    235
    236
    237
    238
    239
    240
    241
    242
    243
    244
    245
    246
    247
    248
    249
    250
    251
    252
    253
    254
    255
    256
    257
    258
    259
    260
    261
    262
    263
    264
    265
    266
    267
    268
    269
    270
    271
    272
    273
    274
    275
    276
    277
    278
    279
    280
    281
    282
    283
    284
    285
    286
    287
    288
    289
    290
    291
    292
    293
    294
    295
    296
    297
    298
    299
    300
    301
    302
    303
    304
    305
    306
    307
    308
    309
    310
    311
    312
    313
    314
    315
    316
    317
    318
    319
    320
    321
    322
    323
    324
    325
    326
    327
    328
    329
    330
    331
    332
    333
    334
    335
    336
    337
    338
    339
    340
    341
    342
    343
    344
    345
    346
    347
    348
    349
    350
    351
    352
    353
    354
    355
    356
    357
    358
    359
    360
    361
    362
    363
    364
    365
    366
    367
    368
    369
    370
    371
    372
    373
    374
    375
    376
    377
    378
    379
    380
    381
    382
    383
    384
    385
    386
    387
    388
    389
    390
    391
    392
    393
    394
    395
    396
    397
    398
    399
    400
    401
    402
    403
    404
    405
    406
    407
    408
    409
    410
    411
    412
    413
    414
    415
    416
    417
    418
    419
    420
    421
    422
    423
    424
    425
    426
    427
    428
    429
    430
    431
    432
    433
    434
    435
    436
    437
    438
    439
    440
    441
    442
    443
    444
    445
    446
    447
    448
    449
    450
    451
    452
    453
    454
    455
    456
    457
    458
    459
    460
    461
    462
    463
    464
    465
    466
    467
    468
    469
    470
    471
    472
    473
    474
    475
    476
    477
    478
    479
    480
    481
    482
    483
    484
    485
    486
    487
    488
    489
    490
    491
    492
    493
    494
    495
    496
    497
    498
    499
    500
    501
    502
    503
    504
    505
    506
    507
    508
    509
    510
    511
    512
    513
    514
    515
    516
    517
    518
    519
    520
    521
    522
    523
    524
    525
    526
    527
    528
    529
    530
    531
    532
    533
    534
    535
    536
    537
    538
    539
    540
    541
    542
    543
    544
    545
    546
    547
    548
    549
    550
    551
    552
    553
    554
    555
    556
    557
    558
    559
    560
    561
    562
    563
    564
    565
    566
    567
    568
    569
    570
    571
    572
    573
    574
    575
    576
    577
    578
    579
    580
    581
    582
    583
    584
    585
    586
    587
    588
    589
    590
    591
    592
    593
    594
    595
    596
    597
    598
    599
    600
    601
    602
    603
    604
    605
    606
    607
    608
    609
    610
    611
    612
    613
    614
    615
    616
    617
    618
    619
    620
    621
    622
    623
    624
    625
    626
    627
    628
    629
    630
    631
    632
    633
    634
    635
    636
    637
    638
    639
    640
    641
    642
    643
    644
    645
    646
    647
    648
    649
    650
    651
    652
    653
    654
    655
    656
    657
    658
    659
    660
    661
    662
    663
    664
    665
    666
    667
    668
    669
    670
    671
    672
    673
    674
    675
    676
    677
    678
    679
    680
    681
    682
    683
    684
    685
    686
    687
    688
    689
    690
    691
    692
    693
    694
    695
    696
    697
    698
    699
    700
    701
    702
    703
    704
    705
    706
    707
    708
    709
    710
    711
    712
    713
    714
    715
    716
    717
    718
    719
    720
    721
    722
    723
    724
    725
    726
    727
    728
    729
    730
    731
    732
    733
    734
    735
    736
    737
    738
    739
    740
    741
    742
    743
    744
    745
    746
    747
    748
    749
    750
    751
    752
    753
    754
    755
    756
    757
    758
    759
    760
    761
    762
    763
    764
    765
    766
    767
    768
    769
    770
    771
    772
    773
    774
    775
    776
    777
    778
    779
    780
    781
    782
    783
    784
    785
    786
    787
    788
    789
    790
    791
    792
    793
    794
    795
    796
    797
    798
    799
    800
    801
    802
    803
    804
    805
    806
    807
    808
    809
    810
    811
    812
    813
    814
    815
    816
    817
    818
    819
    820
    821
    822
    823
    824
    825
    826
    827
    828
    829
    830
    831
    832
    833
    834
    835
    836
    837
    838
    839
    840
    841
    842
    843
    844
    845
    846
    847
    class PDO3 extends PDO2 {
     
    	/* Variables de la classe */
    	//private $_bidon ="truc" ;
     
    	private $_debug		= false	;
    	private $_delimiter	= ',   '; 			// ne pas modifier !
    	private $_type_of_key = array('PRI', 'UNI', 'MUL') ;
    	protected $_pdo		= NULL;				// ne doit pas être "private" car héritage de PDO2
     
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct($pdo)
    	{
    		$pdo_ext = $this->checkPdoExtension();
     
    		if (isset($pdo) && $pdo_ext) {
     
    			$this->_pdo = $pdo;
     
    		}
    		else {
    			throw new PDOException("No pdo connection");
    		}
    	}
     
     
     
    	/* Destructeur */
    	public function __destruct() // nota: ne surtout pas mettre de parametre
    	{
    		$this->close();
    	}
     
     
     
    	/** 28/02/2010
    	* ferme la connexion à la dB (portage dans la classe PDO de la fonction mysql_close)
    	*/
    	public function close()
    	{
    		$this->_pdo = NULL;
    	}
     
    /*
    	private function methode_bidon ($aff = "methode prive bidon")
    	{
    		echo '<br/>'. $aff . ' in this context '.PHP_EOL;
    	}
    */
     
     
    	/* Methode magique */
    	public function __toString($object)
    	{
    		$string = (string) $object ;
    		return $string;
    	}
     
     
    	/* add Field if not exists (Grant Privileges required)*/
    	public function addField($table_name, $field_name, $field_type, $default_value)
    	{
    	// NOTA BENE: la connexion DOIT ETRE OUVERTE avec les privilèges suffisants ...
    	// ex: `product_selling_net_price` decimal(5,2) NOT NULL
    	// ALTER TABLE latable ADD / MODIFY / DROP / RENAME lacolonne [nouvelle définition de la colonne]
    	// NOTA : IF NOT EXISTS can not be used in such a case
     
    		// Init
    		$debug = $this->_debug ;
     
    		// Check if not already existing field before adding
    		$exists = $this->checkFieldExists( $table_name, $field_name );
    		$aff = ($exists)? "ERR: already existing, can not add as new field <b>$field_name</b> to table <b>$table_name</b>" : "Ok, field <b>$field_name</b> can be added to table <b>$table_name</b>";
    		if ($debug) echo "<br/>$aff";
     
    		// Adding field
    		if ( false === $exists) {
     
    			// Parameters check
    			// ######
    			// # ToDo
    			// ######
     
    			// Query
    			$sql = "ALTER TABLE `$table_name` ADD `$field_name` $field_type $default_value";
    			$query = $this->_pdo->prepare($sql) ;
    			$result = $query->execute() ;
    		}
    		else {
    			return false ;
    		}
     
    		if ( $result ) {
    			if ($debug) echo "<br/>Field $field_name added to table $table_name";
    			return true;
    		}
    		else if ( $result === false ){
    			if ($debug) {
    				echo "<br/>Field $field_name <b>NOT added</b> to table $table_name";
    				return false ; //. mysql_error();	
    			}
    			else {
    				return false;
    			}
    		}
     
    	}
     
    	/* Last Id  */
    	public function getLastInsertId()
    	{
    		$last_id = $this->_pdo->lastInsertId(); 
    		return $last_id;
    	}
     
    	/* Min ($field)  */
    	public function getMin( $table_name, $field , $sql_condition="1")
    	{
    		$check_table = $this->checkTableExists($table_name);
    		if ($check_table) {
    			$sql = "SELECT min($field) as min_value FROM $table_name WHERE $sql_condition";
    			$query = $this->_pdo->prepare($sql) ;
    			$query->execute() ;
    			$result = $query->fetch(PDO::FETCH_ASSOC);
    			$min = $result['min_value'];
    			return $min ;
    		}
    		else {
    			return false ;
    		}
    	}
     
     
    	/* Max ($field)  */
    	public function getMax( $table_name, $field, $sql_condition="1" )
    	{
    		$check_table = $this->checkTableExists($table_name);
    		if ($check_table) {
    			$sql = "SELECT max($field) as max_value FROM $table_name WHERE $sql_condition";
    			$query = $this->_pdo->prepare($sql) ;
    			$query->execute() ;
    			$result = $query->fetch(PDO::FETCH_ASSOC);
    			$max = $result['max_value'];
    			return $max ;
    		}
    		else {
    			return false ;
    		}
    	}
     
    	/** 2/03/2010
    	* Vérifie l'existence d'une table dans la database grace à une requete préparée
    	* @param	: (string) $db_name, (string) $table_name
    	* @return	: true or false
    	*/
    	private function checkTableExists( $table_name)
    	{
    		$sql = "SELECT TABLE_NAME
    				FROM INFORMATION_SCHEMA.TABLES
    				WHERE TABLE_SCHEMA = '". $this->_dbname . "' 
    				AND TABLE_NAME = '" . $table_name . "'" ;
    		$query = $this->_pdo->prepare($sql) ;
    		$result = $query->execute() ;
    		//if ($this->_debug && false === $result) echo "<br/>PDO3 debug: pas de connection à la table <b>$table_name</b>";
    		return $result ;
    	}
     
    	/** 9/03/2010
    	* Vérifie l'existence d'un champ dans la table de la database
    	* @param	: (string) $db_name, (string) $table_name
    	* @return	: true or false
    	*/
    	private function checkFieldExists( $table_name, $field_to_check )
    	{
    		// Init
    		$pattern_field	= '/^(?:\<b\>)(\w+)(?:\<\/b\>)(?:\s{0,})\((.+)\)(?:\s{0,})\((.+)?\)$/';
    		$field_name	= $field_format = $field_key = array() ;
     
    		// db fields
    			$db_fields = $this->getFieldsName($table_name) ;
    			if ( empty($db_fields) ) return false ;
    			foreach ( $db_fields as $field) {
    					unset($matches);
    					if ( preg_match($pattern_field, $field, $matches) ) {
    						$field_name[]	= (isset($matches[1]))? trim($matches[1]) : 'name not found' ;
    						$field_format[]	= (isset($matches[2]))? trim($matches[2]) : 'format not found' ;
    						$field_key[]	= (isset($matches[3]))? trim($matches[3]) : 'key not found' ;
    					}
    			}
     
    		// Matching ?
    		$matching = (in_array( trim($field_to_check), $field_name )) ? true : false ;
     
    		if ( false === $matching && $this->_debug) {
    			echo "<br/>DEBUG checkFieldExists: not found <b>$field_to_check</b> in <pre>";
    			print_r($field_name) ;
    			echo "</pre>";
    		}
     
    		return $matching;
     
    	}
     
     
    	/** 2/03/2010
    	* Retourne la clé primaire d'une table
    	* @param	: (string) $table
    	* @return	: (string) $pk ou false
    	*/
    	private function getKey( $table_name, $key_type = 'PRI') 
    	{
     
    		if (!in_array( $key_type, $this->_type_of_key ) ) 
    			return false ;
     
    		$sql = "SELECT COLUMN_NAME
    				FROM INFORMATION_SCHEMA.COLUMNS
    				WHERE TABLE_SCHEMA = '". $this->_dbname . "' 
    				AND TABLE_NAME = '" . $table_name . "' 
    				AND COLUMN_KEY = '". $key_type. "' 
    				ORDER BY ORDINAL_POSITION" ;
    		$select = $this->_pdo->prepare($sql) ;
    		$select->execute() ;
    		$pk = $select->fetch(PDO::FETCH_ASSOC);
    		$pk_name = $pk['COLUMN_NAME'];
    		//if ($this->_debug) echo '<br/>getKey: ' . $pk_name ;
    		return $pk_name ;
    	}
     
     
     
    	/** 2/03/2010
    	* Retourne la clé primaire d'une table ainsi que sa valeur pour une ligne donnée en fonction d'une condition de recherche
    	* @param	: (string) $table, (string) $sql_condition
    	* @return	: (array) $pk ou false
    	*/
    	public function getRecordID($table_name, $sql_condition)
    	{
    		// Verif initiales
    			// Table existante
    				$check_table = $this->checkTableExists($table_name);
    				//if (false === $check_table && $this->_debug) echo '<br/>DEBUG getRecordId err 1';
    				if (false === $check_table) return false ;
     
    			// PK existe
    				$pk_name = $this->getKey( $table_name, 'PRI') ;
    				//if (false === $pk_name && $this->_debug) echo '<br/>DEBUG getRecordId err 2';
    				if (false === $pk_name) return false ;
     
    			// Condition de recherche valide
    				list($condition_field_name, $condition_field_value) = explode("=", $sql_condition);
    				//if ( empty($condition_field_name) || empty($condition_field_name) && $this->_debug) echo '<br/>DEBUG getRecordId err 3';
    				if ( empty($condition_field_name) || empty($condition_field_name) ) return false ;
     
    			// Verif de l'existence du champ de recherche
    			// #######
    			// # to do
    			// #######
     
    		// Requetes SQL et execution
    			$sql = "SELECT $pk_name FROM $table_name WHERE $sql_condition LIMIT 1" ;
    			$select = $this->_pdo->prepare($sql) ;
    			$select->execute() ;
    			$id = $select->fetch(PDO::FETCH_ASSOC);
    			/*
    			if ($this->_debug) { 
    				echo '<br/>DEBUG GetRecordId ';
    				echo '<br/>$sql = ' . $sql;
    				echo '<br/>id found = <pre>'; print_r($id); echo '</pre>' ;
    			}
    			*/
    			$id = $id[$pk_name];
     
    			if ( empty($id) || !isset($id) ) {
    				return false;
    			}
    			else {
    				return $id;
    			}
     
    	}
     
     
    	/**
    	* Retourne la liste résumée des tables de la base
    	* @doc		: http://cedric-duprez.developpez.com/tutoriels/mysql/catalogue/#LV
    	* @return	: (array) $data_array
    	*/
    	public function getFieldsSummary($limit = false)
    	{
    		$data_array = array();
    		$limit = ($limit == false)? '' : 'LIMIT '. (int) $limit;
     
    		// Les tables, dans l'ordre = f(taille)
    		$sql = "SELECT TABLE_NAME, ROUND((DATA_LENGTH + INDEX_LENGTH) / (1024 * 1024), 2) AS SIZE_MO, TABLE_ROWS, AUTO_INCREMENT
    		FROM INFORMATION_SCHEMA.TABLES
    		WHERE TABLE_SCHEMA = '". $this->_dbname . "'
    		ORDER BY DATA_LENGTH + INDEX_LENGTH DESC ". $limit; 
    		$result = $this->_pdo->query($sql);
    		$rows = $result->fetchAll(PDO::FETCH_ASSOC);
     
    		foreach($rows as $key => $value) {
    			$data_array[$key] = $value;
    		}
     
    		return $data_array;
    	}
     
     
     
    	/**
    	* Retourne la liste détaillée des tables de la base
    	* @doc		: http://cedric-duprez.developpez.com/tutoriels/mysql/catalogue/#LV
    	* @return	: (array) $data_array
    	*/
    	public function getFieldsDetails($limit = false)
    	{
    		$data_array = array();
    		$limit = ($limit == false)? false : (int) $limit;
     
    		// Summary: reprise de ces infos comme base de recherche des détails
    		$source_array = $this->getFieldsSummary($limit);
     
    		// Details
    		foreach( $source_array as $key => $field_datas) {
     
    			if (is_array($field_datas)) {
     
    				// Transfert du Summary dans le tableau de détails
    				$data_array[$key] = $field_datas;
     
    				// Details de chaque table
    				$table_encours = $field_datas['TABLE_NAME'];
    				$sql = "SELECT COLUMN_NAME, COLUMN_TYPE, COLUMN_KEY, EXTRA
    						FROM INFORMATION_SCHEMA.COLUMNS
    						WHERE TABLE_SCHEMA = '". $this->_dbname . "'
    						AND TABLE_NAME = '" . $table_encours . "'";
    				$result = $this->_pdo->query($sql);
    				$rows = $result->fetchAll(PDO::FETCH_ASSOC);
     
    				// Field List
    				$field_list= '';
    				$memo_auto_inc = '' ;
    				$count = count($rows);
     
    				foreach($rows as $k => $value) {
    					$memo_auto_inc = ($value['EXTRA'] == 'auto_increment')? $value['COLUMN_NAME'] : $memo_auto_inc ;
    					$field_list .= '<b>'. $value['COLUMN_NAME'] . '</b> (' . $value['COLUMN_TYPE'] . ') ' . ' (' . $value['COLUMN_KEY'] . ')'. $this->_delimiter; 
    				}
     
    				// Marqueur éventuel sur l'AI
    				$data_array[$key]['AI_RESET_NEED']	= ($data_array[$key]['AUTO_INCREMENT'] > $count) ?  'true' : 'false' ;
    				$data_array[$key]['AI_FIELD']		= $memo_auto_inc ;
     
    				// Array Push
    				$data_array[$key]['FIELDS_COUNT']	= $count;
    				$data_array[$key]['FIELDS_LIST']	= $field_list;
    			}
    		}
     
    		return $data_array;
    	}
     
     
     
    	/** 28/02/2010 portage dans la classe PDO de la fonction db_pdo.php
    	 * retourne la liste des champs d'une table sous forme de vecteur colonne
    	 * @param	: (string) $table_name nom de la table dont on veut la liste des champs
    	 * @return	: (array) $fieldNames liste des noms de champs de $table_name
    	 */
    	public function getFieldsName($table_name)
    	{
    		// Field Details: reprise de ces infos comme base de recherche des détails
    		$source_array = $this->getFieldsDetails(false);
     
    		// 1ere méthode: lourde et peu élégante
    		/*
    		$fieldNames = array() ;
    		foreach ($source_array as $key => $table_datas) {
    			if (is_array($table_datas)) {
    				if ($table_datas['TABLE_NAME'] == $table_name) {
    					$fieldNames = explode( $this->_delimiter , $table_datas['FIELDS_LIST'] )  ;
    				}
    			}
    		}
    		array_pop($fieldNames) ;
    		*/
     
    		// 2eme méthode: plus directe et élégante : recupération récursive de la clé correspondante à la table
    		$key = $this->recursiveArraySearch($table_name , $source_array ) ;
    		$fieldNames = explode( $this->_delimiter , $source_array[$key]['FIELDS_LIST'] )  ;
    		array_pop($fieldNames);
     
    		// 3eme methode: brut accès direct aux infos dB sans utiliser d'autres fonctions
    		/*
    		$recordset = $this->_pdo->query("SHOW COLUMNS FROM $table_name");
    		$fields = $recordset->fetchAll(PDO::FETCH_ASSOC);
    		foreach ($fields as $field) {
    			$fieldNames[] = $field['Field'];
    		}
    		*/
     
    		return $fieldNames;
    	}
     
     
     
    	/**
    	* Affecte une valeur dans toute une colonne selon une condition optionnelle
    	* @return	: $count nbre d'enregistrement affectés ou false
    	*/
    	public function updateColumn($table_name, $field_name, $data_cste, $sql_condition=false )
    	{
    		// ex: UPDATE product_supply_base SET `product_selling_net_price` = 300 WHERE `product_ref`=10009
     
    			// Verif existante Table
    			$table_ok = $this->checkTableExists( $table_name) ;
    			if ($this->_debug && false ===  $table_ok) echo '<br/>DEBUG update Column: table Nok';
    			if ( false ===  $table_ok ) return false ;
     
    		// Verif existante Fields
    			$destination_Ok = $this->checkFieldExists( $table_name, $field_name ) ;
    			if ($this->_debug && false ===  $destination_Ok ) echo '<br/>DEBUG update Column: fields Nok';
    			if ( false ===  $destination_Ok ) return false ;
     
    		// Verif data_cste
    			if ($this->_debug && empty($data_cste) ) echo '<br/>DEBUG update Column: champ source et destination identiques: <b>Nok</b>';
    			if ( empty($data_cste) ) return false ;
     
    		// Verif sql condition et champ obligatoire
    			list($condition_field_name, $condition_field_value) = explode("=", $sql_condition);
    			$test = (empty($condition_field_name) || empty($condition_field_name) || false === $this->checkFieldExists( $table_name, $condition_field_name ) );
    			if ($this->_debug && $test ) echo '<br/>DEBUG update Column: sql condition Nok';
    			if ( $test ) return false ;
     
    		// Verif compatibilté du format $data_cste avec le format de la colonne
    		// ######
    		// # ToDo
    		// ######
     
    		// Update Column with constante value under sql_condition
    			$sql = "UPDATE $table_name SET $field_name = $data_cste WHERE $sql_condition" ;
    			$query	= $this->_pdo->prepare($sql) ;
    			$result	= $query->execute() ;
    			$count	= $query->rowCount();
     
    		if ($result) {
    			return $count;
    		}
    		else {
    			if ($this->_debug && false === $result) echo '<br/>DEBUG update Column: result Nok';
    			return false ;
    		}
     
    	}
     
     
     
    	/** 28/02/2010 portage dans la classe PDO de la fonction incluse dans db_pdo.php
    	 * sauve des datas (variables ou array) INSERT ou UPDATE dans la table $table de la dB
    	 * @NOTA 	: fonction dépréciée au profit de la fonction saveData sauf si l'on veut ajouter une condition sur une table sans pk
    	 * @param 	: $table_name, $data_array
    	 * @return	: true si tout s'est bien passé, false sinon
    	 * @doc		: http://fr.php.net/manual/fr/function.mysql-query.php#84962
    	 */
    	public function addData($table_name, $excluded_fields = '', $data_array, $sql_type = 'insert', $sql_condition = NULL )
    	{
     
    			// Init
    			$fields = '';
    			$values = '';
    			$debug = $this->_debug ;
    			$pattern = '/tel|phone|fax|siret|number|numero/';
     
     
    			// format input fields into sql (explode du tableau associatif en requete SQL)
    			foreach ($data_array as $field => $value) {
     
    				if (!strstr($excluded_fields, $field)) {
     
    				  // construction de la sql pour INSERT
    				  if (strtolower($sql_type) == 'insert') {
    					$fields .= "$field, ";
    					$values .= "'$value', ";
    				  }
    				  // construction de la sql pour UPDATE
    				  else {
    					if (!is_numeric($value)) {
    						$fields .= "$field = '$value', ";	// string
    					}
    					else {
    						// exceptions: pour le num de tel/fax/siret qui peut commencer par un zero et peut ainsi être assimilé à un numeric, ce qui n'est pas le cas ds la dB
    						if ( !preg_match( $pattern, $field  ) ) {
    							$fields .= "$field = $value, "; // numeric
    						}
    						else {
    							$fields .= "$field = '$value', ";// si tel/phone/fax/siret etc.. => string
    						}
    					}
    				  }
    				}
    			}
     
    			// remove trailing ", " from $fields and $values
    			$fields = preg_replace('/, $/', '', $fields);
    			$values = preg_replace('/, $/', '', $values);
     
    			// create sql statement
    			if (strtolower($sql_type) == 'insert') {
    				$sql = "INSERT INTO $table_name ($fields) VALUES ($values)";
    			}
    			elseif (strtolower($sql_type) == 'update') {
    				if (!isset($sql_condition)) {
    				  if ($debug) echo 'ERROR: You must enter a sql condition!';
    				  exit;
    				}
    				$sql = "UPDATE $table_name SET $fields WHERE $sql_condition";
     
    			}
    			else {
    				if ($debug) echo 'ERROR: Invalid input for argument $sql_type: must be "insert" or "update"';
    				exit;
    			}
     
    			// prepare & execute sql
    			$query = $this->_pdo->prepare($sql) ;
    			$result = $query->execute() ;
    			//$result->closeCursor(); // not always necessary: depends on SQL driver needs
     
    			if ( $result ) {
    				return true;
    			}
    			else if ( $result === false ){
    				if ($debug) {
    					return 'err sql: ' ; //. mysql_error();	
    				}
    				else {
    					return false;
    				}
    			}
     
    	} // fin addData
     
     
     
    	/** 01/03/2010
    	 * sauve $data_array dans $table (la condition d'UPDATE devenant optionnelle si cette condition ne porte que sur l'id PK)
    	 * @param 	: $table_name destination, $data_array à sauver, $sql_type='insert' ou 'update', $optional_condition
    	 * @doc		: http://www.php.net/manual/fr/pdostatement.execute.php#80499
    	 * @return	: true si tout s'est bien passé, false sinon
    	 */
    	public function saveData($table_name, $data_array, $sql_type='insert', $optional_condition=NULL )
    	{
     
    		// Init
    		$debug			= $this->_debug;
    		$sql			='';
    		$pattern_field	= '/^(?:\<b\>)(\w+)(?:\<\/b\>)(?:\s{0,})\((.+)\)(?:\s{0,})\((.+)?\)$/';
    		$field_name		= $field_format = $field_key = array() ;
     
    		// Initial checks
    			// Data_array in line with table fields
     
    				// db fields
    				$db_fields = $this->getFieldsName($table_name) ;
    				if ( empty($db_fields) ) return false ;
    				foreach ( $db_fields as $field) {
    					unset($matches);
    					if ( preg_match($pattern_field, $field, $matches) ) {
    						$field_name[]	= (isset($matches[1]))? trim($matches[1]) : 'name not found' ;
    						$field_format[]	= (isset($matches[2]))? trim($matches[2]) : 'format not found' ;
    						$field_key[]	= (isset($matches[3]))? trim($matches[3]) : 'key not found' ;
    					}
    				}
     
    				if ($debug) {
    					$this->arr_dump(array('keys'=>$field_key, 'names'=>$field_name, 'formats'=>$field_format) , 'Matching results');
    				}
     
     
    				// Comparison $db_fields with $data_array: can accept missing fields but no mismatching fields (thus avoiding typos in $data_array)
    				$mismatching_fields = array_diff(array_keys($data_array), $field_name);
    				if ( count($mismatching_fields) != 0) {
     
    					if ($debug) {
    						echo '<br/>PDO3 db Vs data fields issue';
    						$this->arr_dump($mismatching_fields, 'mismatch' );
    					}
     
    					return false;
    				}
     
    				// Check data_array format Vs dB_fields format
    				// #########
    				// # To do
    				// #########
     
     
    		// Build SQL statement
    			// INSERT $sql = "INSERT INTO $table_name ($fields) VALUES ($values)";
    				$fields = '';
    				$values = '';
    				if ( strtolower($sql_type) == 'insert') {
    					foreach($data_array as $field => $data) {
    						$fields .= "$field, ";
    						$values .= "'$data', ";
    					}
    					$fields = preg_replace('/, $/', '', $fields);
    					$values = preg_replace('/, $/', '', $values);
    					$sql = "INSERT INTO $table_name ($fields) VALUES ($values)";
    					if ($debug) echo '<br/>sql statement: ' . $sql;
    				}
     
    			// UPDATE
    				if ( strtolower($sql_type) == 'update') {
    					if ($debug) echo '<br/>Starting update....' ;
     
     
    					// Update Condition $sql_condition
    						$sql_condition = '';
     
    						// dB primary key field required
    							$needle = 'PRI';
    							$pk_key = trim($this->recursiveArraySearch($needle, $field_key) ) ;
    							if (false === $pk_key ) {		// NOTA: '===' is a must as $pk_key can be 0 and then understood as 'false'
    								if ($debug) {
    									echo '<br/>PDO3 Err: no primary key found' ;
    									$this->arr_dump( array('pk key'=> $pk_key , 'needle'=>$needle, 'fields' => $field_key), 'Check needle & arrays') ;
    								}
    								return false;
    							}
    							else {
    								if ($debug) {
    									echo '<br/>found primary key in db datas array <b>keys</b> :' . $pk_key;
    									$this->arr_dump( $field_key , 'field_key');
    								}
    							}
     
    						// data_array primary key check
    						$pk_value = (array_key_exists($field_name[$pk_key], $data_array))? $data_array[$field_name[$pk_key]] : false ;
    						if (false === $pk_value || empty($pk_value) )  {
    							if ($debug) $this->arr_dump( array('key'=>$pk_key, 'value'=>$pk_value, 'data'=>$data_array) , 'PDO3 Err: no data content for primary key sql condition');
    							return false ;
    						}
    						else if ($debug) echo '<br/>found primary key value in db datas array <b>values</b>:' . $pk_value;
     
    						// db_field: check if table field of row(pk) exists (in order to being able to properly set the sql_condition)
    						// #########
    						// # to do
    						// #########
     
    						// Sql condition (base)
    						$sql_condition = $field_name[$pk_key] . '=' . $pk_value;
    						if ($debug) echo '<br/>sql update condition base: ' . $sql_condition ;
     
    						// Sql condition (options)
    						// #########
    						// # to do: check if necessary (prepared request) that the optional sql condition is not matching with forbidden conditions like 1=1 or DELETE..
    						// #########
     
    						// Sql condition (global)
    						$sql_condition .= ($optional_condition)? ' AND ' . $optional_condition : '' ;
     
    						// Build sql statement (not necessary to remove PK field from fields to alter)
    						$fields = '';
    						foreach($data_array as $field => $data) {
    							$fields .= "$field = '$data', ";
    						}
    						// remove trailing ','
    						$fields = preg_replace('/, $/', '', $fields);
     
    					// Sql final statement
    						$sql = "UPDATE $table_name SET $fields WHERE $sql_condition";
    						if ($debug) echo '<br/>sql statement: ' . $sql;
    				}
     
     
    		// Prepare & Execute Sql statement (sothat there is no need to use "quote" method for data protection against SQL injection)
    			if ($sql) {
    				$query	= $this->_pdo->prepare($sql) ;
    				$result	= $query->execute() ;
    				$count	= $query->rowCount(); // Check affected rows by INSERT or UPDATE (nota : this methode does not work for SELECT)
    				if (strtolower($sql_type) == 'insert') {
    					$affected_rows = ($count > 0) ? true : false ;
    				}
    				else {
    					$affected_rows = ($count >= 0) ? true : false ;
    				}
    			}
     
    		// Return Value
    			if ($result && $affected_rows) {
    				return true;
    			}
    			else{
    				return false;
    			}
     
     
    	} /* fin saveData */
     
    	/**
    	* Backup d'un champ dans un champ destination (alors écrasé)
    	* @return	: $count nbre d'enregistrement affectés ou false
    	*/
    	public function copyPaste($table_name, $field_source, $field_destination, $sql_condition="1=1") {
    		// ex: UPDATE product_supply_base SET  `previous_selling_price` = product_selling_net_price WHERE `product_ref` = 10009
     
    		// Verif existante Table
    			$table_ok = $this->checkTableExists( $table_name) ;
    			if ($this->_debug && false ===  $table_ok) echo '<br/>DEBUG Copy/Paste: table Nok';
    			if ( false ===  $table_ok ) return false ;
     
    		// Verif existante Fields
    			$source_Ok = $this->checkFieldExists( $table_name, $field_source ) ;
    			$destination_Ok = $this->checkFieldExists( $table_name, $field_destination ) ;
    			if ($this->_debug && (false ===  $destination_Ok || false ===  $source_Ok) ) echo '<br/>DEBUG Copy/Paste: fields Nok';
    			if ( false ===  $destination_Ok || false ===  $source_Ok ) return false ;
     
    		// Verif que Destination différente de la Source
    			$egalite = ($field_source == $field_destination) ;
    			if ($this->_debug && $egalite) echo '<br/>DEBUG Copy/Paste: champ source et destination identiques: <b>Nok</b>';
    			if ($egalite) return false ;
     
    		// Verif sql condition
    			list($condition_field_name, $condition_field_value) = explode("=", $sql_condition);
    			$test = (empty($condition_field_name) || empty($condition_field_name) || false === $this->checkFieldExists( $table_name, $condition_field_name ) );
    			if ($this->_debug && $test ) echo '<br/>DEBUG Copy/Paste: sql condition Nok';
    			if ( $test ) return false ;
     
    		// Update Copy /Paste
    			$sql = "UPDATE $table_name SET $field_destination = $field_source WHERE $sql_condition" ;
    			$query	= $this->_pdo->prepare($sql) ;
    			$result	= $query->execute() ;
    			$count	= $query->rowCount();
     
    		if ($result) {
    			return $count;
    		}
    		else {
    			if ($this->_debug && false === $result) echo '<br/>DEBUG Copy/Paste: result Nok';
    			return false ;
    		}
    	}
     
    	/**
    	* Comptage du nbre enregistrements (par requete préparée)
    	* @return	: $count
    	*/
    	public function countId($table_name, $id_name, $id_value) {
    		// méthode alternative : voir http://php.developpez.com/faq/?page=pdo#pdo-count
     
    		// #####
    		// NOTA: à modifier pour ne selectionner que les enregistrements de la PK (utiliser fonction existante)
    		// #####
     
    		$sql_prep = "SELECT COUNT(*) FROM $table_name WHERE $id_name= :id_value" ;
    		$query	= $this->_pdo->prepare($sql_prep) ;
    		$query->execute(array( ':id_value' => $id_value ));
    		$count = $query->fetchColumn();
    		$query->closeCursor();
    		return $count;
    	}
     
     
    	/**
    	* Comptage du nbre enregistrements DISTINCTS pour un champ donné avec condition (par requete préparée)
    	* @return	: $count
    	*/
    	private function countDistinct($table_name, $field, $sql_condition=false) {
     
    		// Verif sql condition
    		list($condition_field_name, $condition_field_value) = explode("=", $sql_condition);
    		if ( empty($condition_field_name) || empty($condition_field_name) || false === $this->checkFieldExists( $table_name, $condition_field_name )) return false ;
     
    		// Query
    		$sql_prep = "SELECT DISTINCT $field FROM $table_name WHERE $sql_condition" ;
    		$query	= $this->_pdo->prepare($sql_prep) ;
    		$query->execute();
    		$count = count($query->fetchAll());
    		$query->closeCursor();
    		return $count;
    	}
     
    	/**
    	* Comptage du nbre enregistrements DISTINCTS pour un champ donné avec condition (par requete préparée)
    	* @return	: $count
    	*/
     
    	public function isUnique($table_name, $field, $sql_condition) {
    		// méthode alternative : voir http://php.developpez.com/faq/?page=pdo#pdo-count
     
    		$count = $this->countDistinct($table_name, $field, $sql_condition) ;
     
    		$unique = ($count == 1) ? true : false ;
     
    		return $unique;
    	}
     
     
    	/**
    	* Recherche récursive de la clé de niveau 0 d'une valeur située dans un tableau multidimensionnel de profondeur D
    	* @source	: http://fr.php.net/manual/fr/function.array-search.php#91365
    	* @return	: $key($needle)
    	*/
    	private function recursiveArraySearch($needle, $haystack) {
    		foreach($haystack as $key=>$value) {
    			$current_key=$key;
    			if($needle===$value OR (is_array($value) && $this->recursiveArraySearch($needle, $value) !== false)) {
    				return $current_key;
    			}
    		}
    		return false;
    	}
     
     
    	/** 01/02/2010
    	* Vérifie que l'extension pdo est bien chargée et est bien en phase avec le type des requetes émises dans la classe PDO2
    	* @source	: http://php.developpez.com/faq/?page=pdo#pdo-loaded-drivers
    	* @return	: true ou false
    	*/
    	// ######### à ré-implémenter en amont dans PDO2 avec gestion erreur ################
     	private function checkPdoExtension() {
    		if ($this->_debug) echo '<br/>Checking for PDO extension loaded';
    		$pdo_ext = false;
    		foreach(get_loaded_extensions() as $extension) {
    			if(strpos(strtolower($extension), 'pdo') !== 'pdo') {
    				$pdo_ext = true;
    			}
    		}
    		if ($this->_debug && false == $pdo_ext) echo '<br/>PDO extension not loaded';
    		return $pdo_ext;
    	}
     
     
    	/**
    	* fonction de debugage
    	*/	
    	private function arr_dump($array, $text='') {
    		echo '<br/>Dump('.$text.')<pre>'; print_r($array); echo '</pre>' ;
    	}
     
     
    } /* fin de la Classe */
     
    ?>

  15. #35
    Membre expert Avatar de RunCodePhp
    Profil pro
    Inscrit en
    Janvier 2010
    Messages
    2 962
    Détails du profil
    Informations personnelles :
    Localisation : Réunion

    Informations forums :
    Inscription : Janvier 2010
    Messages : 2 962
    Points : 3 947
    Points
    3 947
    Par défaut
    J'ai pris ton code auparavant (l'ancien), je l'ai quelque peu épuré, mais j'obtiens tout de même la même erreur attendue : Un erreur fatal.

    Je remets ce code simplifé (ci-dessous).
    Tu pourrais le tester dans une page vierge, comme je l'ai fais.
    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
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
     
    <?php
    class PDO2 extends PDO {
    	private static $_instance;
    	private static $_dsn	= 'mysql:host=127.0.0.1;dbname=une_bdd';
    	private static $_dbuser	= 'machine';
    	private static $_dbpwd	= 'mysql';
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct() { }
     
    	/* Singleton */
    	public static function getInstance() {
    		if (!isset(self::$_instance)) {
    			try {
    				self::$_instance = new PDO(self::$_dsn , self::$_dbuser , self::$_dbpwd);
    			}
    			catch (PDOException $e) {
    				echo 'Erreur de la classe PDO2 - passer en mode DEBUG';
    				die();
    			}
    		}
    		return self::$_instance; 
    	}
    }
     
     
     
    class PDO3 extends PDO2 {
     
    	/* Variables de la classe */
    	protected $_pdo	= NULL;
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct($pdo) {
    		if (isset($pdo)) {
    			echo 'Classe : '.__CLASS__.'<br />';
    			$this->_pdo = $pdo;
    		}
    		else {
    			throw new PDOException("No pdo connection");
    		}
    	}
    	/**
    	* Comptage du nbre enregistrements DISTINCTS pour un champ donné avec condition (par requête préparée)
    	* @return	: $count
    	*/
    	private function countDistinct() {
    		echo 'PDO2::countDistinct()<br />';
    	}
    }
     
     
    //
    try {
    	$connect = PDO2::getInstance();
    	$pdo = new PDO3($connect);
    	$pdo->countDistinct();
    }
    catch (PDOException $e) {
    	echo ERR_41.': ' .$e->getMessage().'<br/>';
    }
    ?>
    Après, qu'il y est une erreur de conception, ok, mais savoir quand même si cette erreur est liée à l'environnement ou pas me semble pas mal non plus.

  16. #36
    Membre expert Avatar de RunCodePhp
    Profil pro
    Inscrit en
    Janvier 2010
    Messages
    2 962
    Détails du profil
    Informations personnelles :
    Localisation : Réunion

    Informations forums :
    Inscription : Janvier 2010
    Messages : 2 962
    Points : 3 947
    Points
    3 947
    Par défaut
    Avec l'intégralité de ton code, j'obtiens toujours cette erreur fatal :
    Fatal error: Call to private method PDO3::countDistinct() from context


    A savoir aussi que j'ai obtenu en 1er cette erreur :
    Fatal error: Method PDO3::__tostring() cannot take arguments in ... on line 83

    J'ai tout simplement supprimer le $object en argument.
    Il me semble que cette méthode __toString__ ne devrait pas contenir d'argument.

  17. #37
    Membre éclairé
    Profil pro
    Inscrit en
    Mars 2005
    Messages
    625
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mars 2005
    Messages : 625
    Points : 822
    Points
    822
    Par défaut
    Hello,

    Pour rebâtir un peu ton machin (même si ce n'est pas le sujet de thread initial), il faut oublier la piste PDO2/PDO3 (enfin t'es pas obligé mais tu te compliques la vie pour rien) s'inspirer du code proposé par Julp ( http://julien-pauli.developpez.com/t...etiers#LII-B-1 )

    Vu que tu peux pas privatiser le constructeur pour en faire un singleton, il faut passer par une astuce :

    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
    class SinglePDO extends PDO{
     
    	private static $instance;
    	private static $verrou = true;
     
    	public function __construct( $dsn , $uname, $upass , $d_opts = array() ){
    		if( self::$verrou ) trigger_error( 'Construction interdite, mode singleton. Utilisez la méthode SinglePDO::getInstance' , E_USER_ERROR );
    		parent::__construct( $dsn , $uname, $upass , $d_opts );		
    	}
     
    	public static function getInstance( $dsn , $uname , $upass , $d_opts = array() ){
    		if( self::$instance == null ){
    			self::$verrou = false;
    			self::$instance = new SinglePDO( $dsn , $uname , $upass, $d_opts );
    			self::$verrou = true;
    		}
    		return self::$instance;
    	}
     
    	// et tu rajoutes maintenant la suite de tes méthodes de PDO3
     
    }
    Code écrit à l'arrache, non testé et incomplet (pour ne pas repomper totalement le code de julp qui est proprio... )

  18. #38
    Membre éclairé
    Avatar de hornetbzz
    Homme Profil pro
    Directeur commercial
    Inscrit en
    Octobre 2009
    Messages
    482
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : France

    Informations professionnelles :
    Activité : Directeur commercial

    Informations forums :
    Inscription : Octobre 2009
    Messages : 482
    Points : 773
    Points
    773
    Par défaut
    Vraiment un grand merci de m'aider les gars.

    PetitBidon:
    Merci, tu m'as donné le point de départ pour retravailler mes classes, il faut que j'y passe quelques heures (j'aurais dû choisir "PetitNoob" comme nom).

    Run:
    Et bien ce test confirme les précédents, je n'ai pas de Fatal Error.
    Et là, je ne vois qu'une seule différence dans le test : j'utilise la SPL pour charger les classes, scindées en 2 fichiers distincts (+ celui du code principal d'appel). Je ne sais pas si cela peut influencer.

    code principal
    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
    <?php
     
    	require_once("../../lib/fonction_classes.php");
    	spl_autoload_register ('LoadPHPClass');
     
    try {
    	$connect = test_PDO2::getInstance();
    	$pdo = new test_PDO3($connect);
    	$pdo->countDistinct();
    }
    catch (PDOException $e) {
    	echo ERR_41.': ' .$e->getMessage().'<br/>';
    }
     
    ?>
    Classe test_PDO2
    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
    <?php
    class test_PDO2 extends PDO {
    	private static $_instance;
    	private static $_dsn	= 'mysql:host=127.0.0.1;dbname=ma_db' ;
    	private static $_dbuser	= '_user' ;
    	private static $_dbpwd	= '_pwd' ;
     
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct() { }
     
    	/* Singleton */
    	public static function getInstance() {
    		if (!isset(self::$_instance)) {
    			try {
    				self::$_instance = new PDO(self::$_dsn , self::$_dbuser , self::$_dbpwd);
    			}
    			catch (PDOException $e) {
    				echo 'Erreur de la classe PDO2 - passer en mode DEBUG';
    				die();
    			}
    		}
    		return self::$_instance; 
    	}
    }
    ?>
    Classe test_PDO3.php
    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
    <?php
     
    class test_PDO3 extends test_PDO2 {
     
    	/* Variables de la classe */
    	protected $_pdo	= NULL;
     
    	/* Constructeur : héritage public obligatoire par héritage de PDO */
    	public function __construct($pdo) {
    		if (isset($pdo)) {
    			echo 'Classe : '.__CLASS__.'<br />';
    			$this->_pdo = $pdo;
    		}
    		else {
    			throw new PDOException("No pdo connection");
    		}
    	}
    	/**
    	* Comptage du nbre enregistrements DISTINCTS pour un champ donné avec condition (par requête préparée)
    	* @return	: $count
    	*/
    	private function countDistinct() {
    		echo 'execution PDO2::countDistinct()<br />';
    	}
    }
    ?>
    Le retour serveur=> client dans mon browser (200 Ok) :
    Classe : test_PDO3
    execution PDO2::countDistinct()

  19. #39
    Membre à l'essai
    Profil pro
    Inscrit en
    Mars 2010
    Messages
    11
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Mars 2010
    Messages : 11
    Points : 14
    Points
    14
    Par défaut
    Bonjour a tous,

    J'ai simplifié le code et isolé le comportement inattendu.
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    10
    11
    12
    13
    class Foo extends PDO
    {
    	public function __construct( $_dsn , $_dbuser , $_dbpwd) {
    		parent::__construct( $_dsn , $_dbuser , $_dbpwd);
    	}
    	private function bar()
    	{
    		echo 'Foo::bar';
    	}
    }
     
    $instance = new Foo('mysql:host=127.0.0.1;dbname=test' , 'not_root_user' , '');
    $instance->bar();

    C'est une bug de PHP, essaye avec une version supérieur et dit nous ce qu'il en est.

    @++

  20. #40
    Membre éclairé
    Avatar de hornetbzz
    Homme Profil pro
    Directeur commercial
    Inscrit en
    Octobre 2009
    Messages
    482
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 57
    Localisation : France

    Informations professionnelles :
    Activité : Directeur commercial

    Informations forums :
    Inscription : Octobre 2009
    Messages : 482
    Points : 773
    Points
    773
    Par défaut
    Ok merci pour la confirmation.

    Je vais tenter un passage en 5.3 à la prochaine release.

    Je passe en résolu, et merci encore à RunCode et PetitBidon.

+ Répondre à la discussion
Cette discussion est résolue.
Page 2 sur 3 PremièrePremière 123 DernièreDernière

Discussions similaires

  1. [POO] Méthode privée / Attribut public
    Par Benoit.44 dans le forum Général JavaScript
    Réponses: 6
    Dernier message: 17/08/2013, 17h15
  2. Réponses: 6
    Dernier message: 07/01/2009, 11h57
  3. Réponses: 7
    Dernier message: 19/03/2007, 21h25
  4. Réponses: 4
    Dernier message: 17/03/2007, 02h06
  5. Réponses: 8
    Dernier message: 04/06/2004, 10h13

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo