为什么我的输出不正确?

时间:2014-12-20 18:53:24

标签: c data-structures graph

我想用字母构建一个图表,但有些不对劲。

我的代码:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct AdjListNode
{
  char *dest;
  struct AdjListNode* next;
};


struct AdjList
{
  struct AdjListNode *head;  // pointer to head node of list
};

struct Graph
{
  int V;
  struct AdjList* array;
};

struct AdjListNode* newAdjListNode(char *dest){
struct AdjListNode* newNode =
    (struct AdjListNode*) malloc(sizeof(struct AdjListNode));
newNode->dest = dest;
newNode->next = NULL;
return newNode;}

struct Graph* createGraph(int V){
struct Graph* graph = (struct Graph*) malloc(sizeof(struct Graph));
graph->V = V;

// Create an array of adjacency lists.  Size of array will be V
graph->array = (struct AdjList*) malloc(V * sizeof(struct AdjList));

// Initialize each adjacency list as empty by making head as NULL
int i;
for (i = 0; i < V; ++i)
    graph->array[i].head = NULL;

return graph;}

void addEdge(struct Graph* graph, char *src, char *dest){
// Add an edge from src to dest.  A new node is added to the adjacency
// list of src.  The node is added at the beginin
struct AdjListNode* newNode = newAdjListNode(dest);
newNode->next = graph->array[0].head;
graph->array[0].head = newNode;

// Since graph is undirected, add an edge from dest to src also
newNode = newAdjListNode(src);
newNode->next = graph->array[1].head;
graph->array[1].head = newNode;}

void printGraph(struct Graph* graph){
int v;
for (v = 0; v < graph->V; ++v)
{
    struct AdjListNode* pCrawl = graph->array[v].head;
    printf("\n Adjacency list of vertex %d\n head ", v);
    while (pCrawl)
    {
        printf("-> %s", pCrawl->dest);
        pCrawl = pCrawl->next;
    }
    printf("\n");}}

int main(){
// create the graph given in above fugure
int V = 5;
struct Graph* graph = createGraph(V);
addEdge(graph, "a", "b");
addEdge(graph, "a", "e");
addEdge(graph, "b", "c");
addEdge(graph, "b", "d" );
addEdge(graph, "b", "e");
addEdge(graph, "c", "d");
addEdge(graph, "d", "e");


// print the adjacency list representation of the above graph
printGraph(graph);

return 0;}

我的输出是这样的:

E-&GT; D-&GT; E-&GT; D-&GT; A-&GT; B-&GT; C-&GT; B-&GT; A-&GT;

我的输出应该是:

a->b->e 
b->a->c->d->e
c->b->d
d->b->c->e
e->a->b->d

请帮帮我。我又问了一个不同的问题。我想要这个输出。我想用字母

创建adjList

2 个答案:

答案 0 :(得分:0)

这是一个使用线性但有效的2D数组的实现,因为它的大小是已知的,但这只是为了保持一些相似性,否则当V已知时我会声明#define V 5和静态二维数组int graph[V][V]。除了简化之外,我将char*字符串参数更改为int,因为它们代表单个字符。我还将V size参数传递给函数。

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>

void printGraph(int *graph, int V) {
    int v, n;
    for (v=0; v<V; ++v) {
        printf ("%c", v + 'a');
        for (n=0; n<V; ++n) {
            if (graph [v * V + n])
                printf("->%c", n + 'a');
        }
        printf("\n");
    }
}

void addEdge(int *graph, int V, int src, int dest) {
    src  = tolower(src)  - 'a';
    dest = tolower(dest) - 'a';
    if (src < 0 || src >= V || dest < 0 || dest >= V || src == dest)
        return;     // error
    graph [src  * V + dest] = 1;
    graph [dest * V + src ] = 1;
}

int *createGraph (int V) {
    return calloc (V * V, sizeof(int));
}

int main(){
    int V = 5;
    int *graph;
    if (graph = createGraph(V)) {
        addEdge(graph, V, 'a', 'b');  // changed from string to char
        addEdge(graph, V, 'a', 'e');
        addEdge(graph, V, 'b', 'c');
        addEdge(graph, V, 'b', 'd');
        addEdge(graph, V, 'b', 'e');
        addEdge(graph, V, 'c', 'd');
        addEdge(graph, V, 'd', 'e');

        printGraph(graph, V);
        free (graph);
    }
    return 0;
}

节目输出

a->b->e
b->a->c->d->e
c->b->d
d->b->c->e
e->a->b->d

