Добавлено логирование

FormLocomotive.java

@@ -107,7 +107,4 @@ public class FormLocomotive extends JComponent{
         if (_locomotive != null) _locomotive.DrawTransport(g2);
         super.repaint();
     }
-    public static void main(String[] args) {
-        new FormMapWithSetLocomotives();
-    }
 }

FormMapWithSetLocomotives.java

@@ -6,6 +6,7 @@ import java.awt.event.ActionListener;
 import java.awt.image.BufferedImage;
 import java.text.ParseException;
 import java.util.HashMap;
+import java.util.logging.Logger;
 
 public class FormMapWithSetLocomotives extends JComponent {
     private BufferedImage bufferImg = null;
@@ -78,6 +79,7 @@ public class FormMapWithSetLocomotives extends JComponent {
                 return;
             }
             _mapsCollection.AddMap(textFieldNewMapName.getText(), _mapsDict.get(mapSelectComboBox.getSelectedItem().toString()));
+            Logger.getGlobal().info("Map " + textFieldNewMapName.getText() + " added");
             ReloadMaps();
         });
         statusPanel.add(addMapButton);
@@ -87,6 +89,7 @@ public class FormMapWithSetLocomotives extends JComponent {
         listBoxMaps.addListSelectionListener(e -> {
             if(listBoxMaps.getSelectedValue() == null) return;
             bufferImg = _mapsCollection.Get(listBoxMaps.getSelectedValue().toString()).ShowSet();
+            Logger.getGlobal().info("Map switched to " + listBoxMaps.getSelectedValue().toString());
             repaint();
         });
         statusPanel.add(listBoxMaps);
@@ -105,6 +108,7 @@ public class FormMapWithSetLocomotives extends JComponent {
             }
             if(listBoxMaps.getSelectedValue().toString() == null) return;
             _mapsCollection.DelMap(listBoxMaps.getSelectedValue().toString());
+            Logger.getGlobal().info("Map " +listBoxMaps.getSelectedValue().toString() +" deleted");
             ReloadMaps();
             repaint();
         });
@@ -120,16 +124,27 @@ public class FormMapWithSetLocomotives extends JComponent {
                         return;
                     }
                     if (locomotive!=null) {
+                        try {
                             DrawningObjectLocomotive objectLocomotive = new DrawningObjectLocomotive(locomotive);
                             if (_mapsCollection.Get(listBoxMaps.getSelectedValue().toString()).Plus(objectLocomotive)!= -1){
                                 JOptionPane.showMessageDialog(formFrame, "Object added", "Success", JOptionPane.OK_CANCEL_OPTION);
                                 bufferImg = _mapsCollection.Get(listBoxMaps.getSelectedValue().toString()).ShowSet();
+                                Logger.getGlobal().info("Object " + locomotive + " added");
                                 repaint();
                             }
                             else {
                                 JOptionPane.showMessageDialog(formFrame, "Object cannot be added", "Error", JOptionPane.OK_CANCEL_OPTION);
                             }
                         }
+                        catch (StorageOverflowException ex) {
+                            JOptionPane.showMessageDialog(formFrame, "Storage overflow error: "+ ex.getMessage());
+                            Logger.getGlobal().severe("Error " + ex.getMessage());
+                        }
+                        catch (Exception ex) {
+                            JOptionPane.showMessageDialog(formFrame, "Unknown error: "+ ex.getMessage());
+                            Logger.getGlobal().severe("Error " + ex.getMessage());
+                        }
+                    }
                 });
         });
         statusPanel.add(addLocomotiveButton);
