لماذا لا يعمل تطبيق Dijkstra (الرسم البياني)؟
سؤال
لقد قمت بهذا التنفيذ لهذه المشكلة:http://www.spoj.pl/prombs/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();
}
لا أعرف لماذا لا يولد الكود الخاص بي الإجابة الصحيحة لحضور TestCases. إذا كان بإمكان شخص ما مساعدتي في تحسين الرمز ، فيرجى القيام بذلك: د
المحلول
بادئ ذي بدء ، رمز تحليل الرسم البياني غير صحيح. يحدد السطر الأول العرض والارتفاع ، حيث يكون العرض هو عدد الأحرف في كل سطر ، والارتفاع هو عدد الخطوط. لذلك ، مبادلة &a
و &b
في SCANF الأول ، أو تبديل ترتيب المتداخلة for
حلقات (ولكن ليس كلاهما). أيضا ، اضطررت لإضافة دمية scanf("%c", &dummy);
مكالمات في أماكن مختلفة لتصفية الخطوط الجديدة. سيساعد تفريغ بسيط ، مثل هذا ، في تحديد ما إذا كانت خريطتك قد تم تحليلها بشكل صحيح:
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;
}
ملاحظة: أنا أيضا ضبط map[i][j]
إلى 0 لـ 'S' و 'D' ، أيضًا تغيير المتكررة if
بيانات إلى if; else if; else
سلسلة. هذا يجعل الخوارزمية أكثر قوة ، حيث يمكنك إضافة الوقت بشكل عام من المصدر أو الوجهة.
الآن ، إلى الخوارزمية نفسها ....
كل حلقة من زيادات الخوارزمية result
بواسطة وزن موقع الخريطة الحالي. ومع ذلك ، فإن الخوارزمية تبحث عن مسارات متعددة في وقت واحد (أي ، عدد الإدخالات في قائمة انتظار الأولوية) ، وبالتالي 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;
آمل أن يساعد هذا.
نصائح أخرى
لماذا لديك 0 فهارس؟
tempa.x=top.x+X[0];
tempa.y=top.y+Y[0];
tempa.time=map[top.x+X[0]][top.y+Y[0]];
نيتبيك:
bool operator <(const node &s,const node &r)
{
if(s.time>r.time)
return true;
else return false;
}
أليس هذا أكثر قابلية للقراءة:
bool operator <(const node &s,const node &r)
{
return (s.time>r.time);
}
أنت تستخدم X[0]
و Y[0]
بدلاً من X[i]
و Y[i]
في تلك الحلقة الداخلية.
بالمناسبة ، بخلاف أن Dijkstra الخاص بك غير فعال للغاية. أولاً ، أنت تدفع العقد إلى قائمة الانتظار حتى عند زيارتها بالفعل ، وثانياً ، قد يكون لديك العديد من العقدة نفسها في قائمة الانتظار ، فقط بأوقات مختلفة. في نهاية المطاف ، لا يؤثر أي من هذه الأشياء على النتيجة ، لكنك تغير التعقيد.
تحرير: أوه ، tempa.time
يجب أن تساوي top.time
بالإضافة إلى وزن الحافة ، وليس فقط وزن الحافة.