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.

Sign up for GitHub

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

Jump to bottom

221701125_张家榜 #27

Merged
merged 18 commits into from Apr 16, 2020
Merged

221701125_张家榜 #27

merged 18 commits into from Apr 16, 2020

Conversation

dante1024
Copy link
Contributor

221701125_张家榜_软工实践寒假作业（2/2）

@dante1024
221701125
34c669e 
新建目录
@dante1024
基本实现了统计
5c46b4f 
基本实现了读取日志文件并进行统计输出到输出文件，但还未实现指令功能
@dante1024
-log -out两个参数
294b239 
为list指令添加了-log -out两个参数，程序可识别这两个参数
@dante1024
-date参数
140898f 
实现了-date参数，程序会统计所有-date参数指定的日期及之前的疫情，若没有-date参数，则默认统计所有的日志文件
@dante1024
-type参数
bec4683 
实现了-type参数，程序会统计所有-type参数指定的类型，若没有-type参数，则默认统计所有的类型
@dante1024
-province参数
2677af8 
实现了-province参数，程序会统计所有-province参数指定的省份，若没有-province参数，则默认统计全国以及日志文件所涉及的省份疫情
@dante1024
修复bug
5f43148 
修复了不管有没有-province参数，日志文件中涉及到的省份都会输出的bug
@dante1024
修复bug
d79d77e 
代码没有对参数的类型进行判断，例如要统计陕西的疫情，则应该输入指令java InfectStatistic list -log D:/log/ -date 2020-01-22 -out D:/output.txt -province 陕西，但是代码没有对-province进行判断，而是直接识别陕西这个参数，导致输入指令java InfectStatistic list -log D:/log/ -date 2020-01-22 -out D:/output.txt 陕西也可以达到同样的输出，这是不合理的，本次commit将修复此bug
@dante1024
识别参数
cb35822 
可以识别指令里的参数是否都合法
@dante1024
判断路径合法
ac50b16 
用正则表达式判断输出路径是否合法，必须是txt类型的文件
@dante1024
修复bug
8c5445b 
修复了无法识别日志文件中的注释的bug
@dante1024
不产生诱导性
7db2346 
为了使测试文件和输出文件不产生诱导性，要在日志文件/输出文件末尾加上
// 该文档并非真实数据，仅供测试使用
同时发现正则表达式应该用反斜杠
@dante1024
修复bug
25e81ff 
若参数-province没有包含全国，则输出文件中不应该统计全国的疫情。
在日志文件中，应该每输出一行都要在末尾添加换行符
@dante1024
优化代码
84ce468 
应当划分好各功能块，不要把太多功能写到main函数里，可以考虑适当把一些复杂的功能写成函数（方法），增加代码的可读性
@dante1024
优化代码
582f88f 
优化代码，提升性能，文件流记得关闭，删去没用的对象实例，删去没用过的包
@dante1024
优化代码
1c55bb9 
保证每一行代码严格不超过120个字符
@dante1024
优化代码
8894e98 
用正则表达式来判断文件输入是否正确较为麻烦，直接用File类自带的.exist()判断更方便
@dante1024
提交README.md
1918266 
提交README.md
@dante1024 dante1024 changed the title 221701125_张家榜_软工实践寒假作业（2/2） 221701125_张家榜 Feb 20, 2020
@kofyou kofyou merged commit 2d13af1 into kofyou:master Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dante1024 @kofyou