Skip to content

修复了boolean 空指针异常问题。#269

Open
yangbisheng wants to merge 3 commits intomasterfrom
Yangbisheng
Open

修复了boolean 空指针异常问题。#269
yangbisheng wants to merge 3 commits intomasterfrom
Yangbisheng

Conversation

@yangbisheng
Copy link
Copy Markdown
Contributor

  • 这个PR解答了当前仓库中的题目(机器人会自动判题并合并当前PR)
  • 这个PR修复了当前仓库中的一些代码缺陷(机器人不会判题,而是由管理员来处理当前PR)

@hcsp-bot
Copy link
Copy Markdown
Contributor

hcsp-bot commented Aug 4, 2021

🎉 感谢提交Pull Request!请稍等片刻,我们已经将其提交到CI进行检查,一旦有结果会立即通知您!

public boolean isOn() {
return on;
Boolean c = true;
if (on == c) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不必要的条件逻辑。

Boolean c = true;
if (on == c) {
return true;
}else {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'}' 后应有空格。
'else' 前应有空格。

@hcsp-bot
Copy link
Copy Markdown
Contributor

hcsp-bot commented Aug 4, 2021

你的提交 5603cb4 ,似乎失败了:Your tests failed on CircleCI

😅 请不要气馁,仔细分析原因,再接再厉!

return on;
public boolean isOnRawValue() {
Boolean c = true;
if (on == c) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不必要的条件逻辑。

Boolean c = true;
if (on == c) {
return true;
}else {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'}' 后应有空格。
'else' 前应有空格。

@hcsp-bot
Copy link
Copy Markdown
Contributor

hcsp-bot commented Aug 4, 2021

你的提交 9efa11f ,似乎失败了:Your tests failed on CircleCI

😅 请不要气馁,仔细分析原因,再接再厉!

@hcsp-bot
Copy link
Copy Markdown
Contributor

hcsp-bot commented Aug 4, 2021

你的提交 cd63ced ,似乎失败了:Your tests failed on CircleCI

😅 请不要气馁,仔细分析原因,再接再厉!

@hcsp-bot hcsp-bot mentioned this pull request Aug 4, 2021
2 tasks
@hcsp-bot hcsp-bot mentioned this pull request Nov 26, 2021
2 tasks
@hcsp-bot hcsp-bot mentioned this pull request Jan 28, 2022
2 tasks
@hcsp-bot hcsp-bot mentioned this pull request Aug 2, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants