gpt4 book ai didi

c++ - 为什么这个 Dijkstra(图形)实现不起作用?

转载 作者:搜寻专家 更新时间:2023-10-30 23:50:20 24 4
gpt4 key购买 nike

我针对这个问题做了这个实现: http://www.spoj.pl/problems/SHOP/


#include<iostream>
#include<stdio.h>
#include<queue>
#include<conio.h>
#include<string.h>
using namespace std;

struct node
{
int x;
int y;
int time;
};
bool operator <(const node &s,const node &r)
{
if(s.time>r.time)
return true;
else return false;
}
node beg,src,dest,tempa;
int b,a,temp;
int map[25][25];
bool vis[25][25];
int X[]={1,0,-1,0};
int Y[]={0,1,0,-1};


int djs_bfs(node src,node dest)
{
int result=0;
priority_queue<node>pq;
pq.push(src);
while(!pq.empty())
{
node top = pq.top();
pq.pop();
if(top.x==dest.x && top.y==dest.y) return result;
if(top.x<0 || top.x>=a) continue;
if(top.y<0 || top.y>=b) continue;
if(vis[top.x][top.y]) continue;

vis[top.x][top.y]=true;
result+=map[top.x][top.y];
for(int i=0;i<4;i++)
{
tempa.x=top.x+X[0];
tempa.y=top.y+Y[0];
tempa.time=map[top.x+X[0]][top.y+Y[0]];
pq.push(tempa);
}
}
return -1;
}

int main()
{
memset(vis,false,sizeof(vis));
scanf("%d %d",&a,&b);
while(a != 0)
{
for(int i=0;i<a;i++)
for(int j=0;j<b;j++)
{
scanf("%c",&temp);
if(temp=='X') {map[i][j]=0;vis[i][j]=true;}
if(temp=='S') {src.x=i;src.y=j;src.time=0;}
if(temp=='D') {dest.x=i;dest.y=j;dest.time=0;}
else map[i][j]=temp-'0';
}
cout<<djs_bfs(src,dest)<<endl;
scanf("%d %d",&a,&b);
}
return 0;
getch();
}

我不知道为什么我的代码没有为测试用例生成正确的答案。如果有人可以帮助我改进代码,请这样做:D

最佳答案

首先,图解析代码不正确。第一行指定宽度和高度,其中宽度是每行的字符数,高度是行数。因此,在第一个 scanf 中交换 &a&b,或者交换嵌套的 for 循环的顺序(但不能同时交换)。另外,我不得不在各个地方添加虚拟 scanf("%c", &dummy); 调用以过滤掉换行符。像这样的简单转储将有助于确定您的 map 是否已正确解析:

cout << "a=" << a << endl;
cout << "b=" << b << endl;
for (int i=0; i<a; i++) {
for(int j=0; j<b; j++) {
cout << (char)('0' + map[i][j]) << ",";
}
cout << endl;
}

注意:我还将 'S' 和 'D' 的 map[i][j] 设置为 0,还将重复的 if 语句更改为 如果;否则如果; else 链。这使算法更加稳健,因为您通常可以从源或目标添加时间。

现在,关于算法本身......

算法的每个循环都会将 result 增加当前 map 位置权重。然而,该算法同时搜索多条路径(即优先级队列中的条目数),因此 result 最终是所有已处理节点权重的总和,而不是当前路径权重。当前路径权重为 top.temp,因此您可以消除 result 变量并在到达目的地时简单地返回 top.temp

此外,正如其他答案所述,您需要在内部循环中使用 X[i]Y[i] ,否则您只会在一个方向上搜索.

现在,由于 X[i]Y[i] 的加法/减法,您可能会访问 map[][] 超出范围(-1 或 25)。因此,我建议将 if 守卫移动到内部 for 循环中以防止越界访问。这也避免了用非法可能性填充优先级队列。

这里是我的算法版本,有最少的修正,供引用:

priority_queue<node>pq;
pq.push(src);
while(!pq.empty())
{
node top = pq.top();
pq.pop();

if(top.x==dest.x && top.y==dest.y) return top.time;
if(vis[top.x][top.y]) continue;

vis[top.x][top.y]=true;

for(int i=0;i<4;i++)
{
tempa.x=top.x+X[i];
tempa.y=top.y+Y[i];
if(tempa.x<0 || tempa.x>=a) continue;
if(tempa.y<0 || tempa.y>=b) continue;
tempa.time=top.time + map[tempa.x][tempa.y];
pq.push(tempa);
}
}
return -1;

希望对您有所帮助。

关于c++ - 为什么这个 Dijkstra(图形)实现不起作用?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/2192902/

24 4 0
Copyright 2021 - 2024 cfsdn All Rights Reserved 蜀ICP备2022000587号
广告合作:1813099741@qq.com 6ren.com