retour std :: vecteur <std::string> retourne ordures

Question

J'ai eu un comportement bizarre d'un code de pièce aujourd'hui sous windows

std::vector<std::string> getMACs() { 
    std::vector<std::string> macs; 
    for(/*something*/) { 
    char buffer[100]; 
    sprintf_s(buffer, size, "get the mac address here"); 
    std::string s = ""; 
    s.append(buffer); 
    printf("mac=%s\n", s.c_str(); //print the mac address correctly 
    macs.push_back(s); 
    } 
    return macs; 
} 

int main(int, char**) { 
    std::vector<std::string> macs = getMACs(); 
    for (size_t i = 0; i < mac.size(); i++) { 
    printf("mac=%s\n", macs[i]); //prints garbage 
    } 
} 

bien que l'adresse MAC dans la fonction a été correctement imprimé, dans la principale, il imprime des déchets, la seule explication que j'ai, que le vecteur macs est plein de chaînes de déchets, mais comment cela peut-il arriver; un appel à string.append (const char *), bien qu'il passe par référence, la fonction push_back() devrait appeler le constructeur de la chaîne de copie, et ne devrait donc plus pointer vers une référence de chaîne qui deviendra une poubelle après avoir quitté le portée, non?

Answer 1

printf("mac=%s\n", macs[i]); //prints garbage 

Parce que macs[i] est de type std::string et printf qui ne fonctionne pas savoir comment gérer cela. Essayez ceci:

printf("mac=%s\n", macs[i].c_str()); 

Ou ceci:

std::cout << "mac=" << macs[i] << '\n'; 

sécurité Type, FTW

Answer 2

Vous retournez une chaîne:

return s; 

vous devez retourner le vecteur:

return macs; 

Modifier après votre édition, la cause probable de votre problème est une mauvaise utilisation de printf. Vous pouvez parcourir le vecteur et d'imprimer le contenu comme ceci:

std::vector<std::string> macs = getMACs(); 
for (std::vector<std::string>::const_iterator it = macs.begin(); it != mac.end(); ++it) { 
    std::cout << *it << "\n"; 
} 

ou, en C++ 11,

for (const auto& s : macs) { 
    std::cout << s << "\n"; 
}