2016-08-11 2 views
0

У меня нет сомнений в том, что метод contains() работает правильно, но, видимо, я делаю что-то неправильно. Я работаю над Android-приложением, и я обнаруживаю WiFi и перечисляю Wi-Fi в спискеView. Я пытаюсь отфильтровывать повторяющиеся вхождения одной и той же точки доступа, поэтому я создаю массив для отслеживания дубликатов. Вот мой код:Java ArrayList Содержит() метод не работает

public void getWifi(){ 

    ListView wifiList = (ListView)findViewById(R.id.listView); 

    WifiManager wifiManager = (WifiManager) getSystemService(Context.WIFI_SERVICE); 
    List<ScanResult> apList = wifiManager.getScanResults(); 
    wifiManager.startScan(); 

    List<String> wifees = new ArrayList<>(); 

    //split info into arrays 
    for(int i=0; i < apList.size(); i++){ 

     String[] daList = String.valueOf(apList.get(i)).split(","); 

     String info = ""; 
     List<String> SSIDs = new ArrayList<>(); 

     for (String listItem : daList) { 
      String topic = String.valueOf(listItem.split(":")[0]); 
      String val = listItem.split(":")[1]; 
      Log.d(TAG, topic); 
      if (Objects.equals(topic, "SSID")) { 
       if(SSIDs.contains(val)){ 
        Log.d(TAG, "CONTAINS"); 
        break; 
       } else { 
        SSIDs.add(val); 
        info = val; 
       } 
      } else if (Objects.equals(topic, " level")){ 
       String strength = ""; 
       if (Integer.parseInt(val.substring(1)) >= -35) { 
        strength = "100%"; 
       } else if (Integer.parseInt(val.substring(1)) <= -35 && Integer.parseInt(val.substring(1)) >= -65) { 
        strength = "50%"; 
       } else { 
        strength = "1%"; 
       } 
       info += " " + strength; 
      } 
     } 
     wifees.add(info); 
    } 

    assert wifiList != null; 
    ArrayAdapter apter = new ArrayAdapter(this, android.R.layout.simple_list_item_1, wifees); 
    wifiList.setAdapter(apter); 
} 

Это содержимое элемента apList или то, что я получаю, когда я называю apList.get (я):

SSID: SCHOOL-GUEST, BSSID: 04:04:04:04:04:04, capabilities: [ESS], level: -91, frequency: 2412, timestamp: 72500687089, distance: ?(cm), distanceSd: ?(cm), passpoint: no, ChannelBandwidth: 0, centerFreq0: 0, centerFreq1: 0, 80211mcResponder: is not supported 

Вот результат этого кода (примерно потому что это по телефону, и я предпочел бы не включать имена AP по соображениям безопасности):

SCHOOL-WIRELESS  100% 
SCHOOL-SECURE  50% 
SCHOOL-GUEST  50% 
SCHOOL WIRELESS  1% 
SCHOOL_WIRELESS  1% 
SCHOOL-SECURE  1% 
SCHOOL-GUEST  1% 
bxz1872   50% 
... 
... 
... 
(so on and so forth) 

Если бы я мог просто получить второе мнение, что было бы здорово. Я чувствую, что мне не хватает чего-то очевидного. Благодаря!

+1

Вы действительно используете список SSID? Если вы хотите отфильтровать дубликаты, вам лучше использовать набор, а не список. –

+0

Я действительно объявляю это перед этим методом. извините, не включили это. он также добавляет в список, когда имя точки доступа является новым. Я отредактирую. Я также попробую набор. Но я все еще удивляюсь, почему это не сработает ... –

+0

Можете ли вы представить пример того, что может содержать «apList»? Btw. каков должен быть уровень, если вы удалите существующую запись? самый высокий? – JohnnyAW

ответ

2

.contains не работает, потому что вы звоните List<String> SSIDs = new ArrayList<>(); в пределах for(int i=0; i < apList.size(); i++){. Вы в основном создаете новый пустой список для каждой итерации цикла. Вы должны называть его один раз перед внешним контуром:

List<String> SSIDs = new ArrayList<>(); 
for(int i=0; i < apList.size(); i++){ 
    ... 
} 
+0

Спасибо! Я знал, что мне не хватает чего-то очевидного. Это поставило проблему. –

+0

приветствую :) – JohnnyAW

Смежные вопросы