Revize c5111d82

#26 Adding logging mechanism

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/controller/AppController.java

@@ -56 +56 @@
     private ConfigurationService configurationService;
 
     /**
-     *  This method is called by the GET method and initializes
-     *  the main page of the application (index). Loads all projects
-     *  stored in the db and all implemented AP.
+     * This method is called by the GET method and initializes
+     * the main page of the application (index). Loads all projects
+     * stored in the db and all implemented AP.
      *
      * @param model object for passing data to the UI
      * @return html file name for thymeleaf template
@@ -66 +66 @@
     @GetMapping("/")
     public String index(Model model) {
         model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
+        LOGGER.info("@GetMapping(\"/\") - Accessing main page");
         return "index";
     }
 
     /**
-     *  Method for obtaining project by ID.
+     * Method for obtaining project by ID.
      *
-     * @param id project ID
+     * @param id    project ID
      * @param model object for passing data to the UI
      * @return html file name for thymeleaf template
      */
@@ -80 +81 @@
     public String getProjectById(@PathVariable Long id, Model model) {
         model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
         model.addAttribute("project", projectService.getProjectById(id));
+        LOGGER.info("@GetMapping(\"projects/" + id + ")");
         return "project";
     }
 
     /**
      * Method for obtaining all AP.
+     *
      * @return list of AP
      */
     @GetMapping("/anti-patterns")
-    public @ResponseBody List<AntiPattern> getAllAntiPatterns(Model model) {
+    public @ResponseBody
+    List<AntiPattern> getAllAntiPatterns(Model model) {
         model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
+        LOGGER.info("GetMapping(\"/anti-patterns\") and obtaining all APs.");
         return antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns());
     }
 
     /**
      * Method for obtaining AP by ID.
      *
-     * @param id AP ID
+     * @param id    AP ID
      * @param model object for passing data to the UI
      * @return html file name for thymeleaf template
      */
@@ -108 +113 @@
         model.addAttribute("description", antiPatternService.getDescriptionFromCatalogue(id));
         model.addAttribute("operationalizationText", antiPatternService.getOperationalizationText(antiPatternService.getAntiPatternById(id).getAntiPatternModel().getName()));
         model.addAttribute("configurations", configurationService.getConfigurationByName(currentConfigurationName));
+        LOGGER.info("GetMapping(\"/anti-patterns/ " + id + ") and obtaining AP by ID");
         return "anti-pattern";
     }
 
     /**
      * Method that processes requirements for analyzing selected projects and AP.
      *
-     * @param model object for passing data to the UI
-     * @param selectedProjects selected project to analyze
+     * @param model                object for passing data to the UI
+     * @param selectedProjects     selected project to analyze
      * @param selectedAntiPatterns selected AP to analyze
      * @return html file name for thymeleaf template
      */
@@ -130 +136 @@
             model.addAttribute("errorMessage", "No project selected." +
                     " Select at least one project.");
             model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
+            LOGGER.warn("@PostMapping(\"/analyze\") - Processing requirements for analyzing selected projects and AP with an error. " +
+                    "selectedProjects are null!");
             return "index";
         }
 
@@ -137 +145 @@
             model.addAttribute("errorMessage", "No anti-pattern selected." +
                     " Select at least one anti-pattern.");
             model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
+            LOGGER.warn("@PostMapping(\"/analyze\") - Processing requirements for analyzing selected projects and AP with an error. " +
+                    "selectedAntiPatterns are null!");
             return "index";
         }
 
@@ -152 +162 @@
         model.addAttribute("queryResults", results);
         model.addAttribute("recalculationTime", DateTimeFormatter.ofPattern("HH:mm:ss").format(LocalTime.now()));
 
+
+        LOGGER.info("@PostMapping(\"/analyze\") - Processing SUCCESSFULLY requirements for analyzing selected projects and AP.");
         return "result";
     }
 
@@ -172 +184 @@
         model.addAttribute("queryResults", antiPatternService.getResults());
         model.addAttribute("recalculationTime", DateTimeFormatter.ofPattern("HH:mm:ss").format(LocalTime.now()));
 
