PHP|Débutant :: Forums

j'ai pas compris....

Ok, compris.

J'imagine que c'est mieux comme ça :
$sql="SELECT GROUP_CONCAT( `auto` ) AS \"autos\" FROM pieces_historique WHERE categories=\"$cat\" GROUP BY categories";

Ca marche nickel ! merci bcp pour m'avoir fait voir mon erreur d'inatention ...

++
rv

normal, tu ne fait aucune boucle
donc, un seul tuple est lu
while($dataautos = mysql_fetch_assoc( $res )
{
ton code
};

a++

heu j'ai pas compris, tu veut faire quoi avec ta requete ligne 10 ?

parce que bon le where n'est "remplis" qu'en sachant ce que tu veut faire de la requete

si c'est le résultat de la requete précedente c'est $dataautos['autos'] qu'il faut mettre ...

@+

on va faire simple parce que je pige pas trop ta tambouille :d

le résultat de ta requete SQL (avec le group concat) te retourne des lignes avec dedans des chose comme ça : 3/9/15/18/30/59 ?
$dataautos['autos'] est ce que tu récupère avec mysql_fetch_assoc dans la boucle while

ligne 8 tu cherche a savoir si $idvoiture (sortie d'un chapeau magique ?) est dans les valeurs fournis par la requete. Ceci par le biais de explode qui te fournis les valeurs dans un tableau et in_array qui cherche si une valeur est dans un tableau.

Si ton $idauto est ce que tu veut mettre dans la requete sql ligne 10, je vois pas trop à quoi sert le reste.

on n'a pas assez d'info sur ce que tu veut faire, car dans ton cas tu pourrais très bien utiliser le explode après ta requete et regarder la valeur à l'index 3 du tableau
$tableau = explode( '/', $dataautos['autos'] ) ;
//$tableau[3] ==18 si la structure de ce que tu a mis a ton 1er message est toujours la même.

Mon avis sur la question :
Ta table est mal pensée. Je pense que tu doit pouvoir faire 'plus simple' que ce que tu fait actuellement.

si tu nous explique comment tu a créer ta table, peut être pourrions nous mieux comprendre ce que tu souhaite faire (mais la c'est super alambiqué pour pas grand chose j'ai l'impression).

@+

Wé c'est plus clair

Tu a un probleme de modelisation de ta base.

Par exemple pour modifier ton champ tu est obligé de le faire a la main au risque de faire anerie. Imagine la taille du champ pour une rondelle ou les pneus de ma saxo

Il te faut une table auto : ok
Une table pieces : ok ( sans le champ disgracieux avec les id auto)
Une table piecesauto qui va faire la jointure entre les deux premieres
Par exemple
Id clef primaire
Idpieces
Idauto

Pas besoin d'autre chose et cela te permet de faire ta requete facilement

Je te fait un exemple plus concret si besoin quand je rentre

@+

Le principe est simple : 3 tables pour tout faire
une table auto (qui contient les modèle de véhicule)
create table autos(
id int unsigned not null auto_increment,
designation tinytext,
primary key(id)
)type=myisam;
insert into autos values (null,'saxo'),(null,'yaris');
une table pour les pieces détachées
create table pieces (
id int unsigned not null auto_increment,
designation tinytext,
primary key(id)
)type=myisam;

insert into pieces values(null,'pneu saxo'),(null,'penu yaris'),(null, 'frein saxo'),(null,'cardan yaris'),(null,'capot saxo');
une table qui fait la liaison entre les deux premières
create table piecesautos(
id int unsigned not null auto_increment,
idpiece int unsigned not null,
idauto int unsigned not null,
primary key (id)
)type=myisam;
insert into piecesautos values(null,1,1),(null,2,2),(null,3,1),(null,4,2),(null,5,1);

A partir de la si tu choisis une auto dans la 1ère table tu peu récupérer ls pièces qui y sont associées en utilisant une jointure
http://sqlpro.developpez.com/cours/sqlaz/jointures/.

Grosso modo c est quoi une jointure ?
He ben c'est les fait de faire une requete sur deux tables en même temps avec des infos identique entre les deux tables.
Dans notres cas idauto c'est la clef primaire de la table auto. Elle permet d'identifier un véhicule car c'est "id" est UNIQUE (a cause de la primary key ).
Même chose pour idpiece.

si tu choisis une voiture tu aura la clef primaire de la voiture que tu pourra utiliser sur la table piecesautos
select designation, piecesautos.id as idpiecesauto from piecesautos inner join pieces on idpiece=pieces.id where idauto=1;
select designation ,piecesautos.id as idpiecesauto from piecesautos inner join autos on idauto=autos.id where idpiece=6;
ici le 1 représente la valeur fournis par ton 1er formulaire ou lien. (donc $_GET['idauto'] par exemple)).

L'avantage de cette solution c'est que si tu te trompe sur une pice c'est facile a modifier et tu n'est pas obligé de modifier un champ bizard où tu a des chances de faire une annerie

si par exemple tu veut savoir a quelle voiture correspond une pièce (et afficher le nom de la piece) :
select autos.designation as auto, pieces.designation as piece from piecesautos inner join autos on idauto=autos.id inner join pieces on idpiece
=pieces.id where idpiece=6;

Comment aller plus loin ?
Par exemple en créant une table constructeurs qui contiendra .... (renault, citroen, peugeot, bmw etc). Dans la table autos tu ajoutera le idconstructeur
ensuite les requetes sont un poil plus compliqué mais le principe est  la :d

par exemple on peu avoir

create table pays (
id smallint unsigned not null auto_increment,
nom tinytext not null,
primary key(id)
)type=myisam;
insert into pays values (null,'France'),(null,'Japon'),(null,'Allemagne'),(null,'Italie');

create table constructeurs (
id int unsigned not null auto_increment,
idpays smallint unsigned not null,
nom tinytext not null,
primary key(id)
)type=myisam;
insert into constructeurs values (null,1,'Citroën'),(null,1,'Peugeot'),(null,1,'Renault'),(null,2,'toyota'),(null,4,'Ferrari'),(null,4,'maserati'),(null,3,'BMW');

create table autos(
id int unsigned not null auto_increment,
idconstructeur int unsigned not null,
designation tinytext,
primary key(id)
)type=myisam;
insert into autos values (null,1,'saxo'),(null,4,'yaris');

pour avoir les voitures et les constructeurs et pays associés

select autos.designation as auto, constructeurs.nom as constructeur, pays.nom as pays from autos inner join constructeurs on autos.idconstructeur=constructeurs.id inner join pays on constructeurs.idpays=pays.id;
résultat
+-------+--------------+--------+
| auto  | constructeur | pays   |
+-------+--------------+--------+
| saxo  | Citroën      | France |
| yaris | toyota       | Japon  |
+-------+--------------+--------+

tu trouvera à cette adresse http://phpdebutant.org/wall2162.php le fichier sql que j'ai utilisé pour mes tests pour ce que j'ai mis plus haut

amuse toi bien

@+

PS : le lien sur les jointures est a lire en priorité si tu ne connais pas

po de quoi ça m'a fait une p'tite révision :d

@+

Cela vient du fait que tu as des lacunes sur comment interroger une table.
La petite lecture que tu as du faire un peu trop en diagonale : http://phpdebutant.org/article66.php

Sa requête est bonne, mais il n'interprète aucunement la ressource retournée par mysql_query() : cf code de son dernier post.
Donc il a beau avoir une bonne requête il ne sait pas exploiter son résultat.

raaa xTG t'était po clair

@+

Saluton,
En fait, xTG, tu verras, à l'usage, que c'est surtout Moogli qui n'est jamais trop clair.
Hein ranuleto ?

Il serait tout simplement peut-être bon de rappeler à chacun le sens du verbe anglais to fetch