From 99ed8b123307f587a7d356c4367cae42747a0205 Mon Sep 17 00:00:00 2001
From: Bastien Jacquelin <bastien.jacquelin@etu.uca.fr>
Date: Mon, 6 Feb 2023 17:30:44 +0100
Subject: [PATCH] add all class of 4 attributes cards,  edit facotry

---
 src/Console/Console.html         |  6 +++
 src/Console/main.js              | 23 +++++----
 src/Model/Card.js                | 45 ++++--------------
 src/Model/Card4WithoutColor.js   | 27 +++++++++++
 src/Model/Card4WithoutFilling.js | 27 +++++++++++
 src/Model/Card4WithoutNumber.js  | 27 +++++++++++
 src/Model/Card4WithoutOutline.js | 27 +++++++++++
 src/Model/Card4WithoutShape.js   | 27 +++++++++++
 src/Model/Card5.js               | 23 +++++++--
 src/Model/Deck.js                | 68 ++++++++------------------
 src/Model/Factory.js             | 82 ++++++++++++++++++++++++++++++++
 src/index.html                   | 20 --------
 12 files changed, 286 insertions(+), 116 deletions(-)
 create mode 100644 src/Model/Card4WithoutColor.js
 create mode 100644 src/Model/Card4WithoutFilling.js
 create mode 100644 src/Model/Card4WithoutNumber.js
 create mode 100644 src/Model/Card4WithoutOutline.js
 create mode 100644 src/Model/Card4WithoutShape.js
 create mode 100644 src/Model/Factory.js
 delete mode 100644 src/index.html

diff --git a/src/Console/Console.html b/src/Console/Console.html
index f639d2a..9ba88b8 100644
--- a/src/Console/Console.html
+++ b/src/Console/Console.html
@@ -16,8 +16,14 @@
     <script src="../Model/emptyParameterException.js"></script>
     <script src="../Model/unfoundCardException.js"></script>
     <script src="../Model/Card.js"></script>
+    <script src="../Model/Card4WithoutColor.js"></script>
+    <script src="../Model/Card4WithoutFilling.js"></script>
+    <script src="../Model/Card4WithoutNumber.js"></script>
+    <script src="../Model/Card4WithoutOutline.js"></script>
+    <script src="../Model/Card4WithoutShape.js"></script>
     <script src="../Model/Card5.js"></script>
     <script src="../Model/Const.js"></script>
+    <script src="../Model/Factory.js"></script>
     <script src="../Model/Deck.js"></script>
     <script src="main.js"></script>
 </body>
diff --git a/src/Console/main.js b/src/Console/main.js
index b0652e6..2374db8 100644
--- a/src/Console/main.js
+++ b/src/Console/main.js
@@ -3,12 +3,14 @@
 //import {Card} from '../Model/Card';
 
 console.log("~#Test#~");
-let card4 = new Card('red','2','losange','full');
+let card4 = new Card4WithoutOutline('red','2','losange','full');
 console.group('Carte 4 attributes');
 console.log(`carte de 4 elements : ${card4.color}`);
 console.groupEnd();
+
 let card5 = new Card5('blue','2','losange','full','pointillet');
 console.group('Carte 5 attributes');
+console.log(`carte de 5 elements : ${card5.color}`);
 console.log(`carte de 5 elements : ${card5.outline}`);
 console.log(`carte de 5 éléments instance de 5: ${card5 instanceof Card5}`);
 console.log(`carte de 5 éléments accès par méthode idx 0: ${card5.getAttributes()[0]}`);
@@ -26,19 +28,22 @@ try {
     }
 }
 console.groupEnd();
-console.group('Deck');
-let deck  = new Deck([0,1,2,3],4);
 
+
+console.group('Deck');
+let deck  = new Deck([0,1,2,3]);
 console.log(`All cards with 4 attributes size ${deck.allCards.length}`);
 console.log(`size output ${deck.outputCards.length}`);
