2

私はこの問題のためにこの実装を行いました： 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

4

3 に答える 3

4

まず第一に、グラフ解析コードが正しくありません。最初の行は幅と高さを指定します。ここで、幅は1行あたりの文字数、高さは行数です。したがって、&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]「S」と「D」も0に設定し、繰り返されるifステートメントをif; else if; elseチェーンに変更します。これにより、ソースまたは宛先から一般的に時間を追加できるため、アルゴリズムがより堅牢になります。

さて、アルゴリズム自体に移りましょう。

アルゴリズムの各ループはresult、現在のマップ位置の重みで増分します。ただし、アルゴリズムは複数のパスを同時に検索しているため（つまり、優先キュー内のエントリの数）、result現在のパスの重みではなく、処理されたすべてのノードの重みの合計になります。現在のパスの重みはであるため、変数をtop.temp削除して、宛先に到達したときに簡単に戻ることができます。resulttop.temp

また、他の回答が指摘しているように、内側のループでを使用する必要がありX[i]ますY[i]。そうしないと、一方向にしか検索しません。

これで、とからの加算/減算のために、範囲外（-1または25）にアクセスする可能性がありますX[i]。したがって、範囲外のアクセスを防ぐために、ガードを内側のループに移動することをお勧めします。これにより、優先キューが不正な可能性でいっぱいになることも回避されます。Y[i]map[][]iffor

参考までに、最小限の修正を加えた私のバージョンのアルゴリズムを次に示します。

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;

これがお役に立てば幸いです。

于 2010-02-03T21:38:06.927 に答える
2

その内側のループでX[0]andY[0]の代わりにX[i]andを使用しています。Y[i]

ちなみに、あなたのダイクストラは非常に非効率的です。1 つ目は、ノードが既にアクセスされている場合でも、ノードをキューにプッシュしていることです。2 つ目は、キュー内に同じノードがいくつかありますが、時間が異なるだけです。最終的に、これらのどちらも結果に影響を与えませんが、複雑さを変えています。

編集：ああ、エッジの重みだけでなく、エッジの重みを加えたものにtempa.time等しいはずです。top.time

于 2010-02-03T15:14:25.570 に答える
2

なぜインデックスが 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);
}
于 2010-02-03T15:16:00.580 に答える