-
Notifications
You must be signed in to change notification settings - Fork 250
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
Mvvm #8
base: master
Are you sure you want to change the base?
Mvvm #8
Conversation
DataBindingUtil.setContentView() as well as returning the binding, it also sets the given layout as the content view of the given Activity.
setContentView() not required here.
Added handling of activity recreation. For instance on screen rotation
* | ||
*/ | ||
public void mark( int row, int col ) { | ||
public Player mark( int row, int col ) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was really nice!
return playerThatMoved; | ||
} | ||
|
||
public Player valueAtCell(int row, int col) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the point of that method?
@@ -1,3 +1,3 @@ | |||
package com.acme.tictactoe.model; | |||
|
|||
enum Player { X , O } | |||
public enum Player { X , O } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it not only used on /model/
No description provided.