C++で文字列変数を返す

Question

私は便宜上、大文字と小文字を区別しない基本的なユーザーインターフェイスを作ろうとしています。これを行うには、文字列を大文字にするコンバータクラスを作成しましたが、問題が発生しました。このクラスを使用した後、main（）のif文はコンバーターからのメッセージを解釈するはずですが、元の入力を読み込んだだけで、大文字の入力ではなく、変換された文字列を直接コンバータが、それは私をさせません。

#include "stdafx.h" 
#include <iostream> 
#include <string> 

using namespace std; 

string response; 

//converts responses to upper-case 
void convert(string response) { 
    for (int i = 0; i < response.length(); i++) { 
     response[i] = toupper(response[i]); 
    } 
} 

//main dialogue 
int main() { 

    cout << "How are you?: "; 
    getline(cin, response); 
    convert(response); 
    if (response == "GOOD") { 
     cout << "Response 1./l"; 
    } 
     else { 
     cout << "Response 2./l"; 
    } 
} 

私はC++に非常に新しいですので、間違いが修正または私は難易度のソリューションを理解しているかのために簡単なものだった場合、私はお詫び申し上げます。

Answer 1

もう1つの方法は、stringを返すように関数ヘッダーを変更することです。

string convert(const string &inResponse) { 
    string outResponse(inResponse); 
    for (int i = 0; i < inResponse.length(); i++) { 
     outResponse[i] = toupper(inResponse[i]); 
    } 
    return outResponse; 
} 

そしてあなたのようなあなたの主な機能で返された文字列を使用します：つまり別に値の代わりに参照を渡す必要から

.... 
// response is input, outputResponse is output: 
string outputResponse = convert(response); 
.... 

Answer 2

「値渡し」と「参照渡し」見上げて - あなたは「値渡し」しているが、あなたはC++で

を「参照渡し」期待している：あなたのケースのものでvoid convert(string& response) {

@NeilLocketzのコメントで指摘されているように、グローバルresponse、メソッド内にローカルresponseがあります。これは、呼び出しパラメータとして使用しているので、実際はグローバルなものです。適切に処理したい場合は、おそらくresponseをグローバルにしたくないでしょう。

受け入れられた回答にはこれ以上のメモリコピーが残っています。本当の鍵は、値渡しを理解し、参照によって渡し、あなたの状況に適したものを使用することです。

Answer 3

、あなたはC++を使用してみてください - 11機能：

void convert(string &response) { 
    for (auto &c: response) { 
     c = toupper(c); 
    } 
} 

もっとクリーンでシンプルです。