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

2018081309027陶奕兮 #44

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

CWMYStargazer
Copy link

No description provided.

Copy link
Owner

@luckymark luckymark left a comment

Choose a reason for hiding this comment

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

不错啊

@@ -0,0 +1,20 @@
#include<stdio.h>
Copy link
Owner

Choose a reason for hiding this comment

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

文件名应该有明确的含义,不宜使用1.cpp这样的,就像变量名不宜用abc一样;

@luckymark
Copy link
Owner

.suo .db这类由系统自动生成的文件,或者build产生的文件,一般不放到git中管理,1)避免浪费空间;2)减少review的工作量;
可以搜索下.gitignore文件的用法,并实际试验下效果

void BtoA(int x);
void CtoA(int x);

int AtoC(int x)
Copy link
Owner

Choose a reason for hiding this comment

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

这组实现不够简洁,可以再尝试下

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.

2 participants