JeuWeb Créez votre jeu par navigateur !

Plume · (Modification du message : 13-10-2008, 12:18 PM par Sephi-Chan.)

Plop !

C't'encore mwa ! Content de me revoir - déjà - ?

Cette fois-ci, j'ai cette requête :

Code PHP :
<?php 

SELECT

  warehouses.id AS warehouse_id,

  warehouses.capacityMech,

  warehouses.capacityWeight,

  warehouses.countries_id,

  player_has_warehouse.name AS warehouse_name,

//  player_has_warehouse.weight,

//  player_has_warehouse.nb_mechs,

  countries.name AS country_name

FROM

  player_has_warehouse,

  warehouses

JOIN countries

  ON warehouses.countries_id = countries.id

WHERE player_has_warehouse.players_id = $this->playerSess['id']

  AND player_has_warehouse.warehouses_id = warehouses.id

**Sephi-Chan** · 13-10-2008, 12:20 AM

La première idée qui me vient est de simplement intégrer les calculs comme des sous-requêtes. Comme cela :

Code PHP :
<?php 

SELECT    warehouses.id AS warehouse_id,

        warehouses.capacityMech,

        warehouses.capacityWeight,

        warehouses.countries_id,

        player_has_warehouse.name AS warehouse_name,

        (

            (

                SELECT    SUM(armors.weight)

                FROM    armors, players_has_armor

                WHERE    player_has_armor.players_id = $this->playerSess['id']

                AND        player_has_armor.armors_id = armors.id

            )

            +

            (

                SELECT    SUM(weapons.weight)

                FROM    weapons, player_has_weapon

                WHERE    player_has_weapon = $this->playerSess['id']

                AND        player_has_weapon.weapons_id = weapons.id

            )

        ) AS weight,

        (

            SELECT    COUNT(player_has_exomech.id)

            FROM    player_has_exomech

            WHERE    player_has_exomech.players_id = $this->playerSess['id']

        ) AS nb_mechs,

        countries.name AS country_name

FROM    player_has_warehouse,

        warehouses

JOIN    countries

ON        warehouses.countries_id = countries.id

WHERE    player_has_warehouse.players_id = $this->playerSess['id']

AND        player_has_warehouse.warehouses_id = warehouses.id;

Plume · (Modification du message : 13-10-2008, 11:47 AM par Plume.)

Huhu ! Merci les gens ! Encore une fois, c'était plutôt con comme requête.

Pour ce qui est du cache, je verrais quand j'aurais fini quelque chose de fonctionnel. Je ne connais pas les techniques de mises en cache et tout donc j'préfère pas perdre trop de temps, lancer une béta qui sera éprouvée. Pendant cette phase d'épreuve, j'pourrais me pencher sur les optimisations en même temps que les corrections.

Maintenant, j'dis ça mais j'suis ouvert à toutes pistes, propositions,... :]

Encore merci pour le coup de main !

@tchaOo°

[Edit]
J'allais oublié de dire :
Concernant les jointures dans la WHERE Clause, elles sont possibles, elles sont plus simples alors pourquoi s'en priver ?

J'ai cru lire effectivement que quelque part, c'est plus approprié de le faire dans les règles de l'art. Maintenant, si on me convainc du pourquoi du comment, j'suis prêt à faire l'effort. Sans ça, j'vois pas pourquoi je devrais me les casser si j'n'y perds rien ;]

Anthor · 13-10-2008, 11:49 AM

Ben dans le where, tu obliges le moteur de MySQL à réécrire la requêtes pour y mettre les jointures, c'est tout simple Smile

Plume · 13-10-2008, 11:53 AM

Mouarf ! Il est grand, vu la complexité de la jointure, ça doit pas le mettre à genouxx ?

pascal · 13-10-2008, 12:10 PM

Plume a écrit :[Edit]
J'allais oublié de dire :
Concernant les jointures dans la WHERE Clause, elles sont possibles, elles sont plus simples alors pourquoi s'en priver ?

J'ai cru lire effectivement que quelque part, c'est plus approprié de le faire dans les règles de l'art. Maintenant, si on me convainc du pourquoi du comment, j'suis prêt à faire l'effort. Sans ça, j'vois pas pourquoi je devrais me les casser si j'n'y perds rien ;]

d'un point de vue syntaxe, c'est plus correct de mettre les jointures via des JOIN, et de les séparer du WHERE.

pourquoi ?

le WHERE représente des conditions, des filtres sur les données, pas des liens entre tables.

Lorsque je fais une requête, je dessine les tables sur papier, avec les liens entre elles (JOIN) et ensuite je m'intéresse aux conditions (WHERE) c'est à dire quelles lignes récupérer.

c'est une question de lisibilité et de convention de programmation.

A+

Pascal

Plume · 13-10-2008, 12:22 PM

Grumpf !

J'verrais dans l'avenir pour changer mes habitudes. Par contre pas la motivation de revenir sur ce qui est déjà fait ^_^'

Cela dit, j'connais pas beaucoup de monde qui comprenne mieux :

Code PHP :
<?php 

SELECT *

FROM

  player_has_warehouse,

  warehouses

JOIN countries

  ON warehouses.countries_id = countries.id

JOIN player_has_warehouse

  ON player_has_warehouse.warehouses_id = warehouses.id

WHERE player_has_warehouse.players_id = $this->playerSess['id'];

Plume · (Modification du message : 13-10-2008, 12:31 PM par Plume.)

Verdict : Ma requête n'était pas bonne. J'la corrige ^^

Code PHP :
<?php 

/* SELECT

     *  warehouses.id AS warehouse_id,

     *  warehouses.capacityMech,

     *  warehouses.capacityWeight,

     *  warehouses.countries_id,

     *  player_has_warehouse.name AS warehouse_name,

     *  (

     *    (

     *      SELECT SUM(armors.weight)

     *      FROM armors, player_has_armor

     *      WHERE player_has_armor.players_id = $this->playerSess['id']

     *        AND player_has_armor.armors_id = armors.id

     *    )

     *    +

     *    (

     *      SELECT SUM(weapons.weight)

     *      FROM weapons, player_has_weapon

     *      WHERE player_has_weapon = $this->playerSess['id']

     *        AND player_has_weapon.weapons_id = weapons.id

     *    )

     *  ) AS weight,

     *  (

     *    SELECT COUNT(player_has_exomech.id)

     *    FROM player_has_exomech

     *    WHERE player_has_exomech.players_id = $this->playerSess['id']

     *  ) AS nb_mechs,

     *  countries.name AS country_name

     * FROM

     *  warehouses

     * JOIN countries

     *  ON warehouses.countries_id = countries.id

     * JOIN player_has_warehouse

     *  ON player_has_warehouse.warehouses_id = warehouses.id

     * WHERE player_has_warehouse.players_id = $this->playerSess['id']

     */

    $selectWarehouse =

        'SELECT ' .

        ' %s, ' .

        ' %s, ' .

        ' %s, ' .

        ' %s, ' .

        ' %s, ' .

        ' ( ' .

        '   ( ' .

        '     SELECT SUM(%s) ' .

        '     FROM %s, %s ' .

        '     WHERE %s = %d ' .

        '       AND %s = %s ' .

        '   ) ' .

        '   + ' .

        '   ( ' .

        '     SELECT SUM(%s) ' .

        '     FROM %s, %s ' .

        '     WHERE %s = %d ' .

        '       AND %s = %s ' .

        '   ) ' .

        ' ) AS %s, ' .

        ' (' .

        '   SELECT COUNT(%s) ' .

        '   FROM %s ' .

        '   WHERE %s = %d ' .

        ' ) AS %s, ' .

        ' %s ' .

        'FROM %s ' .

        'JOIN %s ' .

        ' ON %s = %s ' .

        'JOIN %s' .

        ' ON %s = %s ' .

        'WHERE %s = %d;';

    $selectWarehouseQuery = sprintf(

      $selectWarehouse,

      $this->tableWarehouses.'.id AS warehouse_id',

      $this->tableWarehouses.'.capacityMech',

      $this->tableWarehouses.'.capacityWeight',

      $this->tableWarehouses.'.countries_id',

      $this->tablePlayerHasWarehouse.'.name AS warehouse_name',

      $this->tableArmors.'.weight',

      $this->tableArmors,

      $this->tablePlayerHasArmor,

      $this->tablePlayerHasArmor.'.players_id',

      $this->playerSess['id'],

      $this->tablePlayerHasArmor.'.armors_id',

      $this->tableArmors.'.id',

      $this->tableWeapons.'.weight',

      $this->tableWeapons,

      $this->tablePlayerHasWeapon,

      $this->tablePlayerHasWeapon.'.players_id',

      $this->playerSess['id'],

      $this->tablePlayerHasWeapon.'.weapons_id',

      $this->tableWeapons.'.id',

      'weight',

      '*',

      $this->tablePlayerHasExomech,

      $this->tablePlayerHasExomech.'.players_id',

      $this->playerSess['id'],

      'nb_mechs',

      $this->tableCountries.'.name AS country_name',

      $this->tableWarehouses,

      $this->tableCountries,

      $this->tableWarehouses.'.countries_id',

      $this->tableCountries.'.id',

      $this->tablePlayerHasWarehouse,

      $this->tablePlayerHasWarehouse.'.warehouses_id',

      $this->tableWarehouses.'.id',

      $this->tablePlayerHasWarehouse.'.players_id',

      $this->playerSess['id']

    );

**Sephi-Chan** · 13-10-2008, 12:49 PM

Je pense que rendre constant le nom des tables et des colonnes améliorerait la lisibilité de l'ensemble, les placeholders (ou marques substitutives xD) ont plutôt intérêt à être utilisés pour les paramètres variables.

Si jamais tes noms de table ou de colonnes changent, tu peux au pire lancer un rechercher et remplacer dans ton modèle : ça ne nuit pas à la maintenabilité du projet.

Sephi-Chan

Plume · 13-10-2008, 02:51 PM

Uè, j'vais modifier ça, ça économisera quelques lignes ^_^'

@tchaOo°

Sujets apparemment similaires…
Sujet		Auteur	Réponses	Affichages	Dernier message
	Besoin d'aide pour le choix du language et conseils pour avancer	hercull	17	6 095	17-12-2012, 08:55 PM Dernier message: Xenos
	Comment construire une application avec Erlang	Sephi-Chan	6	4 057	27-04-2012, 07:45 PM Dernier message: srm
	[SQL] aide pour une requête "récursive"	Ter Rowan	6	4 507	19-03-2012, 02:39 PM Dernier message: Ter Rowan
	aide pour un requête un peu spéciale	php_addict	9	4 118	14-11-2010, 11:11 AM Dernier message: srm
	[Résolu][PDO] Connaître le SQL d'une requête préparée	Ter Rowan	9	3 979	21-12-2009, 08:38 PM Dernier message: Anthor