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

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

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

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

Q&A

解決済

4回答

1237閲覧

C言語 ダブルポインタ

junnnnchan

総合スコア26

C

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

0グッド

1クリップ

投稿2020/06/05 02:07

編集2020/06/05 03:25
#include <stdio.h> #include <string.h> #include <stdlib.h> #define String_Max 80 /*--- 文字列型スタックを実現する構造体 ---*/ typedef struct{ int max; /* スタックの容量 */ int ptr; /* スタックポインタ */ char **stk; /* スタック本体(char* へのポインタ配列 )*/ } StringsStack; int Search(StringsStack *s , char *x){ for(int i = s->ptr ; i <= 0 ; i --){ if(strncmp(s->stk[i], x ,strlen(x)) == 0){//strncmp先頭n文字を比較 return i; } } return -1; } /*--- スタックの初期化 ---*/ int Initialize(StringsStack *s, int max){ s->ptr = 0; if ((s->stk = calloc(max, sizeof(char *))) == NULL) { s->max = 0; /* char* の配列の確保に失敗 */ return -1; } /* char* の配列の確保に成功 */ s->max = max; return 0; } /*--- スタックの後始末 ---*/ void Terminate(StringsStack *s){ if (s->stk != NULL){ while( --s->ptr >= 0) free(s->stk[s->ptr]); /* 動的な文字列保存用配列を解放 */ free(s->stk); } s->max = s->ptr = 0; } /*--- スタックにデータをプッシュ ---*/ int Push(StringsStack *s, char *x){ if (s->ptr >= s->max) return -1; /* スタック満杯 */ if ((s->stk[s->ptr] = calloc(strlen(x)+1, sizeof(char))) == NULL) /* データをコピーするための動的な文字列保存用配列を確保することに失敗 */ return -1; strcpy(s->stk[s->ptr], x); s->ptr++; return 0; } /*--- スタックからデータをポップ ---*/ int Pop(StringsStack *s, char *x){ if (s->ptr <= 0) return -1; /* スタックは空 */ s->ptr--; strcpy(x, s->stk[s->ptr]); free(s->stk[s->ptr]); /* ポップしたので,動的な文字列保存用配列を解放 */ return 0; } /*--- スタックからデータをピーク ---*/ int Peek(StringsStack *s, char *x){ if (s->ptr <= 0) return -1; strcpy(x, s->stk[s->ptr - 1]); return 0; } /*--- スタックの容量 ---*/ int Capacity(const StringsStack *s){ return s->max; } /*--- スタックに積まれているデータ数 ---*/ int Size(const StringsStack *s){ return s->ptr; } /*--- スタックの全データの表示 ---*/ void Print(const StringsStack *s){ int i; for(i = 0; i < s->ptr; i++) printf("%s\n", s->stk[i]); } int main(void){ StringsStack s; int max; printf("スタックの大きさを入力してください"); scanf("%d", &max); if (Initialize(&s, max)==-1){ puts("スタックの生成に失敗しました.\n"); return 1; } while (1) { int menu; char x[String_Max]; printf("現在のデータ数:%d/%d\n",Size(&s), Capacity(&s)); printf("(1)プッシュ (2)ポップ (3)ピーク (4)表示 (5)パターン探索 (0)終了:"); scanf("%d", &menu); if (menu == 0) break; switch (menu) { case 1: /* プッシュ */ printf("プッシュする文字列:"); scanf("%s", x); if (Push(&s, x) == -1) puts("\a エラー:プッシュに失敗しました.\n"); break; case 2: /* ポップ */ if (Pop(&s, x) == -1) puts("\a エラー:ポップに失敗しました.\n"); else{ printf("ポップした文字列は%s,", x); } break; case 3: /* ピーク */ if (Peek(&s, x) == -1) puts("\a エラー:ピークに失敗しました.\n"); else{ printf("ピークした文字列は%s,", x); } break; case 4: /* 表示 */ Print(&s); break; case 5: printf("探索する文字列"); scanf("%s",x); int search = Search(&s,x); if(search == -1) puts("パターンは存在しません\n"); if(search != -1) printf("%dにあります\n",search); break; } } Terminate(&s); return 0; }
#include <stdio.h> #include <string.h> #include <stdlib.h> #define String_Max 80 /*--- 文字列型スタックを実現する構造体 ---*/ typedef struct{ int max; /* スタックの容量 */ int ptr; /* スタックポインタ */ char **stk; /* スタック本体(char* へのポインタ配列 )*/ } StringsStack; int Search(StringsStack *s , char *x){ for(int i = s->ptr ; i <= 0 ; i --){ if(strncmp(s->stk[i], x ,strlen(x)) == 0){//strncmp先頭n文字を比較 return i; } } return -1; } /*--- スタックの初期化 ---*/ int Initialize(StringsStack *s, int max){ s->ptr = 0; if ((s->stk = calloc(max, sizeof(char *))) == NULL) { s->max = 0; /* char* の配列の確保に失敗 */ return -1; } /* char* の配列の確保に成功 */ s->max = max; return 0; } /*--- スタックの後始末 ---*/ void Terminate(StringsStack *s){ if (s->stk != NULL){ while( --s->ptr >= 0) free(s->stk[s->ptr]); /* 動的な文字列保存用配列を解放 */ free(s->stk); } s->max = s->ptr = 0; } /*--- スタックにデータをプッシュ ---*/ int Push(StringsStack *s, char *x){ if (s->ptr >= s->max) return -1; /* スタック満杯 */ if ((s->stk[s->ptr] = calloc(strlen(x)+1, sizeof(char))) == NULL) /* データをコピーするための動的な文字列保存用配列を確保することに失敗 */ return -1; strcpy(s->stk[s->ptr], x); s->ptr++; return 0; } /*--- スタックからデータをポップ ---*/ int Pop(StringsStack *s, char *x){ if (s->ptr <= 0) return -1; /* スタックは空 */ s->ptr--; strcpy(x, s->stk[s->ptr]); free(s->stk[s->ptr]); /* ポップしたので,動的な文字列保存用配列を解放 */ return 0; } /*--- スタックからデータをピーク ---*/ int Peek(StringsStack *s, char *x){ if (s->ptr <= 0) return -1; strcpy(x, s->stk[s->ptr - 1]); return 0; } /*--- スタックの容量 ---*/ int Capacity(const StringsStack *s){ return s->max; } /*--- スタックに積まれているデータ数 ---*/ int Size(const StringsStack *s){ return s->ptr; } /*--- スタックの全データの表示 ---*/ void Print(const StringsStack *s){ int i; for(i = 0; i < s->ptr; i++) printf("%s\n", s->stk[i]); } int main(void){ StringsStack s; int max; printf("スタックの大きさを入力してください"); scanf("%d", &max); if (Initialize(&s, max)==-1){ puts("スタックの生成に失敗しました.\n"); return 1; } while (1) { int menu; char x[String_Max]; printf("現在のデータ数:%d/%d\n",Size(&s), Capacity(&s)); printf("(1)プッシュ (2)ポップ (3)ピーク (4)表示 (5)パターン探索 (0)終了:"); scanf("%d", &menu); if (menu == 0) break; switch (menu) { case 1: /* プッシュ */ printf("プッシュする文字列:"); scanf("%s", x); if (Push(&s, x) == -1) puts("\a エラー:プッシュに失敗しました.\n"); break; case 2: /* ポップ */ if (Pop(&s, x) == -1) puts("\a エラー:ポップに失敗しました.\n"); else{ printf("ポップした文字列は%s,", x); } break; case 3: /* ピーク */ if (Peek(&s, x) == -1) puts("\a エラー:ピークに失敗しました.\n"); else{ printf("ピークした文字列は%s,", x); } break; case 4: /* 表示 */ Print(&s); break; case 5: printf("探索する文字列"); scanf("%s",x); int search = Search(&s,x); if(search == -1) puts("パターンは存在しません\n"); if(search != -1) printf("%dにあります\n",search); break; } } Terminate(&s); return 0; }

