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

Chap0x04 #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Chap0x04 #4

wants to merge 2 commits into from

Conversation

Stephaniesuu
Copy link
Collaborator

No description provided.

@huangyifei226 huangyifei226 self-requested a review June 2, 2021 01:36
Copy link

@huangyifei226 huangyifei226 left a comment

Choose a reason for hiding this comment

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

Accepted

Pros

  • 编写的所有脚本均通过了 shellcheck 的代码规范性检查👍

Task-1

  • 支持命令行参数方式使用不同功能
  • 支持对指定目录下所有支持格式的图片文件进行批处理
  • 支持以下常见图片批处理功能的单独使用或组合使用
    • 支持对jpeg格式图片进行图片质量压缩
    • 支持对jpeg/png/svg格式图片在保持原始宽高比的前提下压缩分辨率
    • 支持对图片批量添加自定义文本水印
    • 支持批量重命名(统一添加文件名前缀或后缀,不影响原始文件扩展名)
    • 支持将png/svg图片统一转换为jpg格式图片

Task-2

  • 统计不同年龄区间范围(20岁以下、[20-30]、30岁以上)的球员数量、百分比
  • 统计不同场上位置的球员数量、百分比
  • 名字最长的球员是谁?名字最短的球员是谁?
  • 年龄最大的球员是谁?年龄最小的球员是谁?

Task-3

  • 统计访问来源主机TOP 100和分别对应出现的总次数
  • 统计访问来源主机TOP 100 IP和分别对应出现的总次数
  • 统计最频繁被访问的URL TOP 100
  • 统计不同响应状态码的出现次数和对应百分比
  • 分别统计不同4XX状态码对应的TOP 10 URL和对应出现的总次数
  • 给定URL输出TOP 100访问来源主机

Cons

  • **–help**在github上渲染失败
  • 实验报告中部分换行失败
  • 实验三中数据统计准确率有待提升

Recommends

  • apt install 一次可以填入多个参数同时安装多个软件,不必一行指令只安装一个软件。所以你在 .travis.yml 里安装软件的代码可以合并为一行: sudo apt install -y p7zip-full imagemagick ghostscript ,使用 -y 参数可以避免 apt 在安装部分软件时出现的交互式提示导致脚本执行被暂停;
  • 任务一的更好测试证明方案建议如下:
    • 支持以下常见图片批处理功能的单独使用或组合使用
      • 这里实际上需要使用 travis-ci 的「持续部署」功能了,对于本次作业来说,可以在所有测试用例通过后,把任务 1 脚本执行后创建的新图片提交到一个专用的 Github 分支来持久化存储本次「构建」的成果。推荐阅读 travis-ci.org 官网帮助之自定义部署方法之 Git

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