
1 zagfai Mar 20, 2023 既然小朋友你这么大胆我也不妨直喷了。。。你这代码质量很差,性能很差,没有意义。 建议,先多看,看够 20 万行代码再写 |
2 KoedaNagi Mar 20, 2023 via Android 既然小朋友你这么大胆我也不妨直喷了。。。你这代码质量很差,性能很差,没有意义。 建议,先多看,看够 40 万行代码再写 |
3 auh Mar 20, 2023 既然小朋友你这么大胆我也不妨直喷了。。。你这代码质量很差,性能很差,没有意义。 建议,先多看,看够 400 万行代码再写 |
4 c6h6benzene Mar 20, 2023 好像有 parseInt()? |
5 msg7086 Mar 20, 2023 首先吧,测试用例呢?你不会以为人们愿意在没有测试覆盖的情况下用你的库吧…… |
6 hefish Mar 20, 2023 这个,还是应该鼓励为主的。 |
7 litchinn Mar 20, 2023 原则上应当以鼓励为主 但是首先,java 的 Integer 类有 Integer.toBinaryString 、Integer.toOctalString 、Integer.toHexString 方法,你可以参考下 其次你的代码真的很不 java |
8 tairan2006 Mar 20, 2023 Java 里面流行的是 hutool 、common-lang3 这种大而全的 utils 集合,这种小功能写个库好像 nodejs 中比较流行,小兄弟不妨考虑换个方向。 |
9 selca Mar 20, 2023 至少功能实现了 |
10 Cola98 Mar 20, 2023 public void init(){ result = ""; atmp = ""; btmp = ""; } 这一段初始化的,Java 不是有一个 get/set 方法嘛?还有就是命名规范有点问题,main 方法里面的测试你可以单独弄成测试单元去跑下 |
11 chocotan Mar 20, 2023 第 89 行 result = new StringBuffer(atmp).reverse().toString(); if(btmp.length() > 1) result += "." + btmp; 这 new 了个 StringBuffer 就是为了 reverse ? |
12 CodeCodeStudy Mar 20, 2023 if 后面一定要跟{},不然不利于阅读 |
13 godleon Mar 20, 2023 建议给 x 做个非空 |
14 dqzcwxb Mar 20, 2023 逆耳良言 |
15 leahoop Mar 20, 2023 好奇问一下,用了共享变量怎么没用多线程控制 |
16 Eiden Mar 20, 2023 我还以为 java 没有内置进制转换 |
17 fiveStarLaoliang Mar 20, 2023 话不多说, 加油 |
18 leonard916 Mar 20, 2023 标准库里有的,不要重新去实现。除非你有更好的算法或数据结构。 当然你作为学习项目,怎么写都可以。作为库就没必要了。 |
19 yaphets666 Mar 20, 2023 其实很好,哪怕自己写的代码不好,发出来就有大佬斧正了,自己就提高了 |
20 sleepyfevniv Mar 20, 2023 既然别人都批评过了,那么我就来按照我公司的规范来讲讲吧。 首先,Transfer 类我可是第一次见还能 import static 的方式(毕竟我不这么用),一般来说都是 import java.lang.Math; 然后 Math.pow() 使用。 第二点,每个方法都不是 static ,意味着每次都需要 new 之后才能使用。 第三,for if 等这种方法,都需要用大括号括起来,不要因为一行语句就不用大括号了,不要吝啬空行。 第四,没有异常处理。 第五,没有注释。 只说了一点点,剩下的交给其他人来补充吧。建议看看别人的一些源码学习学习。 |
21 BigMikey OP 谢谢各位的斧正,本着学习的心态来讨教的哈哈,谢谢啦! |
22 BigMikey OP 喷没有问题哈,实话说我也不介意被喷,主要吧还是想学技术。 |
23 BigMikey OP 只要不是恶意的都没有问题~ |
24 susu200213 Apr 8, 2023 @BigMikey 加油 |