パターンの文字列を指定すると,スタック上の文字列の中から入力したパターンを,
スタックの頂上から底に向かって探索し,最初に見つけたパターンを含む文字列のスタックポインタの値を返す関数 Searchを作りたいです。
パターンの文字列を指定すると,スタック上の文字列の中から入力したパターンを,
スタックの頂上から底に向かって探索し,最初に見つけたパターンを含む文字列のスタックポインタの値を返す関数 Searchを自分でつくりました。

int Search(StringsStack *s , char *x)の引数と型はこのままでお願いします。

<追記>

#include <stdio.h> #include <string.h> #include <stdlib.h> #define String_Max 80 /*--- 文字列型スタックを実現する構造体 ---*/ typedef struct{ int max; /* スタックの容量 */ int ptr; /* スタックポインタ */ char **stk; /* スタック本体(char* へのポインタ配列 )*/ } StringsStack; int Search(StringsStack *s , char *x){ for(int i = s->ptr ; i <= 0 ; i --){ if(strncmp(s->stk[i], x ,strlen(x)) == 0){//strncmp先頭n文字を比較 return i; } } return -1; } /*--- スタックの初期化 ---*/ int Initialize(StringsStack *s, int max){ s->ptr = 0; if ((s->stk = calloc(max, sizeof(char *))) == NULL) { s->max = 0; /* char* の配列の確保に失敗 */ return -1; } /* char* の配列の確保に成功 */ s->max = max; return 0; } /*--- スタックの後始末 ---*/ void Terminate(StringsStack *s){ if (s->stk != NULL){ while( --s->ptr >= 0) free(s->stk[s->ptr]); /* 動的な文字列保存用配列を解放 */ free(s->stk); } s->max = s->ptr = 0; } /*--- スタックにデータをプッシュ ---*/ int Push(StringsStack *s, char *x){ if (s->ptr >= s->max) return -1; /* スタック満杯 */ if ((s->stk[s->ptr] = calloc(strlen(x)+1, sizeof(char))) == NULL) /* データをコピーするための動的な文字列保存用配列を確保することに失敗 */ return -1; strcpy(s->stk[s->ptr], x); s->ptr++; return 0; } /*--- スタックからデータをポップ ---*/ int Pop(StringsStack *s, char *x){ if (s->ptr <= 0) return -1; /* スタックは空 */ s->ptr--; strcpy(x, s->stk[s->ptr]); free(s->stk[s->ptr]); /* ポップしたので,動的な文字列保存用配列を解放 */ return 0; } /*--- スタックからデータをピーク ---*/ int Peek(StringsStack *s, char *x){ if (s->ptr <= 0) return -1; strcpy(x, s->stk[s->ptr - 1]); return 0; } /*--- スタックの容量 ---*/ int Capacity(const StringsStack *s){ return s->max; } /*--- スタックに積まれているデータ数 ---*/ int Size(const StringsStack *s){ return s->ptr; } /*--- スタックの全データの表示 ---*/ void Print(const StringsStack *s){ int i; for(i = 0; i < s->ptr; i++) printf("%s\n", s->stk[i]); } int main(void){ StringsStack s; int max; printf("スタックの大きさを入力してください"); scanf("%d", &max); if (Initialize(&s, max)==-1){ puts("スタックの生成に失敗しました.\n"); return 1; } while (1) { int menu; char x[String_Max]; printf("現在のデータ数:%d/%d\n",Size(&s), Capacity(&s)); printf("(1)プッシュ (2)ポップ (3)ピーク (4)表示 (5)パターン探索 (0)終了:"); scanf("%d", &menu); if (menu == 0) break; switch (menu) { case 1: /* プッシュ */ printf("プッシュする文字列:"); scanf("%s", x); if (Push(&s, x) == -1) puts("\a エラー:プッシュに失敗しました.\n"); break; case 2: /* ポップ */ if (Pop(&s, x) == -1) puts("\a エラー:ポップに失敗しました.\n"); else{ printf("ポップした文字列は%s,", x); } break; case 3: /* ピーク */ if (Peek(&s, x) == -1) puts("\a エラー:ピークに失敗しました.\n"); else{ printf("ピークした文字列は%s,", x); } break; case 4: /* 表示 */ Print(&s); break; case 5: printf("探索する文字列"); scanf("%s",x); int search = Search(&s,x); if(search == -1) puts("パターンは存在しません\n"); if(search != -1) printf("%dにあります\n",search); break; } } Terminate(&s); return 0; }

