Skip to content

Add Solution 123[CPP] #99

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

Merged
merged 10 commits into from
Nov 12, 2018
Merged

Add Solution 123[CPP] #99

merged 10 commits into from
Nov 12, 2018

Conversation

KongJHong
Copy link
Contributor

改了名字,github没有自动merge

@yanglbme yanglbme added the conflict This branch has conflicts that must be resolved label Nov 7, 2018
@KongJHong
Copy link
Contributor Author

更正

@KongJHong
Copy link
Contributor Author

更正上传

@yanglbme
Copy link
Member

yanglbme commented Nov 8, 2018

@KongJHong 0008.String to Integer (atoi)/Solution.java
0008 文件夹已存在,别重复上传,请注意文件夹的空格,谢谢了。

否则 merge 之后会有两个同名文件夹(一个带空格,一个不带空格)

@yanglbme
Copy link
Member

yanglbme commented Nov 8, 2018

提交文件时细心一点哈,不然的话 merge 之后很乱的,希望老哥能理解,维护一个仓库不容易。

@yanglbme yanglbme added the invalid This doesn't seem right label Nov 8, 2018
@yanglbme
Copy link
Member

yanglbme commented Nov 9, 2018

@KongJHong
老哥做题都不按照要求来的吗>_<?
有小伙伴反馈第 41 题复杂度问题,题目限制 O(n),你 sort 一下复杂度就到 O(nlogn) 了噢~

-------------
### 思路:

#### 错误的思路
Copy link
Member

Choose a reason for hiding this comment

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

不要把错误的思路也提交上来哈,容易引导别人往错误的方向思考。

解释: 在这个情况下, 没有交易完成, 所以最大利润为 0。
```
-------------
### 思路:
Copy link
Member

Choose a reason for hiding this comment

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

REAMDE 文件都按统一的规范来吧,参照我前面的提交。

我真的很看重仓库/文档的规范性与统一性。

@KongJHong
Copy link
Contributor Author

KongJHong commented Nov 9, 2018 via email

@KongJHong
Copy link
Contributor Author

KongJHong commented Nov 9, 2018 via email

@KongJHong
Copy link
Contributor Author

更正提交

@yanglbme yanglbme merged commit 2850eca into doocs:master Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conflict This branch has conflicts that must be resolved invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants