Reprise du message précédent :
Dans la pratique, ton instance de Stock va être initialisée en faisant appel à la méthode statique de la classe Voiture, et à-partir de là elle fait sa tambouille toute seule.
Mais à-partir du moment où elle lit ou met à jour des infos concernant des voitures, elle doit passer par la classe Voiture.

Ah non.
Non de non de non.
Alors maintenant c'est la fête du slip, la couche d'accès aux données instancie des objets du modèle...

Bah moi je verrai plutôt un truc comme ça :  

 
Code de merde à la va vite mais ça exprime mon idée quoi. Ca me semble plutôt logique.

ah oui, donc t'as pas vraiment de couche d'accès aux données, donc.
Attends, je te donne ma version...

j'ai dit exemple à la va-vite, j'ai bien sur une couche d'accès aux données dans mes applis ^^
mais ton exemple m'intéresse grandement, j'ai l'impression que j'ai un pbl avec ta définition de ce que doit faire une classe.

IMO les deux sont valables tant que le truc est correctement découplé (genre en utilisant un pattern type observer pour l'écoute de la création des voitures dans le schéma que tu proposes, afin que la classe stock reçoive des updates venant de diverses sources de modifications de stocks et que d'autres classes que celle de la gestion de stock puissent souscrire à des informations sur la création de voitures)

Avec la classe de l'objet  
 
Enfin il me semble qu'en Java ça ne pose aucun problème d'appeler une méthode statique depuis une instance, c'est juste pas des plus utile

Ah non sûrement pas, les deux classes sont totalement indépendantes

Et donc Masklinn, tu confirmes que j'ai tord dans mon approche ?

Bon, noms à la con, hein.
 

class VoitureDataLoader extends MysqlDataLoader{
 
  private function execSql($query){
    $rs = mysql_query($query);
    
    $nbCol = mysql_jesaispasquoiquiretournelenombredecolonnes($rs);
 
    while ($data = mysql_fetch_array($rs))
      {
    $i=0;
    while($i<$nbCol){
      $arrVoiture[$x][$mysql_donne_moi_le_nom_de_la_col($i)] = $data[$i];
      $i++;
    }
      }
  }
 
 
  public function getDispo($marque){
    return $this->runSQL("select * from voiture where marque = '".$marque." and dispo = 1" );
  }
}
 
 
class Voiture
{
    public $marque;
    public $modele;
    public $couleur;
    public $prix
    
    //tu vois le principe pour l'init...
 
 
    public static getDispo($marque){
      $vl = new VoitureDataLoader();
      $data = $vl->getDispo($marque);
      $result = array();
      foreach($data as $key=>$val){
    $result[] = new Voiture($val['marque'], $val['modele'], ...);
      }
    }
    
}
 
class Stock 
{
  private $stockDispo;
 
  function __construct($marque){
    $this->stockDispo = Voiture::getDispo($marque);
  }
 
}

Ouais, ton gestionnaire de stocks "sait" beaucoup trop de choses sur tes voitures

skeye > han t'as même pas fait une CarFactory, t'es trop naze

 
J'ai pas encore reçu mon GOF.

(et ma fonction execSql elle est dans MysqlDataLoader, d'ailleurs.)

 J'espère que vous n'avez pas des délais à respecter si c'est votre boulot de faire du développement php, parce que autant je trouve normal de séparer le code de la présentation etc... autant réinventer le c++ en plus crado avec le php et discuter 107 ans sur comment faire ceci ou celà, je me fais trucider sur place

C'est mon boulot et j'ai des délais à respecter, justement.

 
Ben sérieusement je vois pas comment tu fais, moi tout ce qu'on me demande c'est de torcher un site le plus rapidement possible, ensuite et effectivement ce n'est pas forcément bien, mais on me demande pas "de faire du style", seulement que ca marche

Bon d'après vos dires, corrigez moi si je me trompe, mais je devrais faire une plus grande distinction entre une classe et un objet dans mon pti esprit :  
 
un objet voiture ne connaît pas les autres voitures (un peu comme je disais précédemment),  
MAIS la classe Voiture est tout à fait habilitée à traiter des collections de voiture ?

La différence c'est probablement que c'est moi qui me tape toute la maintenance derrière.
Et je n'ai pas appliqué ces principes dans tout ce que je fais (déjà quand j'ai commencé je connaissais pas...), hein, me fais pas dire ce que je n'ai pas dit...mais c'est seulement parce-que je n'ai pas le temps de tout reprendre à zéro.

 
On dit instance d'objet et classe d'objet.

Bof, dans les langages à métaobjets, les classes sont aussi des instances

Ouais enfin sans discuter du terme employé, c'est bien ça ou pas ?

masklinn> taggle.
 

En gros, oui. Si on parle d'une instance de Voiture, c'est pas logique qu'elle aille chercher des infos concernant d'autres instances de Voitures, a priori.
Mais la classe Voiture, elle, est là pour ça...

Oki vous m'avez presque convaincu
 
Le truc, c'est que j'ai pas l'impression que ça soit fait comme ça ailleurs.  
Par exemple en Java, si je prends la classe JButton : je ne vois aucune méthode statique permettant d'obtenir la liste de tous les JButton.
 
Si vous avez des exemples de classes java qui ont ça : des méthodes statiques qui listent les instances de la classe. Mais j'en trouve pas, donc ça met le doute.

 
m'enfin, pour faire quoi?

bah dans le même esprit d'une méthode statique de la classe User qui liste tous les User. Tu vois pas le parallèle avec une méthode statique de la classe JButton qui liste tous les JButton ?
 
Je donne cet exemple histoire de dire, mais concrètement je ne vois pas de classes dans l'API java qui fait ce que tu dis :  
implémenter dans une classe une méthode statique qui renvoie une collection d'instances de cette classe.

