Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alert.Builder中的方法名不应以大写开头 #17

Closed
superinkfish opened this issue Aug 17, 2021 · 1 comment
Closed

Alert.Builder中的方法名不应以大写开头 #17

superinkfish opened this issue Aug 17, 2021 · 1 comment

Comments

@superinkfish
Copy link

Java中通常方法名是以小写开头的,在Alert.Builder类中,存在两个方法以大写开头,建议修改。
原始代码:

        public Builder AddAllLocArgs(List<String> locArgs) {
            this.locArgs.addAll(locArgs);
            return this;
        }

        public Builder AddLocArg(String locArg) {
            this.locArgs.add(locArg);
            return this;
        }

期望修改后:

        public Builder addAllLocArgs(List<String> locArgs) {
            this.locArgs.addAll(locArgs);
            return this;
        }

        public Builder addLocArg(String locArg) {
            this.locArgs.add(locArg);
            return this;
        }
@Mike-mei
Copy link
Contributor

Mike-mei commented Oct 8, 2021

这个问题新版本中已经解决,参考 5635d3e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants