0

私はcppを初めて使用し、配列に関して質問があります。以下のコードは、strの逆バージョンを作成し、それをnewStrに格納する必要があります。ただし、newStrは常に空になります。strから値を割り当てているのに、なぜこれが発生しているのか誰かに説明してもらえますか?

void reverse (char* str) {
    char* newStr = (char*)malloc(sizeof(str));

    for (int i=0;i<sizeof(str)/sizeof(char);i++) {
       int index = sizeof(str)/sizeof(char)-1-i;
       newStr [i] = str [index];
    }
}

PS:ポインターを移動するか、std :: reverse関数を使用して配列を逆にする方がはるかに効率的であることは知っていますが、上記のコードが機能しない理由に興味があります。

4

4 に答える 4

6

上記のように、コメント投稿sizeof(str)者は文字列の長さを教えてくれません。あなたは使用する必要がありますsize_t len = strlen(str);

void reverse (char* str) {
   size_t len = strlen(str);
   char* newStr = (char*)malloc(len + 1);

   for (int i=0; i<len;i++) {
      int index = len-1-i;
      newStr[i] = str[index];
   }
   newStr[len] = '\0'; // Add terminator to the new string.
}

freeあなたの記憶を忘れないでくださいmalloc。あなたの関数はあなたの新しい文字列を返すと思いますか?

編集:ターミネーター用のスペースを作るために長さを+1します。

于 2012-09-24T00:52:20.357 に答える
3

sizeof 演算子(関数ではありません!)はコンパイル時に評価されます。文字列を保持していると主張するメモリ領域へのポインタを渡します。ただし、この文字列の長さはコンパイル時に固定されていません。sizeof(str)/sizeof(char)アーキテクチャ上のポインタのサイズ、おそらく8または4が常に生成されます。

strlen必要なのは、文字列の長さを決定するために使用することです。

あるいは、これを行うためのより慣用的な方法は、使用することstd::stringです(自分で文字列を逆にすることを主張する場合)

std::string reverse(std::string str) {
  for (std::string::size_type i = 0, j = str.size(); i+1 < j--; ++i) {
    char const swap = str[i];
    str[i] = str[j];
    str[j] = swap;
  }
  return str;
}

暗黙的な変換(オーバーロードを参照)により、プレーンCスタイルのポインターを使用(5)してこの関数を呼び出すこともできることに注意してください。char

于 2012-09-24T00:48:53.093 に答える
2

ここには2つの問題があります。

  1. 演算子は文字列のsizeof長さを教えてくれません。むしろ、それはchar*あなたが使用しているマシン上ののサイズをあなたに与えます。strlen()代わりに使用して、

  2. c文字列はNULL文字で終了します(これがstrlen()文字列の正しい長さを返すことができる理由です)。誤ってNULL文字をソース文字列から宛先文字列の先頭にコピーしていないことを確認する必要があります。また、宛先文字列の最後にNULL文字を追加する必要があります。そうしないと、予期しない出力が発生します。

于 2012-09-24T00:57:47.240 に答える
0
#include <bits/stdc++.h>

using namespace std;

vector<string> split_string(string);

// Complete the reverseArray function below.
vector<int> reverseArray(vector<int> a) {
    return {a.rbegin(), a.rend()};

}

int main()
{
    ofstream fout(getenv("OUTPUT_PATH"));

    int arr_count;
    cin >> arr_count;
    cin.ignore(numeric_limits<streamsize>::max(), '\n');

    string arr_temp_temp;
    getline(cin, arr_temp_temp);

    vector<string> arr_temp = split_string(arr_temp_temp);

    vector<int> arr(arr_count);

    for (int i = 0; i < arr_count; i++) {
        int arr_item = stoi(arr_temp[i]);

        arr[i] = arr_item;
    }

    vector<int> res = reverseArray(arr);

    for (int i = 0; i < res.size(); i++) {
        fout << res[i];

        if (i != res.size() - 1) {
            fout << " ";
        }
    }

    fout << "\n";

    fout.close();

    return 0;
}

vector<string> split_string(string input_string) {
    string::iterator new_end = unique(input_string.begin(), input_string.end(), [] (const char &x, const char &y) {
        return x == y and x == ' ';
    });

    input_string.erase(new_end, input_string.end());

    while (input_string[input_string.length() - 1] == ' ') {
        input_string.pop_back();
    }

    vector<string> splits;
    char delimiter = ' ';

    size_t i = 0;
    size_t pos = input_string.find(delimiter);

    while (pos != string::npos) {
        splits.push_back(input_string.substr(i, pos - i));

        i = pos + 1;
        pos = input_string.find(delimiter, i);
    }

    splits.push_back(input_string.substr(i, min(pos, input_string.length()) - i + 1));

    return splits;
}
于 2020-02-03T09:41:04.247 に答える