Le truc, c'est qu'il faut aussi que ça serve à quelque chose, hein.

Ouais bah doit bien y avoir UNE classe de l'API java pour laquelle ça sert non ?
Disons que j'aimerais juste une confirmation que ce que tu proposes est bien la méthode "officielle" ^^ Si t'as des sources, des exemples extérieurs, autres, ...
 
C'est pas que j'ai pas confiance, juste que j'aimerais être sur avant de modifier ma manière de coder Et que je ne trouve pas de ressources répondant à ces questions.

c'est quoi que tu veux, un exemple d'utilisation d'une méthode statique?

non ^^
Un exemple de classe d'une source "officielle" (API java, code source d'une appli connue, ...) qui fait ce que tu dis : retourner une collection d'instances de sa propre classe par une méthode statique.  
Juste pour être sur que c'est une approche couramment utilisée

Dès que j'ai le temps
 

Moi toute mes factory ont été transformée en méthode 'create' statique de la classe factoryisée PHP c'est tellement lent que tu gagnes à limiter le nombre de classes (et de fichier). Ou alors faudrait foutre la class factory dans le même fichier que la classe...

Ah les questions existentielles ... Model2 ca n'est meme pas un vrai MVC !

Nan mais la base de base c est d utiliser la composition.

Le code avec car c'est du n'importe koi.

La meme chose bien faite utiliserait une interface DAO, qui serait implementee par SQL, FILE qui serait implementee par des classes abstraites qui elle meme serait derivee en Mysqli, Mysql, ADODB.
Chaque classe abstraites a un objet facade en appel statique (factory , $main_usage())
Apres tes objets Model ONT une reference a l objet Helper qui te retourne ce qu il faut quand il faut.

@Florentg : j'utilise depuis le debut des objets helper (facade , factory) de controller a model, appelee en statique. Developper avec la J2ME sur portable ca aide ...

Dites merci a djebel pour le reste

Edit: a quand une section "Architecture d'application" car la ca commence vraiment a chiffrer

Bon je reviens à l'attaque avec mes histoires de qui doit instancier quoi. J'en ai parlé avec mes collègues, et aussi avec des mecs qui bossent pour le SIB (qui gère notamment une des plus grosses db de données biologiques au monde). Et enfin, histoire d'être archi certain, j'en ai parlé par mail avec ce mec : http://www.ece.iastate.edu/~zola/?page=cv_en
 
Bon bah en gros, ils sont tous de mon avis, et aucun de l'avis de Skeye, donc j'en perds un peu mon latin. Ils disent tous que la méthode de Skeye est valable mais archi crade niveau concept objet. Je quote la réponse par mail du dernier gars :  

 
voilà voilà ...

gourage edit quote :x

A class should have only one reason to change

Jlaisse tomber ...

Tu racontes n'importe quoi au gars, Skeye n'a jamais dit que tu devais avoir une méthode statique retournant une collection de Voitures sur ta classe Voiture, il a dit que tu pouvais, ça n'a strictement aucun rapport putain

Accessoirement si tu veux un exemple sur le sujet c'est comme ça que fonctionne ActiveRecord en Ruby:

ActiveRecord est un ORM, c'est à dire qu'il mappe une classes sur une table SQL.

(La définition des propriétés de base d'une classe est tirée directement du modèle SQL par introspection)

# définition de base de notre modèle
class Cars << ActiveRecord::Base # Car hérite de ActiveRecord::Base
end

my_cars = Cars.find(:all) # je récupère toutes les voitures de ma BDD

Donc tu peux (emphase sur la possibilité, pas sur l'obligation) faire comme ça, personne n'a dit que tu y étais obligé

Ce qu'on t'a clairement dit par contre c'est que ce n'était pas à ta classe de Stock de gérer ça, parce que ça induit un couplage fort et inacceptable entre les deux.

edit: et accessoirement il ne dit à aucun moment que c'est "archi crade", il dit qu'il n'apprécie pas ce genre de designs mais peut imaginer pourquoi ce serait fait/utile.
Le plus gros problème dans cette discussion c'est que tu refuses de lire/comprendre ce qui t'es dit en fait.

merci    
 

Oki alors excuse moi mais peux-tu me dire comment on fait autrement ? La solution de la méthode statique ne me plait pas. Je ne vois pas de 3ème solution, si tu peux me mettre sur la voie
 

oki oki, c'est moi qui m'exprime mal en effet. Je trouve archi crade le coup de la méthode statique. Et effectivement, j'avais cru comprendre qu'on me disait que c'était la seule méthode valable. Si on pouvait m'orienter vers une autre solution ça serait bien cool.

Une classe tierce spécialisée dans la gestion de tes collections d'objets (une Factory, en DesignPatternSpeak), c'est d'ailleurs ce dont il te parle dans son mail:

Il ne te dit pas de mettre ta gestion d'objets dans une classe existante dont ce n'est absolument pas le rôle, il te dit d'ajouter une couche d'indirection avec une classe dont l'unique rôle est cette gestion d'objets (cf Factory Method pattern et Abstract Factory pattern)

Oki thx, je potasse ces designs avant de revenir vous gonfler ^^

my_cars = Cars.find(:all) # je récupère toutes les voitures de ma BDD

my_cars = Cars.find_all # je récupère toutes les voitures de ma BDD

 
Y'a une différence entre les deux?

my_cars = Cars.find_all # je récupère toutes les voitures de ma BDD

Non, le second est généré dynamiquement mais aliase sur le premier (je présume via method_missing)
 
cf http://api.rubyonrails.org/classes [...] /Base.html section "Dynamic attribute-based finders"

find_all est d'allieurs déprécié il me semble

Me semble bien, mais j'ai pas le courage de chercher la doc

L'est même pas documentée