+        LOGGER.info("@GetMapping(\"/analyze\") - ??");
         return "result";
     }
 
@@ -183 +196 @@
     @GetMapping("/about")
     public String about(Model model) {
         model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
+        LOGGER.info("@GetMapping(\"/about\") - Accessing about page");
         return "about";
     }
 
@@ -205 +219 @@
     /**
      * Method of saving changes of current configuration
      *
-     * @param model object for passing data to the UI
+     * @param model           object for passing data to the UI
      * @param thresholdValues changed configuration values
-     * @param thresholdNames changed configuration names
+     * @param thresholdNames  changed configuration names
      * @return html file name for thymeleaf template
      */
     @PostMapping(value = "/configuration", params = "configuration-save-button")
     public String configurationSavePost(Model model,
-                                    @RequestParam(value = "thresholdValues", required = false) String[] thresholdValues,
-                                    @RequestParam(value = "thresholdNames", required = false) String[] thresholdNames,
-                                    @RequestParam(value = "antiPatternNames", required = false) String[] antiPatternNames,
-                                    HttpSession session, RedirectAttributes redirectAttributes) {
+                                        @RequestParam(value = "thresholdValues", required = false) String[] thresholdValues,
+                                        @RequestParam(value = "thresholdNames", required = false) String[] thresholdNames,
+                                        @RequestParam(value = "antiPatternNames", required = false) String[] antiPatternNames,
+                                        HttpSession session, RedirectAttributes redirectAttributes) {
 
         String currentConfigurationName = configurationGetFromSession(session);
 
@@ -225 +239 @@
 
         if (wrongParameters.isEmpty()) {
             model.addAttribute("successMessage", "All configuration values have been successfully saved.");
+            LOGGER.info("@PostMapping(value = \"/configuration\", params = \"configuration-save-button\") - " +
+                    "input parameters were successfully processed...");
         } else {
             model.addAttribute("errorMessage", "One or more configuration values are not in correct format, see messages below.");
             antiPatternService.setErrorMessages(query.getAntiPatterns(), wrongParameters);
+            LOGGER.warn("@PostMapping(value = \"/configuration\", params = \"configuration-save-button\") ended with an error!" +
+                    "Some parameters seem to be wrong!");
         }
 
         model.addAttribute("query", query);
 
         model.addAttribute("configurations", configurationService.getConfigurationByName(currentConfigurationName));
+        LOGGER.info("@PostMapping(value = \"/configuration\", params = \"configuration-save-button\") - Save Button was pressed");
         return "configuration";
     }
 
     /**
      * Method for saving full new configuration
      *
-     * @param model object for passing data to the UI
+     * @param model           object for passing data to the UI
      * @param thresholdValues changed configuration values
-     * @param thresholdNames changed configuration names
+     * @param thresholdNames  changed configuration names
      * @return html file name for thymeleaf template
      */
     @PostMapping(value = "/configuration", params = "configuration-save-as-button")
     public String configurationSaveAsPost(Model model,
-                                    @RequestParam(value = "thresholdValues", required = false) String[] thresholdValues,
-                                    @RequestParam(value = "thresholdNames", required = false) String[] thresholdNames,
-                                    @RequestParam(value = "antiPatternNames", required = false) String[] antiPatternNames,
-                                    @RequestParam(value = "configuration-save-as-input", required = false) String newConfigName,
-                                    HttpSession session, RedirectAttributes redirectAttributes) {
+                                          @RequestParam(value = "thresholdValues", required = false) String[] thresholdValues,
+                                          @RequestParam(value = "thresholdNames", required = false) String[] thresholdNames,
+                                          @RequestParam(value = "antiPatternNames", required = false) String[] antiPatternNames,
+                                          @RequestParam(value = "configuration-save-as-input", required = false) String newConfigName,
+                                          HttpSession session, RedirectAttributes redirectAttributes) {
 
         Query query = new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns()));
         String currentConfigurationName = configurationGetFromSession(session);
 
         List<String> allConfigurationNames = configurationService.getAllConfigurationNames();