@@ -149,6 +164,7 @@ public class FormMapWithSetLocomotives extends JComponent {
         JFormattedTextField finalMaskedTextFieldPosition = maskedTextFieldPosition;
         deleteLocomotiveButton.addActionListener(e -> {
             // логика удаления
+            try{
                 if ((String)mapSelectComboBox.getSelectedItem() == null) {
                     return;
                 }
@@ -159,10 +175,19 @@ public class FormMapWithSetLocomotives extends JComponent {
                 if (_mapsCollection.Get(listBoxMaps.getSelectedValue().toString()).Minus(position) != null) {
                     JOptionPane.showMessageDialog(formFrame, "Object removed", "Success", JOptionPane.OK_CANCEL_OPTION);
                     bufferImg = _mapsCollection.Get(listBoxMaps.getSelectedValue().toString()).ShowSet();
+                    Logger.getGlobal().info("Locomotive deleted at position " + position);
                     repaint();
                 }
                 else{
                     JOptionPane.showMessageDialog(formFrame, "Object cannot be removed", "Error", JOptionPane.OK_CANCEL_OPTION);
+                }}
+            catch (LocomotiveNotFoundException ex) {
+                JOptionPane.showMessageDialog(formFrame, "Locomotive not found error: " + ex.getMessage());
+                Logger.getGlobal().severe("Error " + ex.getMessage());
+            }
+            catch (Exception ex) {
+                JOptionPane.showMessageDialog(formFrame, "Unknown error: "+ ex.getMessage());
+                Logger.getGlobal().severe("Error " + ex.getMessage());
             }
         });
         statusPanel.add(deleteLocomotiveButton);
@@ -186,8 +211,14 @@ public class FormMapWithSetLocomotives extends JComponent {
             {
                 return;
             }
+            try {
                 bufferImg = _mapsCollection.Get(listBoxMaps.getSelectedValue().toString()).ShowOnMap();
                 repaint();
+            }
+            catch (Exception ex) {
+                JOptionPane.showMessageDialog(null, "Unknown error: "+ ex.getMessage());
+                Logger.getGlobal().severe("Error " + ex.getMessage());
+            }
         });
         statusPanel.add(showOnMapButton);
 
@@ -271,10 +302,15 @@ public class FormMapWithSetLocomotives extends JComponent {
             fileChooser.setDialogTitle("Saving");
             int result = fileChooser.showSaveDialog(FormMapWithSetLocomotives.this);
             if (result == JFileChooser.APPROVE_OPTION ) {
-                if (_mapsCollection.SaveData(fileChooser.getSelectedFile().getAbsolutePath())) {
+                try {
+                    _mapsCollection.SaveData(fileChooser.getSelectedFile().getAbsolutePath());
                     JOptionPane.showMessageDialog(null, "Save success");
+                    Logger.getGlobal().info("Saved all to " + fileChooser.getSelectedFile().getAbsolutePath());
+                }
+                catch (Exception ex) {
+                    JOptionPane.showMessageDialog(null, "Save error: " + ex.getMessage());
+                    Logger.getGlobal().severe("Error " + ex.getMessage());
                 }
-                else JOptionPane.showMessageDialog(null, "Save failed");
             }
 
         });
@@ -285,13 +321,18 @@ public class FormMapWithSetLocomotives extends JComponent {
             fileChooser.setDialogTitle("Loading");
             int result = fileChooser.showSaveDialog(FormMapWithSetLocomotives.this);
             if (result == JFileChooser.APPROVE_OPTION ) {
-                if (_mapsCollection.LoadData(fileChooser.getSelectedFile().getAbsolutePath())) {
+                try {
+                    _mapsCollection.LoadData(fileChooser.getSelectedFile().getAbsolutePath());
                     JOptionPane.showMessageDialog(null, "Load success");
-                }
+                    Logger.getGlobal().info("Loaded all from " + fileChooser.getSelectedFile().getAbsolutePath());
-                else JOptionPane.showMessageDialog(null, "Load failed");
-            }
                     ReloadMaps();
                     repaint();
+                }
+                catch (Exception ex) {
+                    JOptionPane.showMessageDialog(null, "Load error: " + ex.getMessage());
+                    Logger.getGlobal().severe("Error " + ex.getMessage());
+                }
+            }
         });
         statusPanel.add(loadButton);
 
@@ -306,12 +347,17 @@ public class FormMapWithSetLocomotives extends JComponent {
                     return;
                 }
                 if(listBoxMaps.getSelectedValue().toString() == null) return;
-                if (_mapsCollection.SaveMap(listBoxMaps.getSelectedValue().toString(), fileChooser.getSelectedFile().getAbsolutePath())) {
+                try {
+                    _mapsCollection.SaveMap(listBoxMaps.getSelectedValue().toString(), fileChooser.getSelectedFile().getAbsolutePath());
                     JOptionPane.showMessageDialog(null, "Map saving success");
-                }
+                    Logger.getGlobal().info("Saved map to " + fileChooser.getSelectedFile().getAbsolutePath());
-                else JOptionPane.showMessageDialog(null, "Map saving fail");
                     repaint();
                 }
