Changeset 88

Timestamp:

08/23/07 22:15:34 (6 years ago)

Author:

saua

Message:

add logging to text file

Location:

bbtracker/trunk/src/org/bbtracker/mobile

Files:

• BBTracker.java (modified) (9 diffs)
• FileTrackStore.java (modified) (10 diffs)
• IconManager.java (modified) (1 diff)
• RMSTrackStore.java (modified) (2 diffs)
• TrackManager.java (modified) (4 diffs)
• gui/AboutForm.java (modified) (2 diffs)
• gui/MainCanvas.java (modified) (1 diff)
• gui/OptionsForm.java (modified) (1 diff)
• gui/StatusTile.java (modified) (2 diffs)
• gui/TracksForm.java (modified) (2 diffs)

bbtracker/trunk/src/org/bbtracker/mobile/BBTracker.java

@@ -18 +18 @@
 package org.bbtracker.mobile;
 
+import java.io.OutputStream;
+import java.io.PrintStream;
+import java.util.Date;
 import java.util.Timer;
 
+import javax.microedition.io.Connector;
+import javax.microedition.io.file.FileConnection;
 import javax.microedition.lcdui.Alert;
 import javax.microedition.lcdui.AlertType;
@@ -34 +39 @@
 import org.bbtracker.mobile.TrackStore.TrackStoreException;
 import org.bbtracker.mobile.gui.MainCanvas;
+import org.bbtracker.mobile.gui.OptionsForm;
 import org.bbtracker.mobile.gui.TrackNameForm;
-import org.bbtracker.mobile.gui.OptionsForm;
 import org.bbtracker.mobile.gui.TracksForm;
 
@@ -46 +51 @@
 
         private static BBTracker instance;
+
+        private static PrintStream logStream;
 
         private final TrackManager trackManager;
@@ -80 +87 @@
                         nonFatal(e, "Initializing Location Provider", mainCanvas);
                 }
+
+                initLog();
         }
 
         public void shutdown(final boolean destroy) {
+                log(this, "shutdown " + destroy);
                 if (trackManager != null) {
                         trackManager.shutdown();
@@ -94 +104 @@
                         notifyDestroyed();
                 }
+                if (logStream != null) {
+                        logStream.close();
+                        logStream = null;
+                }
         }
 
@@ -121 +135 @@
 
         public static void nonFatal(final Throwable t, final String action, final Displayable next) {
-                log(t);
+                log(BBTracker.class, t, "non-fatal " + action);
                 final Alert alert = new Alert("Non-fatal Exception", "Non-fatal Exception while " + action + ": " +
                                 t.getMessage(), null, AlertType.WARNING);
@@ -128 +142 @@
 
         public static void fatal(final Throwable t, final String action) {
-                log(t);
+                log(BBTracker.class, t, "fatal " + action);
                 final BBTracker i = getInstance();
                 i.shutdown(false);
@@ -152 +166 @@
         }
 
-        public static void log(final Throwable e) {
+        public static void initLog() {
+                if (logStream != null) {
+                        return;
+                }
+                final String dirName = Preferences.getInstance().getTrackDirectory();
+                final String logUrl = "file:///" + dirName + "debug.txt";
+                try {
+                        final FileConnection fileConnection = (FileConnection) Connector.open(logUrl);
+                        if (!(fileConnection.exists() && fileConnection.canWrite())) {
+                                fileConnection.close();
+                                return;
+                        }
+                        final OutputStream out = fileConnection.openOutputStream();
+                        logStream = new PrintStream(out);
+                } catch (final Throwable e) {
+                        log(BBTracker.class, e, "opening " + logUrl);
+                }
+        }
+
+        public static void setLogActive(final boolean logActive) {
+                if (!logActive && logStream != null) {
+                        logStream.close();
+                        logStream = null;
+                }
+
+                final String dirName = Preferences.getInstance().getTrackDirectory();
+                final String logUrl = "file:///" + dirName + "debug.txt";
+                try {
+                        final FileConnection fileConnection = (FileConnection) Connector.open(logUrl);
+                        if (logActive) {
+                                if (!fileConnection.exists()) {
+                                        fileConnection.create();
+                                }
+                                final OutputStream out = fileConnection.openOutputStream();
+                                logStream = new PrintStream(out);
+                        } else {
+                                if (fileConnection.exists()) {
+                                        fileConnection.delete();
+                                        fileConnection.close();
+                                }
+                        }
+                } catch (final Throwable e) {
+                        log(BBTracker.class, e, "opening " + logUrl);
+                }
+        }
+
+        public static boolean isLogActive() {
+                return logStream != null;
+        }
+
+        public static void log(final Object source, final Throwable e) {
+                log(source, "Exception: " + e.toString());
+                // this is only useful for debugging in the emulator
                 e.printStackTrace();
         }
 
-        public static void log(final String m) {
-                System.err.println(m);
+        public static void log(final Object source, final Throwable e, final String message) {
+                log(source, "Exception <" + message + ">: " + e.toString());
+                // this is only useful for debugging in the emulator
+                e.printStackTrace();
+        }
+
+        public static void log(final Object source, final String m) {
+                final String line = new Date() + ": [" + source + "] " + m;
+                System.err.println(line);
+                if (logStream != null) {
+                        logStream.println(line);
+                }
         }
 
@@ -165 +241 @@
 
         protected void pauseApp() {
+                log(this, "pauseApp");
         }
 
         protected void startApp() throws MIDletStateChangeException {
+                log(this, firstStart ? "first startApp" : "startApp");
                 if (firstStart) {
                         firstStart = false;

bbtracker/trunk/src/org/bbtracker/mobile/FileTrackStore.java

@@ -34 +34 @@
                                         result.addElement(new FileTrackStoreEntry(name, date, fileUrl));
                                 } catch (final IOException e) {
-                                        BBTracker.log("Failed to load info from " + file);
-                                        BBTracker.log(e);
+                                        BBTracker.log(this, e, "loading info from " + file);
                                 } finally {
                                         if (din != null) {
@@ -41 +40 @@
                                                         din.close();
                                                 } catch (final IOException e) {
-                                                        BBTracker.log(e);
+                                                        BBTracker.log(this, e);
                                                 }
                                         }
@@ -50 +49 @@
                         return entries;
                 } catch (final IOException e) {
-                        BBTracker.log(e);
+                        BBTracker.log(this, e, "loading track list");
                         throw new TrackStoreException(e);
                 } finally {
@@ -58 +57 @@
                                 } catch (final IOException e) {
                                         // can't do anything about it
-                                        BBTracker.log(e);
+                                        BBTracker.log(this, e);
                                 }
                         }
@@ -73 +72 @@
                         track.writeToStream(dout);
                 } catch (final IOException e) {
-                        BBTracker.log(e);
+                        BBTracker.log(this, e, "saving track");
                         throw new TrackStoreException(e);
                 } finally {
@@ -81 +80 @@
                                 } catch (final IOException e) {
                                         // can't do anything about it
-                                        BBTracker.log(e);
                                 }
                         }
