Passare una stringa a file.open();

Question

Sono abituato ai linguaggi di livello superiore (java, python ecc.), Dove questo è ovvio. Sto cercando di passare una stringa che l'utente inserisce in cin, il nome di un file da aprire. Sembra che ci sia una sorta di errore di follia del puntatore e il mio codice non verrà compilato. Ho cancellato parte del mio codice per renderlo più chiaro.

#include <iostream> 
    #include <fstream> 
    using namespace std; 

    string hash(string filename); 

    int main(){ 
      cout << "Please input a file name to hash\n"; 
      string filename; 
      cin >> filename; 
      cout <<hash(filename); 
      return 0; 
    } 


    string hash(string filename){ 
      file.open(filename); 
      if(file.is_open()){ 

        file.close(); 
      } 

      return returnval; 
    } 

Ecco l'errore di compilazione.

<code> 
$ g++ md5.cpp 
md5.cpp: In function ‘std::string hash(std::string)’: 
md5.cpp:22: error: no matching function for call to ‘std::basic_ifstream<char, std::char_traits<char> >::open(std::string&)’ 
/usr/include/c++/4.2.1/fstream:518: note: candidates are: void std::basic_ifstream<_CharT, _Traits>::open(const char*, std::_Ios_Openmode) [with _CharT = char, _Traits = std::char_traits<char>] 
</code> 

(lo so che ci sono librerie per hash MD5, ma sto cercando di conoscere come funziona l'hash, e la collisione alla fine hash)

Answer 1

open() prende una stringa in stile C. Utilizzare std::string::c_str() per ottenere questo:

file.open (filename.c_str()); 

Per poter utilizzare solo una stringa, come sottolineato di seguito, è necessario utilizzare un compilatore C++ con 11 di supporto, come il sovraccarico è stato aggiunto per C++ 11.

Il motivo non è come Java, ecc è che proveniva da C. Le classi non esisteva in C (beh, non quasi così come fanno in C++), per non parlare di una classe String. Affinché C++ fornisca una classe stringa e mantenga la compatibilità, devono essere diversi e la classe fornisce un costruttore di conversioni per const char * -> std::string e c_str() per passare dall'altra parte.

Considerare di passare l'argomento (e forse anche il ritorno) come const std::string &; nessuna copia inutile. L'ottimizzazione probabilmente li catturerebbe, ma è sempre bene fare.