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

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

ただいまの
回答率

90.04%

C++でのNULL終端文字列の取り扱い

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 2,108

takano20771

score 64

現在,C言語のプロジェクトをC++のプロジェクトとして書き換えています.詳細としては,ESP-8266というマイコンで,FreeOSというRTOSを使ってwi-fiのアクセスポイントへ接続し,ネットへ接続しようとしております.

サンプルプログラムでは,下記のようなプログラムが与えられており,WIFI_SSIDとWIFI_PASSを書き換えることで問題無く接続が可能となっております.しかし,ファイル名をmain.cからmain.cppへと変更すると,C++ 99 "ssid" outside aggregate initializerなどと表示され,コンパイルが通らない状況です.原因として,C99の機能であり,C++では用いることができないということがわかり,これを書き換えたいと考えております.

    #define WIFI_SSID "hoge"
    #define WIFI_PASS "piyo"

    ~~~~~~
    中略
    ~~~~~~

    struct sdk_station_config config = {
        .ssid = WIFI_SSID,
        .password = WIFI_PASS,
    };

そこで,以下のようにプログラムを書換え,対処を行いました.コンパイルエラー自体はこれで回避することができたのですが,実際にプログラムを実行してみると,アクセスポイントの接続に失敗している状態となっており,文字列の取り扱いに問題があるのではないかと推測できる症状が発生しております.sdk_station_configが定義されているヘッダファイルには,NULL TERMINATED STRINGと書かれており,ヌル終端文字列(?)であるようなのですが,このような場合,どのように書き換えるのが適切なのでしょうか?

    #define WIFI_SSID "hoge"
    #define WIFI_PASS "piyo"

    ~~~~~~
    中略
    ~~~~~~

    struct sdk_station_config config;
    *(config.ssid) = WIFI_SSID;
    *(config.password) = WIFI_PASS;

失礼いたしました.ご指摘いただきました,sdk_station_config構造体の定義を記載いたします.

struct sdk_station_config {
    uint8_t ssid[32];     /* Null terminated string */
    uint8_t password[64]; /* Null terminated string */
    uint8_t bssid_set;    /* One if bssid is used, otherwise zero. */
    uint8_t bssid[6];     /* The BSSID bytes */
};

yohhoyさんよりご指摘いただきました,C++コンパイラのバージョンについて追記です.
以下,

$ gcc --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/c++/4.2.1
Apple LLVM version 9.0.0 (clang-900.0.38)
Target: x86_64-apple-darwin17.2.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

また,使用しているOSはOS X 10.13.1です.

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • takano20771

    2017/11/27 15:47

    失礼いたしました.ご指摘いただきました,内容を追記いたしました.yohhoyさんにいただいたURLの記載と同じです.

    キャンセル

  • yohhoy

    2017/11/28 01:39

    エラーメッセージから察するにC++コンパイラはGCCでしょうか?比較的古いバージョンと予想されますので、GCCバージョン(gcc --version出力結果など)も質問中に明記ください。

    キャンセル

  • takano20771

    2017/11/28 10:17

    ご指摘いただきました,C++コンパイラの情報を追記いたしました.

    キャンセル

回答 4

+3

こんにちは。

yohhoyさんが書かれている構造体定義の場合は、次の記述でできると思います。

#include <iostream>

#define WIFI_SSID "hoge"
#define WIFI_PASS "piyo"

#ifdef    __cplusplus
extern "C" {
#endif

struct sdk_station_config {
    uint8_t ssid[32];     /* Null terminated string */
    uint8_t password[64]; /* Null terminated string */
    uint8_t bssid_set;    /* One if bssid is used, otherwise zero. */
    uint8_t bssid[6];     /* The BSSID bytes */
};
}

int main()
{
    struct sdk_station_config config={WIFI_SSID, WIFI_PASS, {}, {}};
    std::cout << config.ssid << "\n";
    std::cout << config.password << "\n";
}

