ChipsEater/PIbd-23_Polevoy_S.V._SelfPropelledArtilleryUnit_Hard
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Compare commits

base: ChipsEater:60bc67a8f509e534890a5a3739081e0720f1c61b
Branches Tags
ChipsEater:master
ChipsEater:LabWork07
ChipsEater:LabWork06
ChipsEater:LabWork05
ChipsEater:LabWork04
ChipsEater:LabWork02
ChipsEater:LabWork03
ChipsEater:LabWork01
..
compare: ChipsEater:7839cb7aadfa45e18d2fd01be8644fdb6eefdd5a
Branches Tags
ChipsEater:LabWork07
ChipsEater:LabWork06
ChipsEater:LabWork05
ChipsEater:LabWork04
ChipsEater:LabWork02
ChipsEater:LabWork03
ChipsEater:LabWork01
ChipsEater:master

5 Commits
60bc67a8f5 ... 7839cb7aad

Author SHA1 Message Date
Сергей Полевой
7839cb7aad Fixes 2022-12-05 23:43:06 +04:00
Сергей Полевой
4cadb6385c Fixes 2022-12-05 23:42:43 +04:00
Сергей Полевой
3845906a87 Added logging 2022-12-05 23:37:38 +04:00
Сергей Полевой
09e3dfc336 Added exception handling to FormMapWithSetArtilleries 2022-12-05 23:00:06 +04:00
Сергей Полевой
36a610081e Added exceptions to MapsCollection 2022-12-05 22:34:50 +04:00
4 changed files with 114 additions and 67 deletions
Show Stats Expand all files Collapse all files

124
FormMapWithSetArtilleries.java
View File

