Pomos za zadaca

Zdravo, zadacata od SPOJ - https://www.spoj.com/problems/SHPATH/ ja iskucav , ama kodot ne raboti. Kje moze celosna popravka okolu istiot:
Kod:

#include <bits/sdc++.h>

using namespace std;
const int MAX=1e5+10;
const int INF=(1<<30);
int ns;
vector<pair<int,int> > graph[MAX];
int dist[MAX];
bool vis[MAX];
struct node{
    int idx;
    int val;
    node(){}
    node(int _idx,int _val)
    {
        idx=_idx;
        val=_val;
    }

bool operator < (const node &TMP) const{
        return val>TMP.val;

}

};

void dijkstra(int start, int end)
{
    for(int i=0; i<ns; i++)
    {
        dist[i]=INF;
        vis[i]=false;
    }
    dist[start]=0;
    priority_queue<node> q;
    q.push(node(start,0));
    while(!q.empty())
    {
        node curr= q.top();
        q.pop();
        if(vis[curr.idx])
        {
            continue;
        }
        vis[curr.idx]=true;
        for(int i=0; i<(int)graph[curr.idx].size(); i++)
        {
            int sosed=graph[curr.idx][i].first;
            int cost=graph[curr.idx][i].second;
            if(!vis[sosed] && dist[sosed] > curr.val +cost)
            {
                vis[sosed]=true;
                dist[sosed]=curr.val+cost;
                q.push(node(sosed,dist[sosed]));
            }
        }
    }
    cout<<dist[end]<<endl;
}

int main()
{
    int t;
    cin>>t;
    while(t--)
    {

cin>>ns;
        int idx=0;
        mapa<string,int> mapa[ns+1];
        for(int i=0; i<ns; i++)
        {
            string s;
            cin>>s;
            int sosedi;
            cin>>sosedi;
            mapa[s]=idx;
            idx++;
            int cnt=0;
            for(int j=0; j<sosedi; j++)
            {

int a,b;
                cin>>a>>b;
                a--;
                graph[cnt].push_back(make_pair(a,b));
                cnt++;
            }
        }
        int ses;
        cin>>ses;
        vector<pair<string, string > > s;
        for(int i=0; i<ses; i++)
        {
            cin>>s[i].first;
            cin>>s[i].second;
            dijkstra(mapa[s[i]].first]--,mapa[s[i].second]--);
        }

}
    return 0;
}

BATIR wrote:Zdravo, zadacata od SPOJ - https://www.spoj.com/problems/SHPATH/ ja iskucav , ama kodot ne raboti. Kje moze celosna popravka okolu istiot

Ај вака, мислам дека е најдобро ако те насочам само во добра насока, бидејќи за време на натпревари/испити/итн ќе немаш помош некој друг да го гледа/поправа кодот.

1. Програмата што ја имаш прикачено не се ни компајлира. Имаш некои грешки како на пример во линијата mapa<string,int> mapa[ns+1]; која треба да биде map<string,int> mapa; итн.
Тоа треба прво да поправиш

2. Потоа, имаш и други грешки, некои во однос на извршување на програмата, а некои грешки и со алгоритамот.
На пример, имаш вектор "s", и потоа читаш во s[i].first и s[i].second, но тој вектор (кога се прави) има големина 0, па неможеш да читаш во s[0].first на пример (елемент s[0] не постои). Можеш да додаваш со push_back на пример, или уште на почеток да го направиш со соодветна големина.

Има и грешки со алгоритамот Dijkstra. На пример, не треба да имаш vis[sosed]=true; во for циклусот каде што ги поминуваш соседите, бидејќи можеби ќе се открие некој друг (пократок) пат од некое друго теме подоцна.

3. Има и некои делови кои што не треба да ги правиш како на пример mapa[s[i]].first]-- (ова не се ни компајлира, бидејќи има повеќе знаци ']' од '[', но тука не треба ни да има --, бидејќи индексите ти почнуваат од 0, итн.

Sega popraviv nesto , ama seushte ima greska kaj mapa[s]=idx, CE ima. Shto treba sega da popravam?

#include <bits/stdc++.h>

using namespace std;
const int MAX=1e5+10;
const int INF=(1<<30);
int ns;
vector<pair<int,int> > graph[MAX];
int dist[MAX];
bool vis[MAX];
struct node{
    int idx;
    int val;
    node(){}
    node(int _idx,int _val)
    {
        idx=_idx;
        val=_val;
    }

bool operator < (const node &TMP) const{
        return val>TMP.val;

}

};

void dijkstra(int start, int end)
{
    for(int i=0; i<ns; i++)
    {
        dist[i]=INF;
        vis[i]=false;
    }
    dist[start]=0;
    priority_queue<node> q;
    q.push(node(start,0));
    while(!q.empty())
    {
        node curr= q.top();
        q.pop();
        if(vis[curr.idx])
        {
            continue;
        }
        vis[curr.idx]=true;
        for(int i=0; i<(int)graph[curr.idx].size(); i++)
        {
            int sosed=graph[curr.idx][i].first;
            int cost=graph[curr.idx][i].second;
            if(!vis[sosed] && dist[sosed] > curr.val +cost)
            {
                vis[sosed]=true;
                dist[sosed]=curr.val+cost;
                q.push(node(sosed,dist[sosed]));
            }
        }
    }
    cout<<dist[end]<<endl;
}

int main()
{
    int t;
    cin>>t;
    while(t--)
    {

cin>>ns;
        int idx=0;
        map<string,int> mapa[ns+1];
        for(int i=0; i<ns; i++)
        {
            string s;
            cin>>s;
            int sosedi;
            cin>>sosedi;
            mapa[s]=idx;
            idx++;
            int cnt=0;
            for(int j=0; j<sosedi; j++)
            {

int a,b;
                cin>>a>>b;
                a--;
                graph[cnt].push_back(make_pair(a,b));
                cnt++;
            }
        }
        int ses;
        cin>>ses;
        vector<pair<string, string > > s1;
        for(int i=0; i<ses; i++)
        {
            int s3,s4;
            cin>>s3>>s4;
            s1[i].first.push_back(s3);
            s1[i].second.push_back(s4);
            dijkstra(mapa[s1[i]].first],mapa[s1[i].second]);
        }

}
    return 0;
}

Dali go citas errorot?
Eve primer za mapa: http://www.cplusplus.com/reference/map/map/operator[]/
Ti vo svojot kod pravis array so ns+1 mapi. Imas napisano mapa[S]=idx kade S e string sto ne e mozno, bidejki mapa vo tvojot kod e array i mozes samo so broevi da go ineksiras. Smeni go redot map<string,int> mapa[ns+1] vo map<string,int> mapa;.

Всушност, најголема грешка ти е што не го прочита мојот одговор соодветно.

Дури имав напишано точно што треба

која треба да биде map<string,int> mapa; итн.

Исто и при крајот каде што имаш пак s1[i]... Не можеш s1[i] бидејќи s1 иницијално има 0 елементи. Може s1.push_back(...)

Mislam deka odgovoruvas na gresniot covek

This message was edited 1 time. Last update was at 01/12/2018 13:04:32

VlatkoSh wrote:Mislam deka odgovoruvas na gresniot covek

Да да. На BATIR му/и пишував.