他の方も書かれている通りこのような質問の場合、初期化したい変数の型(の関連部分)を正確に記載しないと正しい回答の難度が跳ね上がります。実は私は最初スルーしてました。yohhoyさんの補足依頼を見てちょっと回答してみたという経緯になります。
また、これが分かっていれば他の回答者様全員、恐らく正しい回答をされているだろうと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/28 14:51

    もしGCC 4.9系を使っているのであれば、コンパイラのバグですね。
    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60859

    キャンセル

  • 2017/11/29 00:50

    ご指摘の通りC++11でコンパイルできるよう引数渡すようmakeファイルに追記しました.

    C++は11,14,17が今あると思うのですがが,14以降ではなく11以降と書かれた意図はどういったところなのでしょうか・・・?

    また,コンパイラのバージョンは質問文に追記させていただきましたが,Clang-900.0.38です.

    キャンセル

  • 2017/12/03 14:54

    Appleのカスタマイズが入ったgccってバージョン番号ほんとに意味不明で困る。少なくとも私には読めん。
    あれって__GCC_MINOR__マクロの値とか当てになるんだろうか・・・

    >C++は11,14,17が今あると思うのですがが,14以降ではなく11以降と書かれた意図はどういったところなのでしょうか・・・?

    そういうわけで14使えるのかわからなかったけど流石に11は使えるだろという思いで書きました

    キャンセル

checkベストアンサー

+2

構造体の初期化において、{}ブロックの中での「.変数 = 値」という書式は、C++では対応していませんのでエラーになります。C++での構造体の初期化は、Cの古典的な方法である定義したのと同じ順番で値だけを並べるか、構造体変数を定義した後でメンバ変数に一つ一つ代入していくか(下のコードのような具合ですね)、あるいはC++の機能として適切なコンストラクターを記述するかのいずれかになります。

ご質問で問題となっている下のコードは、*(~)としてしまっているので、メンバ変数ssidpasswordへの設定ではなく、そのポインタの指し示す領域に値を格納してしまっています。当然、それらのメンバ変数の初期化前なので、不正なアドレスへの書き込みになっています。

アスタリスクは不要(というか余計)なので取ってください。

struct sdk_station_config config;
config.ssid = WIFI_SSID;
config.password = WIFI_PASS;

追記
上記コードはssid,passwordをポインタと仮定したものなので、構造体の中でchar配列として定義しているならstrcpy等を使用してください。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/28 10:04

    yumetodoさん、フォローありがとうございます。

    takano20771さん。「キャスト」は理解できますか? C言語には文字列型が存在しないので、文字列の扱いは「ライブラリー依存」となります。ご利用のライブラリーが文字の基本単位を`uint8_t`としている理由は判りませんが、利用する際はそれに合わせなければいけません。
    基本的なことですが、移植作業は本来であれば当該言語の深い理解が必要とされるものです。言語リファレンス等の熟読をお勧めします。

    キャンセル

  • 2017/11/28 10:49

    ご指摘ありがとうございます.
    キャストは既に試しておりましたが,コードにミスがあったようです.
    ご指摘頂きました通り,reinterpret_cast<char*>(config.ssid)で試したところ,うまく行きました.
    また,(char*)(config.ssid)としてもうまくいったので,おそらく昨日は(char*)config.ssidとでもかいていたのではないかと思います・・・・

    catsforepawさんもありがとうございました.C++の理解を深める必要がありそうです・・・

    キャンセル

  • 2017/11/28 10:51

    教訓: C形式のキャストを使うことなかれ

    キャンセル

+1

多分

struct sdk_station_config config;

sdk_station_config config{};

でいけると思いますが、KoichiSugiyamaさん指摘の通り追記お願いします。特に文字列をどう格納しているかや長さをどう見ているかを確認しておきたいので。

なおその初期化記法、一応C++標準化委員会に提案はされたんですけとねぇ・・・

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/27 17:00

    どうせその初期化が使える要件の定義とPODという概念を消すみたいな話らへんが衝突していただけだと邪推(多分違う

    キャンセル

  • 2017/11/27 23:12

    C99の言語機能に比べるとC++では色々と限定されています。おそらくC言語仕様のままでは柔軟性が高すぎる=プログラマにとって"罠"が多かったので、その辺りの整理・合意形成に時間がかかったのでしょう。(特に初期化順序が主要因と思われます)
    https://stackoverflow.com/questions/18731707/
    https://stackoverflow.com/questions/34614308/

    キャンセル

  • 2017/11/28 00:25

    なるほど.言語体系が複雑なC++ならではといったところなのでしょうか.
    補足ありがとうございます.

    キャンセル

+1

構造体の定義が不明なので以下憶測です。

Cのコードで、
.ssid = WIFI_SSID
としているので、ssidはchar*なのではないでしょうか。
とすれば、C++の方では
config.ssid = WIFI_SSID;
と書けば良いように思えます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/27 16:22

    構造体の定義を追記させていただきました.unsigned charの配列です.

    キャンセル

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

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