xinge-api-java icon indicating copy to clipboard operation
xinge-api-java copied to clipboard

PushAppRequest的toString方法有多余的对象开销

Open perlmonk opened this issue 6 years ago • 3 comments

Jsckson的ObjectMapper,全局一份就够了

    public String toString() {
        ObjectMapper mapper = new ObjectMapper();
        String repoStr = null;

        try {
            repoStr = mapper.writeValueAsString(this);
        } catch (Exception var4) {
            ;
        }

        return repoStr;
    }

另外,提供个捷径: http api用Retrofit来实现更简洁

perlmonk avatar Aug 10 '18 04:08 perlmonk

@perlmonk 感谢! 我来fix 下。如果有发现bug,如果您有时间的话,也欢迎提供merge request哈

caofangkun avatar Aug 14 '18 06:08 caofangkun

@perlmonk 关于http api用Retrofit来实现更简洁 当前后台API服务使用的是 Spring MVC + Swagger, 暂时不会更换。

caofangkun avatar Aug 14 '18 07:08 caofangkun

@caofangkun 谢谢回复

我们的一直在使用1.1.8版本SDK,最近一两周推送接口的响应时间变慢,推送经常出现积压,准备升级v3接口,换用1.2.0时发现接口有breaking change。因为内部只需要使用少量子集,暂时自行封装API解决了。

perlmonk avatar Aug 14 '18 07:08 perlmonk