C++ zadatak

Pozdrav!

Evo imam problema s STL-ima, tocnije s vektorom i mappingom, dvije funkcije, pa ako netko ima vremena i volje za pogledat kod i doradit neka se javi na pm.

Thnx unaprijed =)

1. funkcija

-prima vektor stringova, vraca vektor  permutiranih stringova tako da su oni palindromi max duljine(ako za jedan string ima vise  takvih vraca najmanji obzirom na leksikograf uredjaj, velika slova dolaze prie malih), prvo iz svakog stringa izbaci sve sto nije slovo; npr  "abA", "bdc", "Dca", "abecbdac" - "Aba", "b", "D", "abcdcba"

#include<iostream>
#include<string>

#include<cctype>
#include<set>

using namespace std;

vector<string> permutirani_palindromi( vector<string> v){
string s;
vector<string> P;

for(int i=0; i<v.size(); i++) { s=v[i];
   for(int j=0; j<s.size(); j++)
      if(!isalpha(s[j])) s.erase(j); }

for(int i=0; i<v.size(); i++) { s=v[i];
   multiset<char> S;
   for(int j=0; j<s.length(); j++) S.insert(s[j]);   
   multiset<char>::iterator si;  string p;
    for(si=S.begin(); si!=S.end();){
      
        if (!(S.count(*si)%2)) {p.insert(p.length()/2, S.count(*si), (*si) );
                                multiset<char>::iterator temp;
                                for(int k=0; k<S.count(*si); k++) {temp=si; si++;
                                S.erase(temp);}
                                }             
        if((S.count(*si)%2) && (S.count(*si)>1)){ p.insert(p.length()/2, S.count(*si)-1, (*si) );             
                                                multiset<char>::iterator temp;
                                                for(int k=0; k<S.count(*si)-1; k++) {temp=si; si++;
                                                S.erase(temp); }
                                                }
       if(S.count(*si)==1)  si++;
                                }
       for(si=S.begin(); si!=S.end();si++) if( S.count(*si)==1)
                                                 {p.insert(p.length()/2, 1, (*si)); break;}
       P.push_back(p);                                                              }        
return P;     
}

ideja mi je bila za svaki string svaki char stavit u multiskup jer ce se tako odmah sortirat pa ce bit lakse nac max palindrome, al negdje sam ocito pogrijesila jer ne ispisuje sta bi trebala..

Hehe, np, događa se svima.

Što se tiće koda, nisam cijeli pročitao, ali primjetio sam na početku da u for petlji brišeš znakove, a ne smanjuješ brojač - neki znakovi će se preskočiti.

Ovako na brzinu: 

float b =static_cast<int>(a);

Znači zagrade ,a imaš i gubitak podataka pri castanju ....
Ima i boljih castanja i načina na koji se casta ali eto samo sam letimično pogledao...

Hmm, pa ovaj način se preporuča... ima jednostavnijih, ali ovo je "najispravniji".

edit: onda je krivo u knjizi, mora biti ()

demistificirani c++

2. izdanje str.51

mozda sam sto ja nisam skuzio trebao bih pripazit na zagrade