Java conditional statement not working as expected

Question

I'm trying to override getItemViewType and return a different view type for my footer, but integer comparison is failing within the function:

This image shows a debug session where the input position is 21 and the the private List mValues has a size of 21, however when I pushed "step over" the conditional code is skipped.

If both integers are equal, why does position == mValues.size() return false?

I cleaned and rebuilt my project before running.

UPDATE:

It works when the integers are hardcoded:

but not when I use the input parameter:

UPDATE

I added a Log statement as the first line of the function

Log.e("MYAPP", "position: " + position + ", evaluates: " + (position == 21));

and in my monitor I see: position: 21, evaluates: true

However, my breakpoint at return FOOTER_VIEW; is never called and none of the functions related to the footer view are ever called.

UPDATE:

Heres the full Adaptor: https://gist.github.com/fergusom/79c6d5d9b1fd5e8348949d114a0159d5

Note that I updated the code to compute a boolean first, but it still fails even though the boolean prints out true in the logs

What type is mValues (runtime type)? Did you try stepping into mValues.size()? Can you verify that it returns 21? — Ole V.V., Sep 02 '16 at 16:12
If I replace `mValues.size()` with 21, it still fails, added a picture that shows this — Cbas, Sep 02 '16 at 16:14
Don't trust the debugger! Put logs statements inside and outside the `if`. — K Neeraj Lal, Sep 02 '16 at 16:16
The debugger may highlight the wrong line because of code optimizations. — Rediska, Sep 02 '16 at 16:18
ok but if I put my breakpoint `return FOOTER_VIEW`, it never gets triggered, even though `position` clearly reaches 21 — Cbas, Sep 02 '16 at 16:22
Nothing beats a `log.d`. Also, make sure you clean the project when you start seeing weird stuff happening, especially with instant run on. — mbmc, Sep 02 '16 at 16:29
I already cleaned it, in the OP. just added a log but it still implies the conditional statement is failing somehow — Cbas, Sep 02 '16 at 16:31
`getViewTypeCount` isn't a function. for `getItemViewType` I only return 1 or call the super function — Cbas, Sep 02 '16 at 16:34
Try to override `getViewTypeCount` method and return 2 from it. — K Neeraj Lal, Sep 02 '16 at 16:37
@mbmc I replaced `mValue.size()` with a hardcoded 21 and it still fails — Cbas, Sep 02 '16 at 16:38
@KNeerajLal I'm using a `RecyclerView.Adapter`. that function isn't available — Cbas, Sep 02 '16 at 16:40
are you sure the method is actually called? if you put a log statement or breakpoint on entry to the method, do you see the method itself being called? — Kevin Hooke, Sep 02 '16 at 16:42
@ColdFire idk I got that code from here: http://stackoverflow.com/questions/26448717/android-5-0-add-header-footer-to-a-recyclerview — Cbas, Sep 02 '16 at 17:09

score 1 · Answer 1 · answered Sep 02 '16 at 16:43

1

Give the jvm no chance:

if(new Integer(position).equals(new Integer(mValues.size()))

If the Integer constructor fails you'll know what the values were in the exception

answered Sep 02 '16 at 16:43

David Kühner

793
1
7
19

I tried going one step further and creating a `boolean` to use in the conditional and it still failed even when the `boolean` prints out `true` in a log. check out the full gist in the last update – Cbas Sep 02 '16 at 16:54
Wait what? You printed a boolean variable, gives you true and it doesn't goes in the if statement? – David Kühner Sep 02 '16 at 17:13
yes. seems to have been a glitch on my device. I restarted my phone and its working now – Cbas Sep 02 '16 at 17:14

score 1 · Accepted Answer · answered Sep 02 '16 at 17:13

1

I restarted my phone and it started working...

answered Sep 02 '16 at 17:13

Cbas

6,003
11
56
87

that was entertaining :) – mbmc Sep 02 '16 at 19:42

Java conditional statement not working as expected

2 Answers2