int Search(StringsStack *s , char *x)は,スタックに積まれている文字列の中から,パターンの文字列(x の先頭からパターンが入っている)で指定された文字列を,スタックの頂上から底に向かって探
索してし,最初に見つけたパターンを含む文字列のスタックポインタの値を戻り値として返すようにししたいです.ただし,パターンが見つからなかった場合は,-1 を返すようにします。
[2] aa
[1] bb
[0] cc
とスタックが積まれている場合、探索パターンをbとすると、スタックポインタがさしている1を返すようにsearch関数を実装したいです。
しかし自分のコードでは、常に-1が返されるようにsearch関数が実装されてしまっています。
<質問>
エラーは起こらないのですが、search関数がうまく動作していません。 for文の条件が違うとご指摘されたのですが、頂上から底まで探索するならどこが間違っているのか自分ではわかりませんでした。そこを教えていただきたいです。
search関数は最初に見つけたパターンを含む文字列のスタックポインタの値を戻り値として返すようにしたいです。

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

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

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

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

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

ozwk

2020/06/05 02:24 編集

Search()関数内で返している s->stk[i] は char* ですが、この関数の返り値の型はintです。 「文字列のスタックポインタ」とはint型なんですか?
ozwk

2020/06/05 02:24

あと単純にcase 5の下の行のprintfの;が抜けてます
junnnnchan

