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

chao0x01实验报告1提交 #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chao0x01实验报告1提交 #1

wants to merge 1 commit into from

Conversation

Stephaniesuu
Copy link
Collaborator

网络安全课程实验报告1提交

@huangyifei226 huangyifei226 self-requested a review September 28, 2021 08:54
huangyifei226
huangyifei226 approved these changes Sep 28, 2021
View reviewed changes
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

  • 实验报告中每一个步骤都记录的非常详细；
  • 实验完成度较高，所有配置任务均已完成；
  • 每一台虚拟机标注上相应的名称，达到了「见名知意」的效果；

Cons

  • 实验报告中部分换行失败；

Recommends

  • 对于实验环境的说明，表格并不是最适合用来描述的，更推荐的描述方法是绘制一张「网络拓扑图」，并标注虚拟网卡类型、虚拟网卡名称、虚拟网卡ip等。比直接沿用黄大的网络拓扑图，多了更多细节，达到「一图胜千言」的效果。
  • 参考资料建议渲染成无序列表；

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@huangyifei226 huangyifei226 huangyifei226 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Stephaniesuu @huangyifei226