+                catch (Exception ex) {
+                    JOptionPane.showMessageDialog(null, "Map saving error: " + ex.getMessage());
+                    Logger.getGlobal().severe("Error " + ex.getMessage());
+                }
+            }
         });
         statusPanel.add(saveMapButton);
 
@@ -321,13 +367,18 @@ public class FormMapWithSetLocomotives extends JComponent {
             fileChooser.setDialogTitle("Loading");
             int result = fileChooser.showSaveDialog(FormMapWithSetLocomotives.this);
             if (result == JFileChooser.APPROVE_OPTION ) {
-                if (_mapsCollection.LoadMap(fileChooser.getSelectedFile().getAbsolutePath())) {
+                try {
+                    _mapsCollection.LoadMap(fileChooser.getSelectedFile().getAbsolutePath());
                     JOptionPane.showMessageDialog(null, "Load Map success");
-                }
+                    Logger.getGlobal().info("Loaded map from " + fileChooser.getSelectedFile().getAbsolutePath());
-                else JOptionPane.showMessageDialog(null, "Load Map failed");
-            }
                     ReloadMaps();
                     repaint();
+                }
+                catch (Exception ex) {
+                    JOptionPane.showMessageDialog(null, "Load Map error: " + ex.getMessage());
+                    Logger.getGlobal().severe("Error " + ex.getMessage());
+                }
+            }
         });
         statusPanel.add(loadMapButton);
 

LocomotiveNotFoundException.java

@@ -0,0 +1,5 @@
+public class LocomotiveNotFoundException extends Exception{
+    public LocomotiveNotFoundException(int i) {
+        super("Не найден объект по позиции " + i);
+    }
+}

Main.java

@@ -0,0 +1,24 @@
+import javax.swing.*;
+import java.awt.*;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.IOException;
+import java.util.logging.LogManager;
+import java.util.logging.Logger;
+
+public class Main {
+    private static Logger logger = null;
+
+    public static void main(String[] args) {
+        try {
+            FileInputStream stream = new FileInputStream(new File("C:/secondCourse/OOP/PIbd-23_Mochalov_D.V._Locomotive_Hard/log.config"));
+            LogManager.getLogManager().readConfiguration(stream);
+            logger = Logger.getGlobal();
+            stream.close();
+
+        } catch (IOException e) {
+            throw new RuntimeException(e);
+        }
+        new FormMapWithSetLocomotives();
+    }
+}

MapWithSetLocomotivesGeneric.java

@@ -13,9 +13,9 @@ public class MapWithSetLocomotivesGeneric
     /// Высота окна отрисовки
     private final int _pictureHeight;
     /// Размер занимаемого объектом места (ширина)
-    private final int _placeSizeWidth = 210;
+    private final int _placeSizeWidth = 150;
     /// Размер занимаемого объектом места (высота)
-    private final int _placeSizeHeight = 90;
+    private final int _placeSizeHeight = 150;
     public final SetLocomotivesGeneric<T> _setLocomotives;
 
     // Список удаленных объектов
@@ -39,12 +39,12 @@ public class MapWithSetLocomotivesGeneric
     }
 
     /// Добавление
-    public int Plus(T locomotive)
+    public int Plus(T locomotive) throws StorageOverflowException
     {
         return this._setLocomotives.Insert(locomotive);
     }
     /// Удаление
-    public T Minus(int position)
+    public T Minus(int position) throws LocomotiveNotFoundException
     {
         T temp = this._setLocomotives.Remove(position);
         if (temp == null) return null;
@@ -62,7 +62,7 @@ public class MapWithSetLocomotivesGeneric
     }
 
     /// Просмотр объекта на карте
-    public BufferedImage ShowOnMap()
+    public BufferedImage ShowOnMap() throws LocomotiveNotFoundException, StorageOverflowException
     {
         Shaking();
         for (var locomotive : _setLocomotives.GetLocomotives())
@@ -86,7 +86,7 @@ public class MapWithSetLocomotivesGeneric
     }
 
     /// "Взбалтываем" набор, чтобы все элементы оказались в начале
