繁体   English   中英

java - 读取输入线并输出重复

[英]java - reading input lines and outputting duplicates

因此,我正在学习阅读文本文件等,并且我正在尝试制作一个程序,该程序一次读取一行并输出当前行,当且仅当它小于迄今为止读取的任何其他行时。 较小的是相对于字符串的通常顺序,由 String.compareTo() 定义。

当我尝试运行我的代码时,我收到错误“列表无法解析为类型”和“ArrayList 无法解析为类型”。 我很困惑为什么在我的程序中出现这个错误,并且想知道是否还有其他我应该使用的东西?

package comp;

import java.io.BufferedReader;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.PrintWriter;
import java.util.HashSet;

public class Part2 {

    public static void doIt(BufferedReader r, PrintWriter w) throws IOException {
        List<String> allStrings = new ArrayList<>();

        String line;
        String shortest = allStrings.get(0);
        while((line = r.readLine()) != null) {
            for(String s: allStrings) {
                if(s.length()>shortest.length()) {
                    shortest = s;
                    line = shortest;
                }
            }
            allStrings.add(line);

            for (String text: allStrings) {
                w.println(text);
            }
        }
    }

    public static void main(String[] args) {
        try {
            BufferedReader r;
            PrintWriter w;
            if (args.length == 0) {
                r = new BufferedReader(new InputStreamReader(System.in));
                w = new PrintWriter(System.out);
            } else if (args.length == 1) {
                r = new BufferedReader(new FileReader(args[0]));
                w = new PrintWriter(System.out);                
            } else {
                r = new BufferedReader(new FileReader(args[0]));
                w = new PrintWriter(new FileWriter(args[1]));
            }
            long start = System.nanoTime();
            doIt(r, w);
            w.flush();
            long stop = System.nanoTime();
            System.out.println("Execution time: " + 10e-9 * (stop-start));
        } catch (IOException e) {
            System.err.println(e);
            System.exit(-1);
        }
    }
}

您可以只使用 integer 作为最小长度:

public class Part2 {

    public static void doIt(BufferedReader r, PrintWriter w) throws IOException {
        List<String> allStrings = new ArrayList<String>();
        String line;
        int minLength = 0;
        while ((line = r.readLine()) != null) {
            if (minLength == 0 || minLength > line.length()) {
                allStrings.add(line);
                minLength = line.length();
            }
        }

        for (String text : allStrings) {
            w.println(text);
        }
    }

    public static void main(String[] args) {
        try {
            BufferedReader r;
            PrintWriter w;
            if (args.length == 0) {
                r = new BufferedReader(new InputStreamReader(System.in));
                w = new PrintWriter(System.out);
            } else if (args.length == 1) {
                r = new BufferedReader(new FileReader(args[0]));
                w = new PrintWriter(System.out);
            } else {
                r = new BufferedReader(new FileReader(args[0]));
                w = new PrintWriter(new FileWriter(args[1]));
            }
            long start = System.nanoTime();
            doIt(r, w);
            w.flush();
            long stop = System.nanoTime();
            System.out.println("Execution time: " + 10e-9 * (stop - start));
        } catch (IOException e) {
            System.err.println(e);
            System.exit(-1);
        }
    }
}

输入文件

1111111
222222222
3333333333
444

Output

1111111
444

您的代码也有以下错误:

    List<String> allStrings = new ArrayList<>();

 // It throws IndexOutOfBoundsException!!!
    String shortest = allStrings.get(0);

那么这里有几个问题。 如果你运行你的代码,你会在doIt()中得到一个IndexOutOfBoundException因为你的allStrings列表是空的,因此当你调用String shortest = allStrings.get(0); 你打电话给不在你名单上的东西。

此外,逻辑if(s.length()>shortest.length()))有点奇怪,因为本质上您实际上只是比较列表中的字符串,而不是与您从文件中读取的内容进行比较。 因此,您永远无法确定您从文件中读取的行是否小于您已有的任何行。

最后,这段代码... for (String text: allStrings) {w.println(text); } for (String text: allStrings) {w.println(text); }写你的文件在循环中,因此你在每次循环迭代时都写字符串。

您不需要最短变量,只需使用 boolean 来指示您读入的字符串是否比列表中已有的字符串短,如果是,则将其添加到列表中。

public static void doIt(BufferedReader r, PrintWriter w) throws IOException {
    List<String> shortestStrings = new ArrayList<>();

    String line = r.readLine();
    shortestStrings.add(line);

    while(line != null) {
        boolean isShortert = Boolean.TRUE;
        for(String s: shortestStrings) {
            if(line.length() > s.length()) {
                isShortert = Boolean.FALSE;
            }
        }
        if(isShortert) {
            shortestStrings.add(line);
        }
        line = r.readLine();
    }

    for (String text: shortestStrings) {
        w.println(text);
    }
}

暂无
暂无

声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM