czechitas / java-2 Goto Github PK

1. Consider data-th-attributes instead of th:attributes

2. Consider renaming iter to status or stat

3. Make sure you use stat.index instead of stat.count-1 and stat.count instead of stat.index+1

4. Decide whether minimize or maximize size of SpEL ${} when Thymeleaf evaluation can work as well. That is particularly important for single text values like th:text="'sample'" or th:text="${'sample'}" and stick with the strategy

5. Make sure URLs are all absolute using @{/sample}.

6. Decide whether you should use ModelAndView as a return type or Model or ModelMap as an input parameter while returning just String viewName. The other strategy is more validation friendly and probably more common in the wild.

7. Decide whether you want to use .html in the URLs or not and stick with it consistently.

1. Deploy demo apps and homeworks to Heroku, Google Cloud, Amazon AWS, Microsoft Azure or RedHat OpenShift so that students can look at them anytime. Zero maintenance effort required, must be free.

2. Put all links (to the deployed apps etc.) to the documentation via a redirect service (bit.ly, tiny.url, goo.gl etc.) to avoid hardwiring to any service.
   Note: Bit.ly is not a good candidate as the target URLs cannot be changed once set up. There are alternatives:
   https://thenextweb.com/apps/2012/06/02/not-a-fan-of-the-big-bitly-revamp-here-are-9-alternatives/
   https://www.quora.com/Is-it-possible-to-change-the-destination-of-a-bit-ly-link

Students struggle to understand that error details are written in the logs and not on the web page.
Change default error page to display also the error details.

odpovedi[i] instead of odpoved[i]

When the app is started, a lot of redundant messages gets to the console (log). Optimize that output.

novy.html -> input form has 2 identical labels