@@ -89 +87 @@
                                 } catch (final IOException e) {
                                         // can't do anything about it
-                                        BBTracker.log(e);
+                                        BBTracker.log(this, e);
                                 }
                         }
@@ -173 +171 @@
                                 connection.delete();
                         } catch (final IOException e) {
-                                BBTracker.log(e);
+                                BBTracker.log(this, e, "deleting track");
                                 throw new TrackStoreException("Failed to delete track: " + e.getMessage());
                         }
@@ -185 +183 @@
                                 return track;
                         } catch (final IOException e) {
-                                BBTracker.log(e);
+                                BBTracker.log(this, e, "loading track");
                                 throw new TrackStoreException("Failed to load track: " + e.getMessage());
                         } finally {
@@ -193 +191 @@
                                         } catch (final IOException e) {
                                                 // can't do anything about it
-                                                BBTracker.log(e);
+                                                BBTracker.log(this, e);
                                         }
                                 }

bbtracker/trunk/src/org/bbtracker/mobile/IconManager.java

@@ -91 +91 @@
                                 icon = Image.createImage(resourceName);
                         } catch (final IOException e) {
-                                BBTracker.log("Couldn't read \"" + resourceName + "\": " + e.getMessage());
+                                BBTracker.log(this, "Couldn't read \"" + resourceName + "\": " + e.getMessage());
                                 icon = NO_ICON;
                         }

bbtracker/trunk/src/org/bbtracker/mobile/RMSTrackStore.java

@@ -37 +37 @@
                                         dis.close();
                                 } catch (final IOException e) {
-                                        BBTracker.log(e);
+                                        BBTracker.log(this, e);
                                 }
                                 i++;
@@ -55 +55 @@
                         return new TrackStoreEntry[0];
                 } catch (final RecordStoreException e) {
-                        BBTracker.log(e);
+                        BBTracker.log(this, e, "listing tracks");
                         throw new TrackStoreException(e);
                 } finally {

bbtracker/trunk/src/org/bbtracker/mobile/TrackManager.java

@@ -280 +280 @@
                                 break;
                         } catch (final TrackStoreException e) {
-                                BBTracker.log(e);
+                                BBTracker.log(this, e, "saving track");
                                 final String msg = e.getMessage();
                                 error = error == null ? msg : error + "\n" + msg;
@@ -305 +305 @@
                 } catch (final IllegalArgumentException e) {
                         provider.setLocationListener(locationListener, -1, -1, -1);
-                        BBTracker.log(e);
+                        BBTracker.log(this, e);
                 }
         }
@@ -432 +432 @@
                                         state = oldState;
                                 } catch (final LocationException e) {
-                                        BBTracker.log(e);
+                                        BBTracker.log(this, e);
                                         BBTracker.getTimer().schedule(this, DELAY_PER_LEVEL * gpsRecoveryEscalation);
                                         fireStateChanged();
@@ -438 +438 @@
                         }
                 }
