Spinner auf Popup funktioniert nicht

prototype0815 · 10. April 2015

Hey folks, ich habe große Probleme mit meinem Spinner.
[Blockierte Grafik: http://preview.slick.pics/FUnaKXCKJ.png]

Mein Popup wird angezeigt, aber sobald ich den Spinner öffnen will crasht meine App mit folgendem Fehler...

Code

04-10 07:18:02.843    1169-1169/com.example.paul_io E/AndroidRuntime﹕ FATAL EXCEPTION: main
    Process: com.example.paul_io, PID: 1169
    android.view.WindowManager$BadTokenException: Unable to add window -- token android.view.ViewRootImpl$W@52f824f8 is not valid; is your activity running?
            at android.view.ViewRootImpl.setView(ViewRootImpl.java:532)
            at android.view.WindowManagerGlobal.addView(WindowManagerGlobal.java:259)
            at android.view.WindowManagerImpl.addView(WindowManagerImpl.java:69)
            at android.widget.PopupWindow.invokePopup(PopupWindow.java:1019)
            at android.widget.PopupWindow.showAsDropDown(PopupWindow.java:925)
            at android.widget.ListPopupWindow.show(ListPopupWindow.java:625)
            at android.widget.Spinner$DropdownPopup.show(Spinner.java:1110)
            at android.widget.Spinner.performClick(Spinner.java:658)
            at android.view.View$PerformClick.run(View.java:18422)
            at android.os.Handler.handleCallback(Handler.java:733)
            at android.os.Handler.dispatchMessage(Handler.java:95)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5001)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:785)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:601)
            at dalvik.system.NativeStart.main(Native Method)

Alles anzeigen

Ich vermute das ich dem Spinner einen falschen Context beim initialisieren gegeben habe, und zwar habe ich folgendes übergeben...

Code

Variables.mainActivity = MainActivity.this;
.
.
.  


  Activity mContext = Variables.mainActivity;
  Spinner ecb = new Spinner(mContext);

ist es möglich das ich dem Spinner irgendeine View o.ä. vom Popup bzw. PopupWindow geben muss?

prototype0815 · 14. April 2015

Hier noch etwas zusätzliche Info...
para_popup.xml

XML

&lt;?xml version="1.0" encoding="utf-8"?&gt;
&lt;LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
              android:id="@+id/llMain"
              android:layout_width="wrap_content"
              android:layout_height="wrap_content"
              android:orientation="vertical"
              android:background="#EE000000"
              android:visibility="visible"
    &gt;




    &lt;TextView
        android:id="@+id/tvParameterPopup"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:textAppearance="?android:attr/textAppearanceMedium"
        android:layout_marginTop="5dp"
        android:layout_marginLeft="10dp"
        android:layout_marginRight="10dp"
        android:layout_marginBottom="20dp"
        /&gt;


    &lt;GridLayout
        android:id="@+id/glData"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        &gt;
    &lt;/GridLayout&gt;


    &lt;LinearLayout
        android:id="@+id/llButtons"
        android:orientation="horizontal"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        &gt;
    &lt;/LinearLayout&gt;
&lt;/LinearLayout&gt;

Alles anzeigen

jede Row meiner Tabelle bekommt diesen onLoongClickListener...

Java

@Override
  public boolean onLongClick(View v) {
    if (v instanceof TableRow) {
      TableRow r = ((TableRow) v);
      if (r.getChildCount() &gt; 0) {
        final Object paraObj;


        if (Variables.basisVersion &gt; 101)
          paraObj = hashmap.get(((TextView) r.getVirtualChildAt(1)).getText().toString().trim());  //  1 = Spalte ( ID )
        else
          paraObj = hashmap.get(((TextView) r.getVirtualChildAt(0)).getText().toString().trim());  //  0 = Spalte ( ID )


//      °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°   POPUP WINDOW   °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°
        LayoutInflater layoutInflater = (LayoutInflater) mContext.getBaseContext().getSystemService(Variables.mainActivity.LAYOUT_INFLATER_SERVICE);
        View popupView = layoutInflater.inflate(R.layout.para_popup, null);
        final PopupWindow popupWindow = new PopupWindow(
                popupView,
                LinearLayout.LayoutParams.WRAP_CONTENT,
                LinearLayout.LayoutParams.WRAP_CONTENT);
        popupWindow.setFocusable(true);
        popupWindow.showAtLocation(v, Gravity.CENTER, 0, 0);
        ParaEdit pe = new ParaEdit(paraObj, popupWindow) {
          @Override
          public void ok() {
            System.out.println("ParaEdit void ok(); pressed ########################");
            doUpdate();
          }
        };
      }
    }
    return true;
  }