-        if (newConfigName == null || newConfigName.length() == 0 ||  allConfigurationNames.contains(newConfigName)) {
+        if (newConfigName == null || newConfigName.length() == 0 || allConfigurationNames.contains(newConfigName)) {
             model.addAttribute("configurations", configurationService.getConfigurationByName(currentConfigurationName));
             model.addAttribute("errorMessage", "Configuration name is not possible.");
+            LOGGER.error("@PostMapping(value = \"/configuration\", params = \"configuration-save-as-button\") - " +
+                    "new configuration could not be saved because the name was not entered!");
             return "configuration";
         }
 
@@ -268 +289 @@
             model.addAttribute("successMessage", "New configuration has been successfully saved.");
             session.setAttribute("configuration", newConfigName);
             model.addAttribute("configurations", configurationService.getConfigurationByName(newConfigName));
+            LOGGER.warn("@PostMapping(value = \"/configuration\", params = \"configuration-save-as-button\") - " +
+                    "some parameters seem to be wrong!");
         } else {
             model.addAttribute("errorMessage", "One or more configuration values are not in correct format, see messages below.");
             antiPatternService.setErrorMessages(query.getAntiPatterns(), wrongParameters);
             model.addAttribute("configurations", configurationService.getConfigurationByName(currentConfigurationName));
+            LOGGER.info("@PostMapping(value = \"/configuration\", params = \"configuration-save-as-button\") - " +
+                    "parameters were successfully processed...");
         }
 
         model.addAttribute("query", query);
@@ -279 +304 @@
         model.addAttribute("configurationList", configurationsGetList(session));
         model.addAttribute("selectedConfiguration", session.getAttribute("configuration"));
 
+        LOGGER.info("@PostMapping(value = \"/configuration\", params = \"configuration-save-as-button\") - " +
+                "everything successfully processed... Returning configuration.html");
         return "configuration";
     }
 
     /**
      * Method for saving changes of one AP in current configuration
      *
-     * @param model object for passing data to the UI
-     * @param id id of AP
+     * @param model           object for passing data to the UI
+     * @param id              id of AP
      * @param thresholdValues new config values
-     * @param thresholdNames configuration names
-     * @param redirectAttrs attributes for redirection
+     * @param thresholdNames  configuration names
+     * @param redirectAttrs   attributes for redirection
      * @return redirected html file name for thymeleaf template
      */
     @PostMapping("/anti-patterns/{id}")
@@ -305 +332 @@
         List<AntiPattern> antiPatterns = antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns());
 
         AntiPattern antiPattern = antiPatternService.antiPatternToModel(antiPatternService.getAntiPatternById(id));
-        List<String> wrongParameters = configurationService.saveNewConfiguration(antiPatterns, currentConfigurationName,antiPatternNames, thresholdNames, thresholdValues, false);
+        List<String> wrongParameters = configurationService.saveNewConfiguration(antiPatterns, currentConfigurationName, antiPatternNames, thresholdNames, thresholdValues, false);
 
 
         if (wrongParameters.isEmpty()) {
             redirectAttrs.addFlashAttribute("successMessage", "All threshold values has been successfully saved.");
+            LOGGER.info("@PostMapping(\"/anti-patterns/{id}\") - Success");
         } else {
             redirectAttrs.addFlashAttribute("errorMessage", "One or more configuration values are not in correct format, see messages below.");
             antiPatternService.setErrorMessages(antiPattern, wrongParameters);
+            LOGGER.warn("@PostMapping(\"/anti-patterns/{id}\") - Some parameters seem to be wrong!");
         }
 
         model.addAttribute("antiPatterns", antiPattern);
         model.addAttribute("configurations", configurationService.getConfigurationByName(currentConfigurationName));
+
+        LOGGER.info("@PostMapping(\"/anti-patterns/{id}\") - redirecting to current anti pattern id");
         return "redirect:/anti-patterns/{id}";
     }
 
     /**
      * Method for storing operationalization detail for individual AP
      *
-     * @param model object for passing data to the UI
-     * @param id id of AP
-     * @param innerText operationalization text (HTML)
+     * @param model         object for passing data to the UI
+     * @param id            id of AP
+     * @param innerText     operationalization text (HTML)
      * @param redirectAttrs attributes for redirection
      * @return redirected html file name for thymeleaf template
      */
