-
Notifications
You must be signed in to change notification settings - Fork 2
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
Profile gui #62
Profile gui #62
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK de mon côté, si les fautes de typo et les attributs en français sont traduits en anglais
src/main/java/io/github/oliviercailloux/y2018/apartments/gui/LayoutApartmentGUI.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/oliviercailloux/y2018/apartments/gui/LayoutApartmentGUI.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/oliviercailloux/y2018/apartments/gui/LayoutApartmentGUI.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/oliviercailloux/y2018/apartments/gui/LayoutApartmentGUI.java
Outdated
Show resolved
Hide resolved
src/main/java/io/github/oliviercailloux/y2018/apartments/gui/LayoutApartmentGUI.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
À terminer…
src/main/java/io/github/oliviercailloux/y2018/apartments/gui/AskOpinionForUtility.java
Outdated
Show resolved
Hide resolved
(Apartment c, Apartment d) -> | ||
-Double.compare(linearAVF.getSubjectiveValue(c), linearAVF.getSubjectiveValue(d))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Plus élégant d’utiliser Comparator#Comparing (OPT)
Bonjour, nous avons réalisé tous les changements nécessaires à la prise en compte de cette PR. |
Bonjour,
suite aux nombreux problèmes concernant QuestionPriceArea, nous avons fait une version compilant et se lançant correctement. La seule subtilité réside dans l'utilisation du Object à la place de QuestionPriceArea. Dans cette version, nous ne pouvons voir les questions dans le GUI Profile Questions.
Vous pourrez trouver ci-joint des captures d'écran de ce qui a été réalisé.
Cette PR n'a pas pour but d'être merge en l'état actuel. Une fois que la PR 45 aura été fusionnée, nous ferons les modifications nécessaires. Nous sommes prêt à finaliser cette tâche même après les examens et les derniers rendus afin de pouvoir aboutir ce travail longuement réfléchi.
Nous restons ouverts à vos propositions concernant cette PR, en considérant qu'il ne s'agit pas de la version finale. Vous pourrez tout de même y retrouver le travail que nous avons réalisé avec @MorganeFt.
Bonne journée