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

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

ただいまの
回答率

90.49%

  • C

    3707questions

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

C言語 strstr関数の自作をしています

解決済

回答 4

投稿

  • 評価
  • クリップ 1
  • VIEW 493

P_Beginner

score 13

前回の質問でstrstr関数の自作を試みたのですが、色々だめだったらしいので
新しくプログラムをたてたのですが、セグメントエラーが出るのでどうしたらよいでしょうか。

可能な限り改善、改良できるところをあげてほしいです。

str1にstr2が複数含まれる場合は最初の先頭位置のみわかればよいです。
一応入れてませんがmain関数の方に
strlen(str1)<strlen(str2)
のif文を追加するつもりです。

#include<stdio.h>

char *strPointSearch(char *str1,char *str2);

int main(void)
{
char str1[]={},str2[]={},point;

printf("str1を入力---->");
gets(str1);
printf("str2を入力---->");
gets(str2);

point=*strPointSearch(str1,str2);
printf("%d番目\n",point);

return 0;
}

char *strPointSearch(char *str1, char *str2)
{
int i, j, k;

if(*str2 == '\0')
return str1;
else
{
for(i = 0;*(str1 + i) != '\0';i++)
{
if(*(str1 + i) == *str2)
{
for(j = i, k = 0;*(str1 + j) == *(str2 + k);j++, k++);
if(*(str2 + k) == '\0')
return str1 + i;
}
}
return NULL;
}
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • Zuishin

    2018/04/29 21:14

    https://teratail.com/questions/123972 同じ質問を繰り返さないでください。

    キャンセル

  • Zuishin

    2018/04/29 21:16

    strstr は自作しなくてもあります。他人に作ってもらっても構わないなら標準のものをつくってください。あるいはソースを見ればそれで解決でしょう。

    キャンセル

  • Zuishin

    2018/04/29 21:16

    自作ということなら自分で作ってください。

    キャンセル

回答 4

checkベストアンサー

0

google で "c strstr 実装" で検索して、実装例をいろいろさがして、自分のコードと比較してみてください。

そして、ぜひ テストコードを書いてみてください。
考えられるいろいろいろなケースを列挙し、実行結果が仕様と一致するかを確認するテストプログラムを書くのです。

テストコード例は以下でみることができます。
https://sourceware.org/git/?p=glibc.git;a=blob;f=string/test-strstr.c;h=acf6ff8224608737701046a421faed2be9f52f68;hb=HEAD

https://github.com/gcc-mirror/gcc/blob/e11be3ea01eaf8acd8cd86d3f9c427621b64e6b4/gcc/testsuite/gcc.c-torture/execute/builtins/strstr.c

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

私なら以下のように作ります。動作未確認。

int myCompare(const char* p, const char* q)
{
  for(; q; p++, q++){
    if(*p != *q){
      return 0;
    }
  }
  return 1;
}
char* myStrstr(const char* p, const char* const q)
{
  for(; p;p++){
    if(myCompare(p, q) ){
      return p;
    }
  }
  return NULL;
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

可能な限り改善、改良できるところをあげてほしいです。 

まず、インデントしてください。
記述方法に好き嫌いはあるでしょうが、全く無いと見ずらいです。(バグが見つけにくい)
*(str1 + i)  => str1[i] と書いた方が見やすいですが、拘りがありますか?
見つからなかった場合、NULL を返しますが、その場合、mainがではどうなるでしようか? => ほぼ、確実にエラー

strlen(str1)<strlen(str2)

無くても問題ない筈。 => 見つからないだけ。
とりあえず気づいた範囲で。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

急がば回れではないですが、GWですので、
是非、名著「プログラミング言語C」共立出版、カーニハン著・石田晴久訳を読んでみることをお勧めします。(石田先生はあちらにいかれてしまったのですね)
標準関数strxxxなどを、どのように作るかの説明もあったような気がします(うろ覚えです。間違っていたらすみません。本を探したけどありませんでした)。
私はこの本で教えられ、教えもしました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/29 22:03

    > 名著「プログラミング言語C」
    止めましょう。 名著かも知れませんが、古すぎ。文法も違いすぎ。記憶に間違いなければ、関数の宣言も古い形式の筈

    キャンセル

  • 2018/04/29 22:14

    了解しました。

    キャンセル

  • 2018/04/29 22:59

    著者名にリッチーの名が無い…

    > pepperleaf さん
    第二版であればANSI C(C89/C90)準拠であるため、それほど古いとは言えません。互換性や特殊な環境でC99対応コンパイラがないなどの理由で未だにC90が使われる場合があります。と言っても、Cを全く知らない初心者が読むような本じゃないのは確かですけど。

    キャンセル

  • 2018/04/30 10:06

    raccyさん、
    確かに。第二版、忘れてました。話は聞いたけど、もういいや、だったのも確かです。
    ただ、初心者に、C90とか、 C99 と言っても意味不明では? それと、現実のプログラムは、使っているコンパイラの仕様に依存するので、最近、興味無くなっています。(と言うより、追いかけきれない)

    キャンセル

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

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

関連した質問

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

  • C

    3707questions

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