2020/06/05 02:27

あ、ほんとですね(笑)
episteme

2020/06/05 02:45

そんなエラー出んのやけど...もしかして質問コードを編集した?
fana

2020/06/05 02:47

この質問みたく,質問内容やコードを逐次変えていくやり方,ほんとやめてほしい.
episteme

2020/06/05 02:50

↑やるならせめて"追記"して積み上げておくれよ...
episteme

2020/06/05 02:53

おっと、こんどは質問がなくなったぞ?
junnnnchan

2020/06/05 02:58

追記とはここに書くということですか?
DreamTheater

2020/06/05 03:00

評価下げる理由、何にすれば良いのやらWw
episteme

2020/06/05 03:00

もとの質問はそのまま、書き換えではなく"書き足せ"ってこと。 で、質問はなんですか?
junnnnchan

2020/06/05 03:07

エラーは起こらないのですが、search関数がうまく動作していません。 for文の条件が違うとご指摘されたのですが、頂上から底まで探索するならどこが間違っているのか自分ではわかりませんでした。そこを教えていただきたいです。
episteme

2020/06/05 03:07

編集するなら: 質問・回答・修正...の「一連の流れが追えるように」してくれってことなんだが。 # それができんなら元質問を編集するな、なんだが。
junnnnchan

2020/06/05 03:08

承知しました。ご指摘ありがとうございます
episteme

2020/06/05 03:13 編集

> エラーは起こらないのですが、search関数がうまく動作していません。... なぜそれを質問に書かんのだ!? うまく動作したらどうなるハズで、実際にはどうなるんだ? 「うまく動作しない」だけじゃ通じないぞ。症状を説明しないとお医者は薬が出せないぞ。
junnnnchan

2020/06/05 03:13

更新しました。 この書き方でだいじょうぶでしょうか?
episteme

2020/06/05 03:16 編集

ダメ。「期待するふるまい」と「実際のふるまい」を明記せよ。 関数 Search の”仕様”を説明せよ。それはあなたしか書けない。
guest

回答4

0

Search は「スタックポインタの値を返す」のに、なぜ定義が

C

1int Search(...)

で正しいと思えるのですか?

int はポインタではありませんよ?
だから「integer を返す定義なのに、pointer を返してるけど、キャストしなくていいの?」と警告が出ているのです。

