OSDN Git Service

Removed some unused comments and some System.out.println calls.
authorSylvain Vedrenne <svedrenne@users.sourceforge.jp>
Wed, 15 Feb 2012 23:24:45 +0000 (00:24 +0100)
committerSylvain Vedrenne <svedrenne@users.sourceforge.jp>
Wed, 15 Feb 2012 23:24:45 +0000 (00:24 +0100)
README
src/classes/net/jankenpoi/i18n/I18n.java
src/classes/net/jankenpoi/sudokuki/controller/GridController.java
src/classes/net/jankenpoi/sudokuki/generator/suexg/SuexgJava.java
src/classes/net/jankenpoi/sudokuki/preferences/UserPreferences.java
src/classes/net/jankenpoi/sudokuki/ui/swing/ActionsRepository.java
src/classes/net/jankenpoi/sudokuki/ui/swing/CheckUpdateDialog.java
src/classes/net/jankenpoi/sudokuki/ui/swing/CustomGridAction.java
src/classes/net/jankenpoi/sudokuki/ui/swing/OpenGridAction.java
src/classes/net/jankenpoi/sudokuki/ui/swing/SwingGrid.java
src/classes/net/jankenpoi/sudokuki/ui/swing/SwingView.java

diff --git a/README b/README
index 6b9fb0c..e730961 100644 (file)
--- a/README
+++ b/README
@@ -2,9 +2,10 @@ Sudokuki - essential sudoku game
 
 -- Changelog --
 
-
-- Esperanto translations for Sudokuki 1.1 (from Sylvain Vedrenne)
-- German translations for Sudokuki 1.1 from Claus Poerschke
+- Fixed issue where "ENTER" doesn't close language dialog, and minor sync issue 
+- Portuguese and russian translations for Sudokuki 1.1 (by Dmitry Salnikov) 
+- Esperanto translations for Sudokuki 1.1 (by Sylvain Vedrenne)
+- German translations for Sudokuki 1.1 ( byClaus Poerschke)
 
 * 2012-02-11 - released Sudokuki 1.1.Beta11
 - Fixed #3485100 Cancelling 4 grids generation doesn't have expected effect
index b643bf4..7a51393 100644 (file)
@@ -53,7 +53,6 @@ public class I18n {
                                        catalog = null;\r
                                }\r
                        }\r
-                       System.out.println("I18n using locale: " + currentLocale);\r
                }\r
        }\r
 \r
index 8bcea09..ce09e96 100644 (file)
@@ -81,7 +81,6 @@ public class GridController {
        }
        
        public void notifyEnterCustomGridMode() {
-//             System.out.println("GridController.notifyEnterCustomGridMode()");
                model.enterCustomGridMode();
        }
        
