android - onTouchListener detecting wrong dynamically created cardview -


i have learnt how create cardview dynamically. i'm using button press create cardview , have set ontouchlistener it. when i'm creating 1 card ontouchlistener working i'm creating 1 more card ontouchlistener isn't working expected. when have 2 cards , try move 1st card it's moving 2nd 1 instead of 1st.

public class mainactivity extends appcompatactivity     implements navigationview.onnavigationitemselectedlistener {  context context; imageview imageview; relativelayout relativelayout; static int pos=0;  @override protected void oncreate(bundle savedinstancestate) {     super.oncreate(savedinstancestate);     setcontentview(r.layout.activity_main);     toolbar toolbar = (toolbar) findviewbyid(r.id.toolbar);     setsupportactionbar(toolbar);      context=this;     relativelayout=(relativelayout)findviewbyid(r.id.rl_layout);      drawerlayout drawer = (drawerlayout) findviewbyid(r.id.drawer_layout);     actionbardrawertoggle toggle = new actionbardrawertoggle(             this, drawer, toolbar, r.string.navigation_drawer_open, r.string.navigation_drawer_close);     drawer.setdrawerlistener(toggle);     toggle.syncstate();      navigationview navigationview = (navigationview) findviewbyid(r.id.nav_view);     navigationview.setnavigationitemselectedlistener(this); }    @override public void onbackpressed() {     drawerlayout drawer = (drawerlayout) findviewbyid(r.id.drawer_layout);     if (drawer.isdraweropen(gravitycompat.start)) {         drawer.closedrawer(gravitycompat.start);     } else {         super.onbackpressed();     } }  @override public boolean oncreateoptionsmenu(menu menu) {     // inflate menu; adds items action bar if present.     getmenuinflater().inflate(r.menu.main, menu);     return true; }  @override public boolean onoptionsitemselected(menuitem item) {     int id = item.getitemid();      //noinspection simplifiableifstatement     if (id == r.id.action_settings) {          createcardviewwithoutimageview();          return true;     }      return super.onoptionsitemselected(item); }  @suppresswarnings("statementwithemptybody") @override public boolean onnavigationitemselected(menuitem item) {     return true; }  public void createcardviewwithoutimageview(){      final cardview cardview=new cardview(context);      cardview.setid(++pos);     cardview.setlayoutparams((new cardview.layoutparams(600, 600)));     cardview.setcontentpadding(25,25,25,25);     cardview.setcardbackgroundcolor(color.magenta);      relativelayout.addview(cardview);  }  private final view.ontouchlistener mlistener = new view.ontouchlistener() {     @override     public boolean ontouch(view card, motionevent event) {         // view (v) here cardview ontouchlistener set         relativelayout.layoutparams layoutparamsrl = (relativelayout.layoutparams) ((cardview) card).getlayoutparams();         float dx = 0, dy = 0, x = 0, y = 0;          switch (event.getaction()) {             case motionevent.action_down: {                 card.bringtofront();                 dx = event.getrawx() - layoutparamsrl.leftmargin;                 dy = event.getrawy() - layoutparamsrl.topmargin;             }             break;             case motionevent.action_move: {                 x = event.getrawx();                 y = event.getrawy();                 layoutparamsrl.leftmargin = (int) (x - dx);                 layoutparamsrl.topmargin = (int) (y - dy);                 card.setlayoutparams(layoutparamsrl);             }             break;             case motionevent.action_up: {              }             break;         }         return true;     } };

}

it's because cardview variable global. each time create new cardview it's replaced listener using last cardview created. should like:

// note: way, should not use capital letter name function public void createcardviewwithoutimageview(){      //cardview     final cardview cardview = new cardview(context);      //....  }

you going have same issue layoutparamsrl should local too.

edit