@@ -345 +376 @@
             writer.close();
             redirectAttrs.addFlashAttribute("successMessage", "Operationalization detail has been successfully saved.");
         } catch (Exception e) {
+            LOGGER.error("@PostMapping(\"/anti-patterns/{id}/operationalization\") - " +
+                    "An error has occurred while trying to create new BufferedWriter or FileWriter!");
         }
         return "redirect:/anti-patterns/{id}";
     }
@@ -357 +390 @@
      * @throws Exception If image is not in the folder
      */
     @GetMapping("/operationalizations/images/{imageName}")
-    public @ResponseBody byte[] imageGet(@PathVariable String imageName) throws Exception {
+    public @ResponseBody
+    byte[] imageGet(@PathVariable String imageName) throws Exception {
         File f = new File(antiPatternService.getOperationalizationImageFilePath(imageName));
         return Files.readAllBytes(f.toPath());
     }
@@ -369 +403 @@
      * @return result
      */
     @PostMapping("/uploadImage")
-    public ResponseEntity<?> handleFileUpload(@RequestParam("file") MultipartFile file ) {
+    public ResponseEntity<?> handleFileUpload(@RequestParam("file") MultipartFile file) {
 
         String fileName = file.getOriginalFilename();
 
-        if(new File(antiPatternService.getOperationalizationImageFilePath(fileName)).isFile()){
+        if (new File(antiPatternService.getOperationalizationImageFilePath(fileName)).isFile()) {
             return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).build();
         }
 
         try {
             file.transferTo(new File(antiPatternService.getOperationalizationImageFilePath(fileName)));
         } catch (Exception e) {
+            LOGGER.error("An error with uploading images has occurred!");
             return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR).build();
         }
         return ResponseEntity.ok("File uploaded successfully.");
@@ -391 +426 @@
      * @return html file name for thymeleaf template
      */
     @GetMapping("/login")
-    public String login(Model model){
+    public String login(Model model) {
         model.addAttribute("query", new Query(projectService.getAllProjects(), antiPatternService.antiPatternsToModel(antiPatternService.getAllAntiPatterns())));
         return "login";
     }
 
     /**
      * Processing login details
-     * @param session current session
-     * @param nameInput input field for name
-     * @param passInput input field for password
+     *
+     * @param session       current session
+     * @param nameInput     input field for name
+     * @param passInput     input field for password
      * @param redirectAttrs redirect attributes for adding flash message
      * @return html file name for thymeleaf template
      */
@@ -410 +446 @@
                                @RequestParam(value = "passInput", required = false) String passInput,
                                RedirectAttributes redirectAttrs) {
 
-        if(userAccountService.checkCredentials(nameInput, passInput)){
+        if (userAccountService.checkCredentials(nameInput, passInput)) {
             session.setAttribute("user", nameInput);
             return "redirect:/";
         }
@@ -421 +457 @@
 
     /**
      * Getting the name of the logged user from the session
-     * @param model object for passing data to the UI
+     *
+     * @param model   object for passing data to the UI
      * @param session current session
      * @return html file name for thymeleaf template
      */
     @GetMapping("/userLogged")
-    public String userLogged(Model model, HttpSession session){
+    public String userLogged(Model model, HttpSession session) {
         String user = (String) session.getAttribute("user");
         model.addAttribute("user", user);
         return "redirect:/";
@@ -434 +471 @@
 
     /**
      * Processing logout, removing user from session
-     * @param model object for passing data to the UI
+     *
+     * @param model   object for passing data to the UI
      * @param session current session
      * @return html file name for thymeleaf template
      */
     @GetMapping("/logout")
-    public String userLogout(Model model, HttpSession session, RedirectAttributes redirectAttrs){
+    public String userLogout(Model model, HttpSession session, RedirectAttributes redirectAttrs) {
         session.removeAttribute("user");
         session.removeAttribute("configuration");
         return "redirect:/";
@@ -447 +485 @@
 
     /**
      * Model attribute for getting list of configurations
+     *
      * @param session current session
      * @return list of configurations
      */
     @ModelAttribute("configurationList")
-    public List<String> configurationsGetList(HttpSession session){
+    public List<String> configurationsGetList(HttpSession session) {
         List<String> configurationList;
 
-        if(session.getAttribute("user") != null)
+        if (session.getAttribute("user") != null)
             configurationList = configurationService.getAllConfigurationNames();
         else
             configurationList = configurationService.getDefaultConfigurationNames();
@@ -464 +503 @@
 
     /**
      * Model attribute for getting current configuration
+     *
      * @param session current session
      * @return name of current configuration
      */
     @ModelAttribute("selectedConfiguration")
-    public String configurationGetFromSession(HttpSession session){
-        if(session.getAttribute("configuration") != null)
+    public String configurationGetFromSession(HttpSession session) {
+        if (session.getAttribute("configuration") != null)
             return session.getAttribute("configuration").toString(); // return configuration stored in session
-        else{
+        else {
             List<String> configurationList = configurationService.getDefaultConfigurationNames();
-            if(configurationList.size() == 0)
+            if (configurationList.size() == 0)
                 return null;
 
             return configurationList.get(0); // return first item from the list of default configurations
@@ -482 +522 @@
 
     /**
      * Processing select of configuration from select box
-     * @param session current session
+     *
+     * @param session            current session
      * @param selectedConfigName name of selected configuration
-     * @param redirectAttrs attributes for redirection
+     * @param redirectAttrs      attributes for redirection
      * @return html page for redirect
      */
     @PostMapping("/setSelectedConfiguration")
     public String setSelectedConfiguration(HttpSession session,
                                            @RequestParam(value = "current-configuration-select", required = false) String selectedConfigName,
-                                           RedirectAttributes redirectAttrs){
+                                           RedirectAttributes redirectAttrs) {
 
         session.setAttribute("configuration", selectedConfigName); // storing selected configuration to session
 
@@ -499 +540 @@
 
     /**
      * Checking if configuration is default and editable by user
-     * @param model object for passing data to the UI
-     * @param session current session
+     *
+     * @param model             object for passing data to the UI
+     * @param session           current session
      * @param configurationName name of the curren configuration
      * @return html template
      */
     @GetMapping("/isConfigEditable/{configurationName}")
-    public String isConfigEditableForUser(Model model, HttpSession session, @PathVariable String configurationName){
-       List<String> defaultConfigurationNames = configurationService.getDefaultConfigurationNames();
+    public String isConfigEditableForUser(Model model, HttpSession session, @PathVariable String configurationName) {
+        List<String> defaultConfigurationNames = configurationService.getDefaultConfigurationNames();
 
-        for(int i = 0; i < defaultConfigurationNames.size(); i++){
-            if(defaultConfigurationNames.get(i).equals(configurationName))
+        for (int i = 0; i < defaultConfigurationNames.size(); i++) {
+            if (defaultConfigurationNames.get(i).equals(configurationName))
                 model.addAttribute("default", "true"); // configuration is default
         }
 

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/detecting/DatabaseConnection.java

@@ -4 +4 @@
 import cz.zcu.fav.kiv.antipatterndetectionapp.spring.ApplicationProperties;
 import cz.zcu.fav.kiv.antipatterndetectionapp.spring.SpringApplicationContext;
 import cz.zcu.fav.kiv.antipatterndetectionapp.utils.Utils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.sql.*;
 import java.util.ArrayList;
@@ -14 +16 @@
  * A class that takes care of database connections and running queries.
  */
 public class DatabaseConnection {
+    private final Logger LOGGER = LoggerFactory.getLogger(DatabaseConnection.class);
 
     private Connection databaseConnection;
 
@@ -45 +48 @@
 
         } catch (SQLException e) {
             e.printStackTrace();
+            LOGGER.error("Connection with DB could not be created");
         }
         return conn;
     }
@@ -58 +62 @@
         try {
             this.databaseConnection.close();
         } catch (SQLException e) {
+            LOGGER.error("Connection to DB could not be closed!");
             e.printStackTrace();
         }
     }
@@ -96 +101 @@
 
             }
         } catch (SQLException e) {
+            LOGGER.error("DB query could not be performed!");
             e.printStackTrace();
         }
         return resultSet;
@@ -129 +135 @@
 
             }
         } catch (SQLException e) {
+            LOGGER.error("DB query with multiple results could not be performed!");
             e.printStackTrace();
         }
 

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/model/types/Percentage.java

@@ -1 +1 @@
 package cz.zcu.fav.kiv.antipatterndetectionapp.model.types;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * This class represents percentage as a positive float number between 0 an 100.
  */
@@ -10 +13 @@
 
     private final float value;
 
+    private final Logger LOGGER = LoggerFactory.getLogger(Percentage.class);
+
     public Percentage(float value) throws NumberFormatException {
         if (value > MAX_VALUE || value < MIN_VALUE) {
+            LOGGER.error("Invalid number entered!");
             throw new NumberFormatException("Percentage should be between 0 and 100");
         }
         this.value = value;

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/model/types/PositiveFloat.java

@@ -1 +1 @@
 package cz.zcu.fav.kiv.antipatterndetectionapp.model.types;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * This class represents percentage as a positive float number between 0 and MAX_FLOAT.
  */
@@ -9 +12 @@
 
     private final float value;
 
+    private final Logger LOGGER = LoggerFactory.getLogger(Percentage.class);
+
     public PositiveFloat(float value) throws NumberFormatException {
         if (value > MAX_VALUE || value < MIN_VALUE) {
+            LOGGER.error("Invalid number entered!");
             throw new NumberFormatException("Positive integer should be between 0 and infinity");
         }
         this.value = value;

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/model/types/PositiveInteger.java

@@ -1 +1 @@
 package cz.zcu.fav.kiv.antipatterndetectionapp.model.types;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * This class represents percentage as a positive int number between 0 and MAX_INT.
  */
@@ -10 +13 @@
 
     private final int value;
 
+    private final Logger LOGGER = LoggerFactory.getLogger(PositiveInteger.class);
+
     public PositiveInteger(int value) throws NumberFormatException {
         if (value > MAX_VALUE || value < MIN_VALUE) {
+            LOGGER.error("Invalid number entered!");
             throw new NumberFormatException("Positive integer should be between 0 and infinity");
         }
         this.value = value;

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/repository/AntiPatternRepository.java

@@ -182 +182 @@
             APCatalogueFileName = node.get("catalogueFileName") != null ? node.get("catalogueFileName").asText() : null; // optional field
         }
         catch(Exception e){
+            LOGGER.error(e.getMessage());
             return null;
         }
 
@@ -202 +203 @@
                 thresholdErrorMess = tmpNode.get("thresholdErrorMess").asText();
             }
             catch(Exception e){
+                LOGGER.error(e.getMessage());
                 return null;
             }
 
@@ -260 +262 @@
         try {
             return new FileSystemResource(OPERATIONALIZATION_DIR).getFile().getAbsolutePath();
         } catch (Exception e) {
+            LOGGER.error(e.getMessage());
             e.printStackTrace();
         }
         return null;
@@ -269 +272 @@
         try {
             return new FileSystemResource(OPERATIONALIZATION_IMG_DIR).getFile().getAbsolutePath();
         } catch (Exception e) {
+            LOGGER.error(e.getMessage());
             e.printStackTrace();
         }
         return null;

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/repository/ConfigurationRepository.java

@@ -66 +66 @@
                 }
             }
             catch(Exception e){
-                LOGGER.error("Cannot read configuration from file " + fileEntry.getName());
+                LOGGER.error("Cannot read configuration from file " + fileEntry.getName(), e);
                 continue;
             }
 
@@ -183 +183 @@
                         }
                         catch(NumberFormatException e){
                             incorrectParameters.add(thresholdNames[i]);
+                            LOGGER.error(e.getMessage());
                         }
                     }
                     else if(antiPattern.getThresholds().get(thresholdNames[i]).getType().equals("Percentage")){
@@ -191 +192 @@
                         }
                         catch(NumberFormatException e){
                             incorrectParameters.add(thresholdNames[i]);
+                            LOGGER.error(e.getMessage());
                         }
                     }
                     else if(antiPattern.getThresholds().get(thresholdNames[i]).getType().equals("PositiveInteger")){
@@ -199 +201 @@
                         }
                         catch(NumberFormatException e){
                             incorrectParameters.add(thresholdNames[i]);
+                            LOGGER.error(e.getMessage());
                         }
                     }
                     else if(antiPattern.getThresholds().get(thresholdNames[i]).getType().equals("PositiveFloat")){
@@ -207 +210 @@
                         }
                         catch(NumberFormatException e){
                             incorrectParameters.add(thresholdNames[i]);
+                            LOGGER.error(e.getMessage());
                         }
                     }
                     else if(antiPattern.getThresholds().get(thresholdNames[i]).getType().equals("Float")){
@@ -215 +219 @@
                         }
                         catch(NumberFormatException e){
                             incorrectParameters.add(thresholdNames[i]);
+                            LOGGER.error(e.getMessage());
                         }
                     }
                     else if(antiPattern.getThresholds().get(thresholdNames[i]).getType().equals("Double")){
@@ -223 +228 @@
                         }
                         catch(NumberFormatException e){
                             incorrectParameters.add(thresholdNames[i]);
+                            LOGGER.error(e.getMessage());
                         }
                     }
                     else if(antiPattern.getThresholds().get(thresholdNames[i]).getType().equals("String")){

src/main/java/cz/zcu/fav/kiv/antipatterndetectionapp/service/AntiPatternServiceImpl.java

@@ -8 +8 @@
 import cz.zcu.fav.kiv.antipatterndetectionapp.model.QueryResult;
 import cz.zcu.fav.kiv.antipatterndetectionapp.repository.AntiPatternRepository;
 import org.jsoup.Jsoup;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
@@ -19 +21 @@
 
 @Service
 public class AntiPatternServiceImpl implements AntiPatternService {
+    private final Logger LOGGER = LoggerFactory.getLogger(AntiPatternServiceImpl.class);
 
     @Autowired
     private AntiPatternRepository antiPatternRepository;
@@ -141 +144 @@
         }
         catch (Exception e){
             /* If html from catalogue is not extracted, the description from anti-pattern configuration is returned */
+            LOGGER.error(e.getMessage());
             return ap.getDescription();
         }
 
@@ -175 +179 @@
         try {
             content = new String (Files.readAllBytes(Paths.get(filePath)));
         } catch (IOException e) {
+            LOGGER.error(e.getMessage());
             e.printStackTrace();
         }
         return content;

src/main/resources/application.properties

@@ -12 +12 @@
 
 # admin account password:
 account.user.password=
+
+# logger
+logging.level.root=INFO
+logging.level.cz.zcu.fav.kiv.antipatterndetectionapp=TRACE

src/main/resources/logback.xml

@@ +1 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<configuration>
+
+    <property name="LOGS" value="./logs" />
+
+    <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
+        <encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder">
+            <pattern>%d{HH:mm:ss.SSS} %-5level %logger{36} - %msg%n</pattern>
+        </encoder>
+    </appender>
+
+    <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
+        <file>${LOGS}/appController.log</file>
+
+        <encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder">
+            <pattern>%-30(%d{HH:mm:ss} [%thread]) %-5level %logger{10} [%file:%line] %msg%n</pattern>
+        </encoder>
+
+        <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
+            <fileNamePattern>
+                ${LOGS}/archived/appController-%d{yyyy-MM-dd}.%i.log
+            </fileNamePattern>
+            <timeBasedFileNamingAndTriggeringPolicy class="ch.qos.logback.core.rolling.SizeAndTimeBasedFNATP">
+                <maxFileSize>1MB</maxFileSize>
+            </timeBasedFileNamingAndTriggeringPolicy>
+        </rollingPolicy>
+    </appender>
+
+    <root level="info">
+        <appender-ref ref="STDOUT" />
+    </root>
+
+    <logger name="cz.zcu.fav.kiv.antipatterndetectionapp" level="INFO" additivity="false">
+        <appender-ref ref="FILE" />
+        <appender-ref ref="STDOUT" />
+    </logger>
+
+</configuration>