Местоположение BlackBerry Refresh через GPS или сотовую вышку

Итак, хотя мой первоначальный ответ был верным, опубликованный вами новый код имеет некоторые другие проблемы, поэтому я публикую второй ответ. Было достаточно вещей, которые выглядели неправильно, поэтому я просто переписал ваш handleeGPS класс. Я объясню основные изменения, которые я сделал, одно за другим:

1. Попробуйте использовать соглашения об именах Java. Так нам будет проще вам помочь. Прежде чем вы отправили код в свой класс handleeGPS, я думал, что это переменная, потому что имена в нижнем регистре обычно используются для переменных, а не для классов.

2. Избегайте дублирования кода. В классе handleeGPS было много кода, который нужно было прочитать, но большая его часть была кодом для получения местоположения из веб-службы Google, который вы продублировали в двух местах. Просто создайте метод, содержащий только этот код, и вызовите его дважды.

3. Я переименовал ваш класс handleeGPS в GPSHandler. Я не уверен, было ли handlee ошибкой или это слово на другом языке, которое вы использовали. В любом случае, имя должно начинаться как минимум с заглавной буквы.

4. Избегайте множества static переменных и методов. Иногда действительно должен быть только один из чего-то. Класс обработки GPS, вероятно, является хорошим примером этого, потому что устройство имеет только одну систему GPS. Но чтобы обеспечить соблюдение этой конструкции кода, не помечайте все как static. Просто сделайте класс Singleton, что предполагает создание только одного static переменная-член (_instance) и один статический метод (getInstance()). В моем коде вы получите доступ к классу следующим образом: GPSHandler gps = GPSHandler.getInstance();.

5. Я полагаю, что проверка того, были ли установлены карты BB, на самом деле была задом наперед. Вы просматривали модуль net_rim_bb_lbs, и если он был больше нуля (что означает, что BB Maps установлены), то вы обращались непосредственно к веб-сервису Google. Я думаю, вы хотите, чтобы было наоборот (попробуйте устройство GPS, если установлены карты BB). Кроме того, начиная с 6.0, вам также необходимо проверять наличие net_rim_bb_maps.

6. Прежде чем вы опубликовали обновление, я думал, что ваши методы getLatitude() и getLongitude() на самом деле получают местоположение устройства. Это было плохим предположением с моей стороны. Они просто преобразовывали числа в строки. Таким образом, нет причин делать это в фоновом режиме (с Thread). Вы уже написали свой класс handleeGPS для использования фонового потока, и это хорошо. Достаточно одного фонового потока. Пользовательский интерфейс, использующий информацию о местоположении, также не должен нуждаться в фоне Thread. Я изменил код, чтобы добавить интерфейс GPSListener. Этот интерфейс должен быть реализован вашим кодом пользовательского интерфейса, чтобы получать обновления местоположения. Нет причин продолжать зацикливаться, спрашивая, не равно ли местоположение {0.0, 0.0}. Это неэффективно. С моим кодом вы просто получите уведомление, когда местоположение действительно изменится.

7. Исходный код не был потокобезопасным. Переменные handleeGPS latitude и longitude были установлены в фоновом потоке и доступны в потоке пользовательского интерфейса. Это не безопасно. Два потока не должны одновременно читать и записывать один и тот же фрагмент данных. Изменив код для отправки данных о местоположении в GPSListener, можно избежать этой проблемы.

8. Я раскомментировал код Dialog.alert(), который у вас был внутри вашего класса handleeGPS, который не сработал бы для вас, потому что вам не разрешено выполнять вызовы пользовательского интерфейса из фона. Я окружил эти звонки UiApplication.getUiApplication().invokeLater(), чтобы сделать их безопасными.

Чтобы использовать этот класс где-то в коде вашего пользовательского интерфейса, вы должны сделать это вместо использования Thread для запуска вашего метода refreshCoordinates():

public void fieldChanged(Field field, int context) 
    // this is called when your location refresh button is clicked
    GPSHandler.getInstance().setListener(this); 
    GPSHandler.getInstance().requestLocationUpdates();
    busyDialog.setEscapeEnabled(false);  
    busyDialog.show();
}

...

public void onLocationReceived(Coordinates location) {
    lblLatitude.setText(Double.toString(location.getLatitude()));               
    lblLongitude.setText(Double.toString(location.getLongitude()));               
    busyDialog.cancel();               
}     

