#412 规范代码,修复变量名使用错误的问题

This commit is contained in:
Binary Wang
2017-12-23 12:22:43 +08:00
parent 94ae938a2b
commit 0a21fe2934
2 changed files with 40 additions and 43 deletions

View File

@ -51,7 +51,7 @@ public class WxMpInRedisConfigStorage extends WxMpInMemoryConfigStorage {
@Override @Override
public String getAccessToken() { public String getAccessToken() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(accessTokenKey); return jedis.get(this.accessTokenKey);
} }
} }
@ -65,70 +65,70 @@ public class WxMpInRedisConfigStorage extends WxMpInMemoryConfigStorage {
@Override @Override
public synchronized void updateAccessToken(String accessToken, int expiresInSeconds) { public synchronized void updateAccessToken(String accessToken, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(accessTokenKey, expiresInSeconds - 200, accessToken); jedis.setex(this.accessTokenKey, expiresInSeconds - 200, accessToken);
} }
} }
@Override @Override
public void expireAccessToken() { public void expireAccessToken() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.expire(accessTokenKey, 0); jedis.expire(this.accessTokenKey, 0);
} }
} }
@Override @Override
public String getJsapiTicket() { public String getJsapiTicket() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(jsapiTicketKey); return jedis.get(this.jsapiTicketKey);
} }
} }
@Override @Override
public boolean isJsapiTicketExpired() { public boolean isJsapiTicketExpired() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.ttl(jsapiTicketKey) < 2; return jedis.ttl(this.jsapiTicketKey) < 2;
} }
} }
@Override @Override
public synchronized void updateJsapiTicket(String jsapiTicket, int expiresInSeconds) { public synchronized void updateJsapiTicket(String jsapiTicket, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(jsapiTicketKey, expiresInSeconds - 200, jsapiTicket); jedis.setex(this.jsapiTicketKey, expiresInSeconds - 200, jsapiTicket);
} }
} }
@Override @Override
public void expireJsapiTicket() { public void expireJsapiTicket() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.expire(jsapiTicketKey, 0); jedis.expire(this.jsapiTicketKey, 0);
} }
} }
@Override @Override
public String getCardApiTicket() { public String getCardApiTicket() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(cardapiTicketKey); return jedis.get(this.cardapiTicketKey);
} }
} }
@Override @Override
public boolean isCardApiTicketExpired() { public boolean isCardApiTicketExpired() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.ttl(cardapiTicketKey) < 2; return jedis.ttl(this.cardapiTicketKey) < 2;
} }
} }
@Override @Override
public synchronized void updateCardApiTicket(String cardApiTicket, int expiresInSeconds) { public synchronized void updateCardApiTicket(String cardApiTicket, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(cardapiTicketKey, expiresInSeconds - 200, cardApiTicket); jedis.setex(this.cardapiTicketKey, expiresInSeconds - 200, cardApiTicket);
} }
} }
@Override @Override
public void expireCardApiTicket() { public void expireCardApiTicket() {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.expire(cardapiTicketKey, 0); jedis.expire(this.cardapiTicketKey, 0);
} }
} }
} }

View File