@@ -127,8 +126,6 @@ public class GridController {
        private int lastCO = 4;
 
        public void notifyFocusPositionChanged(int li, int co) {
-               System.out.println("GridController.notifyGridPositionChanged() li:"
-                               + li + " co:" + co);
                lastLI = li;
                lastCO = co;
        }
index e4486a8..39c9312 100644 (file)
@@ -147,8 +147,6 @@ class SuexgJava extends SuexgGenerator {
        gridGenerate(final int seed, final int requestedRatingMin, final int requestedRatingMax, final int[] grid, final int[] rating,\r
                        final int[] grid_with_clues) {\r
 \r
-               // System.out.println("SuexgJava.grid_generate()");\r
-\r
                /*\r
                 * boolean values used to simulate the infamous goto used in the\r
                 * original C suexg algorithm\r
@@ -177,19 +175,11 @@ class SuexgJava extends SuexgGenerator {
                        for (y = 1; y <= 9; y++) {\r
                                for (s = 1; s <= 9; s++) {\r
                                        r++;\r
-                                       // System.out.println("r : " + r);\r
                                        Cols[r] = 4;\r
                                        Col[r][1] = x * 9 - 9 + y;\r
-                                       // System.out.println("Col[" + r + "][1] : " + Col[r][1]);\r
                                        Col[r][2] = (B[x * 9 - 9 + y] - 48) * 9 - 9 + s + 81;\r
-                                       // System.out.println("(B[" + (x * 9 - 9 + y) + "]-48)*9-9+"\r
-                                       // + s + "+81");\r
-                                       // System.out.println("Col[" + r + "][2] : " + Col[r][2]);\r
                                        Col[r][3] = x * 9 - 9 + s + 81 * 2;\r
-                                       // System.out.println("Col[" + r + "][3] : " + Col[r][3]);\r
                                        Col[r][4] = y * 9 - 9 + s + 81 * 3;\r
-                                       // System.out.println("Col[" + r + "][4] : " + Col[r][4]);\r
-\r
                                }\r
                        }\r
                }\r
@@ -197,10 +187,8 @@ class SuexgJava extends SuexgGenerator {
                        Rows[c] = 0;\r
                }\r
                for (r = 1; r <= n; r++) {\r
-                       // System.out.println("r : " + r);\r
                        for (c = 1; c <= Cols[r]; c++) {\r
                                a = Col[r][c];\r
-                               // System.out.println("a:" + a);\r
                                Rows[a]++;\r
                                Row[a][Rows[a]] = r;\r
                        }\r
@@ -211,11 +199,9 @@ class SuexgJava extends SuexgGenerator {
                while (true) {\r
                        // m0s:\r
                        if (gotoM0S) {\r
-                               // System.out.println("m0s");\r
                                gotoM0S = false;\r
                                sam1++;\r
                                if (sam1 > samples) {\r
-                                       // printf(".\n");\r
                                        return 0;\r
                                } else {\r
                                        gotoM0 = true;\r
@@ -224,7 +210,6 @@ class SuexgJava extends SuexgGenerator {
 \r
                        // m0:\r
                        if (gotoM0) {\r
-                               // System.out.println("m0");\r
                                gotoM0 = false;\r
                                for (i = 1; i <= 81; i++) {\r
                                        A[i] = 0;\r
@@ -234,7 +219,6 @@ class SuexgJava extends SuexgGenerator {
 \r
                        // mr1:\r
                        if (gotoMR1) {\r
-                               // System.out.println("mr1");\r
                                gotoMR1 = false;\r
                                i1 = (int) (MWC() >> 8) & 127;\r
                                if (i1 > 80) {\r
@@ -251,7 +235,6 @@ class SuexgJava extends SuexgGenerator {
 \r
                        // mr3:\r
                        if (gotoMR3) {\r
-                               // System.out.println("mr3");\r
                                gotoMR3 = false;\r
                                s = (int) (MWC() >> 9) & 15;\r
                                if (s > 8) {\r
@@ -260,8 +243,6 @@ class SuexgJava extends SuexgGenerator {
                                }\r
                                s++;\r
                                A[i1] = s;\r
-                               // printf("\nA[i1:%d] = s:%d\n", i1, s);\r
-                               // System.out.println("A[i1:"+i1+"] = s:"+s);\r
                                m2 = solve();\r
 \r
                                // add a random clue and solve it. No solution ==> remove it\r
@@ -282,7 +263,6 @@ class SuexgJava extends SuexgGenerator {
                                // now we have a unique-solution sudoku. Now remove clues to\r
                                // make it minimal\r
                                {// EXPERIMENTAL: here is the grid with clues in it\r
-                                       // System.out.println("SuexgJava.grid_generate() EXPERIMENTAL...");\r
                                        for (i = 1; i <= 81; i++) {\r
                                                grid_with_clues[i - 1] = A[i];\r
                                        }\r
@@ -291,7 +271,6 @@ class SuexgJava extends SuexgGenerator {
 \r
                                        x = i;\r
                                        while (x >= i) {\r
-                                               // System.out.println("mr4");\r
                                                // mr4:\r
                                                x = (int) (MWC() >> 8) & 127;\r
                                        }\r
@@ -313,7 +292,6 @@ class SuexgJava extends SuexgGenerator {
                                                solve();\r
                                                nt += nodes;\r
                                        }\r
-                                       // printf ( "new grid, rating:%6i", nt );\r
                                        \r
                                        if (nt < requestedRatingMin|| requestedRatingMax < nt) {\r
                                                gotoM0 = true;\r
@@ -342,7 +320,6 @@ class SuexgJava extends SuexgGenerator {
                                                                // (more\r
                // than one sol.)\r
 \r
-               // System.out.println("SuexgJava.solve()");\r
                /*\r
                 * boolean values used to simulate the infamous goto used in the\r
                 * original C suexg algorithm\r
@@ -358,7 +335,6 @@ class SuexgJava extends SuexgGenerator {
                clues = 0;\r
                for (i = 1; i <= 81; i++) {\r
                        if (A[i] != 0) {\r
-                               // System.out.println("clues:"+clues);\r
                                clues++;\r
                                r = i * 9 - 9 + A[i];\r
                                for (j = 1; j <= Cols[r]; j++) {\r
@@ -393,23 +369,18 @@ class SuexgJava extends SuexgGenerator {
 \r
                        // m2: //////////\r
                        if (gotoM2) {\r
-                               // System.out.println("M2 ");\r
                                gotoM2 = false;\r
 \r
                                i++;\r
                                I[i] = 0;\r
                                min = n + 1;\r
 \r
-                               // System.out.println("i:"+i+" m0:"+m0+" ");\r
                                if (i > 81 || m0 != 0) {\r
-                                       // System.out.println("SuexgJava.solve() ICI(1) ");\r
                                        gotoM4 = true;\r
                                        continue; // simulates: goto m4;\r
                                }\r
-                               // System.out.println("m1:"+m1+" ");\r
                                if (m1 != 0) {\r
                                        C[i] = m1;\r
-                                       // System.out.println("gotoM3 ");\r
                                        gotoM3 = true;\r
                                        continue; // simulates: goto m3;\r
                                }\r
@@ -419,7 +390,6 @@ class SuexgJava extends SuexgGenerator {
                                        if (Uc[c] == 0) {\r
                                                if (V[c] < 2) {\r
                                                        C[i] = c;\r
-                                                       // System.out.println("gotoM3 ");\r
                                                        gotoM3 = true;\r
                                                        continue whileloop; // simulates: goto m3;\r
                                                }\r
@@ -440,37 +410,30 @@ class SuexgJava extends SuexgGenerator {
 \r
                        // /mr:\r
                        if (gotoMR) {\r
-                               // System.out.println("MR ");\r
                                gotoMR = false;\r
 \r
                                c2 = (int) (MWC() & Two[w]);\r
                                if (c2 >= w) {\r
                                        gotoMR = true;\r
-                                       // System.out.println("gotoMR ");\r
                                        continue; // simulates: goto mr;\r
                                }\r
                                C[i] = W[c2 + 1];\r
                                gotoM3 = true;\r
-                               // System.out.println("gotoM3 ");\r
                        }\r
 \r
                        // m3: //////\r
                        if (gotoM3) {\r
-                               // System.out.println("M3 ");\r
                                gotoM3 = false;\r
 \r
                                c = C[i];\r
                                I[i]++;\r
                                if (I[i] > Rows[c]) {\r
-                                       // System.out.println("SuexgJava.solve() ICI(2) ");\r
                                        gotoM4 = true;\r
-                                       // System.out.println("gotoM4 ");\r
                                        continue; // simulates: goto m4;\r
                                }\r
                                r = Row[c][I[i]];\r
                                if (Ur[r] != 0) {\r
                                        gotoM3 = true;\r
-                                       // System.out.println("gotoM3 ");\r
                                        continue; // simulates: goto m3;\r
                                }\r
                                m0 = 0;\r
@@ -506,13 +469,11 @@ class SuexgJava extends SuexgGenerator {
                                        break; // simulates: goto m9;\r
                                }\r
                                gotoM2 = true;\r
-                               // System.out.println("gotoM2 ");\r
                                continue; // simulates: goto m2;\r
                        } // end of m3 if\r
 \r
                        // m4: ////\r
                        if (gotoM4) {\r
-                               // System.out.println("M4 ");\r
                                gotoM4 = false;\r
 \r
                                i--;\r
@@ -538,7 +499,6 @@ class SuexgJava extends SuexgGenerator {
 \r
                                if (i > clues) {\r
                                        gotoM3 = true;\r
-                                       // System.out.println("gotoM3 ");\r
                                        continue; // simulates: goto m3;\r
                                }\r
                        } // end of m4 if\r
@@ -546,8 +506,6 @@ class SuexgJava extends SuexgGenerator {
                        break;\r
                } // outer while loop\r
 \r
-               // System.out.println();\r
-               // System.out.println("solve=>"+solutions);\r
                // m9: /////\r
                return solutions;\r
        }\r
index 7671053..71736dd 100644 (file)
@@ -23,7 +23,6 @@ public class UserPreferences {
        \r
        public final Integer getInteger(String name, Integer defaultInteger) {\r
                Object obj = map.get(name);\r
-               System.out.println("UserPreferences.getInteger("+name+") found obj:"+obj);\r
                if (!(obj instanceof Integer)) {\r
                        return defaultInteger;\r
                }\r
@@ -33,7 +32,6 @@ public class UserPreferences {
        \r
        public final Boolean getBoolean(String name, Boolean defaultBoolean) {\r
                Object obj = map.get(name);\r
-               System.out.println("UserPreferences.getBoolean("+name+") found obj:"+obj);\r
                if (!(obj instanceof Boolean)) {\r
                        return defaultBoolean;\r
                }\r
index 4b8d018..6e1b421 100644 (file)
@@ -9,9 +9,7 @@ public class ActionsRepository {
        private final HashMap<String, Action> map = new HashMap<String, Action>();
        
        final Action get(String name) {
-               System.out.println("ActionsRepository.get() name:"+name);
                Action action = map.get(name);
-               //System.out.println("ActionsRepository.get() found action:"+action);
                return action;
        }
        
index 6cd63e8..a345254 100644 (file)
@@ -73,20 +73,9 @@ public class CheckUpdateDialog extends JDialog {
                        try {
                                String httpVersionString = get();
 
-                               System.out
-                                               .println("SwingApp.getHttpLatestVersionString() (embedded) Version.versionString:"
-                                                               + Version.versionString);
-                               System.out
-                                               .println("SwingApp.getHttpLatestVersionString()  retrieved         versionString:"
-                                                               + httpVersionString);
-
                                if (httpVersionString.equals(Version.versionString)) {
-                                       System.out
-                                                       .println("SwingApp.getHttpLatestVersionString() This version is up-to-date");
                                        result = 0;
                                } else if (httpVersionString.startsWith("Sudokuki")) {
-                                       System.out
-                                                       .println("SwingApp.getHttpLatestVersionString() This version is outdated. Please download the latest version from Sourceforge.");
                                        checkUpdateAction.notifyNewVersionFound();
                                        result = 1;
                                }
@@ -133,7 +122,6 @@ public class CheckUpdateDialog extends JDialog {
                                        e.printStackTrace();
                                }
                        }
-                       System.out.println("getHttpLatestVersionString() line:" + line);
                        String versionString = "";
                        if (line != null) {
                                String[] strs = line.split(" is the latest version.");
@@ -192,10 +180,7 @@ public class CheckUpdateDialog extends JDialog {
        }
 
        private void cancelButtonClicked() {
-               System.out.println("CheckUpdateDialog.cancelButtonClicked()");
                boolean cancelled = worker.cancel(true);
-               System.out.println("CheckUpdateDialog.cancelButtonClicked() cancelled:"
-                               + cancelled);
        }
 
        /**
index 41f93ae..ee73a54 100644 (file)
@@ -33,7 +33,6 @@ public class CustomGridAction extends AbstractAction {
        public CustomGridAction(JFrame parent, String text, Icon icon,
                        String desc, Integer mnemonic, GridView view) {
                super(text, icon);
-               System.out.println("CustomGridAction.CustomGridAction()");
                
                putValue(SHORT_DESCRIPTION, desc);
                putValue(MNEMONIC_KEY, mnemonic);
@@ -43,7 +42,6 @@ public class CustomGridAction extends AbstractAction {
 
        @Override
        public void actionPerformed(ActionEvent e) {
-               System.out.println("CustomGridAction.actionPerformed()");
                view.getController().notifyEnterCustomGridMode();
        }
 
index a91f155..05f7e2f 100644 (file)
@@ -74,8 +74,6 @@ public class OpenGridAction extends AbstractAction {
                        @Override
                        public boolean accept(File f) {
                                String extension = getExtension(f);
-                               System.out
-                                               .println("OpenGridAction ext:"+extension+"|");
                                if (f.isDirectory() || "skg".equals(extension)) {
                                        return true;
                                }
@@ -112,11 +110,9 @@ public class OpenGridAction extends AbstractAction {
                        try {
                                int lo = fis.read();
                                int hi = fis.read();
-//                             System.out.println("OpenGridAction.actionPerformed() hi:"+hi+" lo:"+lo);
                                short together = (short) (hi << 8 | lo);
 
                                externalCellInfos[i] = (short) together;
-//                             System.out.println("OpenGridAction.actionPerformed() externalCellInfos["+i+"]:"+externalCellInfos[i]);
                        } catch (IOException e1) {
                                // TODO Auto-generated catch block
                                e1.printStackTrace();
index 01d2b14..7dadb64 100644 (file)
@@ -546,7 +546,6 @@ public class SwingGrid extends JPanel implements Printable {
        private class InnerKeyListener extends KeyAdapter {
                @Override
                public void keyPressed(KeyEvent ke) {
-                       System.out.println("SwingGrid.InnerKeyAdapter.keyPressed() ke:"+ke);
                        int code = ke.getKeyCode();
                        boolean hasMoved = false;
                        if (code == KeyEvent.VK_KP_DOWN || code == KeyEvent.VK_DOWN || code == KeyEvent.VK_J) {
@@ -578,16 +577,12 @@ public class SwingGrid extends JPanel implements Printable {
                                }
                        }
                        if (hasMoved) {
-                               System.out
-                               .println("SwingGrid.InnerKeyListener.keyPressed() has moved...");
                                view.getController().notifyFocusPositionChanged(posY, posX);
                        }
                }
 
                @Override
                public void keyReleased(KeyEvent ke) {
-                       System.out.println("SwingGrid.InnerKeyAdapter.keyReleased() ke:"
-                                       + ke);
                        int code = ke.getKeyCode();
                        if (code == KeyEvent.VK_SPACE) {
                                Point pos = getTopLeftPoint(posY, posX);
index f98d0ab..d40e3aa 100644 (file)
@@ -77,14 +77,12 @@ public class SwingView extends GridView {
 
        @Override
        public void display() {
-               System.out.println("SwingView.display()");
                javax.swing.SwingUtilities.invokeLater(new Runnable() {
                        public void run() {
                                frame.setVisible(true);
                                frame.setSize(frame.getPreferredSize());
                        }
                });
-               System.out.println("SwingView.display()/");
        }
 
        public void gridChanged(final GridChangedEvent event) {
@@ -158,7 +156,6 @@ public class SwingView extends GridView {
 
        private void refreshClearAllMovesAction(GridModel model) {
                if (actions == null) {
-                       System.out.println("SwingView.refreshClearAllMovesAction() actions is null...");
                        return;
                }
                Action clearAllMovesAction = actions.get("ClearAllMoves"); 
@@ -175,7 +172,6 @@ public class SwingView extends GridView {
 
        protected void refreshResolveAction(GridModel model) {
                if (actions == null) {
-                       System.out.println("SwingView.refreshResolveAction() actions is null...");
                        return;
                }
                Action resolveAction = actions.get("ResolveGrid"); 
@@ -184,7 +180,6 @@ public class SwingView extends GridView {
        
        private void refreshCustomGridAction(GridModel model) {
                if (actions == null) {
-                       System.out.println("SwingView.refreshCustomGridAction() actions is null...");
                        return;
                }
                Action customGridAction = actions.get("CustomGrid"); 
@@ -193,7 +188,6 @@ public class SwingView extends GridView {
 
        private void refreshPlayCustomGridAction(GridModel model) {
                if (actions == null) {
-                       System.out.println("SwingView.refreshPlayCustomGridAction() actions is null...");
                        return;
                }
                Action playCustomGridAction = actions.get("PlayCustomGrid");