KX
Messages postés16733Date d'inscriptionsamedi 31 mai 2008StatutModérateurDernière intervention31 janvier 2024127 30 oct. 2014 à 22:57
Bonsoir,
J'ai lu en diagonale ton code, je n'ai pas testé non plus. Je regarderais peut être ça plus en détail ce week-end. D'emblée je trouve dommage que tu n'ais pas mis un pom.xml (pour Maven), ça aurait évité de transporter tes jar dans les lib.
Autre point un peu critique, c'est que tu utilises du XML, cela t'obliges à te promener avec des @XmlElement c'est assez compliqué. Avec le REST on a plutôt tendance à le faire en JSON, c'est plus simple pour le client, et pour le serveur, en plus tu as les jars qu'il te faut dans les lib (jettison, jackson...)
Petite astuce pour MenuClient, au lieu de mettre localhost, mets ton adresse IP locale, cela permettra aux autre ordinateurs de ton réseau de communiquer avec toi, ce que ne permet pas localhost. Par contre il n'y a aucun intérêt à exposer plusieurs fois les URL dans MenuClient, c'est les annotations de ServiceX qui vont te le faire tout seul.
Il vaudrait peut-être mieux que tu consolides le code existant avant de commencer ce que tu as mis en perspective. Mais c'est un début ;-)
30 oct. 2014 à 22:57
J'ai lu en diagonale ton code, je n'ai pas testé non plus. Je regarderais peut être ça plus en détail ce week-end. D'emblée je trouve dommage que tu n'ais pas mis un pom.xml (pour Maven), ça aurait évité de transporter tes jar dans les lib.
Autre point un peu critique, c'est que tu utilises du XML, cela t'obliges à te promener avec des @XmlElement c'est assez compliqué. Avec le REST on a plutôt tendance à le faire en JSON, c'est plus simple pour le client, et pour le serveur, en plus tu as les jars qu'il te faut dans les lib (jettison, jackson...)
Petite astuce pour MenuClient, au lieu de mettre localhost, mets ton adresse IP locale, cela permettra aux autre ordinateurs de ton réseau de communiquer avec toi, ce que ne permet pas localhost. Par contre il n'y a aucun intérêt à exposer plusieurs fois les URL dans MenuClient, c'est les annotations de ServiceX qui vont te le faire tout seul.
Il vaudrait peut-être mieux que tu consolides le code existant avant de commencer ce que tu as mis en perspective. Mais c'est un début ;-)