Open
Conversation
Owner
MIKEILOLITESLA
commented
Aug 29, 2019
- 上传了修改后的 代码
- 更新了部分代码
- 修改了上个版本中被指出的问题
- 分析数据的 title 改为汉语表示
- 增加了在当前文件夹中生成 result.txt 文件的功能,使结果更方便观看
MIKEILOLITESLA
commented
Sep 5, 2019
MIKEILOLITESLA
commented
Sep 6, 2019
Owner
Author
MIKEILOLITESLA
left a comment
There was a problem hiding this comment.
可以通过在后面加 arg 的形式来改变分析的表格
- '-A' 或者 '--Table_A' 后面加文件名替换 Applestore.csv 文件
- '-B' 或者 '--Table_B' 后面加文件名替换 googleplaystore.csv 文件
加入了 main 函数
可以用 ./ 的形式执行不用再前缀 python3
未加入文件类型识别功能
ikbear
reviewed
Sep 6, 2019
Collaborator
There was a problem hiding this comment.
__main__ 语句之后不能有其它函数定义,以及非函数定义的语句了,除非是在 __main__ 自己的作用于之内。
第 100 行 print('##导入数据完成##') 开始以后的还需要再整理成函数封装,并且放到 __main__ 语句的前面。
Collaborator
There was a problem hiding this comment.
前面三条语句可以直接合并成一条:file_handle.write('##开始对数据的清理##\n\n')
Collaborator
There was a problem hiding this comment.
前面三条语句直接合并成一条:print('##导入数据完成##\n##开始对数据的清理##')
再去 Google 上搜索一下 print() 语句的用法。
Collaborator
There was a problem hiding this comment.
-5 又是什么参数?
应该先把它赋值给一个变量,再以参数的形式传给函数 display_table,这样别人读起来更直观。
Collaborator
There was a problem hiding this comment.
-
把这个模块的代码整理成函数。
-
在计算方法的代码之前注释一下它的算法。
Owner
Author
本次更新的内容
遇到的问题
接下来的计划
|
ikbear
reviewed
Sep 22, 2019
Collaborator
|
This file contains hidden or 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
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.