Comment convertir cette programmation procédurale en programmation orientée objet?

Toutes sortes de Odeurs peut être trouvé dans ce code. Si vous voulez vraiment faire un code agréable de celui-ci, lisez le chapitre refactoring de Fowler par chapitre et appliquez toutes les connaissances acquises immédiatement.

Une autre approche est simplement ne le touche pas, "ne réparez pas le moteur de travail".

Cela ressemble au jeu de mémoire classique, à l'aide de cartes. Je vois quatre classes principales:

1. jeu li>
2. Deck Li>
3. carte li>
4. grille li> ol>

   Voici quelques propriétés et méthodes suggérées pour chacun. C'est incomplet, mais cela devrait vous faire aller. P>

   Game
     previewDuration
     selectedRow
     selectedColumn
     selectedMatchRow
     selectedMatchColumn
     deck
     grid
     main()
     promptForColumn()
     promptForRow()
     preview()
     loadGrid()
   
   Deck
     cardCount
     shuffle()
     getNextCard()
   
   Card
     caption
     equals()
   
   Grid
     width
     height
     getCardAtPos()
   

Cette réponse illustre une refactorisation possible que vous pouvez faire pour rendre votre approche plus orientée objet. Il ne traite pas du domaine problématique (cartes, ponts, etc.), qui sont traités par d'autres réponses. Au lieu de cela, il décrit comment vous pouvez utiliser des principes OO pour faire face à la mise en œuvre programmatique de manière plus OO.

La première étape que je ferais est de faire sur ce que je vais appeler la construction de NovedLoop. Vous avez une boucle do / tandis que vous appuyez sur l'utilisateur, qu'il s'agisse de courir ou de ne pas exécuter oui, il fait une action.

Prenez cette fonctionnalité et encapsulez-la. Faites de la carte un objet runnable et laissez-la de l'exécuter à plusieurs reprises. xxx

alors, il ne peut s'agir d'une seule instance d'une action de jeu et ne doit pas avoir à Intégrer la fonctionnalité de boucle. Vous avez ensuite la possibilité de réutiliser le message NOUVELDLOOP pour d'autres programmes et vous pouvez lire plus facilement .Play comme vous n'êtes pas distrait par le fait que cela pourrait être bouclé.

Avant de vous inquiéter des objets, vous devez d'abord séparer au moins des sections en méthodes nommées, il est donc plus clair que chaque partie fait. C'est important que ce soit OO ou non. Par exemple:

    for (int i =0; i<4;i++){
        for (int j=0;j<4;j++){

            System.out.print(cards[i][j]);
            System.out.print(" ");
        }
        System.out.println("");
        System.out.println("");
    }

Mais je ne sais pas quoi faire avec constructeur et quelles variables sont censées être privées. P>

Peu importe ce qui peut être, devrait être privé code>. Pour déterminer cela, il est souvent plus facile de marquer tout em> privé code>, puis lorsque vous frappez un "Point de douleur", promouvez-le à public code>. p>

Quant aux cours dont vous avez besoin, jetez un coup d'œil au code existant pour "odeurs" tels que: p> xxx pré>

vous avez une "primitive nommée" 1 sup> là - et il est utilisé assez souvent. Cela en fait un nom important à votre programme. Encapsulez-le dans une classe: p> xxx pré>

maintenant, cherchez où vous manipulez la primitive nommée: P>

public class Cards {
    private int width;
    private int length;
    private int[][] cards;

   public Cards(int length, int width) {
      this.length = length;
      this.width = width;
      this.cards = new int[length][width];
   }

   public void shuffle() {
      // existing shuffle method goes here - but works with private cards
   }

   public void print() {
      for (int i = 0; i < length; i++){
        for (int j = 0; j < width; j++){
           System.out.print(cards[i][j]);
           System.out.print(" ");
        }
      System.out.println("");
      System.out.println("");
   }
}