Skip to content

JAVA课程项目_简单版本控制工具项目_第一阶段任务#5

Open
bigbeats wants to merge 4 commits intomainfrom
dev1
Open

JAVA课程项目_简单版本控制工具项目_第一阶段任务#5
bigbeats wants to merge 4 commits intomainfrom
dev1

Conversation

@bigbeats
Copy link
Copy Markdown
Collaborator

@bigbeats bigbeats commented Dec 8, 2020

之前的代码被误操作merge了,这是新的提交@js00070 恳请学长指正!
JAVA课程项目_简单版本控制工具项目_第一阶段任务的代码:
文件夹中包含三个类:Hash类,Blob类,Test类。第二阶段任务将补充Tree类,用于实现将一个文件夹转化为key的工作。
①Hash类用于提供生成文件或文件夹或字符串对应key值的方法。
②Blob类用于生成blob对象,主要提供为存储中添加key-value的方法。
③Test类用于单元测试,提供给定key值找到对应value值的方法。

重新提交一次commits
重新提交一次commits
重新提交一次commits
Comment thread gitcontroltool/Blob.java
@1. ����value����洢�����Ӷ�Ӧ��key-value
step1 �ڹ������½�һ��file���󣬲�д���ļ����ݣ�����̨���롢������д��һ�����е��ļ���
step2 Ϊ���file����������Ϊ���ļ���hashֵ��������
/* 任务一的理解:主要是针对blob对象
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

其实Tree对象也是一样,也是通过key-value接口进行读写的,Tree和Blob本质上没什么不同,都是有自己的value值,然后value的哈希值作为key,只是Blob和Tree的value的具体含义不一样

Comment thread gitcontroltool/Test.java
}
}

public static void getValue(String path, String encoding) throws IOException {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

getValue方法是不是应该整合到功能代码里?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

代码逻辑的拆分有改进空间,最好是拆分出一个类,专门做key-value的get、set接口,文件的读写代码全部封装在这个类里

明白学长的意思!今晚会再做修改。

@js00070
Copy link
Copy Markdown

js00070 commented Dec 9, 2020

代码逻辑的拆分有改进空间,最好是拆分出一个类,专门做key-value的get、set接口,文件的读写代码全部封装在这个类里

@js00070
Copy link
Copy Markdown

js00070 commented Dec 9, 2020

主分支似乎得git reset然后git push -f一下,这样才不会有冲突

@bigbeats
Copy link
Copy Markdown
Collaborator Author

bigbeats commented Dec 9, 2020

主分支似乎得git reset然后git push -f一下,这样才不会有冲突

感谢学长指点解决方案!

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.

各位可以提一提对于项目中的任务一和任务二的通俗理解。

3 participants