Commit 160a6436 authored by Petr Adamek's avatar Petr Adamek
Browse files

Proper error logging added

parent f0751fa6

src/main/java/cz/muni/fi/pv168/project/model/db/AbstractDataModel.java

+6 −0
Original line number Diff line number Diff line
@@ -4,10 +4,13 @@ import cz.muni.fi.pv168.project.ui.dialog.error.ErrorDialog; 
import cz.muni.fi.pv168.project.ui.i18n.I18N;



import javax.swing.DefaultListModel;

import java.util.logging.Level;

import java.util.logging.Logger;



public abstract class AbstractDataModel<E> extends DefaultListModel<E> {



    private static final I18N I18N = new I18N(AbstractDataModel.class);

    private static final Logger log = Logger.getLogger(AbstractDataModel.class.getName());



    @Override

    public void setElementAt(E element, int index) {
@@ -15,6 +18,7 @@ public abstract class AbstractDataModel<E> extends DefaultListModel<E> { 
            update(element);

            super.setElementAt(element, index);

        } catch (Exception e) {

            log.log(Level.SEVERE, "Error when updating entity " + element, e);

            ErrorDialog.show(I18N.getString("editingEntError") + " " + element, e);

        }

    }
@@ -25,6 +29,7 @@ public abstract class AbstractDataModel<E> extends DefaultListModel<E> { 
            add(element);

            super.addElement(element);

        } catch (Exception e) {

            log.log(Level.SEVERE, "Error when adding entity " + element, e);

            ErrorDialog.show(I18N.getString("addingEntError") + " " + element, e);

        }

    }
@@ -36,6 +41,7 @@ public abstract class AbstractDataModel<E> extends DefaultListModel<E> { 
            delete((E) obj);

            return super.removeElement(obj);

        } catch (Exception e) {

            log.log(Level.SEVERE, "Error when removing entity " + obj, e);

            ErrorDialog.show(I18N.getString("deletingEntError") + " " + obj, e);

        }

        return false;

src/main/java/cz/muni/fi/pv168/project/model/db/TaskModel.java

+4 −0
Original line number Diff line number Diff line
@@ -8,10 +8,13 @@ import cz.muni.fi.pv168.project.ui.i18n.I18N; 


import java.util.ArrayList;

import java.util.function.Predicate;

import java.util.logging.Level;

import java.util.logging.Logger;



public class TaskModel extends UpdatableDataModel<Task> {



    private static final I18N I18N = new I18N(TaskModel.class);

    private static final Logger log = Logger.getLogger(TaskModel.class.getName());



    public TaskModel(DataAccessObject<Task> dao) {

        super(dao);
@@ -28,6 +31,7 @@ public class TaskModel extends UpdatableDataModel<Task> { 
            }

            addAll(tasks);

        } catch (DataAccessException e) {

            log.log(Level.SEVERE, "Error loading all tasks from database", e);

            ErrorDialog.show(I18N.getString("queryAllError"), e);

        }

    }

src/main/java/cz/muni/fi/pv168/project/model/db/UpdatableDataModel.java

+5 −0
Original line number Diff line number Diff line
@@ -4,9 +4,13 @@ import cz.muni.fi.pv168.project.db.interfaces.DataAccessObject; 
import cz.muni.fi.pv168.project.ui.dialog.error.ErrorDialog;

import cz.muni.fi.pv168.project.ui.i18n.I18N;



import java.util.logging.Level;

import java.util.logging.Logger;



public class UpdatableDataModel<E> extends AbstractDataModel<E> {



    protected final DataAccessObject<E> dataAccessObject;

    private static final Logger log = Logger.getLogger(UpdatableDataModel.class.getName());



    private static final I18N I18N = new I18N(UpdatableDataModel.class);
@@ -35,6 +39,7 @@ public class UpdatableDataModel<E> extends AbstractDataModel<E> { 
        try {

            addAll(dataAccessObject.getAll());

        } catch (Exception e) {

            log.log(Level.SEVERE, "Error loading all entities from database", e);

            ErrorDialog.show(I18N.getString("queryAllError"), e);

        }

    }