all #49
Open
all #49
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
exe等二进制文件，一般不提交到git中；一般来说，凡是可以build生成的文件都不提交到git |
|
忘记gitnore了 |
luckymark
reviewed
Mar 20, 2019
course-C/PI.cpp
Outdated
| #define MAX_RANGE /*(2 << 29 - 1)*/ 100000 | ||
| double arccot(int n,int digit)//所以要怎么控制迭代经度呢？如果用浮点数就会产生问题 | ||
| { | ||
| for() |
There was a problem hiding this comment.
还没实现完成吧？ 如果是没完成的代码，就不要commit，更不要push了
There was a problem hiding this comment.
没有，完成的会弄在level2的文件夹里面，这个course-c文件夹只是暂存。
level1/p04_ narcissus/narcissus.cpp
Outdated
| @@ -0,0 +1,29 @@ | |||
| #include <stdio.h> | |||
| void get_threeDigit_Number(const int targetnumber,int *digit_number) | |||
There was a problem hiding this comment.
下划线风格或驼峰风格，用一种就好吧；当然全大写的常量用下划线分割一下不错
|
|
||
| for (; i < m; i++, p++) | ||
| { | ||
| p->weight = 0; |
There was a problem hiding this comment.
这里不是重复，是哈夫曼树的初始化，因为变量p和w申明在前面的，所以前面初始化后，后面的指针还可以再用
There was a problem hiding this comment.
我不是指功能重复了，而是指代码重复了；建议看看Martin Fowler的《重构》
There was a problem hiding this comment.
好，这里我真没理解俸爷的意思，我去看下这本书然后考虑下怎么改掉这里。
| p->parent = -1; | ||
| } | ||
| //构造HUFFMAN TREE | ||
| for (i = n; i < m; i++) |
| [神经网络入门](http://blog.csdn.net/zzwu/article/details/574931) | ||
|
|
||
| 说实话，用C++写挺麻烦的，数据集也要自己生成。一般来说神经网络倒不是特别难写，不过要借助opencv的库。。。还是先把五子棋做完再来看看吧。 |
NONOThingC
force-pushed
the
master
branch
2 times, most recently
from
1039ec2
to
e0154d9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
补课补学分doge的作业