-
         }
 }

bbtracker/trunk/src/org/bbtracker/mobile/gui/AboutForm.java

@@ -27 +27 @@
 
 public class AboutForm extends Form implements CommandListener {
+        private final Command backCommand;
+
+        private final Command activeDebugCommand;
+
+        private final Command deactiveDebugCommand;
+
         public AboutForm() {
                 super("About " + BBTracker.getName());
@@ -33 +39 @@
                                 "bbTracker is released under the GNU General Public License v2. See http://www.gnu.org/licenses/."));
                 append("Icons have been taken (and sometimes modified) from the Tango Project (http://tango-project.org/) and the Human Icon Theme (Copyright 2004-2006 Canonical Ltd.). Both projects release their icons under the Creative Commons Attribution-ShareAlike 2.5 license. Any modifications I did on those icons are released under the same license.");
-                addCommand(new Command("Back", Command.BACK, 0));
+                backCommand = new Command("Back", Command.BACK, 0);
+                activeDebugCommand = new Command("Activate Debug Log", Command.SCREEN, 1);
+                deactiveDebugCommand = new Command("Deactivate Debug Log", Command.SCREEN, 1);
+                addCommand(backCommand);
+                updateDebugCommands();
                 setCommandListener(this);
         }
 
+        private void updateDebugCommands() {
+                removeCommand(activeDebugCommand);
+                removeCommand(deactiveDebugCommand);
+                addCommand(BBTracker.isLogActive() ? deactiveDebugCommand : activeDebugCommand);
+        }
+
         public void commandAction(final Command command, final Displayable source) {
-                BBTracker.getInstance().showMainCanvas();
+                if (command == backCommand) {
+                        BBTracker.getInstance().showMainCanvas();
+                } else if (command == deactiveDebugCommand) {
+                        BBTracker.setLogActive(false);
+                        updateDebugCommands();
+                } else if (command == activeDebugCommand) {
+                        BBTracker.setLogActive(true);
+                        updateDebugCommands();
+                }
         }
 

bbtracker/trunk/src/org/bbtracker/mobile/gui/MainCanvas.java

@@ -292 +292 @@
                                 nextDisplayable = this;
                         } else {
-                                BBTracker.log("Unknown command: " + command + " <" + command.getLabel() + "/" + command.getLongLabel() +
-                                                ">");
+                                BBTracker.log(this, "Unknown command: " + command + " <" + command.getLabel() + "/" +
+                                                command.getLongLabel() + ">");
                                 nextDisplayable = this;
                         }

bbtracker/trunk/src/org/bbtracker/mobile/gui/OptionsForm.java

@@ -194 +194 @@
                         } catch (final NumberFormatException e) {
                                 // should not happen
-                                BBTracker.log(e);
+                                BBTracker.log(this, e, "parsing sampleInterval: " + sampleField.getString());
                         }
                         pref.setStartAction(startTypeGroup.getSelectedIndex());
                         pref.setTrackDirectory(directoryField.getString());
+                        BBTracker.initLog();
 
                         for (int i = 0; i < Preferences.EXPORT_FORMATS.length; i++) {

bbtracker/trunk/src/org/bbtracker/mobile/gui/StatusTile.java

@@ -89 +89 @@
                 twoLineLayout = fitsTwoLineLayout(width);
                 if (!twoLineLayout && width < ((MARGIN + latWidth) * 2 + MINIMAL_GAP)) {
-                        BBTracker.log("onResize: Setting Font size to small, because even three lines overlap!");
+                        BBTracker.log(this, "onResize: Setting Font size to small, because even three lines overlap!");
                         setFontSize(Font.SIZE_SMALL);
                 }
@@ -199 +199 @@
                         lineCount = 3;
                         if (!fitsThreeLineLayout(width)) {
-                                BBTracker.log("getPreferredHeight: Setting Font size to small, because even three lines overlap!");
+                                BBTracker
+                                                .log(this, "getPreferredHeight: Setting Font size to small, because even three lines overlap!");
                                 setFontSize(Font.SIZE_SMALL);
                         }

bbtracker/trunk/src/org/bbtracker/mobile/gui/TracksForm.java

@@ -107 +107 @@
                                 loadEntries();
                         } catch (final TrackStoreException e) {
-                                BBTracker.log(e);
+                                BBTracker.log(this, e, "deleting track and listing tracks");
                                 final Alert alert = new Alert("Couldn't delete Track.", "The track " + tse.getName() +
                                                 " couldn't be deleted: " + e.getMessage(), null, AlertType.INFO);
@@ -125 +125 @@
                                 track = tse.loadTrack();
                         } catch (final TrackStoreException e) {
-                                BBTracker.log(e);
+                                BBTracker.log(this, e, "loading track");
                                 final Alert alert = new Alert("Couldn't load Track.", "The track " + tse.getName() +
                                                 " couldn't be loaded: " + e.getMessage(), null, AlertType.INFO);