Alles anzeigen

in Zeile 22 wird ein ParaEdit erstellt, das ist eine Klasse die das GridLayout der para_popup.xml füllt...
Hier nun die wichtigsten Ausschnitte aus der ParaEdit Klasse...

Bei den "/////////////////" hab ich unwichtigen Code ausgelassen, da meine ParaEdit 595 Zeilen hat.

Java

Activity mContext = Variables.mainActivity;
PopupWindow popupWindow;
Spinner ecb = new Spinner(mContext);


////////////////////


GridLayout glData = (GridLayout) popupWindow.getContentView().findViewById(R.id.glData);
glData.removeAllViews();
glData.setColumnCount(2);
setToPopup(glData, 1, 0, ecb);


///////////////////


  private synchronized void setToPopup(GridLayout layout, int col, int row, View ob) {
    GridLayout.LayoutParams lp = new GridLayout.LayoutParams();
    lp.height = GridLayout.LayoutParams.WRAP_CONTENT;
    lp.width = GridLayout.LayoutParams.WRAP_CONTENT;
    lp.leftMargin = 10;
    lp.setGravity(Gravity.FILL_HORIZONTAL);
    lp.columnSpec = GridLayout.spec(col);
    lp.rowSpec = GridLayout.spec(row);
    ob.setLayoutParams(lp);
    layout.addView(ob, lp);
  }


///////////////////


      String[] names = ((EPara) obj).getNames();
      ArrayList&lt;String&gt; EParaList = new ArrayList&lt;String&gt;();


      for (int i = 0; i &lt; names.length; i++)
        EParaList.add(new String(names[i]));


      ArrayAdapter&lt;String&gt; adapter = new ArrayAdapter&lt;String&gt;(mContext, android.R.layout.simple_spinner_item, EParaList);
      adapter.setDropDownViewResource(R.layout.custom_spinner_checkbox_item);
      ecb.setAdapter(adapter);
      ecb.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
        @Override
        public void onItemSelected(AdapterView&lt;?&gt; parent, View view, int position, long id) {
        }


        @Override
        public void onNothingSelected(AdapterView&lt;?&gt; parent) {


        }
      });


      ecb.setSelection(((EPara) obj).getAct());

Alles anzeigen

Marco Feltmann · 14. April 2015

Ist denn der Kontext derjenige, der in dieser Ansicht auch wirklich gezeigt wird?
Die Fehlermeldung lässt darauf schließen, dass Du einfach irgendeinen Kontext übergibst, damit sich der Spinner initialisieren lässt.
Da das PopUp sich direkt an das View hängt müsstest Du vermutlich dessen Kontext an den Spinner übergeben.

Ben · 14. April 2015

ja also Variables.mainActivity sieht für mich sehr krude aus. Du solltest so wenig statische methoden wie möglich verwenden.

Und wie marko schon sagte, du müsstest den context der view verwenden (sollte in der regel die activitiy sein wo diese view gesetzt wird)

prototype0815 · 15. April 2015

die MainActivity tauscht in meinem Fall über FragmentManager -> FragmentTransaction -> replace() die Fragmente aus, die aktuell angezeigt werden. Es ist also immer so das der gezeigt Context, der von meiner MainActivity ist !? Ausser ich hab da was falsch verstanden.

Auf meinem aktuell angezeigten Fragment (ParaMain) liegt ein MapPagerAdapter, dieser zeigt wiederum Fragmente an (ParaFragment), auf diesen Fragmenten liegt ein TableLayout, jede Row bekommt einen onLongKlickListener.
Ab hier sind wir dann wieder an der oben geschilderten Stelle, durch den LongClick wird ein popupView erstellt, para_popup.xml als Layout definiert, dessen GridLayout wird durch die Klasse ParaEdit gefüllt.