-    private void Shaking()
+    private void Shaking() throws LocomotiveNotFoundException, StorageOverflowException
     {
         int j = _setLocomotives.Count() - 1;
         for (int i = 0; i < _setLocomotives.Count(); i++)
@@ -157,8 +157,8 @@ public class MapWithSetLocomotivesGeneric
     /// Метод прорисовки объектов
     private void DrawLocomotives(Graphics2D g)
     {
-        int width = _pictureWidth / _placeSizeWidth;
+        int width = _pictureWidth / _placeSizeWidth - 1;
-        int height = _pictureHeight / _placeSizeHeight;
+        int height = _pictureHeight / _placeSizeHeight - 1;
 
         int curWidth = 0;
         int curHeight = 0;
@@ -166,7 +166,7 @@ public class MapWithSetLocomotivesGeneric
         for (var locomotive : _setLocomotives.GetLocomotives())
         {
             // установка позиции
-            if (locomotive != null) locomotive.SetObject(curWidth * _placeSizeWidth + 10, curHeight * _placeSizeHeight + 18, _pictureWidth, _pictureHeight);
+            if (locomotive != null) locomotive.SetObject(curWidth * _placeSizeWidth + 10, curHeight * _placeSizeHeight + 80, _pictureWidth, _pictureHeight);
             if (locomotive != null) locomotive.DrawningObject(g);
             if (curWidth < width) curWidth++;
             else
@@ -189,7 +189,7 @@ public class MapWithSetLocomotivesGeneric
     }
 
     /// Загрузка списка из массива строк
-    public void LoadData(String[] records)
+    public void LoadData(String[] records) throws StorageOverflowException
     {
         Collections.reverse(Arrays.asList(records));
         for (var rec : records)

MapsCollection.java

@@ -2,6 +2,8 @@ import java.io.*;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
+import java.util.logging.Logger;
+import java.util.zip.DataFormatException;
 
 public class MapsCollection {
     /// Словарь (хранилище) с картами
@@ -51,59 +53,53 @@ public class MapsCollection {
     }
 
     /// Сохранение информации по локомотивам в хранилище в файл
-    public boolean SaveData(String filename)
+    public void SaveData(String filename) throws IOException
     {
         File file = new File(filename);
         if (file.exists())
         {
             file.delete();
         }
-        try(BufferedWriter bw = new BufferedWriter(new FileWriter(filename))) {
+        BufferedWriter bw = new BufferedWriter(new FileWriter(filename));
         bw.write("MapsCollection\n");
         for (var storage : _mapStorages.entrySet()) {
             bw.write("" + storage.getKey() + separatorDict + storage.getValue().GetData(separatorDict.charAt(0), separatorData) + "\n");
         }
-        }
+        bw.flush();
-        catch (IOException ex) {
-            System.out.println(ex.getMessage());
-        }
-        return true;
     }
 
     // Сохранение одной выбранной карты
-    public boolean SaveMap(String map_name, String filename) {
+    public void SaveMap(String map_name, String filename) throws IOException{
         File file = new File(filename);
         MapWithSetLocomotivesGeneric<IDrawningObject, AbstractMap> map = Get(map_name);
         if (file.exists())
         {
             file.delete();
         }
-        try(BufferedWriter bw = new BufferedWriter(new FileWriter(filename))) {
+        BufferedWriter bw = new BufferedWriter(new FileWriter(filename));
         bw.write("SingleMap\n");
         bw.write("" + map_name + "\n");
         bw.write("" + map.getClass() + "\n");
         for (var locomotive : map._setLocomotives.GetLocomotives()) {
             bw.write("" + locomotive.getInfo() + "\n");
         }
-        }
+        bw.flush();
-        catch (IOException ex) {
+
-            System.out.println(ex.getMessage());
-        }
-        return true;
     }
 
     // Загрузка одной карты
-    public boolean LoadMap(String filename){
+    public void LoadMap(String filename) throws StorageOverflowException, IOException, DataFormatException{
             File file = new File(filename);
             if (!file.exists())
             {
-            return false;
+                Logger.getGlobal().severe("FileNotFoundException " + filename);
+                throw new FileNotFoundException("Файл не найден");
             }
-        try {
             BufferedReader br = new BufferedReader(new FileReader(filename));
             String curLine = br.readLine();
             if (!curLine.contains("SingleMap")) {
-                return  false;
+                Logger.getGlobal().severe("FileFormatException " + filename);
+                throw new DataFormatException("Неверный формат данных");
             }
             String mapName = br.readLine();
             String mapClass = br.readLine();
@@ -113,7 +109,6 @@ public class MapsCollection {
                     _mapStorages.get(mapName)._setLocomotives.Insert(DrawningObjectLocomotive.Create(curLine));
                 }
                 _mapStorages.get(mapName)._setLocomotives.ReversePlaces();
-                return true;
             }
             AbstractMap map = null;
             switch (mapClass)
@@ -133,33 +128,29 @@ public class MapsCollection {
                 _mapStorages.get(mapName)._setLocomotives.Insert(DrawningObjectLocomotive.Create(curLine));
             }
             _mapStorages.get(mapName)._setLocomotives.ReversePlaces();
-
-        } catch (IOException ex) {
-            System.out.println(ex.getMessage());
-        }
-        return true;
     }
 
     /// Загрузка информации по локомотивам в депо из файла
-    public boolean LoadData(String filename)
+    public void LoadData(String filename) throws DataFormatException, IOException, StorageOverflowException
     {
         File file = new File(filename);
         if (!file.exists())
         {
-            return false;
+            Logger.getGlobal().severe("FileNotFoundException " + filename);
+            throw new FileNotFoundException("Файл не найден");
         }
-        try {
+
         BufferedReader br = new BufferedReader(new FileReader(filename));
         String curLine = br.readLine();
         if (!curLine.contains("MapsCollection")) {
-                return  false;
+            Logger.getGlobal().severe("FileFormatException " + filename);
+            throw new DataFormatException("Неверный формат данных");
         }
 
         _mapStorages.clear();
         while ((curLine = br.readLine()) != null) {
             var elems = curLine.split(separatorDict);
             AbstractMap map = null;
-
             switch (elems[1])
             {
                 case "class SimpleMap":
@@ -176,10 +167,6 @@ public class MapsCollection {
             _mapStorages.put(elems[0], new MapWithSetLocomotivesGeneric<IDrawningObject, AbstractMap>(_pictureWidth, _pictureHeight, map));
             _mapStorages.get(elems[0]).LoadData(elems[2].split(Character.toString(separatorData)));
         }
-        } catch (IOException ex) {
-            System.out.println(ex.getMessage());
-        }
-        return true;
     }
 
 

SetLocomotivesGeneric.java

@@ -1,5 +1,6 @@
 import java.util.ArrayList;
 import java.util.Collections;
+import java.util.logging.Logger;
 
 public class SetLocomotivesGeneric <T>
 {
@@ -18,20 +19,28 @@ public class SetLocomotivesGeneric <T>
         _places = new ArrayList<>();
     }
 
-    public int Insert (T locomotive) {
+    public int Insert (T locomotive) throws StorageOverflowException{
         return Insert(locomotive, 0);
     }
 
-    public int Insert (T locomotive, int position) {
+    public int Insert (T locomotive, int position) throws StorageOverflowException{
-        if (position >= _maxCount|| position < 0) return -1;
+       if (position < 0) return -1;
+       if (Count() >= _maxCount) {
+           Logger.getGlobal().warning("StorageOverflowException");
+           throw new StorageOverflowException(_maxCount);
+       }
        _places.add(position, locomotive);
        return position;
     }
 
-    public T Remove (int position) {
+    public T Remove (int position) throws LocomotiveNotFoundException {
         if (position >= _maxCount || position < 0) return null;
+        if (_places.get(position) == null) {
+            Logger.getGlobal().warning("LocomotiveNotFoundException at " + position);
+            throw new LocomotiveNotFoundException(position);
+        }
         T result = _places.get(position);
-        _places.remove(position);
+        _places.set(position, null);
         return result;
     }
 

StorageOverflowException.java

@@ -0,0 +1,5 @@
+public class StorageOverflowException extends Exception{
+    public StorageOverflowException(int count) {
+        super("В наборе превышено допустимое количество: " + count);
+    }
+}

log.config

@@ -0,0 +1,7 @@
+handlers = java.util.logging.FileHandler
+java.util.logging.FileHandler.level     = INFO
+java.util.logging.FileHandler.formatter = java.util.logging.SimpleFormatter
+java.util.logging.SimpleFormatter.format= %4$S %5$S (%1$td.%1$tm.%1$tY) %n
+java.util.logging.FileHandler.append    = true
+java.util.logging.FileHandler.pattern   = log.txt
+java.util.logging.FileHandler.limit = 100000