从文本文件中查找最大值

Question

我一直在努力提高我的 Java 技能，但仍然遇到一些问题。对于这个程序，除了我的最终输出之外，一切似乎都运行正常。本质上，一旦它读入 numbers.txt，它应该将值放入一个数组中，然后打印出最大的数字。但是，我的逻辑一定有缺陷，因为目前它只输出列表中的顶部数字，而不是找到最大的数字。这是我的程序，我在我认为我的逻辑有缺陷的地方发表了评论。

import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;

public class ProcessDataFile {

    public static void main(String[] args) throws IOException {

        String fileName = "numbers.txt";
        String temp;
        int max = Integer.MIN_VALUE;
        int i = 0;
        int[] numbers = new int[100000];

        try (BufferedReader br = new BufferedReader(new FileReader(fileName))) {
            while ((temp = br.readLine()) != null) {
                if (temp.isEmpty())
                    break;
                numbers[i++] = Integer.parseInt(temp);
            }
        }
        // I think my logic error is somewhere in this section. 
        for (i = 0; i < numbers.length; i++)
            if (max < numbers[i])
                max = numbers[i];



        System.out.println("The largest number is: " + max);
    }
}

Answer 1

是的，有一个错误，但不是你的逻辑，以前缺少for 括号，所以for 之后的第一行就在迭代中。用正确的标识查看您的代码

// I think my logic error is somewhere in this section. 
for (i = 0; i < numbers.length; i++)
    if (max < numbers[i])

max = numbers[i]; // this line is out of the for scope!!!

只需添加{ 和} 即可定义循环范围

// I think my logic error is somewhere in this section. 
for (i = 0; i < numbers.length; i++){
    if (max < numbers[i])
        max = numbers[i];
}

Answer 2

问题出在您的 numbers.txt 文件中。将取决于您所在的平台以及预期的行尾，但我相信代码有效。

我在 Mac 上运行此代码，在 IntelliJ 中使用包含随机数的 txt 文件。它必须触发 temp.isEmpty() 条件并在读取第一项后退出。

检查文件，尤其是行尾。

另外，如果您无法在调试模式下单步执行代码，请尝试添加一些 System.out.println 语句以澄清发生了什么。

此代码对我有用。