小白自学，大家帮忙看看这个程序是否正确？

[weiyuzhou]

门外汉自学入门，刚刚看到 if 分支语句，做完了书上的习题输入三个数并输出最大数的习题后我就在想能不能更难一点，输入四个数试试?然后我就瞬间感觉脑子不够用了，反反复复写了近半个小时写出了下面的代码，测试了一下好像也没有什么问题，但是总感觉怪怪的...

想请教下大家这段代码对不对或者是否有任何需要改进的地方？ 我知道肯定有更简单的算法，但是鉴于别的我也看不懂，所以就限定是用判断语句并且不考虑特殊情况。 代码如下：

import java.util.Scanner;

public class Largest {
    public static void main(String[] args) {
        Scanner input = new Scanner(System.in);
        System.out.print("Please enter four numbers: ");
        int number1 = input.nextInt();
        int number2 = input.nextInt();
        int number3 = input.nextInt();
        int number4 = input.nextInt();
        int largest = 0;

        if (number1 > number2) {
            if (number1 > number3) {
                if (number1 > number4) {
                    largest = number1;
                }else {
                    largest = number4;
                }
            }else {
                if (number3 > number4) {
                    largest = number3;
                }else {
                    largest = number4;
                }
            }
        }else if (number2 > number3) {
            if (number2 > number4) {
                largest = number2;
            }else {
                largest = number4;
            }
        }else {
            if (number3 > number4) {
                largest = number3;
            }else {
                largest = number4;
            }
        }
        System.out.println(largest);
    }
}

[weiyuzhou]

忘了说，学的是 java 语言。

[takanasi]

>>> a=[1,2,3,4]
>>> max(a)
4

人生苦短，快用 python

[weiyuzhou]

@takanasi 这是 Python 数组吗?看不懂啊…为什么都没人回复我… 😭

[yejinmo]

List<int> list= new List<int>(){1, 2, 3, 4};
list.Sort();
int res = list[list.Count - 1];

人生苦短，试试 C#

[mjikop1231]

明明 java 也有 sort
（不过 java 代码好长啊好长啊好长啊）

[shengkunz]

应该先排序，然后在输出最大的那个。

[misaka19000]

2 楼有点搞笑了，这种东西用库写谁不会写啊

楼主写的有点麻烦了，简单点的话可以这样：

int[] array = new int[]{number1, number2, number3, number4};
for (int i : array) {
if (i > largest)
largest = i;
}
System.out.println(largest);

[hcnhcn012]

>>> print(max([i for i in range(1,5)]))
4

[hcnhcn012]

人生苦短，快用 python

[qiayue]

说个你比较能理解的，你先三个数比较，找出最大数，然后跟第四个数比较

[nyanyh]

List<Integer> nums = new ArrayList<>();
nums.add(10);
nums.add(20);
nums.add(45);
nums.add(50);
nums.add(51);

int max = Collections.max(nums); // 51

[nyanyh]

算法第四版 Java 实现的一开始就介绍了 Comparable 如何使用，所以我认为使用 Collections 并无任何不妥

[weiyuzhou]

@qiayue 这个靠谱，能理解！也谢谢大家，感觉脑子不够用，稍微难点的就不知道怎么算了，不知道合不合适学编程…

[weiyuzhou]

@misaka19000 这个暂时还没学过…谢谢！

[AlwaysBehave]

@nyanyh +1
看自学主要方向吧
算法的话直接看底层怎么实现的（数据结构是前提？），应用的话就直接学会快速读懂手册有流程意识？（有拆分问题）

[LxExExl]

楼上排序和用数组的都是二逼 答非所问

if(a>b && a>c && a>d) largest=a;

同理 写其他三行就行了

[ipwx]

@LxExExl 用 >= 更好。

[misaka19000]

@LxExExl
@Livid
16 楼人身攻击

[weiyuzhou]

@LxExExl 与运算，这个我懂，怎么没想到，多谢！

[JamesMackerel]

不需要这么复杂啊。
用数组存起来，假设是 num，然后你就：

largest = num[0];
for(int t: num){
if(largest > t) largest = t;
}

楼上说的那些先排序再取最大的，考虑正常的排序算法，它们的时间复杂度最优也要 nlogn，而这个方法是 n。

[acgnsstech]

@takanasi
@yejinmo

请看我大 C 艹版本

int a[4] = { 4,2,3,1 };
cout<<*max_element(a, a+4)<<endl;

[thet]

可以搜搜排序算法看看

[x7395759]

你们能不能专心解决问题！！！

其实编码是一个表现思路的过程，一个个比较还是排序还是冒泡啥的，都是思路。

不必担心代码的难易，前期要专注于思维的培养。

[sagaxu]

IntStream.range(0, 4).map(i -> input.nextInt()).max().ifPresent(System.out::println);

[zjp]

同意 #23 楼
楼上很多方法都能解决问题（用其他语言的过分了啊 。
#7 #2
仅仅求最大值，对所有数字排序明显多余。对简单的问题用不简洁的做法多半是思维跑偏了