    // context context; // don't need store context here in activity //    cardview cardview; // don't need global //    relativelayout.layoutparams layoutparamsrl; // don't need global     imageview imageview;     relativelayout relativelayout;     static int pos = 0; // sure needs static field?      @override     protected void oncreate(bundle savedinstancestate) {         super.oncreate(savedinstancestate);         setcontentview(r.layout.activity_main);         toolbar toolbar = (toolbar) findviewbyid(r.id.toolbar);         setsupportactionbar(toolbar);  //        context=this;         relativelayout = (relativelayout) findviewbyid(r.id.rl_layout);          drawerlayout drawer = (drawerlayout) findviewbyid(r.id.drawer_layout);         actionbardrawertoggle toggle = new actionbardrawertoggle(                 this, drawer, toolbar, r.string.navigation_drawer_open, r.string.navigation_drawer_close);         drawer.setdrawerlistener(toggle);         toggle.syncstate();          navigationview navigationview = (navigationview) findviewbyid(r.id.nav_view);         navigationview.setnavigationitemselectedlistener(this);     }       @override     public void onbackpressed() {         drawerlayout drawer = (drawerlayout) findviewbyid(r.id.drawer_layout);         if (drawer.isdraweropen(gravitycompat.start)) {             drawer.closedrawer(gravitycompat.start);         } else {             super.onbackpressed();         }     }      @override     public boolean oncreateoptionsmenu(menu menu) {         // inflate menu; adds items action bar if present.         getmenuinflater().inflate(r.menu.main, menu);         return true;     }      @override     public boolean onoptionsitemselected(menuitem item) {         int id = item.getitemid();          //noinspection simplifiableifstatement         if (id == r.id.action_settings) {              createcardviewwithoutimageview();              return true;         }          return super.onoptionsitemselected(item);     }      @suppresswarnings("statementwithemptybody")     @override     public boolean onnavigationitemselected(menuitem item) {         return true;     }      public void createcardviewwithoutimageview() {          //cardview         cardview cardview = new cardview(mainactivity.this);          cardview.setid(++pos);         cardview.setlayoutparams((new cardview.layoutparams(600, 600)));         cardview.setcontentpadding(25, 25, 25, 25);         cardview.setcardbackgroundcolor(color.magenta);         cardview.setontouchlistener(mlistener);          relativelayout.addview(cardview);     }      private final view.ontouchlistener mlistener = new view.ontouchlistener() {          float dx, dy; // moved outside ontouch method in order them keep value.          @override         public boolean ontouch(view card, motionevent event) {             log.d("ontouchlistener", "don't touch me :p");              // view (v) here cardview ontouchlistener set             relativelayout.layoutparams layoutparamsrl = (relativelayout.layoutparams) ((cardview) card).getlayoutparams();             float x = 0, y = 0;              switch (event.getaction()) {                 case motionevent.action_down: {                     card.bringtofront();                     dx = event.getrawx() - layoutparamsrl.leftmargin;                     dy = event.getrawy() - layoutparamsrl.topmargin;                 }                 break;                 case motionevent.action_move: {                     x = event.getrawx();                     y = event.getrawy();                     layoutparamsrl.leftmargin = (int) (x - dx);                     layoutparamsrl.topmargin = (int) (y - dy);                     card.setlayoutparams(layoutparamsrl);                 }                 break;                 case motionevent.action_up: {                  }                 break;             }             return true;         }     };

Comments

Post a Comment

Popular posts from this blog

android - InAppBilling registering BroadcastReceiver in AndroidManifest -

i'm implementing inapp billing in android app. works well, however, i'm trying decouple broadcast receiver activity manifest. suggestion in android's trivialdrive sample: // important: dynamically register broadcast messages updated purchases. // register receiver here instead of <receiver> in manifest // because call getpurchases() @ startup, therefore can ignore // broadcasts sent while app isn't running. // note: registering listener in activity bad idea, done here // because sample. regardless, receiver must registered after // iabhelper setup, before first call getpurchases(). currently there's class extends broadcastreceiver : public class iabbroadcastreceiver extends broadcastreceiver { /** * intent action receiver should filter for. */ public static final string action = "com.android.vending.billing.purchases_updated"; private final iabbroadcastlistener mlistener; public iabbroadcastreceiver(iabbroadcastlistener listener) { m...
Read more

python Tkinter Capturing keyboard events save as one single string -

is possible store keyboard events 1 string? code below stores 1 char , prints it. card reader or bar code reader, contains collection of character/ string not 1 character @ time. goal save char pressed text variable. from tkinter import * root = tk() def key(event): text= event.char text+= event.char print ("pressed", text) def callback(event): frame.focus_set() print ("clicked at", event.x, event.y) frame = frame(root, width=100, height=100) frame.bind("<key>", key) frame.bind("<button-1>", callback) frame.pack() root.mainloop() currently, creating text variable , everytime key() function gets called, , text stores last character typed. you can define text module level variable , , use module level text inside key function - text = '' def key(event): global text text+= event.char print("pressed", text)
Read more

sql server - Why does Linq-to-SQL add unnecessary COUNT()? -

here tables: create table customer ( cid int primary key, name varchar(32) ) create table ord ( oid int primary key, cid int foreign key references customer, address varchar(20) ) here's linq-to-sql statement: var aaa = c in db.customer select new { c, o = c.ord.tolist() }; and here's inexplicable query generated linq-to-sql: select [t0].[cid] [cid], [t0].[name] [name], [t1].[oid] [oid], [t1].[cid] [cid2], [t1].[address] [address], (select count(*) [dbo].[ord] [t2] [t2].[cid] = [t0].[cid]) [value] [dbo].[customer] [t0] left outer join [dbo].[ord] [t1] on [t1].[cid] = [t0].[cid] order [t0].[cid], [t1].[oid] i want understand how rid of count(*) part. it's entirely uncalled for! linq needs count(*) in order determine number of entries returned o = c.ord.tolist() .
Read more