QA - Spolupráce s průmyslem (SSP)

Backend

• Sonar link

• Kritické chyby
  • Dodgy - Write to static field from instance method [62]
  • Bad practice - Class defines equals() and uses Object.hashCode() [10]
    • Equals Hash Code [10]
  • Avoid Catching Throwable [5] 
  • Security - Array is stored directly [2] 
  • Dodgy - Dead store to local variable [1]
  • Dodgy - Method uses the same code for two branches [1]

Dodgy - Write to static field from instance method

• Ukázkový kód (ostatní jsou stejné)
  • Metodu nelze označit jako statickou, protože by se rozbil mechanismus vytváření Spring contextu

• Duplikace kódu
  • V balíčku cz.cvut.fit.bpm.api.utils.service jsou všechny třídy identické (až na typ hodnoty, kterou drží)
  • Nelze vyřešit generickou třídou, protože Java nepodporuje tímto způsobem určovat typ statické proměnné

Řešení

• Akceptovat/neřešit
• Přehodnotit návrh rozhraní mezi backendem a frontendem

Bad practice - Class defines equals() and uses Object.hashCode() / Equals Hash Code

• Ukázkový kód (ostatní jsou podobné)
  • Bude dělat problémy pokud budou instance třídy vkládané do HashTable, nebo HashMap

Řešení

• Akceptovat/neřešit
• Pro jistotu implementovat metodu hashCode() v souladu s equals()

Avoid Catching Throwable

• Kód

Frontend

• Sonar link