@ -1,3 +1,5 @@
import org.apache.logging.log4j.Logger;
import javax.swing.*; import javax.swing.*;
import javax.swing.filechooser.FileNameExtensionFilter; import javax.swing.filechooser.FileNameExtensionFilter;
import javax.swing.text.DefaultFormatterFactory; import javax.swing.text.DefaultFormatterFactory;
@ -41,6 +43,13 @@ public class FormMapWithSetArtilleries extends JFrame {
private final MapsCollection _mapsCollection; private final MapsCollection _mapsCollection;
private final Stack<IDrawingObject> deletedObjects = new Stack<>(); private final Stack<IDrawingObject> deletedObjects = new Stack<>();
private Logger logger;
public FormMapWithSetArtilleries(Logger logger) {
this();
this.logger = logger;
}
public FormMapWithSetArtilleries() { public FormMapWithSetArtilleries() {
this.setTitle("Artillery"); this.setTitle("Artillery");
this.setContentPane(paneArtilleries); this.setContentPane(paneArtilleries);
@ -67,13 +76,13 @@ public class FormMapWithSetArtilleries extends JFrame {
dialog.showSaveDialog(this); dialog.showSaveDialog(this);
try { try {
if (_mapsCollection.saveData(dialog.getSelectedFile().getAbsolutePath())) { _mapsCollection.saveData(dialog.getSelectedFile().getAbsolutePath());
JOptionPane.showMessageDialog(this, "Сохранение прошло успешно", "Успех", JOptionPane.INFORMATION_MESSAGE); logger.info("Сохранение в файл \"" + dialog.getSelectedFile().getAbsolutePath() + "\"");
} else { JOptionPane.showMessageDialog(this, "Сохранение прошло успешно", "Успех", JOptionPane.INFORMATION_MESSAGE);
JOptionPane.showMessageDialog(this, "Не сохранилось", "Провал", JOptionPane.INFORMATION_MESSAGE);
} } catch (Exception ex) {
} catch (IOException ex) { logger.error("Ошибка сохранения в файл: " + ex.getMessage());
ex.printStackTrace(); JOptionPane.showMessageDialog(this, "Не сохранилось: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
} }
}); });
fileMenu.add(saveMenuItem); fileMenu.add(saveMenuItem);
@ -85,14 +94,13 @@ public class FormMapWithSetArtilleries extends JFrame {
dialog.showOpenDialog(this); dialog.showOpenDialog(this);
try { try {
if (_mapsCollection.loadData(dialog.getSelectedFile().getAbsolutePath())) { _mapsCollection.loadData(dialog.getSelectedFile().getAbsolutePath());
reloadMaps(); reloadMaps();
JOptionPane.showMessageDialog(this, "Загрузка прошла успешно", "Успех", JOptionPane.INFORMATION_MESSAGE); logger.info("Загрузка из файла \"" + dialog.getSelectedFile().getAbsolutePath() + "\"");
} else { JOptionPane.showMessageDialog(this, "Загрузка прошла успешно", "Успех", JOptionPane.INFORMATION_MESSAGE);
JOptionPane.showMessageDialog(this, "Не загрузилось", "Провал", JOptionPane.INFORMATION_MESSAGE); } catch (Exception ex) {
} logger.error("Ошибка загрузки из файла: " + ex.getMessage());
} catch (IOException ex) { JOptionPane.showMessageDialog(this, "Не загрузилось: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
ex.printStackTrace();
} }
}); });
fileMenu.add(loadMenuItem); fileMenu.add(loadMenuItem);
@ -104,13 +112,12 @@ public class FormMapWithSetArtilleries extends JFrame {
dialog.showSaveDialog(this); dialog.showSaveDialog(this);
try { try {
if (_mapsCollection.saveMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse(""), dialog.getSelectedFile().getAbsolutePath())) { _mapsCollection.saveMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse(""), dialog.getSelectedFile().getAbsolutePath());
JOptionPane.showMessageDialog(this, "Сохранение прошло успешно", "Успех", JOptionPane.INFORMATION_MESSAGE); logger.info("Сохранение карты в файл \"" + dialog.getSelectedFile().getAbsolutePath() + "\"");
} else { JOptionPane.showMessageDialog(this, "Сохранение прошло успешно", "Успех", JOptionPane.INFORMATION_MESSAGE);
JOptionPane.showMessageDialog(this, "Не сохранилось", "Провал", JOptionPane.INFORMATION_MESSAGE); } catch (Exception ex) {
} logger.error("Ошибка сохранения карты в файл: " + ex.getMessage());
} catch (IOException ex) { JOptionPane.showMessageDialog(this, "Не сохранилось: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
ex.printStackTrace();
} }
}); });
fileMenu.add(saveMapMenuItem); fileMenu.add(saveMapMenuItem);
@ -122,14 +129,13 @@ public class FormMapWithSetArtilleries extends JFrame {
dialog.showOpenDialog(this); dialog.showOpenDialog(this);
try { try {
if (_mapsCollection.loadMap(dialog.getSelectedFile().getAbsolutePath())) { _mapsCollection.loadMap(dialog.getSelectedFile().getAbsolutePath());
reloadMaps(); reloadMaps();
JOptionPane.showMessageDialog(this, "Загрузка прошла успешно", "Успех", JOptionPane.INFORMATION_MESSAGE); logger.info("Загрузка карты из файла \"" + dialog.getSelectedFile().getAbsolutePath() + "\"");
} else { JOptionPane.showMessageDialog(this, "Загрузка прошла успешно", "Успех", JOptionPane.INFORMATION_MESSAGE);
JOptionPane.showMessageDialog(this, "Не загрузилось", "Провал", JOptionPane.INFORMATION_MESSAGE); } catch (Exception ex) {
} logger.error("Ошибка загрузки карты из файла: " + ex.getMessage());
} catch (IOException ex) { JOptionPane.showMessageDialog(this, "Не загрузилось: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
ex.printStackTrace();
} }
}); });
fileMenu.add(loadMapMenuItem); fileMenu.add(loadMapMenuItem);
@ -143,15 +149,18 @@ public class FormMapWithSetArtilleries extends JFrame {
buttonAddMap.addActionListener(e -> { buttonAddMap.addActionListener(e -> {
if (comboBoxMapSelector.getSelectedIndex() == -1 || textFieldMapName.getText() == null || textFieldMapName.getText().isEmpty()) { if (comboBoxMapSelector.getSelectedIndex() == -1 || textFieldMapName.getText() == null || textFieldMapName.getText().isEmpty()) {
logger.warn("Не удалось добавить карту: Не все данные заполнены");
JOptionPane.showMessageDialog(this, "Не все данные заполнены", "Ошибка", JOptionPane.ERROR_MESSAGE); JOptionPane.showMessageDialog(this, "Не все данные заполнены", "Ошибка", JOptionPane.ERROR_MESSAGE);
return; return;
} }
if (!_mapsDict.containsKey((String)comboBoxMapSelector.getSelectedItem())) { if (!_mapsDict.containsKey((String)comboBoxMapSelector.getSelectedItem())) {
logger.warn("Не удалось добавить карту: Нет такой карты");
JOptionPane.showMessageDialog(this, "Нет такой карты", "Ошибка", JOptionPane.ERROR_MESSAGE); JOptionPane.showMessageDialog(this, "Нет такой карты", "Ошибка", JOptionPane.ERROR_MESSAGE);
return; return;
} }
_mapsCollection.addMap(textFieldMapName.getText(), _mapsDict.get((String)comboBoxMapSelector.getSelectedItem())); _mapsCollection.addMap(textFieldMapName.getText(), _mapsDict.get((String)comboBoxMapSelector.getSelectedItem()));
reloadMaps(); reloadMaps();
logger.info("Добавлена карта \"" + textFieldMapName.getText() + "\" типа " + comboBoxMapSelector.getSelectedItem());
}); });
buttonDeleteMap.addActionListener(e -> { buttonDeleteMap.addActionListener(e -> {
@ -161,6 +170,7 @@ public class FormMapWithSetArtilleries extends JFrame {
if (JOptionPane.showConfirmDialog(this, "Удалить карту " + listBoxMaps.getSelectedValue() + "?", "Удаление", JOptionPane.YES_NO_OPTION) == JOptionPane.YES_OPTION) { if (JOptionPane.showConfirmDialog(this, "Удалить карту " + listBoxMaps.getSelectedValue() + "?", "Удаление", JOptionPane.YES_NO_OPTION) == JOptionPane.YES_OPTION) {
_mapsCollection.deleteMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")); _mapsCollection.deleteMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse(""));
logger.info("Удалена карта \"" + listBoxMaps.getSelectedValue() + "\"");
reloadMaps(); reloadMaps();
} }
}); });
@ -169,6 +179,7 @@ public class FormMapWithSetArtilleries extends JFrame {
if (listBoxMaps.getSelectedValue() != null) { if (listBoxMaps.getSelectedValue() != null) {
bufferedImage = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).showSet(); bufferedImage = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).showSet();
repaint(); repaint();
logger.info("Переход на карту \"" + listBoxMaps.getSelectedValue() + "\"");
} }
}); });
@ -180,16 +191,23 @@ public class FormMapWithSetArtilleries extends JFrame {
} }
if (artillery != null) { if (artillery != null) {
DrawingObjectArtillery objectArtillery = new DrawingObjectArtillery(artillery); try {
if (_mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).addArtillery(objectArtillery) != -1) DrawingObjectArtillery objectArtillery = new DrawingObjectArtillery(artillery);
{ if (_mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).addArtillery(objectArtillery) != -1) {
JOptionPane.showMessageDialog(this, "Объект добавлен", "Успех", JOptionPane.INFORMATION_MESSAGE); JOptionPane.showMessageDialog(this, "Объект добавлен", "Успех", JOptionPane.INFORMATION_MESSAGE);
bufferedImage = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).showSet(); bufferedImage = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).showSet();
repaint(); repaint();
} logger.info("Добавлен новый объект");
else } else {
{ logger.warn("Не удалось добавить объект");
JOptionPane.showMessageDialog(this, "Не удалось добавить объект", "Провал", JOptionPane.INFORMATION_MESSAGE); JOptionPane.showMessageDialog(this, "Не удалось добавить объект", "Провал", JOptionPane.INFORMATION_MESSAGE);
}
} catch (StorageOverflowException ex) {
logger.warn("Ошибка переполнения хранилища: " + ex.getMessage());
JOptionPane.showMessageDialog(this, "Ошибка переполнения хранилища: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
} catch (Exception ex) {
logger.fatal("Неизвестная ошибка: " + ex.getMessage());
JOptionPane.showMessageDialog(this, "Неизвестная ошибка: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
} }
} }
}); });
@ -208,14 +226,24 @@ public class FormMapWithSetArtilleries extends JFrame {
int position = Integer.parseInt(text); int position = Integer.parseInt(text);
IDrawingObject deleted = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).removeArtilleryAt(position); try {
if (deleted != null) { IDrawingObject deleted = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).removeArtilleryAt(position);
deletedObjects.push(deleted); if (deleted != null) {
JOptionPane.showMessageDialog(this, "Объект удалён", "Успех", JOptionPane.INFORMATION_MESSAGE); deletedObjects.push(deleted);
bufferedImage = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).showSet(); JOptionPane.showMessageDialog(this, "Объект удалён", "Успех", JOptionPane.INFORMATION_MESSAGE);
repaint(); bufferedImage = _mapsCollection.getMap(Optional.ofNullable(listBoxMaps.getSelectedValue()).orElse("")).showSet();
} else { repaint();
JOptionPane.showMessageDialog(this, "Не удалось удалить объект", "Провал", JOptionPane.INFORMATION_MESSAGE); logger.info("Удалён объект");
} else {
logger.warn("Не удалось удалить объект по позиции " + position + ". Объект равен null");
JOptionPane.showMessageDialog(this, "Не удалось удалить объект", "Провал", JOptionPane.INFORMATION_MESSAGE);
}
} catch (ArtilleryNotFoundException ex) {
logger.warn("Ошибка удаления: " + ex.getMessage());
JOptionPane.showMessageDialog(this, "Не удалось найти артиллерию по позиции: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
} catch (Exception ex) {
logger.error("Неизвестная ошибка: " + ex.getMessage());
JOptionPane.showMessageDialog(this, "Неизвестная ошибка: " + ex.getMessage(), "Провал", JOptionPane.INFORMATION_MESSAGE);
} }
}); });

