Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feedback #1

Open
wants to merge 6 commits into
base: feedback
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 59 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -26,3 +26,62 @@ hs_err_pid*
# Eclipse
.classpath
.project

# IntellJ
.idea/*

src/main/java

# Created by https://www.toptal.com/developers/gitignore/api/maven
# Edit at https://www.toptal.com/developers/gitignore?templates=maven

### Maven ###
target/
pom.xml.tag
pom.xml.releaseBackup
pom.xml.versionsBackup
pom.xml.next
release.properties
dependency-reduced-pom.xml
buildNumber.properties
.mvn/timing.properties
# https://github.com/takari/maven-wrapper#usage-without-binary-jar
.mvn/wrapper/maven-wrapper.jar

# Eclipse m2e generated files
# Eclipse Core
.project
# JDT-specific (Eclipse Java Development Tools)
.classpath

# End of https://www.toptal.com/developers/gitignore/api/maven

# Created by https://www.toptal.com/developers/gitignore/api/vim
# Edit at https://www.toptal.com/developers/gitignore?templates=vim

### Vim ###
# Swap
[._]*.s[a-v][a-z]
!*.svg # comment out if you don't need vector files
[._]*.sw[a-p]
[._]s[a-rt-v][a-z]
[._]ss[a-gi-z]
[._]sw[a-p]

# Session
Session.vim
Sessionx.vim

# Temporary
.netrwhist
*~
# Auto-generated tag files
tags
# Persistent undo
[._]*.un~

# End of https://www.toptal.com/developers/gitignore/api/vim

.settings
pom.xml
memo.txt
8 changes: 8 additions & 0 deletions .idea/.gitignore

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

11 changes: 11 additions & 0 deletions ALCOL.iml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
<?xml version="1.0" encoding="UTF-8"?>
<module type="JAVA_MODULE" version="4">
<component name="NewModuleRootManager" inherit-compiler-output="true">
<exclude-output />
<content url="file://$MODULE_DIR$">
<sourceFolder url="file://$MODULE_DIR$/src" isTestSource="false" />
</content>
<orderEntry type="inheritedJdk" />
<orderEntry type="sourceFolder" forTests="false" />
</component>
</module>
43 changes: 43 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,46 @@
# estore

Projet pour l'enseignement des composants logiciels

Réponses google forms (en cours) :

https://github.com/Master-DL/m2-estore-pojo-r

BAYCHELIER Rémi PEYRICHOUX Rémy

Classes :
- Account : withdraw()
- Bank: transfert(), dépendance à Account (et jeu de données set-up dans le constructeur)
- Cart: addItems(), dépendance à Client
- Client: Jeu de données dans le constructeur et scénarios, dépendance à Store
- ItemInStock : changeQuantity, dépendance à Provider
- Order : addItem() et computeAmount(), dépendance à Client
- Provider : order()
- Store : isAvailable(), addItemToCart(), pay(), oneShotOrder(), treatOrder(), dépendance à Bank et Provider

Lecture des attributs du code, distinction des méthodes de type getter/setter aux méthodes qui font du traitement plus complexe sur les objets et qui sont pour la plupart publiques.

Voir dépendances dans la question 1

- Account : withdraw() - soustrait un montant d'un compte
- Bank: transfert() - virement d'un montant d'un compte vers un autre
- Cart: addItem() - ajout d'un produit dans le panier
- ItemInStock : changeQuantity() - changer l'inventaire d'un produit en stock
- Order : addItem() - ajout du prix d'un produit dans la commande et change l'inventaire de ce produit, computeAmount() - calculer le montant de la commande
- Provider : order() - retourne une valeur aléatoire dépendant d'une quantité
- Store : isAvailable() - retourne vrai si l'objet donné est en stock
addItemToCart() - ajoute un produit dans le panier avec une quantité
pay() - crée la commande, calcule son montant et effectue le virement entre les comptes
oneShotOrder() - commande d'un seul type de produit avec une quantité

IBankService : Bank, Account
IItemService: Cart, ItemInStock, Order
ISellerService : Provider, Store

IBankService : transfert(), withdraw(), credit()
IItemService : addItem(), setQuantity(), computeAmount()
ISellerService : order(), pay()

8 (+ quelques méthodes uniques à certains services)

Je ne comprends pas la question, ce ne sont pas les mêmes services que ceux exprimés deux questions plus tôt ?
57 changes: 0 additions & 57 deletions src/core/Bank.java

This file was deleted.

36 changes: 0 additions & 36 deletions src/core/Cart.java

This file was deleted.

86 changes: 0 additions & 86 deletions src/core/Client.java

This file was deleted.

45 changes: 0 additions & 45 deletions src/core/ItemInStock.java

This file was deleted.

Loading