[zjp]

@zjp 手滑
同意 #23 楼
楼上很多方法都能解决问题（安利其他语言的过分了啊 。
#7 #20 用数组
#11 用集合排序
#24 用 Java8 的流
仅仅求最大值，对所有数字排序明显多余。对简单的问题用不简洁的做法多半是思维跑偏了

[zjp]

@zjp 23333 说那么多原来只能用 if 语句，逗比了🌚

[cstome]

歪个楼；
我觉得入门写代码的话其实可以从简单的语言，比如 python 甚至 JS 入门，等你熟悉基本语法后，再考虑学习算法。

这样还有个好处，不如当你遇到某些语言没有 sort 时，你也会想着怎么构造个 sort 方法，而不是这样写判断。
前期如果为了实现某个功能，用一堆 if else for 来实现，我觉得是没有意义的。

[cstome]

@cstome PS 我觉得从主流语言的语法的角度，我更建议从 JS 入门编程。

[weiyuzhou]

@cstome 谢谢你的建议，我会参考。先试试看 Java 学不下去就转战 python。

[weiyuzhou]

经过大家的提点我已经我已经重写了代码，虽然仍然很笨拙，但是至少代码结构更加清晰了。不管怎么刚刚入门嘛也是一种解决方法。
代码现在是这样子：
```java
import java.util.Scanner;

public class MaxNumber {
public static void main(String[] args) {
Scanner input = new Scanner(System.in);
System.out.print("Please enter four numbers: ");
int number1 = input.nextInt();
int number2 = input.nextInt();
int number3 = input.nextInt();
int number4 = input.nextInt();
int max = 0;
if (number1 > number2 && number1 > number3 && number1 > number4) {
max = number1;
}else if (number2 > number1 && number2 > number3 && number2 > number4) {
max = number2;
}else if (number3 > number1 && number3 > number2 && number3 > number4) {
max = number3;
}else if (number4 > number1 && number4 > number2 && number4 > number3) {
max = number4;
}else {
System.out.print("有特殊情况！");
}
System.out.print(max);
}
}
```

[mario85]

其实哪用那么麻烦，既然你有四个输入变量、一个用来存最大的变量和 if，那直接冒泡出来就好了：
/*前面输入 blah blah blah*/
largest=number1;
if(number2>largest) largest=number2;
if(number3>largest) largest=number3;
if(number4>largest) largest=number4;
//然后就可以
System.out.print(largest); //了

[weiyuzhou]

@mario85 厉害啦...表示智商很受伤 /cry

[mario85]

@weiyuzhou 其实楼上的兄弟已经写出来啦，我只是考虑到只学到 if 能看懂，把只用 if 的写法写出来而已

[qwertyegg]

如果我写的话我会用 Math.max(Math.max(Math.max(number4, number3), number2), number1);

或者拆一下 Math.max(Math.max(number4, number3), Math.max(number2, number1));

[seognilV2]

只要对比 N-1 次 小的丢弃（也就是大的存到一个变量里向后传递） 就行了
就像是冒泡排序（的一部分）

[wxyz]

@qwertyegg 很别致

[yongjing]

论简洁，自然是我大 JS Math.max(num1,num2,num3,num4)

[oaix]

java 没有那么复杂
IntStream.of(1, 2, 3, 4).max();

[oaix]

@oaix 当然你要先升级到 java8...

[leemove]

人生太长我用 js.
```js
Math.max(num1,num2,num3,num4)
```

[vincenth520]

$arr = [1,2,3,4];
echo max($arr);
人生苦短，php 也不错

[meepo3927]

if 的用法一般用于控制逻辑分支。

找出最大值适合用 for 循环来做。

这个练习就不用深究了噻

[mosliu]

定义
public int bigger(int in1,int in2){
return in1>in2?in1:in2
}

然后调用
System.out.println(bigger(bigger(num1,num2),bigger(num3,num4)));

[Makcy]

[2,1,3].sort[-1] ruby 也不错 :)

[zgx030030]

就是冒一次泡，循环判断一次。

[denano]

lz 你也说了做完了 3 个数里找最大值，想试试从 4 个数里找最大值。那再进一步，是不是就是从 n 个数里找最大值呢？如果沿用你现在的思路，需要写多少行代码，穷举多少种情况才能完成？

[qiuyk]

@denano 然后楼主成功掌握手动展开决策树的精髓 233333

[lneoi]

运行正确就正确了呗。语法还没看完就继续看语法，不碍事。
然后回头再来改成 N 个数找最大值

[fortunezhang]

```
def maxNum(a,b):
return a > b ? a: b

max = maxNum(maxNum(number1,number2),maxNum(number3,number4))
```

[weiyuzhou]

想不到我的帖子会这么火！惭愧，真应该早点学习编程～感谢大家！

[ppwangs]

冒泡，二分。

[ThinkCat]

多看看 java8，可以这样：

int max = Stream.of(number1,number2,number3,number4).reduce((a, b) -> a>b?a:b).get();

[poorcai]

顺带学习了