Dieses GridLayout wird dynamisch erstellt, je nachdem welcher Art mein Parameter ist, es werden also mal nur TextViews und EditText´s angezeigt, mal TextViews mit einer Gruppe RadioButtons, und eben mal TextViews mit einem Spinner.
In allen Fällen wird immer der selbe Context übergeben, nämlich "mContext", und es funktioniert auch alles, bis auf das aufklappen dieses doofen Spinners.

Marco Feltmann · 15. April 2015

In dem Spinner scheinst Du ein PopUp Listview anzuzeigen. (Ich glaube, das ist die Standardfunktionalität des Spinner beim Anklicken.)
Allerdings befindest Du Dich bereits in einem PopUp View.

Gemäß StackOverflow darf das Parent eines PopUp kein PopUp sein.
Du solltest also zunächst vergleichen, ob der Parent Deines Spinners dieselbe Instanz ist wie Dein PopUp View.
Und falls dem so ist das irgendwie ändern.

prototype0815 · 15. April 2015

wie vergleich man so was? Mit equals oder == oder instanceOf ???

Wenn ich den Debugger anhalte und die Eigenschaften meines Spinners anschaue steht bei mParent GridLayout, dessen mParent ist LinearLayout, und dessen mParent ist android.view.ViewRootImpl

Marco Feltmann · 15. April 2015

Und beim PopUp View?

Ich vermute, der Ansatz mit den zwei PopUp Views ist hier der Urheber der Probleme.

prototype0815 · 15. April 2015

Zur Laufzeit zeigt es mir NULL an, darum hab ich es mal mit einem System.out.println() versucht...

Java

@Override
  public boolean onLongClick(View v) {
    if (v instanceof TableRow) {
      TableRow r = ((TableRow) v);
      if (r.getChildCount() &gt; 0) {
        final Object paraObj;


        paraObj = hashmap.get(((TextView) r.getVirtualChildAt(1)).getText().toString().trim());  //  1 = Spalte ( ID )


//      °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°   POPUP WINDOW   °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°
        LayoutInflater layoutInflater = (LayoutInflater) mContext.getBaseContext().getSystemService(Variables.mainActivity.LAYOUT_INFLATER_SERVICE);
        View popupView = layoutInflater.inflate(R.layout.para_popup, null);
        final PopupWindow popupWindow = new PopupWindow(
                popupView,
                LinearLayout.LayoutParams.WRAP_CONTENT,
                LinearLayout.LayoutParams.WRAP_CONTENT);
        popupWindow.setFocusable(true);
        popupWindow.showAtLocation(v, Gravity.CENTER, 0, 0);
        System.out.println();
        ParaEdit pe = new ParaEdit(paraObj, popupWindow) {
          @Override
          public void ok() {
            System.out.println("ParaEdit void ok(); pressed ########################");
            doUpdate();
          }
        };
        System.out.println(popupView.getParent());
        System.out.println(popupWindow.getContentView());
      }
    }
    return true;
  }

Alles anzeigen

Ausgabe:

Code

