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

修复因找不到类NewsFragment的无参构造函数导致的force close #48

Merged
merged 1 commit into from
May 7, 2013

Conversation

king0769
Copy link

@king0769 king0769 commented May 7, 2013

测试用例:
1、打开eoe程序。
2、将系统语言改成其他语言,如从中文切换成英文。
3、再次打开eoe程序。
修改改,上述测试用例未能通过,会产生exception,内容为can't instantiate class cn.eoe.app.view.NewsFragment; no empty constructor;在类NewsFragment增加无参构造函数后,此问题未有出现。
相信在很多场景下均会出现此问题,请修复之。

com360 added a commit that referenced this pull request May 7, 2013
修复因找不到类NewsFragment的无参构造函数导致的force close
@com360 com360 merged commit e6b654b into eoecn:master May 7, 2013
@com360
Copy link
Contributor

com360 commented May 7, 2013

其的无无参构造器的会陆续加上

@mjkohoh
Copy link
Contributor

mjkohoh commented Oct 2, 2013

发现这和issue #40 是同样的问题。@king Law的做法是增加了一个空的构造函数。在issue #40 中@ rockybean提议在fragment中增加newInstance方法。大家对于这两种方法有什么看法。


下面列出同样出同样有这个bug的fragment

  • UserCollectFragment
  • UserCollectListFragment
  • UserIntroFragment
  • UserLogOutFragment
  • WikiFragment

@mjkohoh mjkohoh mentioned this pull request Oct 2, 2013
@mjkohoh mjkohoh mentioned this pull request Oct 22, 2013
@king0769
Copy link
Author

效果都一样,效率也没什么区别。个人觉得。

2013/10/2 KOHOH [email protected]

发现这和issue#40是同样的问题。@king https://github.com/KingLaw的做法是增加了一个空的构造函数。在issue#40中@

rockybean提议在fragment中增加newInstance方法。大家对于这两种方法有什么看法。

下面列出同样出同样有这个bug的fragment

  • UserCollectFragment
  • UserCollectListFragment
  • UserIntroFragment
  • UserLogOutFragment
  • WikiFragment


Reply to this email directly or view it on GitHubhttps://github.com//pull/48#issuecomment-25517002
.

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