java - 在数组末尾打印字符串

Question

抱歉；我不知道如何命名这个问题，所以这个问题有点粗制滥造。

我正在完成一个小程序以获得额外的学分作业，有两个非常小的细节毁了它。

该程序的目的是读取一些数据，将其分为客户编号和航类号，然后确定该航类是否合适。我有文件 here 的图像屏幕截图，因为简短详细地解释起来有点麻烦。抱歉，不得不发送链接。

这是我的代码:

import java.io.File;
import java.io.FileNotFoundException;
import java.util.Scanner;

public class prog475a {
    static Scanner inFile = null;

    public static void main(String[] args) {
        // make the flight plan array
        boolean[][] flightPlan = new boolean[7][7];

        // fill the true
        flightPlan[1][2]=true;
        flightPlan[1][3]=true;
        flightPlan[1][6]=true;
        flightPlan[2][1]=true;
        flightPlan[2][3]=true;
        flightPlan[2][6]=true;
        flightPlan[3][4]=true;
        flightPlan[4][2]=true;
        flightPlan[4][5]=true;
        flightPlan[4][6]=true;
        flightPlan[5][2]=true;
        flightPlan[5][4]=true;
        flightPlan[6][2]=true;
        flightPlan[6][5]=true;

        // read for file
        try {
            // create scanner to read file
            inFile = new Scanner(new File ("prog475a.dat"));
        } catch (FileNotFoundException e) {
            System.out.println("File not found!");
            System.exit(0);
        }

        /*
         * read customer number
         * method to translate flight plan into coordinates
         * 
         */

        int customer = 0;
        int flight = 0;

        while (inFile.hasNext()) {
            customer = inFile.nextInt();
            flight = inFile.nextInt();
            translator(customer, flight, flightPlan);
        }
    }

    public static void translator(int c, int f, boolean[][] fl) {
        System.out.println("Customer Number " + c + "\tFlight Plan " + f);
        // change int f into individual numbers?
        int[] coo = new int[6];

        int a = 10000;
        // boolean ok = true; 

        for (int x = 0; x < coo.length - 1; x ++) {
            coo[x] = (f / a) % 10;
            a /= 10;    
        }

        // test if your array has all the right numbers

//      for (int x = 0; x < coo.length - 1; x++) {
//          
//          System.out.println(coo[x]);
//          
//      }

        // instantiate variables to act as parameters to navigate boolean

        int n = 0;  // the actual coordinate
        int p = 0;  // placeholders
        int q = 0;

        while (q < coo.length) {    // q has to end when it equals length of coo
            p = coo[n]; 
            q = coo[n + 1];

            if (fl[p][q]) {
                System.out.println(p + "\t" + q + "\t" + "Available");
                n++;
            } else {        
                System.out.println(p + "\t" + q + "\t" + "Unavailable\t Flight Plan invalid");
                break;  // if unavailable, break
            }
        }

        System.out.println("");
    }
}

这是数据:

10123 13426
11305 62000
13427 42320
18211 34212
19006 65426
20831 52500
21475 32000
22138 13424
24105 65231
24216 34250
25009 43621

我遇到的问题非常简单——我的程序运行得很好，只是我尝试了各种不同的方法在程序末尾打印“飞行计划有效”，但没有成功。我已经诚实地做了一切；我添加了一个额外的 if/else 语句来解释 q = coo[coo.length - 1] 的情况，并尝试将其附加到 while 循环的外部。我就是无法让它发挥作用。

我遇到的另一个问题是，对于航类 34212，任意打印出一行额外的行，表示从 2 到 0 的航类，而该航类号中根本不存在该行。这是唯一出现此问题的航类。

我真的不确定我做错了什么，我希望有人可以就如何修复我的代码提供一些建议，以确保这些错误得到某种程度的处理。再次感谢您抽出时间。我有点笨，所以我花了很长时间试图弄清楚我做错了什么，我感到有点沮丧。

Answer 1

好吧，translator 函数中有很多逻辑错误，所以我只列出它们，然后提供一些修复的代码。然而，由于这听起来像是针对类(class)的，所以不要只是复制和粘贴。我认为实际解决程序中的一些错误将非常有益。

1. coo 的长度为 6。这可能是有意为之，但没有必要。
2. 当您填写coo时。您只填满 6 个位置中的 5 个。
3. 您永远不会更改 ok 变量。
4. 您从不使用n变量。
5. 我认为在 while 循环的第一部分中，与 q 进行比较是一个拼写错误/逻辑错误，我认为您的意思是与 n< 进行比较。
6. 在 while 循环中使用检查 ok 和 break 语句是多余的。
7. 您没有添加 if 来根据 ok 的状态运行代码。

有了所有这些错误，我认为你绝对应该向你的老师或其他人寻求帮助，因为在一篇文章中需要改变很多。不管怎样，我希望你不要滥用这段代码；)。这是新的 translate 函数:

public static void translator(int c, int f, boolean[][] fl) {
    System.out.println("Customer Number " + c + "    Flight Plan " + f);

    int[] coo = new int[5];
    int a = 10000;

    for (int x = 0; x < coo.length; x++) {
        coo[x] = (f / a) % 10;
        a /= 10;    
    }

    for (int x = 0; x < coo.length - 1; x++) {
        int p = coo[x]; 
        int q = coo[x + 1];

        if (fl[p][q]) {
            System.out.print(p + "\t" + q + "\tAvailable");

            if (x == coo.length - 2) {
                System.out.print("\tFlight Plan is Valid");
            }

            System.out.println();
        }
        else {
            System.out.println(p + "\t" + q + "\tUnavailable\tFlight Plan is Invalid");
            break;
        }
    }
}

说明:在最后一个 for 循环之前，我使用了基本相同的代码，只是将 coo 的长度更改为 5 并制作了第一个 for循环运行5次。您还会注意到我删除了 ok ，因为它是多余的。这是因为一旦我有一个糟糕的航类，我只是打印无效的航类消息然后中断。那里不需要变量。对于您可以看到的最后一个循环，我使用 for 循环而不是 while 循环。我为什么要这样做？几乎只是品味的原因。当我知道要循环多少次时，我通常使用 for 循环；当数字不明确时，我通常使用 while 。另外，在本例中，由于您无论如何都使用了计数变量 n，所以这正是 for 循环的优点(哈哈)。代码的其余部分非常不言自明。如果航类有效，您可以打印该航类。如果这是最后一个循环，因此 if (x == coo.length - 2) (这会检查它是否是最后一个循环)，那么您将打印飞行计划是否有效。否则你打印它是无效的并且你会破坏。我已经用这段代码做了一些测试，到目前为止它是有效的。我希望这可以帮助你!