Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change switch by if #214

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Chrysweel
Copy link

If you are in a library you have to change all the switch/case statements to if/else blocks from ADT version 14.

@Chrysweel
Copy link
Author

@pocmo
Copy link
Owner

pocmo commented Apr 9, 2016

Yeah, this is annoying if you are using a library. Are you using Yaaic as a library? What app are you building with Yaaic as a library?

Regarding the code: This should be a long list of if-else statements not nested, like:

final int id = v.getId()
if (id == R.id.foo) {
    ...
} else if (id = R.id.bar) {
    ...
} else if (id == R.id.foobar) {
    ...
}

Do you want to update the pull request? I'd be happy to accept it after that. :)

…ents to if/else blocks from ADT version 14.

refix ident and mix else if
@Chrysweel Chrysweel force-pushed the change_Switch_By_If branch from ca8a21e to 52ca2f6 Compare April 12, 2016 07:56
@Chrysweel
Copy link
Author

I Solved.

My app is a MVP, using yaaic into my app, with my features login, register ...
I want learn about use yaaic as library, and can programing without modify code of yaaic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants