V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
godleon
V2EX  ›  程序员

Java 语言代码实现一个最优写法。

  •  
  •   godleon ·
    aokleon · 134 天前 · 3943 次点击
    这是一个创建于 134 天前的主题,其中的信息可能已经有所发展或是发生改变。

    需求

    JAVA 语言, 提供一个静态方法接收动态 Integer 参数,把接收到所有 Integer 不为 Null 的相加 返回总和。

    支持 stream ,需要一个最优的写法! 可以评论一下,看看大佬们的代码功底!让我这个菜鸟采纳一下~

    50 条回复    2022-04-01 11:28:50 +08:00
    Leviathann
        1
    Leviathann  
       134 天前
    动态 Integer 参数
    指的是 vararg ?
    dcsuibian
        2
    dcsuibian  
       134 天前
    reduce
    godleon
        3
    godleon  
    OP
       134 天前
    @Leviathann 就是调一次这个方法可以传入 N 个 Integer
    BiChengfei
        4
    BiChengfei  
       134 天前
    ```
    @Test
    public void test022() {
    System.out.println(count(1, 2, 3, 4, 5));
    }

    public int count(int... input) {
    return Arrays.stream(input).sum();
    }
    ```
    这个可以吗
    BiChengfei
        5
    BiChengfei  
       134 天前
    静态就自己加个 static
    privatetan
        6
    privatetan  
       134 天前
    你是来白嫖代码的吧
    godleon
        7
    godleon  
    OP
       134 天前
    @BiChengfei 差不多了 ,你这个差一个非空效验,可以写

    return Arrays.stream(input).mapToInt(n -> (n != null ? n : 0)).sum();

    这个应该算挺优了
    micean
        8
    micean  
       134 天前
    少了并行计算
    BiChengfei
        9
    BiChengfei  
       134 天前
    @godleon 入参用的 int ,不可能为 null
    godleon
        10
    godleon  
    OP
       134 天前
    @BiChengfei 需求就是 Integer
    gitdoit
        11
    gitdoit  
       134 天前
    public static void main(String[] args) {
    System.out.println(sum(1,2,4,null,3,5));
    }
    public static int sum(Integer ... number) {
    return Stream.of(number).filter(Objects::nonNull).reduce(0, Integer::sum);
    }
    BiChengfei
        12
    BiChengfei  
       134 天前
    @godleon 基本类型自动拆箱,方法入参你可以写成 Integer
    count(new Integer(1), new Integer(2), new Integer(3))
    godleon
        13
    godleon  
    OP
       134 天前
    @gitdoit 老哥稳
    Cbdy
        14
    Cbdy  
       134 天前
    public static int sumInt(@Nullable Integer @NotNull ... ints) {
    var sum = 0;
    for (var it : ints) {
    if (it != null) {
    sum += it;
    }
    }
    return sum;
    }
    anzu
        15
    anzu  
       134 天前
    如果只有 null ,我是说 Integer[] arr = null; 的情况
    所有上来就用 stream 的必报错,让你们优雅,哈哈哈
    oneisall8955
        16
    oneisall8955  
       134 天前 via Android
    @anzu 那为什么不反思下 null 时候要调呢,接口定义可以协定不处理 null 的。
    这到底是调用者还是被调用者的问题?
    Leviathann
        17
    Leviathann  
       134 天前
    @anzu 什么意思,vararg 哪来的 null?
    Saxton
        18
    Saxton  
       134 天前
    方法几百种,没有什么最优的,只有最适合你的自己的,可能你写的这个别人认为这里不行,别人写的你认为那里不行,每个人有每个人的写作风格。
    FrankHB
        19
    FrankHB  
       134 天前
    ?现在 Java 普及了 Babel 类似物了么?不卡源语言版本的?
    (要能用 transpiler 一般也是全体 JVM 语言,用不着特指 Java 。)
    yazinnnn
        20
    yazinnnn  
       134 天前
    fun sum(vararg array: Int?): Int {
    return array.filterNotNull().fold(0) { acc, i -> acc + i }
    }
    molicloud
        21
    molicloud  
       134 天前   ❤️ 1
    accuracy 的值越大,精确度越高

    public static long sum(Integer ... numbers) throws InterruptedException {
    long accuracy = 1000L;
    long startTime = System.currentTimeMillis();
    for (Integer number : numbers) {
    Thread.sleep(number * accuracy);
    }
    return (System.currentTimeMillis() - startTime) / accuracy;
    }
    yazinnnn
        22
    yazinnnn  
       134 天前
    kotlin 对 Int 的集合做了优化,直接这样就行

    fun sum(vararg array: Int?): Int {
    return array.filterNotNull().sum()
    }
    lululau
        23
    lululau  
       134 天前
    IntStream.of(1, 2, 3).filter(Objects::nonNull).sum()
    yazinnnn
        24
    yazinnnn  
       134 天前
    fun sum(vararg array: Int?): Int {
    return array.sumOf { it?:0 }
    }
    redorblacck886
        25
    redorblacck886  
       134 天前
    List<Integer> objects = Lists.newArrayList();
    objects.stream().filter(Objects::nonNull).mapToInt(Integer::intValue).sum();
    zzh7982
        26
    zzh7982  
       134 天前
    private static int sum(Integer... i){
    return Arrays.stream(i).filter(Objects::nonNull).mapToInt(x -> x).sum();
    }
    Cbdy
        27
    Cbdy  
       134 天前
    @Leviathann 你好,有可能为 null 的
    hepin1989
        28
    hepin1989  
       134 天前
    for 循环效率最高
    hepin1989
        29
    hepin1989  
       134 天前
    import java.util.Objects._
    def sum(ints:Int *):Int = ints.filter(nonNull).sum

    sum(1,2,3,4)
    hepin1989
        30
    hepin1989  
       134 天前
    hepin1989
        31
    hepin1989  
       134 天前
    def sum(ints:Int *):Int =
    (for(intValue <- ints if intValue ne null) yield intValue).sum
    pengtdyd
        32
    pengtdyd  
       134 天前
    我记得阿里巴巴 java 开发手册里面有这么一句话,调用者应该负责参数的校验,谁调用谁负责,所以有 null 应该由调用者处理吧
    hepin1989
        33
    hepin1989  
       134 天前
    def sum(ints:Integer *):Int =
    ints.foldLeft(0)((a, b) => if (b eq null) a else a + b)
    Leviathann
        34
    Leviathann  
       134 天前
    @Cbdy void func(Integer ...args) { System.out.println(args.toString()); }

    直接 func() 调用都没问题,args 不可能为 null ,怎么导致 stream 报错

    如果是直接传一个 null 参数,那在调用的地方就爆了和函数里面用什么方法处理也没关系
    hepin1989
        35
    hepin1989  
       134 天前
    public static BigInteger sum(Integer ... ints) {
    BigInteger sum = BigInteger.ZERO;
    for (Integer anInt : ints) {
    if (anInt != null) {
    sum = sum.add(BigInteger.valueOf(anInt));
    }
    }
    return sum;
    }
    anzu
        36
    anzu  
       134 天前
    @Leviathann 你确定?这里输出 args is null 后报错

    public class MyClass {
    public static void main(String args[]) {
    Integer[] arr = null;
    func(arr);
    }
    static void func(Integer ...args) {
    if (args == null) {
    System.out.println("args is null");
    }
    System.out.println(args.toString());
    }
    }
    Leviathann
        37
    Leviathann  
       134 天前
    @anzu 的确 我看走眼了,回去翻了下 jshell 运行记录,由于函数体偷懒没写,而调用时那一长串文字原来是警告不是异常,
    再次说明 2022 年还在使用一门连空安全都做不到的语言实在是浪费生命
    Cbdy
        38
    Cbdy  
       134 天前
    @Leviathann 36L 给出了可能为 null 的情况,我在 14L 给给出了我的答案,你可以注意一下到我加的两个注解
    lawler
        39
    lawler  
       134 天前   ❤️ 1
    至此,没有我认同的最佳答案,相反都是最差的回答。最优不等于语法糖。
    lawler
        40
    lawler  
       134 天前   ❤️ 1
    public static int sum(Integer num, Integer... nums) {
    int result = num == null ? 0 : num;
    for (int i = 0; i < (nums == null ? 0 : nums.length); i++) {
    result += sum(nums[i]);
    }
    return result;
    }
    sumulige
        41
    sumulige  
       134 天前
    @privatetan 人家问个问题就白嫖了?
    douyacai911
        42
    douyacai911  
       134 天前 via iPhone
    代码不会写,但 采纳 这词不是这样用的吧
    resol341
        43
    resol341  
       134 天前
    @douyacai911 你不要不识抬举
    arvinsilm
        44
    arvinsilm  
       134 天前
    你们能不能加个 Markdown 代码格式
    aguesuka
        45
    aguesuka  
       134 天前
    mapToInt(x -> x).sum() 比 reduce(0, Integer::sum) 好
    mapToInt(Integer::intValue) 比 mapToInt(x -> x) 好
    filter(Objects::nonNull) 比 n != null ? n : 0 好
    sum(Integer ... numbers) 比 sum(Integer num, Integer... nums) 好, 因为过滤 null 后的加法是阿贝尔群, 允许无参返回单位元很合理
    不考虑参数数组为 null 比数组为 null 返回 0 好
    循环比一定比 stream 好, 因为 stream 可以方便地使用多线程
    aguesuka
        46
    aguesuka  
       134 天前
    @aguesuka "循环比一定比 stream 好" -> "循环不一定比 stream 好"
    Bigglesworth
        47
    Bigglesworth  
       134 天前
    @douyacai911 #42 应该笑纳😺
    intmax2147483647
        48
    intmax2147483647  
       134 天前
    什么牛鬼蛇神的评论区
    godleon
        49
    godleon  
    OP
       133 天前
    @Saxton 每个人有每个人的风格不错,但是在绝对的实力面前你的风格显得黯然失色!
    godleon
        50
    godleon  
    OP
       133 天前
    @Saxton 我觉得代码的风格应该是一些样式问题,不应该是程序的代码的设计问题,确保程序的流畅运行,有时候你不得不放弃你的风格 选择可以达到效率最高的代码
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2443 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 40ms · UTC 03:24 · PVG 11:24 · LAX 20:24 · JFK 23:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.