投稿2020/06/05 02:13

tacsheaven

総合スコア13703

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

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

DreamTheater

2020/06/05 02:30

> キャストしなくていいの? 付け加えると、intにキャストすれば良いという意味ではありません。
episteme

2020/06/05 02:56 編集

> 「integer を返す定義なのに、pointer を返してるけど、キャストしなくていいの?」と警告が出ているのです。 ちがうよ...あ、コードがコロコロ書き換えられてるのか。
Daregada

2020/06/05 03:38

はい。最初は `s->stk[i]` がreturnされていました。
episteme

2020/06/05 03:48

質問主に振り回されてる感www
guest

0

ベストアンサー

C

1int Search(StringsStack* s, const char* x) { 2 for (int i = s->ptr-1; i >= 0; i--) { 3 if (strncmp(s->stk[i], x, strlen(x)) == 0) {//strncmp先頭n文字を比較 4 return i; 5 } 6 } 7 return -1; 8}

投稿2020/06/05 03:33

episteme

総合スコア16614

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

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

junnnnchan

2020/06/05 03:41

s->ptrはpushのあとプラスされて、s->ptrはいま何も入っていないからiの初期値をs->ptr-1にしているということでしょうか?
episteme

2020/06/05 03:46 編集

s->ptr は初期値0でPushのたびに+1されるから、p->ptrは要素数に等しいよね。 要素数がNのとき、stk[0]~stk[N-1] に要素が詰まってるよね。 「だから」です。
junnnnchan

2020/06/05 03:52

なるほど!理解できました。 ありがとうございます。
guest

0

この人のコードの「スタックポインタ」はインデックス的な整数値なので、Searchが成功したときにポインターを返しているのがそもそもの間違いで、「return i;」にすりゃいいんじゃないですかね。
なお、現在のところ、for文の条件式を見ると、そのreturnには絶対に到達しないけど。

投稿2020/06/05 02:33

Daregada

総合スコア11990

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

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

junnnnchan

2020/06/05 02:35

絶対に到達しないとはどういうこどでしょうか?
Daregada

2020/06/05 02:42

構造体のメンバー ptr は、各関数を読むと非負の整数値の範囲で変動しますね。Search内のfor文が利用する変数 i は、それを初期値とするので、やはり非負の整数値です。あとは自分で条件式を見てください。
junnnnchan

2020/06/05 02:45

pushするとs->ptrがプラスされるので、スタック頂上から探索するには、s->ptrから0までループさせる必要があると考えました。
ozwk

2020/06/05 02:47

不等号の向き
Daregada

2020/06/05 02:48

考え方はあっているよ。非負の整数値から0までループさせるための継続条件の条件式が間違っている。
guest

0

main.c: In function ‘Search’: main.c:16:16: warning: return makes integer from pointer without a cast [-Wint-conversion] return s->stk[i];

ワーニングメッセージのとおりです。s->stk[i]char *型ですが、この関数の返り値はint型です。

何を返したいのですか?

投稿2020/06/05 02:12

maisumakun

総合スコア145121

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

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

junnnnchan

2020/06/05 02:20

return をint型に変更しました。 しかし、searchのエラーが消えません。。。
maisumakun

2020/06/05 02:22

何をどう書き換えたのですか?
junnnnchan

2020/06/05 02:26

更新されてませんでした。。。 いま更新しました。
junnnnchan

2020/06/05 02:30

...Program finished with exit code 137 Press ENTER to exit console.main.c:126:25: warning: passing argument 2 of ‘Search’ from incompatible pointer type [-Wincompatible-pointer-types] main.c:12:5: note: expected ‘char *’ but argument is of type ‘char (*)[80]’ エラーは消えたのですが、うまく処理がされません
maisumakun

2020/06/05 02:30

ワーニングメッセージのとおりです。Searchの引数の型が正しくありません(&xは「char (*)[80]」型になってしまいます)。
maisumakun

2020/06/05 02:31

どこまで実行されているのか、デバッガやprintfなどで追いかけてみましょう。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問