Android Kontakte auslesen und in Liste darstellen

[tobi193]

Hallo,

ich möchte in einer Android App alle Kontakte auslesen und einer Liste (als Alert) darstellen.
Das klappt mit nachfolgendem Code auch soweit. Allerdings ist der absolut ineffizient. Er lädt da selbst bei einem Top Gerät eine halbe Ewigkeit für die (wenigen) Kontakte.
Außerdem werden mir alle mehrfach angezeigt.

Auch die Darstellung mit einem Umweg über eine LinkedList ist ziemlich suboptimal. Aber ich habe absolut keine Idee wie man es anders lösen könnte. Weiß da jemand Rat ?

 public void contactPicker(View view) {  
    if(list.isEmpty()){
        Cursor rawContacts = managedQuery(RawContacts.CONTENT_URI, projection, null, null, null);

        if(rawContacts.moveToFirst()){
            while(!rawContacts.isAfterLast()){		
                int contactId = rawContacts.getInt(rawContacts.getColumnIndex(RawContacts.CONTACT_ID));
                Cursor contact = managedQuery(Contacts.CONTENT_URI, contact_value, Contacts._ID + "=?", new String[]{String.valueOf(contactId)}, null);
                if(contact.moveToFirst()){
                    Cursor phone = managedQuery(Phone.CONTENT_URI, contact_phone, Data.CONTACT_ID + "=?",new String[]{String.valueOf(contactId)}, null);
                    if(phone.moveToFirst()) {

                        while(!phone.isAfterLast()){
                            list.add(new Kontakt(contact.getString(contact.getColumnIndex(Contacts.DISPLAY_NAME)), phone.getString(phone.getColumnIndex(Phone.NUMBER))));
                            //message(contact.getString(contact.getColumnIndex(Contacts.DISPLAY_NAME)) + " " + phone.getString(phone.getColumnIndex(Phone.NUMBER)));
                            phone.moveToNext();
                        }
                    }
                    phone.close();               
                    contact.close();
                }
                rawContacts.moveToNext();			
            }
        }

        CharSequence[] cs = new CharSequence[list.size()];
        int i = 0;
        for(Kontakt k : list){
            cs[i] = k.getName();
            i++;
        }
        builder = new AlertDialog.Builder(this);
        builder.setTitle("Kontakte");
        builder.setCancelable(true);
        builder.setItems(cs, new DialogInterface.OnClickListener() {
            public void onClick(DialogInterface dialog, int item) {
                setContact(list.get(item));
            }
        });
        alert = builder.create();
    }
    alert.show();
  }

 

[diel2001]

Verwende einfach ein Set !
Wenn die equals methode von Kontakt überschrieben ist, dann dürftest du keine doppelten Einträge haben.

 

[nillehammer]

Ich kenn mich mit der Android-API nicht aus, aber die vielen Cursor-Objekte in den vielen verschachtelten Schleifen sehen "smelly" aus. Versuch mal, ob Du die Daten nicht direkt aus einem der Returnvalues (z.B. rawContact, contact etc.) herausholen kannst, ohne wieder eine Query mit der ID zu machen.

Ansonsten zum Profiling baue jeweils als letzte Anweisung jeder Schleife ein Log-Statement mit der aktuellen Zeit ein, um zu sehen, wo die Zeit verbraten wird.

 

[mjdv]

Warum ließt du denn alle Raw Kontakte ein? Das macht doch keinen Sinn, lies nur die normalen Kontakte ein, bei denen die Informationen schon zusammengefasst sind.

Des weiteren ließt du jede Nummer ein, aber zeigst nur die Namen an. Ließ die Nummern nur ein wenn du sie auch brauchst. Also erst wenn du den Kontakt mit seinen Daten auch anzeigst.

Dann hast du nur einen Cursor durch den du durch gehst und der Spaß dauert nur ein paar ms.

 

[tobi193]

So, ich habe es jetzt etwas optimiert bekommen.
Die Ladezeit hat sich deutlich verkürzt und es werden alle Kontakte auch korrekt und nur einmal angezeigt. Außerdem sind sie so auch in alphabetischer Reihenfolge sortiert. Nur die unschöne Sache mit der Übertragung der LinkedList in ein Array ist noch vorhanden. Hier mein Quellcode dazu:

if(list.isEmpty()){
        cr = getContentResolver();
        Cursor cur = cr.query(ContactsContract.Contacts.CONTENT_URI, 
                              new String[]{ContactsContract.Contacts._ID, ContactsContract.Contacts.DISPLAY_NAME, ContactsContract.Contacts.HAS_PHONE_NUMBER},  // Diese Spalten sollen ausgelesen werden
                              ContactsContract.Contacts.IN_VISIBLE_GROUP + " = 1", 
                              null, 
                              ContactsContract.Contacts.DISPLAY_NAME);
        
        if(cur.getCount() > 0){
	    while(cur.moveToNext()) {
	        String id = cur.getString(cur.getColumnIndex(ContactsContract.Contacts._ID));
		String name = cur.getString(cur.getColumnIndex(ContactsContract.Contacts.DISPLAY_NAME));
 		
                if(Integer.parseInt(cur.getString(cur.getColumnIndex(ContactsContract.Contacts.HAS_PHONE_NUMBER))) > 0) {
 		    Cursor pCur = cr.query(ContactsContract.CommonDataKinds.Phone.CONTENT_URI,new String[]{Phone.NUMBER, Phone.TYPE},ContactsContract.CommonDataKinds.Phone.CONTACT_ID +" = ?",new String[]{id}, null);
                    while(pCur.moveToNext()){
                        if(pCur.getInt(pCur.getColumnIndex(Phone.TYPE)) == Phone.TYPE_MOBILE){
                            list.add(new Kontakt(name, pCur.getString(pCur.getColumnIndex(Phone.NUMBER))));
                            break;
                        }
                    }
                    pCur.close();
 	        }
            }
 	}

        CharSequence[] cs = new CharSequence[list.size()];
        int i = 0;
        for(Kontakt k : list){
            cs[i] = k.getName();
            i++;
        }
        builder = new AlertDialog.Builder(this);
        builder.setTitle("Kontakte (" + list.size() + ")");
        builder.setCancelable(true);
        builder.setItems(cs, new DialogInterface.OnClickListener(){
            public void onClick(DialogInterface dialog, int item){
                setContact(list.get(item));
            }
        });
        alert = builder.create();
    }
    alert.show();

 

[mjdv]

Bist du dir sicher, dass die nicht mehrmals angezeigt werden, wenn ein Kontakt mehrere Mobil Nummern hat?

Was ist an dem Array schlimm? Ansonsten mach dir nen Custom Dialog und nutze gleich einen SimpleCursorAdapter für eine ListView.

 

[tobi193]

Also der Code sucht sich aktuell nur die Mobilnummer raus und beachtet alle anderen nicht. Wenn jemand mehrere hat, dann würde nur die erste angezeigt werden. Von daher ist es nicht ganz optimal, aber wer hat denn schon mehrere Mobilnummern ?

 

[tobi193]

Jetzt fehlt nur noch eine Möglichkeit den SMS Verlauf des ausgewählten Kontakts zu öffnen sofern dieser denn vorhanden ist. Geht das irgendwie ? Ich habe bei Google nichts gefunden.

 

[wottpal]

How can I read SMS messages from the inbox programmatically in Android?

Grüße