29
MapsCollection.java
View File

@ -1,4 +1,5 @@
import java.io.*; import java.io.*;
import java.nio.file.NoSuchFileException;
import java.util.HashMap; import java.util.HashMap;
import java.util.Set; import java.util.Set;
@ -44,7 +45,7 @@ public class MapsCollection {
} }
@SuppressWarnings("ResultOfMethodCallIgnored") @SuppressWarnings("ResultOfMethodCallIgnored")
public boolean saveData(String filename) throws IOException { public void saveData(String filename) throws IOException {
File file = new File(filename); File file = new File(filename);
if (file.exists()) { if (file.exists()) {
@ -60,22 +61,20 @@ public class MapsCollection {
writer.println(String.format("%s%c%s", storage.getKey(), separatorDict, storage.getValue().getData(separatorDict, separatorData))); writer.println(String.format("%s%c%s", storage.getKey(), separatorDict, storage.getValue().getData(separatorDict, separatorData)));
} }
} }
return true;
} }
public boolean loadData(String filename) throws IOException { public void loadData(String filename) throws IOException {
File file = new File(filename); File file = new File(filename);
if (!file.exists()) { if (!file.exists()) {
return false; throw new FileNotFoundException("Файл не найден");
} }
try (BufferedReader reader = new BufferedReader(new FileReader(file))) { try (BufferedReader reader = new BufferedReader(new FileReader(file))) {
String currentLine = reader.readLine(); String currentLine = reader.readLine();
if (currentLine == null || !currentLine.contains("MapsCollection")) { if (currentLine == null || !currentLine.contains("MapsCollection")) {
return false; throw new FileFormatException("Неверный формат файла");
} }
_mapsStorage.clear(); _mapsStorage.clear();
@ -92,12 +91,10 @@ public class MapsCollection {
_mapsStorage.get(elements[0]).loadData(elements[2].split(separatorData + "\n?")); _mapsStorage.get(elements[0]).loadData(elements[2].split(separatorData + "\n?"));
} }
} }
return true;
} }
@SuppressWarnings("ResultOfMethodCallIgnored") @SuppressWarnings("ResultOfMethodCallIgnored")
public boolean saveMap(String mapName, String filename) throws IOException { public void saveMap(String mapName, String filename) throws Exception {
File file = new File(filename); File file = new File(filename);
if (file.exists()) { if (file.exists()) {
@ -109,7 +106,7 @@ public class MapsCollection {
MapWithSetArtilleriesGeneric<IDrawingObject, AbstractMap> map = _mapsStorage.getOrDefault(mapName, null); MapWithSetArtilleriesGeneric<IDrawingObject, AbstractMap> map = _mapsStorage.getOrDefault(mapName, null);
if (map == null) { if (map == null) {
return false; throw new IndexOutOfBoundsException();
} }
try (PrintWriter writer = new PrintWriter(file)) { try (PrintWriter writer = new PrintWriter(file)) {
@ -121,22 +118,20 @@ public class MapsCollection {
writer.println(artillery.getInfo()); writer.println(artillery.getInfo());
} }
} }
return true;
} }
public boolean loadMap(String filename) throws IOException { public void loadMap(String filename) throws IOException {
File file = new File(filename); File file = new File(filename);
if (!file.exists()) { if (!file.exists()) {
return false; throw new FileNotFoundException("Файл не найден");
} }
try (BufferedReader reader = new BufferedReader(new FileReader(file))) { try (BufferedReader reader = new BufferedReader(new FileReader(file))) {
String currentLine = reader.readLine(); String currentLine = reader.readLine();
if (currentLine == null || !currentLine.contains("Map")) { if (currentLine == null || !currentLine.contains("Map")) {
return false; throw new FileFormatException("Неверный формат файла");
} }
String mapName = reader.readLine(); String mapName = reader.readLine();
@ -145,7 +140,7 @@ public class MapsCollection {
if (_mapsStorage.containsKey(mapName)) { if (_mapsStorage.containsKey(mapName)) {
map = _mapsStorage.get(mapName); map = _mapsStorage.get(mapName);
if (!map.getMap().getClass().getSimpleName().equals(reader.readLine())) { if (!map.getMap().getClass().getSimpleName().equals(reader.readLine())) {
return false; throw new FileFormatException("Неверный формат файла");
} }
map._setArtilleries.clear(); map._setArtilleries.clear();
} else { } else {
@ -162,7 +157,5 @@ public class MapsCollection {
_mapsStorage.put(mapName, map); _mapsStorage.put(mapName, map);
} }
return true;
} }
} }

3
Program.java
View File

@ -1,8 +1,9 @@
import javax.swing.*; import javax.swing.*;
import org.apache.logging.log4j.*;
public class Program { public class Program {
public static void main(String[] args) { public static void main(String[] args) {
FormMapWithSetArtilleries form = new FormMapWithSetArtilleries(); FormMapWithSetArtilleries form = new FormMapWithSetArtilleries(LogManager.getLogger(Program.class));
form.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE); form.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
} }
} }

25
log4j2.xml Normal file
View File

@ -0,0 +1,25 @@
<?xml version="1.0" encoding="UTF-8"?>
<Configuration status="info" >
<Appenders>
<File name="error" fileName="error.log" append="true">
<PatternLayout>
<Pattern>%-5level %msg (%d{dd.MM.yyyy})%n</Pattern>
</PatternLayout>
<ThresholdFilter level="WARN" onMatch="ACCEPT" onMismatch="DENY"/>
</File>
<File name="info" fileName="info.log" append="true">
<PatternLayout>
<Pattern>%-5level %msg (%d{dd.MM.yyyy})%n</Pattern>
</PatternLayout>
<ThresholdFilter level="WARN" onMatch="DENY" onMismatch="ACCEPT"/>
</File>
</Appenders>
<Loggers>
<Root level="INFO">
<AppenderRef ref="info" />
<AppenderRef ref="error" />
</Root>
</Loggers>
</Configuration>