Android resource IDs suddenly not final, switch()'es broken

Question

PREAMBLE: this question is quite obsolete, it was written when the preferred Android dev environment was Eclipse with the Android plugin.

I had a Java Android project for a while. Today, I've updated the Android dev tools to the Google's latest. And the project broke - I get a bunch of "case expressions must be constant expressions" compilation error messages.

Turns out that the R.java file is being now generated differently. Formerly, it would have a bunch of

public static final int MyID=0x12340000;

statements; now, it looks (after a clean/rebuild) like this:

public static int MyID=0x12340000;

final is gone. So all switches on resource IDs that I had (and I had a few) are wrong. What happened, please? Is it just me? What's the rationale here? Is it documented anywhere? Can I bring final back somehow?

score 43 · Accepted Answer · answered Oct 20 '11 at 19:11

43

This happened about yesterday, when the SDK/ADT 14 got released:

As of ADT 14, resource constants in library projects are no longer final. This is explained in greater detail in http://tools.android.com/tips/non-constant-fields

There's a quickfix available from ADT 14: http://tools.android.com/recent/switchstatementconversion

To quote from the rationale:

When multiple library projects are combined, the actual values of the fields (which must be unique) could collide. Before ADT 14, all fields were final, so as a result, all libraries had to have all their resources and associated Java code recompiled along with the main project whenever they were used. This was bad for performance, since it made builds very slow. It also prevented distributing library projects that didn't include the source code, limiting the usage scope of library projects.

The reason the fields are no longer final is that it means that the library jars can be compiled once and reused directly in other projects. As well as allowing distributing binary version of library projects (coming in r15), this makes for much faster builds.

answered Oct 20 '11 at 19:11

Philipp Reichart

20,771
6
58
65

2

Makes sense, actually. I see where they're coming from. The fact that Android libraries were just linked sources as opposed to, well, compiled libraries, has been a minor pain point for me for some time. Also, kudos to Google for providing a magic refactorer. – Seva Alekseyev Oct 20 '11 at 19:27
10

This is just a consequence of an idiotic design decision by the Android team. Whenever you see some generated file with a bunch of magic number constants, you should look at it funny. Thanks, Google for breaking a bunch of your developers' source code due to complete lack of foresight. Public static ints. Much better?! – Nate Nov 12 '11 at 01:15
4

This answer is marked as solution - but I don't see it... Very nice discussions, suggestions, documentation links and etc only... What is concrete solution? What should I change in my code to fix the issue - please in a few words? Or no solution? - then please answer in one word: "no". ? – Vitaly Dec 25 '20 at 05:56
1

@RussiaDroneFlights Did you read any of the links? They say to use if-else clauses instead of switch statements if you need to do someting conditionally with resource IDs. Switch statements would require the IDs to be final, which they haven't been for slightly over nine years now. – Philipp Reichart Dec 26 '20 at 11:40
This will makes you do a lot of extra work and do a lot of unnecessary modify of your code, just add `android.nonFinalResIds=false` in the gradle.properties file will working. – applixy Jul 29 '23 at 00:33
Bad Android design, ids must be final. – user2305886 Aug 19 '23 at 01:40

A P · Answer 2 · 2022-08-13T09:20:10.967

13

You could switch over to using If/Else statements and the warning will go away.

Sample:

    @Override
    public void onClick(final View v) {
        //finds which button was pressed
        final int buttonView = v.getId();
        String current = fromEditText.getText().toString();
        if (buttonView == R.id.bA) {
            current += getString(R.string.a);
        } 
  }

edited Aug 13 '22 at 09:20

answered Nov 05 '20 at 09:20

A P

2,131
2
24
36

9

❤️ Good one! – ONE Nov 11 '20 at 18:12

score 13 · Answer 3 · answered Jun 12 '22 at 07:05

13

Just add parentheses:

switch (view.getId()) {
    case (R.id.view1):
        break;
    case (R.id.view2):
        break;
}

answered Jun 12 '22 at 07:05

M. Marmor

390
4
9

4

Why does this work? And is it safe to do? – SMBiggs Dec 05 '22 at 22:49
@SMBiggs I think it makes it some kind of constant, not only a reference. I use it some months in production mode with many customers and yet i've not gotten any crash or complaint. – M. Marmor Dec 06 '22 at 09:54
The case statements are hardcoded at compile time. What you need is the resource to be environment constant and it is not. with or without parentheses.. This is similar with: "final int VIEW_ID_1 = R.id.view1;" The switch will fail if the resource will be changed later because it will no longer match the hardcoded resource. – Chris Feb 20 '23 at 08:33
2

The only thing this does is potentially make the detector miss this case. But it **doesn't** solve the case when the ids will actually be non-final (the Java compiler will not compile this code then). So it's a band-aid at best (all it does it silence the warning, but it doesn't fix the problem). – Joachim Sauer Apr 28 '23 at 10:32

score 6 · Answer 4 · answered Oct 04 '21 at 15:42

Google recommends you use if/else conditions

http://tools.android.com/tips/non-constant-fields

To change them automatically you can place the caret on the switch keyword and press Alt + Enter on Windows (Option + Enter on Mac) and select Replace 'switch' with 'if'

score 1 · Answer 5 · answered Nov 10 '20 at 06:12

1

You should use view binding!

android {
    ...
    viewBinding {
        enabled = true
    }
}

answered Nov 10 '20 at 06:12

Muse

422
3
7

1

Doesn't work with menus. The original question was mostly about menu IDs. – Seva Alekseyev May 19 '21 at 20:33
1

Adds a lot of classes and also doesn't actually fix the problem, it just uses another way to identify and work with views. – LordWabbit Aug 18 '21 at 19:24

score 0 · Answer 6 · answered Jul 29 '23 at 00:37

0

This is because in new AGP verion, Google make all rescours ids non final, add android.nonFinalResIds=false in your gradle.properties file in the root directory of your project to make sure the AGP generate final res ids will make you avoid modify your code.

answered Jul 29 '23 at 00:37

applixy

873
6
5

This happened in 2011. – Seva Alekseyev Jul 30 '23 at 00:17

Andrei Andrushchak · Answer 7 · 2020-10-15T17:52:44.450

-1

Just add this snipped in your module-level build.gradle file:

android {
    ...
    lintOptions {
        disable 'NonConstantResourceId'
    }
}

More: https://developer.android.com/studio/write/lint#gradle

edited Oct 15 '20 at 17:52

answered Oct 15 '20 at 17:47

Andrei Andrushchak

25
3

Any disadvantage of doing disabling lint option. – Shubham AgaRwal Jan 28 '21 at 09:32
18

Careful, this just disables the Lint Warning, it does not actually fix the problem in itself. – Cosmin Radu Jan 30 '21 at 16:06

Android resource IDs suddenly not final, switch()'es broken

7 Answers7

Linked

Related