04-15 15:12:50.913    4456-4456/com.example.paul_io I/System.out﹕ android.view.ViewRootImpl@530a90dc
04-15 15:12:50.913    4456-4456/com.example.paul_io I/System.out﹕ android.widget.LinearLayout{530a86ac V.E..... ......I. 0,0-0,0 #7f0a0073 app:id/llMain}

Marco Feltmann · 15. April 2015

Beide müssen denselben Parent haben.
Es nützt also nix beim ersten Lauf nach dem Einen und beim zweiten Lauf nach dem Andern zu sehen.

Du musst beides bei einem Durchlauf betrachten.
Und dann nicht nur auf die Klasse schauen (android.view.ViewRootImpl) sondern auf die Instanz (@530a90dc)

prototype0815 · 16. April 2015

Zeile 27 + 28

Java

@Override
  public boolean onLongClick(View v) {
    if (v instanceof TableRow) {
      TableRow r = ((TableRow) v);
      if (r.getChildCount() &gt; 0) {
        final Object paraObj;


        paraObj = hashmap.get(((TextView) r.getVirtualChildAt(1)).getText().toString().trim());  //  1 = Spalte ( ID )


//      °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°   POPUP WINDOW   °°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°°
        LayoutInflater layoutInflater = (LayoutInflater) mContext.getBaseContext().getSystemService(Variables.mainActivity.LAYOUT_INFLATER_SERVICE);
        View popupView = layoutInflater.inflate(R.layout.para_popup, null);
        final PopupWindow popupWindow = new PopupWindow(
                popupView,
                LinearLayout.LayoutParams.WRAP_CONTENT,
                LinearLayout.LayoutParams.WRAP_CONTENT);
        popupWindow.setFocusable(true);
        popupWindow.showAtLocation(v, Gravity.CENTER, 0, 0);
        System.out.println();
        ParaEdit pe = new ParaEdit(paraObj, popupWindow) {
          @Override
          public void ok() {
            System.out.println("ParaEdit void ok(); pressed ########################");
            doUpdate();
          }
        };
        System.out.println("onLongClick popupView:     ###########  " + popupView.getParent());
        System.out.println("onLongClick popupWindow:   ###########  " + ((LinearLayout)popupWindow.getContentView()).getParent());
      }
    }
    return true;
  }

Alles anzeigen

Zeile 7 + 8

Java

String[] names = ((EPara) obj).getNames();
      ArrayList&lt;String&gt; EParaList = new ArrayList&lt;String&gt;();


      for (int i = 0; i &lt; names.length; i++)
        EParaList.add(new String(names[i]));


      System.out.println("ParaEdit init Spinner:     ###########  " + ((LinearLayout)((GridLayout)ecb.getParent()).getParent()).getParent());
      System.out.println("ParaEdit init popupWindow: ###########  " + ((LinearLayout)popupWindow.getContentView()).getParent());


      ArrayAdapter&lt;String&gt; adapter = new ArrayAdapter&lt;String&gt;(mContext, android.R.layout.simple_spinner_item, EParaList);
      adapter.setDropDownViewResource(R.layout.custom_spinner_checkbox_item);
      ecb.setAdapter(adapter);
      ecb.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
        @Override
        public void onItemSelected(AdapterView&lt;?&gt; parent, View view, int position, long id) {
        }


        @Override
        public void onNothingSelected(AdapterView&lt;?&gt; parent) {


        }
      });


      ecb.setSelection(((EPara) obj).getAct());

Alles anzeigen

Ausgabe:

Code

System.out﹕ ParaEdit init Spinner:     ###########  android.view.ViewRootImpl@5312a4e0
System.out﹕ ParaEdit init popupWindow: ###########  android.view.ViewRootImpl@5312a4e0
System.out﹕ ParaEdit init Spinner:     ###########  android.view.ViewRootImpl@5312a4e0
System.out﹕ ParaEdit init popupWindow: ###########  android.view.ViewRootImpl@5312a4e0
System.out﹕ onLongClick popupView:     ###########  android.view.ViewRootImpl@5312a4e0
System.out﹕ onLongClick popupWindow:   ###########  android.view.ViewRootImpl@5312a4e0

Marco Feltmann · 16. April 2015

+hm+
Da müssen dann mal ein paar reine Android Entwickler schauen, ob und wie sich so was realisieren lässt.
Ich bin noch mindestens bis Mitte des Jahres zeittechnisch mit iOS ausgelastet.

prototype0815 · 16. April 2015

Ich dank Dir dennoch für deine Mühe

Psychosadistic · 16. April 2015

Hast du das mal versucht oder wäre die Lösung auf diesem Wege nicht unbedingt so toll?

http://stackoverflow.com/quest…dtokene/22310732#22310732

(Auf obigen Link bin ich von hier http://stackoverflow.com/quest…window-when-i-clickurgent gekommen)

Oder hier eine allgemeine Anleitung mit obiger Lösung: http://android-er.blogspot.de/…r-inside-popupwindow.html

Spinner auf Popup funktioniert nicht

Jetzt mitmachen!

Teilen