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

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

新規登録して質問してみよう
ただいま回答率
85.37%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

String

Stringは、ゼロ以上の文字から連続してできた文字の集合を扱うデータ型です。基本的にテキストを表すために使われます。

structure

このタグは、プログラム言語におけるデータ型structure(構造体)に関するタグです。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

Q&A

解決済

2回答

5685閲覧

strcatでコンパイルエラー(c言語)

gravy

総合スコア10

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

String

Stringは、ゼロ以上の文字から連続してできた文字の集合を扱うデータ型です。基本的にテキストを表すために使われます。

structure

このタグは、プログラム言語におけるデータ型structure(構造体)に関するタグです。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

0グッド

0クリップ

投稿2016/12/15 09:48

###前提・実現したいこと
c言語でstrcatを用いて、構造体型の配列を繋げようとしてます。

###発生している問題・エラーメッセージ
コンパイルエラーがなぜ出ているのか、どう改善したらいいのかがわかりません。

21:20: warning: passing 'const char *const' to parameter of type 'char *' discards qualifiers [-Wincompatible-pointer-types-discards-qualifiers] char *s = strcat(tests[1].frst, tests[1].scnd); ^~~~~~~~~~~~~ /usr/include/secure/_string.h:125:27: note: expanded from macro 'strcat' __builtin___strcat_chk (dest, src, __darwin_obsz (dest)) ^~~~

###該当のソースコード
デバッグするため、簡略したので、testsは全要素使っていません。

#include <stdio.h> #include <stdlib.h> #include <string.h> struct testcase { const char *frst; const char *scnd; const char *result; }; const struct testcase tests[] = { {"pine", "apple", "pineaplle"}, {"sea", "shell", "seashell"}, {"break", "fast", "breakfast"} }; int main(){ char *s = strcat(tests[1].frst, tests[1].scnd); printf("%s\n",s); return 0; }

###試したこと
constを消してみました。
結果、Abort trap 6が出てコンパイルエラーでした。

###補足情報(言語/FW/ツール等のバージョンなど)
macでgccを使っています。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

frstは初期化時に文字列リテラルに初期化されています。文字列リテラルは内容を破壊してはならない場所に存在していると思ってください。constを取り去り無理にコンパイルを通し破壊してはならない領域を更新してしまったことが原因だと思います。

文字列の連結を行いたい場合は別途更新してもよい充分なサイズのcharの配列領域を用意してそこで行ってください。

c

1char buff[100]; // 100は適当です。 2strcpy(buff, tests[1].frst); 3strcat(buff, tests[1].scnd); 4printf("%s\n", buff);

投稿2016/12/15 10:16

編集2016/12/15 10:16
KSwordOfHaste

総合スコア18400

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

これ、配列を壊してますよ。
strcatは配列の後ろに配列を結合しますが、結果を格納する側には十分な余裕が必要です。
ですので、tests[].frstは十分なスペースが無いはずですから別の配列を用意して、これにstrcpy()で初めの配列をコピーしてその後ろにtests[].scndをstrcatして処理しないと。

#include <stdio.h> #include <stdlib.h> #include <string.h> struct testcase { const char *frst; const char *scnd; const char *result; }; const struct testcase tests[] = { {"pine", "apple", "pineaplle"}, {"sea", "shell", "seashell"}, {"break", "fast", "breakfast"} }; int main(){ char temp[20]; strcpt(temp,tests[1].frst); strcat(temp,test[1].scnd); printf("%s\n",temp); return 0; }

みたいに。

投稿2016/12/15 10:10

MasahikoHirata

総合スコア3747

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.37%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問