Убедитесь, что класс, в который вы поместили этот код (выше), также implements GPSListener, который является интерфейсом, определенным здесь:

public interface GPSListener {
    public void onLocationReceived(Coordinates location);
}

и, наконец, GPSHandler:

public class GPSHandler { 

   private GPSThread _gpsThread;
   private Coordinates _location;
   private boolean _gotLocation;
   private GPSListener _listener;

   /** this class will be a Singleton, as the device only has one GPS system */
   private static GPSHandler _instance;

   /** @return the Singleton instance of the GPSHandler */
   public static GPSHandler getInstance() {
      if (_instance == null) {
         _instance = new GPSHandler();
      }
      return _instance;
   }

   /** not publicly accessible ... use getInstance() */
   private GPSHandler() { 
   }

   /** call this to trigger a new location fix */
   public void requestLocationUpdates() {
       if (_gpsThread == null || !_gpsThread.isAlive()) {
          _gpsThread = new GPSThread();
          _gpsThread.start(); 
       }
   }

   public void setListener(GPSListener listener) {
      // only supports one listener this way
      _listener = listener;   
   }

   private void setLocation(final Coordinates value) {
      _location = value;
      if (value.getLatitude() != 0.0 || value.getLongitude() != 0.0) {
         _gotLocation = true;
         if (_listener != null) {
            // this assumes listeners are UI listeners, and want callbacks on the UI thread:
            UiApplication.getUiApplication().invokeLater(new Runnable() {
               public void run() {
                  _listener.onLocationReceived(value);
               }
            });
         }
      }
   }

   private class GPSThread extends Thread {

      private void getLocationFromGoogle() {
         try { 
            int cellID = GPRSInfo.getCellInfo().getCellId(); 
            int lac = GPRSInfo.getCellInfo().getLAC(); 

            String urlString2 = "http://www.google.com/glm/mmap"; 

            // Open a connection to Google Maps API  
            ConnectionFactory connFact = new ConnectionFactory(); 
            ConnectionDescriptor connDesc; 
            connDesc = connFact.getConnection(urlString2); 

            HttpConnection httpConn2; 
            httpConn2 = (HttpConnection)connDesc.getConnection(); 
            httpConn2.setRequestMethod("POST"); 

            // Write some custom data to Google Maps API  
            OutputStream outputStream2 = httpConn2.openOutputStream();//getOutputStream(); 
            writeDataGoogleMaps(outputStream2, cellID, lac); 

            // Get the response   
            InputStream inputStream2 = httpConn2.openInputStream();//getInputStream(); 
            DataInputStream dataInputStream2 = new DataInputStream(inputStream2); 

            // Interpret the response obtained  
            dataInputStream2.readShort(); 
            dataInputStream2.readByte(); 

            final int code = dataInputStream2.readInt(); 
            UiApplication.getUiApplication().invokeLater(new Runnable() {
               public void run() {
                  Dialog.alert(code + "");   
               }
            });                        

            if (code == 0) { 
               final double latitude = dataInputStream2.readInt() / 1000000D; 
               final double longitude = dataInputStream2.readInt() / 1000000D; 
               setLocation(new Coordinates(latitude, longitude, 0.0f));

               UiApplication.getUiApplication().invokeLater(new Runnable() {
                  public void run() {
                     Dialog.alert(latitude+"-----"+longitude);   
                  }
               });

               dataInputStream2.readInt(); 
               dataInputStream2.readInt(); 
               dataInputStream2.readUTF(); 
            } else { 
               System.out.println("Error obtaining Cell Id "); 
            } 
            outputStream2.close(); 
            inputStream2.close(); 
         } catch (Exception e) { 
            System.out.println("Error: " + e.getMessage()); 
         } 
      }

      private void tryGetLocationFromDevice() {
         _gotLocation = false;
         try {
            Criteria myCriteria = new Criteria(); 
            myCriteria.setCostAllowed(false); 
            LocationProvider myLocationProvider = LocationProvider.getInstance(myCriteria); 

            try { 
               Location myLocation = myLocationProvider.getLocation(300); 
               setLocation(myLocation.getQualifiedCoordinates()); 
            } catch ( InterruptedException iex ) { 
               System.out.println(iex.getMessage());
            } catch ( LocationException lex ) { 
               System.out.println(lex.getMessage());
            } 
         } catch ( LocationException lex ) { 
            System.out.println(lex.getMessage());
         }

         if (!_gotLocation) { 
            getLocationFromGoogle();
         } 
      }