@ -7,7 +7,6 @@ import redis.clients.jedis.JedisPool;
* @author <a href="https://github.com/007gzs">007</a> * @author <a href="https://github.com/007gzs">007</a>
*/ */
public class WxOpenInRedisConfigStorage extends WxOpenInMemoryConfigStorage { public class WxOpenInRedisConfigStorage extends WxOpenInMemoryConfigStorage {
private final static String COMPONENT_VERIFY_TICKET_KEY = "wechat_component_verify_ticket:"; private final static String COMPONENT_VERIFY_TICKET_KEY = "wechat_component_verify_ticket:";
private final static String COMPONENT_ACCESS_TOKEN_KEY = "wechat_component_access_token:"; private final static String COMPONENT_ACCESS_TOKEN_KEY = "wechat_component_access_token:";
@ -16,7 +15,6 @@ public class WxOpenInRedisConfigStorage extends WxOpenInMemoryConfigStorage {
private final static String JSAPI_TICKET_KEY = "wechat_jsapi_ticket:"; private final static String JSAPI_TICKET_KEY = "wechat_jsapi_ticket:";
private final static String CARD_API_TICKET_KEY = "wechat_card_api_ticket:"; private final static String CARD_API_TICKET_KEY = "wechat_card_api_ticket:";
protected final JedisPool jedisPool; protected final JedisPool jedisPool;
private String componentVerifyTicketKey; private String componentVerifyTicketKey;
private String componentAccessTokenKey; private String componentAccessTokenKey;
@ -32,46 +30,46 @@ public class WxOpenInRedisConfigStorage extends WxOpenInMemoryConfigStorage {
@Override @Override
public void setComponentAppId(String componentAppId) { public void setComponentAppId(String componentAppId) {
super.setComponentAppId(componentAppId); super.setComponentAppId(componentAppId);
componentVerifyTicketKey = COMPONENT_VERIFY_TICKET_KEY.concat(componentAppId); this.componentVerifyTicketKey = COMPONENT_VERIFY_TICKET_KEY.concat(componentAppId);
componentAccessTokenKey = COMPONENT_ACCESS_TOKEN_KEY.concat(componentAppId); this.componentAccessTokenKey = COMPONENT_ACCESS_TOKEN_KEY.concat(componentAppId);
authorizerRefreshTokenKey = AUTHORIZER_REFRESH_TOKEN_KEY.concat(componentAppId); this.authorizerRefreshTokenKey = AUTHORIZER_REFRESH_TOKEN_KEY.concat(componentAppId);
authorizerAccessTokenKey = AUTHORIZER_ACCESS_TOKEN_KEY.concat(componentAppId); this.authorizerAccessTokenKey = AUTHORIZER_ACCESS_TOKEN_KEY.concat(componentAppId);
jsapiTicketKey = JSAPI_TICKET_KEY.concat(componentAppId); this.jsapiTicketKey = JSAPI_TICKET_KEY.concat(componentAppId);
cardApiTicket = CARD_API_TICKET_KEY.concat(componentAppId); this.cardApiTicket = CARD_API_TICKET_KEY.concat(componentAppId);
} }
@Override @Override
public String getComponentVerifyTicket() { public String getComponentVerifyTicket() {
try (Jedis jedis = jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(componentVerifyTicketKey); return jedis.get(this.componentVerifyTicketKey);
} }
} }
@Override @Override
public void setComponentVerifyTicket(String componentVerifyTicket) { public void setComponentVerifyTicket(String componentVerifyTicket) {
try (Jedis jedis = jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.set(componentVerifyTicketKey, componentVerifyTicket); jedis.set(this.componentVerifyTicketKey, componentVerifyTicket);
} }
} }
@Override @Override
public String getComponentAccessToken() { public String getComponentAccessToken() {
try (Jedis jedis = jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(componentAccessTokenKey); return jedis.get(this.componentAccessTokenKey);
} }
} }
@Override @Override
public boolean isComponentAccessTokenExpired() { public boolean isComponentAccessTokenExpired() {
try (Jedis jedis = jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.ttl(componentAccessTokenKey) < 2; return jedis.ttl(this.componentAccessTokenKey) < 2;
} }
} }
@Override @Override
public void updateComponentAccessTokent(String componentAccessToken, int expiresInSeconds) { public void updateComponentAccessTokent(String componentAccessToken, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(componentAccessTokenKey, expiresInSeconds - 200, componentAccessToken); jedis.setex(this.componentAccessTokenKey, expiresInSeconds - 200, componentAccessToken);
} }
} }
@ -82,99 +80,98 @@ public class WxOpenInRedisConfigStorage extends WxOpenInMemoryConfigStorage {
@Override @Override
public String getAuthorizerRefreshToken(String appId) { public String getAuthorizerRefreshToken(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(getKey(authorizerRefreshTokenKey, appId)); return jedis.get(this.getKey(this.authorizerRefreshTokenKey, appId));
} }
} }
@Override @Override
public void setAuthorizerRefreshToken(String appId, String authorizerRefreshToken) { public void setAuthorizerRefreshToken(String appId, String authorizerRefreshToken) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.set(getKey(authorizerRefreshTokenKey, appId), authorizerRefreshToken); jedis.set(this.getKey(this.authorizerRefreshTokenKey, appId), authorizerRefreshToken);
} }
} }
@Override @Override
public String getAuthorizerAccessToken(String appId) { public String getAuthorizerAccessToken(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(getKey(authorizerAccessTokenKey, appId)); return jedis.get(this.getKey(this.authorizerAccessTokenKey, appId));
} }
} }
@Override @Override
public boolean isAuthorizerAccessTokenExpired(String appId) { public boolean isAuthorizerAccessTokenExpired(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.ttl(getKey(authorizerAccessTokenKey, appId)) < 2; return jedis.ttl(this.getKey(this.authorizerAccessTokenKey, appId)) < 2;
} }
} }
@Override @Override
public void expireAuthorizerAccessToken(String appId) { public void expireAuthorizerAccessToken(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.expire(getKey(authorizerAccessTokenKey, appId), 0); jedis.expire(this.getKey(this.authorizerAccessTokenKey, appId), 0);
} }
} }
@Override @Override
public void updateAuthorizerAccessToken(String appId, String authorizerAccessToken, int expiresInSeconds) { public void updateAuthorizerAccessToken(String appId, String authorizerAccessToken, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(getKey(authorizerAccessTokenKey, appId), expiresInSeconds - 200, authorizerAccessToken); jedis.setex(this.getKey(this.authorizerAccessTokenKey, appId), expiresInSeconds - 200, authorizerAccessToken);
} }
} }
@Override @Override
public String getJsapiTicket(String appId) { public String getJsapiTicket(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(getKey(jsapiTicketKey, appId)); return jedis.get(this.getKey(this.jsapiTicketKey, appId));
} }
} }
@Override @Override
public boolean isJsapiTicketExpired(String appId) { public boolean isJsapiTicketExpired(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.ttl(getKey(jsapiTicketKey, appId)) < 2; return jedis.ttl(this.getKey(this.jsapiTicketKey, appId)) < 2;
} }
} }
@Override @Override
public void expireJsapiTicket(String appId) { public void expireJsapiTicket(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.expire(getKey(jsapiTicketKey, appId), 0); jedis.expire(this.getKey(this.jsapiTicketKey, appId), 0);
} }
} }
@Override @Override
public void updateJsapiTicket(String appId, String jsapiTicket, int expiresInSeconds) { public void updateJsapiTicket(String appId, String jsapiTicket, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(getKey(jsapiTicketKey, appId), expiresInSeconds - 200, jsapiTicket); jedis.setex(this.getKey(this.jsapiTicketKey, appId), expiresInSeconds - 200, jsapiTicket);
} }
} }
@Override @Override
public String getCardApiTicket(String appId) { public String getCardApiTicket(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.get(getKey(jsapiTicketKey, appId)); return jedis.get(this.getKey(this.cardApiTicket, appId));
} }
} }
@Override @Override
public boolean isCardApiTicketExpired(String appId) { public boolean isCardApiTicketExpired(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
return jedis.ttl(getKey(cardApiTicket, appId)) < 2; return jedis.ttl(this.getKey(this.cardApiTicket, appId)) < 2;
} }
} }
@Override @Override
public void expireCardApiTicket(String appId) { public void expireCardApiTicket(String appId) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.expire(getKey(cardApiTicket, appId), 0); jedis.expire(this.getKey(this.cardApiTicket, appId), 0);
} }
} }
@Override @Override
public void updateCardApiTicket(String appId, String cardApiTicket, int expiresInSeconds) { public void updateCardApiTicket(String appId, String cardApiTicket, int expiresInSeconds) {
try (Jedis jedis = this.jedisPool.getResource()) { try (Jedis jedis = this.jedisPool.getResource()) {
jedis.setex(getKey(cardApiTicket, appId), expiresInSeconds - 200, cardApiTicket); jedis.setex(this.getKey(this.cardApiTicket, appId), expiresInSeconds - 200, cardApiTicket);
} }
} }
} }