解决并发空指针问题 #28
Closed
解决并发空指针问题 #28
Conversation
This file contains 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
|
本身设计上是要求自己保证线程安全的。 还有其它方法都要加synchronized, 如truncate_file, modify_file等等。并且重用也有风险 而且由于不支持断线重连（如server端重启了），TrackerServer里面的socket不是null, 这样StorageClient其实已经不能用了。。外面加try-catch貌似可行? 所以StorageClient重用风险很大。 故基于当前的代码库，唯一的办法，就是每次用的时候new一个。 外加try catch, 出错的话重传。 |
|
@stonyz 正解！ |
Closed
|
竟然重新NEW了， 为什么还要加Synchronized同步锁，storageServer每次也是新的 |
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.
详细内容见 #27