From 7e1868bbb8fcfdafed04b928f1179fedce62a8ec Mon Sep 17 00:00:00 2001 From: Chris Date: Sat, 18 Feb 2012 21:51:58 +0000 Subject: [PATCH] - CheckStyle. --- src/main/java/forge/deck/OldDeckParser.java | 94 ++++++++++----------- 1 file changed, 47 insertions(+), 47 deletions(-) diff --git a/src/main/java/forge/deck/OldDeckParser.java b/src/main/java/forge/deck/OldDeckParser.java index 61ecfba86d6..a4d7a51de6c 100644 --- a/src/main/java/forge/deck/OldDeckParser.java +++ b/src/main/java/forge/deck/OldDeckParser.java @@ -33,8 +33,8 @@ public class OldDeckParser { public boolean accept(final File dir, final String name) { return name.endsWith(".bdk"); } - }; - + }; + /** * TODO: Write javadoc for Constructor. * @param file @@ -51,7 +51,7 @@ public class OldDeckParser { cube = cube2; draft = draft2; } - + protected final IFolderMap getSealed() { return sealed; } @@ -80,17 +80,15 @@ public class OldDeckParser { convertConstructedAndSealed(); convertDrafts(); } - - private void convertDrafts() - { - for( File f : deckDir.listFiles(bdkFileFilter)) - { + + private void convertDrafts() { + for (File f : deckDir.listFiles(bdkFileFilter)) { boolean gotError = false; Deck human = Deck.fromFile(new File(f, "0.dck")); final DeckSet d = new DeckSet(human.getName()); d.setHumanDeck(human); - - + + for (int i = 1; i < DeckSetSerializer.MAX_DRAFT_PLAYERS; i++) { Deck nextAi = Deck.fromFile(new File(f, i + ".dck")); if (nextAi == null) { @@ -99,7 +97,7 @@ public class OldDeckParser { } d.addAiDeck(nextAi); } - + boolean mayDelete = !gotError; if (!gotError) { draft.add(d); @@ -107,93 +105,95 @@ public class OldDeckParser { String msg = String.format("Draft '%s' lacked some decks.%n%nShould it be deleted?"); mayDelete = JOptionPane.YES_OPTION == JOptionPane.showConfirmDialog(null, msg, "Draft loading error", JOptionPane.YES_NO_OPTION); } - - if( mayDelete ) { - for( File f1 : f.listFiles() ) + + if (mayDelete) { + for (File f1 : f.listFiles()) { f1.delete(); + } f.delete(); } } } - - private void convertConstructedAndSealed() - { + + private void convertConstructedAndSealed() { boolean allowDeleteUnsupportedConstructed = false; - Map>> sealedDecks = new TreeMap>>(String.CASE_INSENSITIVE_ORDER); - - for( File f : deckDir.listFiles(DeckSerializer.DCK_FILE_FILTER)) - { + Map>> sealedDecks = new TreeMap>>(String.CASE_INSENSITIVE_ORDER); + + for (File f : deckDir.listFiles(DeckSerializer.DCK_FILE_FILTER)) { boolean importedOk = false; - + List fileLines = FileUtil.readFile(f); Map> sections = SectionUtil.parseSections(fileLines); DeckFileHeader dh = DeckSerializer.readDeckMetadata(sections); String name = dh.getName(); - if ( dh.isCustomPool() ) { continue; } - + if (dh.isCustomPool()) { + continue; + } + switch(dh.getDeckType()) { case Constructed: try { constructed.add(Deck.fromLines(fileLines)); importedOk = true; - } catch( NoSuchElementException ex) { - if( !allowDeleteUnsupportedConstructed ) { + } catch (NoSuchElementException ex) { + if (!allowDeleteUnsupportedConstructed) { String msg = String.format("Can not convert deck '%s' for some unsupported cards it contains. %n%s%n%nMay Forge delete all such decks?", name, ex.getMessage()); allowDeleteUnsupportedConstructed = JOptionPane.YES_OPTION == JOptionPane.showConfirmDialog(null, msg, "Problem converting decks", JOptionPane.YES_NO_OPTION); } } - if( importedOk || allowDeleteUnsupportedConstructed ) + if (importedOk || allowDeleteUnsupportedConstructed) { f.delete(); + } break; - + case Sealed: boolean isAi = dh.getPlayerType() == PlayerType.COMPUTER; name = name.startsWith("AI_") ? name.replace("AI_", "") : name; - - Pair> stored = sealedDecks.get(name); - if( null == stored ) - stored = ImmutablePair.of(new DeckSet(name), MutablePair.of((File)null, (File)null)); - + + Pair> stored = sealedDecks.get(name); + if (null == stored) { + stored = ImmutablePair.of(new DeckSet(name), MutablePair.of((File) null, (File) null)); + } + Deck deck = Deck.fromLines(fileLines); - if( isAi ) { + if (isAi) { stored.getLeft().addAiDeck(deck); stored.getRight().setRight(f); } else { stored.getLeft().setHumanDeck(deck); stored.getRight().setLeft(f); } - - if( stored.getLeft().getHumanDeck() != null && !stored.getLeft().getAiDecks().isEmpty() ) { + + if (stored.getLeft().getHumanDeck() != null && !stored.getLeft().getAiDecks().isEmpty()) { // have both parts of sealed deck, may convert sealed.add(stored.getLeft()); stored.getRight().getLeft().delete(); stored.getRight().getRight().delete(); - + // there stay only orphans sealedDecks.remove(name); - } else + } else { sealedDecks.put(name, stored); + } break; } } - + // advise to kill orphaned decks - if( !sealedDecks.isEmpty() ) - { + if (!sealedDecks.isEmpty()) { StringBuilder sb = new StringBuilder(); - for(Pair> s : sealedDecks.values()) { + for (Pair> s : sealedDecks.values()) { String missingPart = s.getRight().getLeft() == null ? "human" : "computer"; sb.append(String.format("Sealed deck '%s' has no matching '%s' deck.%n", s.getKey().getName(), missingPart)); } sb.append(System.getProperty("line.separator")); sb.append("May Forge delete these decks?"); int response = JOptionPane.showConfirmDialog(null, sb.toString(), "Some of your sealed decks are orphaned", JOptionPane.YES_NO_OPTION); - if( response == JOptionPane.YES_OPTION ) - { - for(Pair> s : sealedDecks.values()) { - if( s.getRight().getLeft() != null ) { s.getRight().getLeft().delete(); } - if( s.getRight().getRight() != null ) { s.getRight().getRight().delete(); } + if (response == JOptionPane.YES_OPTION) { + for (Pair> s : sealedDecks.values()) { + if (s.getRight().getLeft() != null) { s.getRight().getLeft().delete(); } + if (s.getRight().getRight() != null) { s.getRight().getRight().delete(); } } } }