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

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

ただいまの
回答率

90.49%

  • C

    3707questions

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

C言語で文字列の文字を検出したい

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 162

aiueoaiueoaiue

score 74

実現したい事

C言語でa(〇,〇)といった文字列を読み込んで〇の部分の文字を表示させたいと考えているのですが、実行した際に処理が止まってしまって上手くいきません。エラーも出てくれないので何がいけないのかわからないので、原因と解決策が分かる方回答お願いします。

ソース

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(void){
    char word1[20] , word2[20]; 
    char *a, *b, *ac, *bc;
    char *decidea1, *decidea2, *decideb1, *decideb2, *kekka;
    int i, j, k;

    strcpy(word1, "a(x,f(y))");
    strcpy(word2, "b(y,z)");

    ac = (char*)malloc(10);
    bc = (char*)malloc(10);
    a = word1;
    b = word2;

    ac = strchr(a, (int)',');
    bc = strchr(b, (int)',');

    i = ac - a-1;
    j = ac - a+1;


    if(a[i] >= 'a' && a[i] <= 'z'){
        printf("%c", a[i]);
        decidea1[0] = a[i];
    printf("%d,%d", i, j);
    }
    else{
        sprintf(decidea1, "%c%c%c%c ", a[i-3], a[i-2], a[i-1], a[i]);
    }
    printf("aの前は%sです\n", decidea1);

    k = j+1;
    if(a[j] >= 'a' && a[j] <= 'z'){
        while(a[k] != '\0'){
        printf("%c\n", a[j]);
            if(a[k] == '('){
                sprintf(decidea2, "%s ", decidea1);
                printf("%s\n", decidea2);
                break;
            }
            else{
                decidea2[0] = a[j];
            }
            k++;
        }
    }
    else{
        sprintf(decidea2, "%c%c%c%c ", a[j], a[j+1], a[j+2], a[j+3]);
    }
    printf("aの後ろは%sです\n", decidea2);

    i = bc - b-1;
    j = bc - b+1;
    if(b[i] >= 'a' && b[i] <= 'z'){
        decideb1[0] = b[i];
    }
    else{
        sprintf(decideb1, "%c%c%c%c ", b[i-3], b[i-2], b[i-1], b[i]);
    }
    printf("bの前は%sです\n", decideb1);

    k = j+1;
    if(b[j] >= 'a' && b[j] <= 'z'){
        while(b[k] != '\0'){
            if(b[k] == '('){
                sprintf(decidea2, "%c%c%c%c ", b[j], b[j+1], b[j+2], b[j+3]);
                break;
            }
            else{
                decideb2[0] = b[j];
                printf("%s\n", decideb2);
            }
            k++;
        }
    }
    else{
        sprintf(decideb2, "%c%c%c%c ", b[j], b[j+1], b[j+2], b[j+3]);
    }
    printf("bの後ろは%sです\n", decideb2);
    sprintf(kekka, "%s  %s, %s %s", decidea1, decideb1, decidea2, decideb2);
    return 0;    
}

補足

ソースは現段階ではベタ打ちです。
ソースの例ではxとf(y)とyとzが表示されてほしいです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • episteme

    2018/07/21 14:42 編集

    a(b(x,y),c(z,t)) にも対応せんならん? だとしたらかなり厄介

    キャンセル

回答 4

+1

まず、decidea1[0] に値を入れていますが、 decidea1は、 char * として宣言されているだけなので、動作は不明(普通は、エラーで落ちると思います)
と思ったら、

char *decidea1, *decidea2, *decideb1, *decideb2, *kekka;

が全て領域確保無しで、使われています。

動作的には問題無いと思うが、、、
ac, bc に malloc()で領域確保していますが、その直後に、 "ac = strchr(.." として値をセットしているので、 malloc()が無意味(リーク)

とりあえず、ここまで。


ちょっと追ってみたのでのですが、結構、面倒。デバッガを使って、動きを追いましょう、、って言いたくなりましたが、それでは回答になりませんので、いくつか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

Windowsでやってるなら、VisualStudioとかEclipseとか、その他にもC言語用デバッガ・統合環境はいろいろあるからそういうのを使ってデバッグしてみれ。
任意のところで実行を止めて変数の値をモニタするとかできるぞ。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

pepperleaf さんが指摘された、5つのポインタ変数を初期化せずに使っている件はどうなったでしょうか?

大げさかもしれないが、あまりに明らかな低品質コードに危機感(笑)を覚えたので、対策のひとつとして手っ取り早いと思う案を示します。

int main(void) {
    char word1[20], word2[20]; 
    char *a, *b, *ac, *bc;
    char *decidea1, *decidea2, *decideb1, *decideb2, *kekka;
    int i, j, k;

    // 次の5行で、ポインタ変数が指すメモリを割当てる(ポインタ初期化)
    decidea1 = malloc(20);
    decidea2 = malloc(20);
    decideb1 = malloc(20);
    decideb2 = malloc(20);
    kekka = malloc(20);
    // 以下省略


それぞれ20バイトも必要無さそうですが、word1[], word2[] の大きさに合わせてみました。
追記
朝ごはん食べたら、よくわかった(苦笑)。こう変えるだけでしたね。

    char decidea1[20], decidea2[20], decideb1[20], decideb2[20], kekka[20];

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

check解決した方法

-1

decidea1[0] = a[i];のような部分を

sprintf(decidea1, "%c", a[i]);のようにすると上手くいきました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/20 23:09

    pepperleaf さんが指摘された、5つのポインタ変数を初期化せずに使っている件はどうなったでしょうか?
    これがそのままなら、解決できたと思うのは早計です。

    「〜のようにすると上手くいきました」・・・これだけで動作するようになったのであれば、たまたま運がよかったにすぎません。どこのメモリ内容を壊すか分からない、他の人のコンピュータでは動作しない可能性の高い、間違ったプログラムです。現に私の手元ではコアダンプを起こし動作しません。

    キャンセル

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

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

関連した質問

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

  • C

    3707questions

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