您使用“a”到“e”标记顶点。在我的回答中,我将这些更改为'a'thru'e'并使用这些标签来索引5x5数组并操纵这些顶点标签,使它们在0到4的范围内。我使用tolower()来处理标签使用“A”到“E”,但由于没有使用它们,因此没有必要。然后我减去'a'以将标签范围从0开始。作为int,值'a'为97,因此您可以看到'a' - 'a' = 0'e' - 'a' = 4

答案 1 :(得分:0)

我评论了改变的部分并解释了原因。我没有更改您的代码,因此您可以轻松理解。

试试这个:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

struct AdjListNode
{
    char *dest;
    struct AdjListNode* next;
};

struct AdjList
{
    struct AdjListNode *head;  // pointer to head node of list
};

struct Graph
{
    int V;
    struct AdjList* array;
};

struct AdjListNode* newAdjListNode(char *dest)
{
    struct AdjListNode* newNode = (struct AdjListNode*) malloc(sizeof(struct AdjListNode));
    newNode->dest = dest;
    newNode->next = NULL;
    return newNode;
}

struct Graph* createGraph(int V)
{
    struct Graph* graph = (struct Graph*) malloc(sizeof(struct Graph));
    graph->V = V;

    // Create an array of adjacency lists.  Size of array will be V
    graph->array = (struct AdjList*) malloc(V * sizeof(struct AdjList));

    // Initialize each adjacency list as empty by making head as NULL
    int i;
    for (i = 0; i < V; ++i)
        graph->array[i].head = NULL;

    return graph;
}

void addEdge(struct Graph* graph, char *src, char *dest)
{
    // Add an edge from src to dest.  A new node is added to the adjacency
    // list of src.  The node is added at the beginin

    struct AdjListNode* newNode = newAdjListNode(dest);

    //***Changed. you need to add edge in src node. But you were always adding in 0
    newNode->next = graph->array[src[0]-'a'].head;
    graph->array[src[0]-'a'].head = newNode;

    // Since graph is undirected, add an edge from dest to src also
    newNode = newAdjListNode(src);

    //***Changed. you need to add edge in dest node. But you were always adding in 1
    newNode->next = graph->array[dest[0]-'a'].head;
    graph->array[dest[0]-'a'].head = newNode;
}

void printGraph(struct Graph* graph)
{
    int v;
    for (v = 0; v < graph->V; ++v)
    {
        struct AdjListNode* pCrawl = graph->array[v].head;
        printf("\n Adjacency list of vertex %d\n head %c", v, v + 'a');
        while (pCrawl)
        {
            printf("-> %s", pCrawl->dest);
            pCrawl = pCrawl->next;
        }
        printf("\n");
    }
}

int main()
{
    // create the graph given in above fugure
    int V = 5;
    struct Graph* graph = createGraph(V);
    addEdge(graph, "a", "b");
    addEdge(graph, "a", "e");
    addEdge(graph, "b", "c");
    addEdge(graph, "b", "d" );
    addEdge(graph, "b", "e");
    addEdge(graph, "c", "d");
    addEdge(graph, "d", "e");


    // print the adjacency list representation of the above graph
    printGraph(graph);

    return 0;
}

它会给你输出:

a-> e-> b
b-> e-> d-> c-> a
c-> d-> b
d-> e-> c-> b
e-> d-> b-> a

如果你想获得完全相同的输出:

a-> b-> e
b-> a-> c-> d-> e
c-> b-> d
d-> b-> c-> e
e-> a-> b-> d

只需按照以下方式更改addEdge()函数:

void addEdge(struct Graph* graph, char *src, char *dest)
{
    // Add an edge from src to dest.  A new node is added to the adjacency
    // list of src.  The node is added at the beginin

    struct AdjListNode* newNode = newAdjListNode(dest);

    //***Changed. you need to add adge in src node. But you were always adding in 0
    struct AdjListNode* temp=graph->array[src[0]-'a'].head;

    if(temp==NULL)  // First element of the list
      graph->array[src[0]-'a'].head=newNode;
    else
    {
        while(temp->next!=NULL) // finding the last element of the list
            temp=temp->next;
        temp->next=newNode; // Adding current element to the last
    }

    // Since graph is undirected, add an edge from dest to src also
    newNode = newAdjListNode(src);

    //***Changed. you need to add adge in dest node. But you were always adding in 1
    temp = graph->array[dest[0]-'a'].head;

    if(temp==NULL) // First element of the list
      graph->array[dest[0]-'a'].head=newNode;
    else
    {
        while(temp->next!=NULL) // finding the last element of the list
            temp=temp->next;
        temp->next=newNode; // Adding current element to the last
    }
}

第一个代码将在前面添加新边缘,第二个代码将在最后添加它 希望它有所帮助:)