質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.79%

stringのoperator+が働かない?

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,299

前提、実現したいこと

C++とNativeActivityを使ったAndroidアプリの開発。
ファイル検索をするための文字列を生成したい。

発生している問題、エラーメッセージ

stringクラスのoperator+が正常な値を返さず、結合する前の文字列を返してしまう。
(下記のコードではstartに「/storage/emulated/0/Android/data/(アプリのパッケージ名)/files」が入っている場合、findstartにもstartと同じ文字列が入ってしまう)
なおoperator+=も同様に文字列を結合してくれない。
また、他の関数では正常に結合された文字列を返すので、この関数のみで起こっている問題の模様。

事象の発生しているコード

    vector<string> EnumFiles(const string& start)
    {
        static const string addfindstart = u8"/*";
        static const string adddirsplit = u8"/";

        vector<string> res;
        res.clear();

        DWORD_PTR handle;
        FILEINFO filebuf;

        if (start.empty()) return res;

        //ここで文字列結合が正常に行われない
        string findstart = start + addfindstart;

        handle = FileRead_findFirst(findstart.c_str(), &filebuf);

        if (handle == -1) return res;

        do
        {
            if (filebuf.DirFlag)
            {
                string dirname = filebuf.Name;

                if (dirname.compare(".") != 0 && dirname.compare("..") != 0)
                {
                    vector<string> addfile = EnumFiles(start + adddirsplit + dirname);
                    for (size_t i = 0, is = addfile.size(); i < is; i++) res.push_back(addfile[i]);
                }
            }
            else res.push_back(start + adddirsplit + filebuf.Name);
        } while (DXFUNC_BOOL(FileRead_findNext(handle, &filebuf)));

        FileRead_findClose(handle);

        return res;
    }

試したこと

Visual Studioを最新版に更新した。
プラットフォームツールセットをClang 5.0に変更した。

補足情報(言語/FW/ツール等のバージョンなど)

Visual Studio 2017の2018年4月3日現在の最新版を使用。
テストに使用した端末はXperia XZ、Androidのバージョンは8.0。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+2

string::operator+ が働いていないかのように見えるサンプル:

#include <iostream>
#include <string>

int main() {
  using namespace std;
  string strA = "abcd"; strA += '\0';
  string strB = "efgh"; strB += '\0';

  string strC = strA + strB;

  cout << "strC  [";
  for ( char ch : strC ) { cout << (ch == '\0' ? '_' : ch); };
  cout << "]\n";

  cout << "c_str [" <<  strC.c_str() << "]\n";

}

/* 実行結果
strC  [abcd_efgh_]
c_str [abcd]*/

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

+1

こんにちは。

簡単に試してみましたが、ちゃんと結合されました。

startに「/storage/emulated/0/Android/data/(アプリのパッケージ名)/files」

std::stringには\0も入りますので、上記文字列を入れる際に何かのミスで最後の\0までstd::stringに入っているということはないでしょうか?
その場合、文字列を追加しても途中に\0が入っています。
取り出し方によっては、その\0で文字列が終端してしまい残りの文字が表示されない場合があります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/04 08:25

    結合後のstringに対し、size()/length()で大きさを調べてみるといいよ。
    途中に'\0'が挟まっていると、strlen(findstart.c_str()) より大きな値が返ってくる。

    キャンセル

  • 2018/04/04 18:11

    Chironianさん、epistemeさんのご指摘の通り文字列に'\0'が混入していてc_str()を関数に渡す過程で文字列が途中で切れてしまっていたようです。(DXライブラリのためc_str()で文字列を渡さなければならない)
    そもそもパスの文字列はGetExternalDataPathという関数で取得するパスをstringに強引に入れていた(バッファオーバーランとパスが途中で切れることを防ぐため)のですが、その過程で'\0'までstringに入ってしまっていたようです。
    パスを取得する過程で末尾の'\0'を消す処理を入れたところ、正常にoperator+が動作したことを確認しました。
    ベストアンサーは回答が早かった方とさせていただきます。
    お二方ともありがとうございました!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.79%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る