      public void run() { 
         int bbMapsHandle = CodeModuleManager.getModuleHandle("net_rim_bb_lbs");    // OS < 6.0
         int bbMapsHandle60 = CodeModuleManager.getModuleHandle("net_rim_bb_maps"); // OS 6.0+
         if (bbMapsHandle > 0 || bbMapsHandle60 > 0) { 
            tryGetLocationFromDevice();
         } else {
            getLocationFromGoogle();
         }
      } 
   } 

   private void writeDataGoogleMaps(OutputStream out, int cellID, int lac) throws IOException { 
      DataOutputStream dataOutputStream = new DataOutputStream(out); 
      dataOutputStream.writeShort(21); 
      dataOutputStream.writeLong(0); 
      dataOutputStream.writeUTF("en"); 
      dataOutputStream.writeUTF("Android"); 
      dataOutputStream.writeUTF("1.0"); 
      dataOutputStream.writeUTF("Web"); 
      dataOutputStream.writeByte(27); 
      dataOutputStream.writeInt(0); 
      dataOutputStream.writeInt(0); 
      dataOutputStream.writeInt(3); 
      dataOutputStream.writeUTF(""); 

      dataOutputStream.writeInt(cellID); 
      dataOutputStream.writeInt(lac); 

      dataOutputStream.writeInt(0); 
      dataOutputStream.writeInt(0); 
      dataOutputStream.writeInt(0); 
      dataOutputStream.writeInt(0); 
      dataOutputStream.flush(); 
   } 

} 

Есть много кода, который мы не видим (например, getLatitude(), getLongitude(), refreshDetails()). Так что там может быть что-то не так. Кроме того, я не вижу никакого экрана загрузки в опубликованном вами коде, поэтому я не могу сказать, почему он не отображается.

Но вот что-то не так:

        synchronized (Application.getEventLock())
        {
            busyDialog.show();
        }

Если вы читали этот вопрос на форуме BlackBerry, вы увидите, что попытка синхронизировать блокировку события приложения из основного (UI) потока может привести к зависанию вашего приложения. Метод public void fieldChanged(Field field, int context) всегда вызывается в потоке пользовательского интерфейса, поскольку именно поток пользовательского интерфейса отслеживает нажатия кнопок и вызывает обработчики кликов, например fieldChanged().

Вы также можете прочитать документы BlackBerry API для приложений, в которых объясняется, что getEventLock() предназначен для рабочих (также известных как фоновых) потоков, а не для основных (он же пользовательский интерфейс).

Таким образом, нет необходимости использовать специальные методы для получения блокировки событий в коде, который уже выполняется в потоке пользовательского интерфейса. Вместо приведенного выше кода просто сделайте следующее:

            busyDialog.show();

Обе эти техники:

        synchronized (Application.getEventLock())
        {
            busyDialog.show();
        }

or

        UiApplication.getUiApplication().invokeLater(new Runnable() {
            public void run() {
               busyDialog.show();  
            }
        });

способы безопасного вызова методов пользовательского интерфейса из фонового потока. Но вы не должны использовать их в коде, который, как вы знаете, уже работает в потоке пользовательского интерфейса.

Попробуйте исправить это и посмотрите, исчезнет ли ваша проблема.

Редактировать. Кроме того, ваш код проверяет имя пользователя и пароль перед обновлением местоположения. Это действительно то, чего вы хотите? Я не думаю, что это имеет какое-то отношение к вашей проблеме, но обычно я не ожидаю, что мне понадобится имя пользователя или пароль для доступа к службам определения местоположения. Конечно, я не знаю вашего приложения, так что это просто комментарий с моей стороны.

Я думаю, что здесь простой ответ - ошибка:

((longi == "0.0" || lati == "0.0") || (longi.length() == 0 || lati.length()==0));

Вы должны использовать String.equals() вместо оператора ==.

После первого вызова longi и lati имеют значение "0.0". Но == вернет false, потому что он сравнивает ссылки по умолчанию, и они разные, потому что это разные объекты.