Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
superrain51
apollo
提交
f1f72459
apollo
项目概览
superrain51
/
apollo
与 Fork 源项目一致
从无法访问的项目Fork
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
apollo
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
未验证
提交
f1f72459
编写于
5月 03, 2018
作者:
J
Jason Song
提交者:
GitHub
5月 03, 2018
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #1072 from wuwen5/properties#preform
优化Properties#stringPropertyNames处理,jdk9以下版本此方法cpu消耗相对过高.
上级
c4946900
96efd8ff
变更
2
显示空白变更内容
内联
并排
Showing
2 changed file
with
50 addition
and
1 deletion
+50
-1
apollo-client/src/main/java/com/ctrip/framework/apollo/internals/DefaultConfig.java
...a/com/ctrip/framework/apollo/internals/DefaultConfig.java
+15
-1
apollo-client/src/test/java/com/ctrip/framework/apollo/internals/DefaultConfigTest.java
...m/ctrip/framework/apollo/internals/DefaultConfigTest.java
+35
-0
未找到文件。
apollo-client/src/main/java/com/ctrip/framework/apollo/internals/DefaultConfig.java
浏览文件 @
f1f72459
...
...
@@ -8,6 +8,7 @@ import java.util.Map;
import
java.util.Objects
;
import
java.util.Properties
;
import
java.util.Set
;
import
java.util.HashMap
;
import
java.util.concurrent.atomic.AtomicReference
;
import
org.slf4j.Logger
;
...
...
@@ -101,7 +102,20 @@ public class DefaultConfig extends AbstractConfig implements RepositoryChangeLis
return
Collections
.
emptySet
();
}
return
properties
.
stringPropertyNames
();
return
stringPropertyNames
(
properties
);
}
private
Set
<
String
>
stringPropertyNames
(
Properties
properties
)
{
//jdk9以下版本Properties#enumerateStringProperties方法存在性能问题,keys() + get(k) 重复迭代, jdk9之后改为entrySet遍历.
Map
<
String
,
String
>
h
=
new
HashMap
<>();
for
(
Map
.
Entry
<
Object
,
Object
>
e
:
properties
.
entrySet
())
{
Object
k
=
e
.
getKey
();
Object
v
=
e
.
getValue
();
if
(
k
instanceof
String
&&
v
instanceof
String
)
{
h
.
put
((
String
)
k
,
(
String
)
v
);
}
}
return
h
.
keySet
();
}
@Override
...
...
apollo-client/src/test/java/com/ctrip/framework/apollo/internals/DefaultConfigTest.java
浏览文件 @
f1f72459
...
...
@@ -11,6 +11,8 @@ import java.io.File;
import
java.util.Calendar
;
import
java.util.Date
;
import
java.util.Properties
;
import
java.util.Set
;
import
java.util.Collections
;
import
java.util.concurrent.TimeUnit
;
import
org.junit.After
;
...
...
@@ -645,6 +647,39 @@ public class DefaultConfigTest {
assertEquals
(
PropertyChangeType
.
ADDED
,
newKeyChange
.
getChangeType
());
}
@Test
public
void
testGetPropertyNames
()
{
String
someKeyPrefix
=
"someKey"
;
String
someValuePrefix
=
"someValue"
;
//set up config repo
someProperties
=
new
Properties
();
for
(
int
i
=
0
;
i
<
10
;
i
++)
{
someProperties
.
setProperty
(
someKeyPrefix
+
i
,
someValuePrefix
+
i
);
}
when
(
configRepository
.
getConfig
()).
thenReturn
(
someProperties
);
DefaultConfig
defaultConfig
=
new
DefaultConfig
(
someNamespace
,
configRepository
);
Set
<
String
>
propertyNames
=
defaultConfig
.
getPropertyNames
();
assertEquals
(
10
,
propertyNames
.
size
());
assertEquals
(
someProperties
.
stringPropertyNames
(),
propertyNames
);
}
@Test
public
void
testGetPropertyNamesWithNullProp
()
{
when
(
configRepository
.
getConfig
()).
thenReturn
(
null
);
DefaultConfig
defaultConfig
=
new
DefaultConfig
(
someNamespace
,
configRepository
);
Set
<
String
>
propertyNames
=
defaultConfig
.
getPropertyNames
();
assertEquals
(
Collections
.
emptySet
(),
propertyNames
);
}
private
void
checkDatePropertyWithFormat
(
Config
config
,
Date
expected
,
String
propertyName
,
String
format
,
Date
defaultValue
)
{
assertEquals
(
expected
,
config
.
getDateProperty
(
propertyName
,
format
,
defaultValue
));
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录