静态字段java,java – 从实例方法写入静态字段

我的代码如下.我知道了

public MyClass{

private static DataSource dataSource = null;

private static DataSource getDataSource(){

if (dataSource == null) {

try {

dataSource = // something.

} catch (Exception e) {

// some exception.

}

}

return dataSource;

}

public List doSomething(){

// ...

if(dataSource == null){

dataSource = getDataSource();

}

dataSource.getConnection();

// ...

}

}

我在声纳anaylsis中看到以下消息.

Dodgy - Write to static field from instance method

This instance method writes to a static field. This is tricky to get correct if multiple instances are being manipulated, and generally bad practice.

findbugs:ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD Sep12 Reliability > Architecture

我看到在这个实现中一切都没问题,只是我们正在改变doSomething方法中的静态变量.我们如何解决这个问题?

解决方法:

不确定您的静态分析工具如何工作,但是 –

尝试通过静态setter写入您的值:

private synchronized static void setDataSource(DataSource ds) {

dataSource = ds;

}

这样你就可以做到

if(dataSource == null){

setDataSource(getDataSource());

}

标签:java,sonarqube

来源: https://codeday.me/bug/20190825/1714621.html

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值