-// Display all cards
-// deck.allCards.forEach(e => {
-//    console.log(e.color,e.number,e.shape,e.filling);
-// });
-console.log(`Output cards`);
-deck.outputCards.forEach(e => {
+//Display all cards
+console.log(`All cards`);
+console.log(deck.allCards)
+deck.allCards.forEach(e => {
    console.log(e.color,e.number,e.shape,e.filling);
 });
+console.log(`Output cards`);
+// deck.outputCards.forEach(e => {
+//    console.log(e.getAttributes());
+// });
 console.log(`set already made ${deck.setMade}`);
 
 
diff --git a/src/Model/Card.js b/src/Model/Card.js
index 3ee1a44..45c5972 100644
--- a/src/Model/Card.js
+++ b/src/Model/Card.js
@@ -1,40 +1,15 @@
 class Card{
-    constructor(color, number, shape, filling, outline){
-        if(color=''){
-            throw new EmptyParamaterException('Color');
-        }
-        if(number=''){
-            throw new EmptyParamaterException('Number');
-        }
-        if(shape==''){
-            throw new EmptyParamaterException('Shape');
-        }
-        if(filling==''){
-            throw new EmptyParamaterException('Filling');
-        }
-        if(outline==''){
-            throw new EmptyParamaterException('Outline');
-        }
-        this.color=color;
-        this.number=number;
-        this.shape=shape;
-        this.filling=filling;
-        this.outline=outline;
-    }
+    constructor(){}
     /**
-     * @returns array of all attributes : 
-        idx 1 : color
-        idx 2 : number 
-        idx 3 : shape
-        idx 4 : filling
-        idx 5 : null
-        * @author Bastien Jacquelin
+     * 
+     * @returns all attributes of a card
      */
-    getAttributes(){
-        return [this.color,this.number,this.shape,this.filling,this.outline];
-    }
-    equals(card){
-        return this.color===card.color && this.number===card.number && this.shape===card.shape && this.filling===card.filling && this.outline===card.outline ;
-    }
+    getAttributes(){}
+    /**
+     * 
+     * @param {*} card card to be compared with the current obj
+     * @returns boolean
+     */
+    equals(card){}
     
 }//export {Card}
\ No newline at end of file
diff --git a/src/Model/Card4WithoutColor.js b/src/Model/Card4WithoutColor.js
new file mode 100644
index 0000000..cbad689
--- /dev/null
+++ b/src/Model/Card4WithoutColor.js
@@ -0,0 +1,27 @@
+class Card4WithoutColor extends Card{
+    constructor(number, shape, filling, outline){
+        super();
+        if(number==''){
+            throw new EmptyParamaterException('Number');
+        }
+        if(shape==''){
+            throw new EmptyParamaterException('Shape');
+        }
+        if(filling==''){
+            throw new EmptyParamaterException('Filling');
+        }
+        if(outline==''){
+            throw new EmptyParamaterException('Outline');
+        }
+        this.number=number;
+        this.shape=shape;
+        this.filling=filling;
+        this.outline=outline;
+    }
+    getAttributes(){
+        return [this.number,this.shape,this.filling,this.outline];
+    }
+    equals(card){
+        return this.number===card.number && this.shape===card.shape && this.filling===card.filling && this.outline===card.outline ;
+    }
+}
\ No newline at end of file
diff --git a/src/Model/Card4WithoutFilling.js b/src/Model/Card4WithoutFilling.js
new file mode 100644
index 0000000..b895c07
--- /dev/null
+++ b/src/Model/Card4WithoutFilling.js
@@ -0,0 +1,27 @@
+class Card4WithoutFilling extends Card{
+    constructor(color,number, shape, outline){
+        super();
+        if(number==''){
+            throw new EmptyParamaterException('Number');
+        }
+        if(shape==''){
+            throw new EmptyParamaterException('Shape');
+        }
+        if(color==''){
+            throw new EmptyParamaterException('Color');
+        }
+        if(outline==''){
+            throw new EmptyParamaterException('Outline');
+        }
+        this.number=number;
+        this.shape=shape;
+        this.color=color;
+        this.outline=outline;
+    }
+    getAttributes(){
+        return [this.number,this.shape,this.color,this.outline];
+    }
+    equals(card){
+        return this.number===card.number && this.shape===card.shape && this.color===card.color && this.outline===card.outline ;
+    }
+}
\ No newline at end of file
diff --git a/src/Model/Card4WithoutNumber.js b/src/Model/Card4WithoutNumber.js
new file mode 100644
index 0000000..95abc61
--- /dev/null
+++ b/src/Model/Card4WithoutNumber.js
@@ -0,0 +1,27 @@
+class Card4WithoutNumber extends Card{
+    constructor(color, shape, filling, outline){
+        super();
+        if(color==''){
+            throw new EmptyParamaterException('Color');
+        }
+        if(shape==''){
+            throw new EmptyParamaterException('Shape');
+        }
+        if(filling==''){
+            throw new EmptyParamaterException('Filling');
+        }
+        if(outline==''){
+            throw new EmptyParamaterException('Outline');
+        }
+        this.color=color;
+        this.shape=shape;
+        this.filling=filling;
+        this.outline=outline;
+    }
+    getAttributes(){
+        return [this.color,this.shape,this.filling,this.outline];
+    }
+    equals(card){
+        return this.color===card.color && this.shape===card.shape && this.filling===card.filling && this.outline===card.outline ;
+    }
+}
\ No newline at end of file
diff --git a/src/Model/Card4WithoutOutline.js b/src/Model/Card4WithoutOutline.js
new file mode 100644
index 0000000..c52c2f1
--- /dev/null
+++ b/src/Model/Card4WithoutOutline.js
@@ -0,0 +1,27 @@
+class Card4WithoutOutline extends Card{
+    constructor(color, number, shape, filling){
+        super();
+        if(number==''){
+            throw new EmptyParamaterException('Number');
+        }
+        if(shape==''){
+            throw new EmptyParamaterException('Shape');
+        }
+        if(filling==''){
+            throw new EmptyParamaterException('Filling');
+        }
+        if(color==''){
+            throw new EmptyParamaterException('Color');
+        }
+        this.number=number;
+        this.shape=shape;
+        this.filling=filling;
+        this.color=color;
+    }
+    getAttributes(){
+        return [this.number,this.shape,this.filling,this.color];
+    }
+    equals(card){
+        return this.number===card.number && this.shape===card.shape && this.filling===card.filling && this.color===card.color;
+    }
+}
\ No newline at end of file
diff --git a/src/Model/Card4WithoutShape.js b/src/Model/Card4WithoutShape.js
new file mode 100644
index 0000000..bf0d36c
--- /dev/null
+++ b/src/Model/Card4WithoutShape.js
@@ -0,0 +1,27 @@
+class Card4WithoutShape extends Card{
+    constructor(color,number, filling, outline){
+        super();
+        if(color==''){
+            throw new EmptyParamaterException('Color');
+        }
+        if(number==''){
+            throw new EmptyParamaterException('Number');
+        }
+        if(filling==''){
+            throw new EmptyParamaterException('Filling');
+        }
+        if(outline==''){
+            throw new EmptyParamaterException('Outline');
+        }
+        this.color=color;
+        this.number=number;
+        this.filling=filling;
+        this.outline=outline;
+    }
+    getAttributes(){
+        return [this.number,this.color,this.filling,this.outline];
+    }
+    equals(card){
+        return this.number===card.number && this.color===card.color && this.filling===card.filling && this.outline===card.outline ;
+    }
+}
\ No newline at end of file
diff --git a/src/Model/Card5.js b/src/Model/Card5.js
index bead780..a22e47a 100644
--- a/src/Model/Card5.js
+++ b/src/Model/Card5.js
@@ -2,10 +2,26 @@
 
 class Card5 extends Card {
     constructor(color, number, shape, filling, outline){
-        super(color,number,shape,filling);
-        if(!outline){
+        super();
+        if(color==''){
+            throw new EmptyParamaterException('Color');
+        }
+        if(number==''){
+            throw new EmptyParamaterException('Number');
+        }
+        if(shape==''){
+            throw new EmptyParamaterException('Shape');
+        }
+        if(filling==''){
+            throw new EmptyParamaterException('Filling');
+        }
+        if(outline==''){
             throw new EmptyParamaterException('Outline');
         }
+        this.color=color;
+        this.number=number;
+        this.shape=shape;
+        this.filling=filling;
         this.outline=outline;
     }
     /**
@@ -18,8 +34,7 @@ class Card5 extends Card {
         * @author Bastien Jacquelin
     */
     getAttributes(){
-        // return [this.color,this.number,this.shape,this.filling,this.outline];
-        return super.getAttributes().concat(this.outline);
+        return [this.color,this.number,this.shape,this.filling,this.outline];
     }
     equals(card){
         return this.color===card.color && this.number===card.number && this.shape===card.shape && this.filling===card.filling && this.outline===card.outline ;
diff --git a/src/Model/Deck.js b/src/Model/Deck.js
index 40c365c..1c13a92 100644
--- a/src/Model/Deck.js
+++ b/src/Model/Deck.js
@@ -3,38 +3,17 @@ class Deck{
      * 
      * @param {*} attributes : array with the attributes index for the cards
      */
-    constructor(attributes,nbAttributes){
-        let attributesRequired=this.attributesRequiredFun(attributes);
-        attributesRequired.forEach(e=>{
-            for(let i=0;i<e.length;i++){
-                console.log(e[i]);
-            }
-            console.log(`lenght: ${e.length}`);
-            console.log('------');
-        });
-        this.allCards=this.createCards(attributesRequired,nbAttributes);// All the cards in the game
+    constructor(attributes){
+        console.log(attributes);
+        this.allCards=this.createCards(attributes);// All the cards in the game
         this.remainingCards=this.allCards;// cards in the stack
         this.outputCards=[];// 12 cards lay on the table 
         this.setMade=[];// array with all the set already mades (array of set)
-        //this.createDeck();
-    }
-    attributesRequiredFun(attributes){
-        let attributesRequiredTmp=[];
-        let nullArray=[0,0,0,0,0];
-        for(let i=0;i<5;i++){
-            let find=false;
-            for (let j=0;j<attributes.length;j++){
-                if(i==attributes[j]){
-                    attributesRequiredTmp.push(ATTRIBUTES[j]);
-                    find=true;
-                }
-            }
-            if(!find){
-                attributesRequiredTmp.push(nullArray);
-            }
-        }
-        return attributesRequiredTmp;
+        this.createDeck();
     }
+    /**
+     * @brief creation of the deck : call factory to create the good cards
+     */
     createDeck(){
         for (let i=0; i<12; i++){
             const rand = this.getRandCard();
@@ -42,33 +21,22 @@ class Deck{
             this.remainingCards.splice(rand,1);
         }
     }
+    /**
+     * 
+     * @returns random number in range of the array size 
+     */
     getRandCard(){
         const random = Math.floor(Math.random() * this.remainingCards.length);
         return random;
     }
     /**
      * 
-     * @param nbAttributes : attributes of the card, by default = 4 
-     * @returns all cards: 81 in case of 4 attributes and 1224
+     * @param attributes : index of the attributes used
+     * @returns all cards: 81 in case of 4 attributes and 1024 if 5 attributes
      */
-    createCards(attributes,nbAttributes){
-        let tabOfAllCards=[];
-
-        for (let c=0; c<this.lower(attributes[0].length,nbAttributes)-1; c++){
-            for (let n=0; n<this.lower(attributes[1].length,nbAttributes)-1; n++){
-                for (let s=0; s<this.lower(attributes[2].length,nbAttributes)-1; s++){
-                    for (let f=0; f<this.lower(attributes[3].length,nbAttributes)-1; f++){
-                        for (let o=0; o<this.lower(attributes[4].length,nbAttributes)-1; o++){
-                            tabOfAllCards.push(new Card(attributes[0][c],attributes[1][n],attributes[2][s],attributes[3][f],attributes[4][o]));
-                        }
-                    }
-                }
-            }
-        }
-        return tabOfAllCards;
-    }
-    lower(length,nbAttributes){
-        return nbAttributes<length?nbAttributes:length;
+    createCards(attributes){
+        let factory = new Factory(attributes)
+        return factory.concreteCardCreation();
     }
     checkSet(selectedCards){
         if(true){//isSet(selectedCards)){
@@ -77,6 +45,10 @@ class Deck{
             });
         }
     }
+    /**
+     * 
+     * @param {*} selectedCards wehn a set is made, need to remove the card from the array remainingCards
+     */
     removeFromRemainingCards(selectedCards){//better check of card type more opti
         let set=[];
         for(let i=0; i<this.allCards.length;i++){
diff --git a/src/Model/Factory.js b/src/Model/Factory.js
new file mode 100644
index 0000000..e85a853
--- /dev/null
+++ b/src/Model/Factory.js
@@ -0,0 +1,82 @@
+class Factory{
+    constructor(arrayOfAttributes){
+        let length=arrayOfAttributes.length
+        this.product=this.concreteCardCreation(arrayOfAttributes,length);
+    }
+    attributesRequiredFun(arrayOfAttributes,length){
+        let attributesRequiredTmp=[];
+        let nullArray=[0,0,0,0,0];
+        for(let i=0;i<5;i++){
+            let find=false;
+            for (let j=0;j<length;j++){
+                if(i==arrayOfAttributes[j]){
+                    attributesRequiredTmp.push(ATTRIBUTES[j]);
+                    find=true;
+                }
+            }
+            if(!find){
+                attributesRequiredTmp.push(nullArray);
+            }
+        }
+        return attributesRequiredTmp;
+    }
+    concreteCardCreation(arrayOfAttributes, length){
+        let tabOfAllCards=[];
+        let attributes=this.attributesRequiredFun(arrayOfAttributes,length);
+        let nbAttributes=length
+        if(nbAttributes==3){
+            for (let c=0; c<nbAttributes-1; c++){
+                for (let n=0; n<nbAttributes-1; n++){
+                    for (let s=0; s<nbAttributes-1; s++){
+                        tabOfAllCards.push(new Card3(ATTRIBUTES[0][c],ATTRIBUTES[1][n],ATTRIBUTES[2][s]));
+                    }
+                }
+            }
+        }
+        else if(nbAttributes==4){
+            for (let n=0; n<nbAttributes-1; n++){
+                for (let s=0; s<nbAttributes-1; s++){
+                    for (let f=0; f<nbAttributes-1; f++){
+                        for (let o=0; o<nbAttributes-1; o++){
+                            if(attributes[0][0]===0){
+                                tabOfAllCards.push(new Card4WithoutColor(attributes[1][n],attributes[2][s],attributes[3][f],attributes[4][o]));
+                            }
+                            else if(attributes[1][0]===0){
+                                tabOfAllCards.push(new Card4WithoutNumber(attributes[0][n],attributes[2][s],attributes[3][f],attributes[4][o]));
+                            }
+                            else if(attributes[2][0]===0){
+                                tabOfAllCards.push(new Card4WithoutShape(attributes[0][n],attributes[1][s],attributes[3][f],attributes[4][o]));
+                            }
+                            else if(attributes[3][0]===0){
+                                console.log(attributes[0][n],attributes[1][s],attributes[2][f],attributes[4][o]);
+                                tabOfAllCards.push(new Card4WithoutFilling(attributes[0][n],attributes[1][s],attributes[2][f],attributes[4][o]));
+                            }
+                            else if(attributes[4][0]===0){
+                                console.log('rentre la');
+                                console.log(attributes[0][n],attributes[1][s],attributes[2][f],attributes[3][o]);
+                                tabOfAllCards.push(new Card4WithoutOutline(attributes[0][n],attributes[1][s],attributes[2][f],attributes[3][o]));
+                            }
+                        }
+                    }
+                }
+            }
+        }
+        else if(nbAttributes==5){
+            for (let c=0; c<nbAttributes-1; c++){
+                for (let n=0; n<nbAttributes-1; n++){
+                    for (let s=0; s<nbAttributes-1; s++){
+                        for (let f=0; f<nbAttributes-1; f++){
+                            for (let o=0; o<nbAttributes-1; o++){
+                                tabOfAllCards.push(new Card5(attributes[0][c],attributes[1][n],attributes[2][s],attributes[3][f],attributes[4][o]));
+                            }
+                        }
+                    }
+                }
+            }
+        }
+        else{
+            //ERROR
+        }
+        return tabOfAllCards
+    }
+}
\ No newline at end of file
diff --git a/src/index.html b/src/index.html
deleted file mode 100644
index b32303c..0000000
--- a/src/index.html
+++ /dev/null
@@ -1,20 +0,0 @@
-<!DOCTYPE html>
-
-<html lang="en">
-
-<head>
-    <title>My awesome blog</title>
-</head>
-
-<body>
-    <h1>My awesome blog</h1>
-</body>
-
-<script src="./Model/Card.js"></script>
-<script src="./algo.js"></script>
-
-<script>
-    main();    
-</script>
-
-</html>
\ No newline at end of file