0

我已经对以下程序进行了编码以递归地实现四色映射定理（简而言之，任何映射都只能用 4 种颜色着色，而没有任何相邻区域的颜色相同）。一切都可以编译，但我的输出给了我错误的数据（每个区域的颜色为-1，而不是现在的值 0-3）。我最大的问题是为什么输出不正确。

对于那些想知道的人，输入文件是一个邻接矩阵，如下所示：

0 1 0 1  
1 0 1 0  
0 1 0 1  
1 0 1 0

这是我的代码：

public class FourColorMapThm 
{
    public static boolean acceptable(int[] map, int[][] matrix, int r, int c)
    {
        for(int i = 0; i < map.length; i++)
        {
            if(matrix[r][i] == 1)
            {
                if(map[i] == c) return false;
            }
        }
        return true;
    }

    public static boolean colorTheMap(int[] map, int[][] matrix, int r, int c)
    {
        boolean q = false;
        int i = 0;

        if(!acceptable(map, matrix, r, i)) return false;

        map[r] = i;
        boolean done = true;

        for(int j = 0; j < map.length; j++)
        {
            if(map[j] == -1)
            {
                done = false;
            }
        }
        if(done) return true;

        do
        {
            i++;
            q = colorTheMap(map, matrix, r+1, i);
            if(q) return true;
        }
        while(i <= 3);

        map[r] = -1;
        return false;
    }

    public static void main(String[] args) throws IOException
    {
        Scanner in = new Scanner(System.in);
        String snumRegions, fileName;
        int numRegions;

        System.out.print("Enter the number of regions: ");
        snumRegions = in.nextLine();
        numRegions = Integer.parseInt(snumRegions);

        System.out.print("\nEnter filename for adjacency matrix: ");
        fileName = in.nextLine();
        in.close();

        Scanner inFile = new Scanner(new FileReader(fileName));
        PrintWriter outFile = new PrintWriter("coloredMap.txt");
        int[][] matrix = new int[numRegions][numRegions];
        int[] map = new int[numRegions];

        //initializing matrix from file
        for(int row = 0; row < matrix.length; row++)
        {
            for(int col = 0; col < matrix.length; col++)
            {
                matrix[row][col] = inFile.nextInt();
            }
        }
        inFile.close();

        //initialize map vals to -1
        for(int i = 0; i < map.length; i++)
        {
                map[i] = -1;
        }

        colorTheMap(map, matrix, 0, 0);

        outFile.println("Region\t\tColor");
        for(int i = 0; i < map.length; i++)
        {
            outFile.println(i+1 + "\t\t" + map[i]);
        }
        outFile.close();
    }
}
4

1 回答 1

1

您没有使用cin colorTheMap，而且您可能想要使用。

您获得map所有条目的原因-1是：

    int i = 0;
    if(!acceptable(map, matrix, r, i)) return false;
    map[r] = i;
    .
    .
    .
    do
    {
        i++;
        q = colorTheMap(map, matrix, r+1, i);
        if(q) return true;
    }
    while(i <= 3);
    map[r] = -1;

每次调用都会在它命中的那一刻colorTheMap(map, matrix, r+1, i)返回false

    int i = 0;
    if(!acceptable(map, matrix, r, i)) return false;

如果它的任何邻居已经被着色0（因为你从不使用c，如果你到达那条线，你总是会分配0给map[r]in map[r] = i;），所以在返回之后，我们false在分配任何颜色之前立即返回r。我假设你的输入图是连接的，所以任何一个调用colorTheMap(map, matrix, r, c)都会找到一个r有色的邻居，0甚至没有设置map[r]为其他任何东西，因为if(!acceptable(map, matrix, r, i)) return false;它会立即返回，或者它只接收q = falsein 的分配

    do
    {
        i++;
        q = colorTheMap(map, matrix, r+1, i);
        if(q) return true;
    }
    while(i <= 3);

并撤消map[r] = -1;循环后面的行中的着色。

另一个注意事项：我假设您正在尝试实现贪婪着色，但这不是最佳的，即您可能以这种方式最终得到未着色的区域。四色问题比简单的“只需用没有指定任何邻居的颜色给所有东西上色就可以了”要复杂得多，否则不会花费一个多世纪的时间来证明四种颜色就足够了出现。下面是一个需要二次时间的正确算法的轮廓（引用以防链接丢失：Neil Robertson、Daniel P. Sanders、Paul Seymour 和 Robin Thomas。1996 年。高效的四色平面图。在 Proceedings 中第 28 届年度 ACM 计算理论研讨会 (STOC '96)。美国计算机协会，纽约，纽约，571-575。DOI：https://doi.org/10.1145/237814.238005）。在证明四个着色平面图总是可能的20年后，它出现了。

于 2015-09-18T09:01:44.133 回答