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

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

ただいまの
回答率

90.49%

  • C

    3805questions

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

文字列中任意の文字を削除

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 1,364

ShunyaKogure

score 1

C言語を使って入力された文字列の中から数字のみを削除する、というプログラムを書こうとしてるのですが、数字が連続するとちゃんと動きません。どのように直せば良いのでしょうか?

#include <stdio.h>

int del_digit(char s[]) {
  int j, i = 0;
  int flag = 0;

  while (s[i]) {
    if (s[i] - '0' < 10) {
      if (s[i+1] - '0' < 10) flag = 1;
      do {
        j = i;
        while (s[j]) {
          s[j] = s[j+1];
          j++;
        }
        if ((s[i+1] - '0' > 10) && (s[i] - '0' > 10)) flag = 0;
      } while(flag == 1);
    }
    i++;
  }
  return 0;
}

int main (void) {
  char w[50];
  int j;

  printf("文字列を入力してください");
  scanf("%s", w);

  del_digit(w);

  printf("%s\n", w);
  return 0;
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

checkベストアンサー

+1

どういう手順(ロジック)で処理しようとしているか、考えを整理してみましょう
恐らくやりたいこと以上に難しく考えているのではないでしょうか

コードからは
・文字列の先頭から順に文字の終わりが出るまで処理を繰り返す
・判定文字が数字だったら、次の文字が数字であるかも判定する
・文字を前に詰める(実質的な文字削除)
・次の文字も数字だったらもう一度前に詰める
・今回判定位置の文字と次の文字のどちらも数字以外であれば、さらに次の文字の処理へ進む
ということをしようとしているように読み取れます
・・・
なんだか少しややこしいうえに、1文字だけだったり、数字だけだったりの場合など
いろいろうまくいかないケースがありそうです

やりたいことは
・入力文字列から数字を抜き出した文字列を出力する
ですね

シンプルに手順を考えてみましょう

・文字列の先頭から順に文字の終わりが出るまで処理を繰り返す
・処理対象文字が数字だったら、次文字から後ろの文字を1つずつ前に詰める

なんだか、これだけでうまくいきそうですね

コードを見てみましょう

while (s[i]) {
  if (s[i] - '0' < 10) { // ←数字かどうかを判定(1)
    if (s[i+1] - '0' < 10) flag = 1; //←次の文字が数字かどうかを判定(2)
    do {
      j = i;
      while (s[j]) {; //←前詰処理(実質的に数字を削除)(3)
        s[j] = s[j+1];
        j++;
      }
      if ((s[i+1] - '0' > 10) && (s[i] - '0' > 10)) flag = 0; //←次の文字と今回文字が数字以外なら前詰処理を抜ける(4)
    } while(flag == 1);
  }
  i++;
}

(1) と (3) は必要な処理ですが、(2) と (4) はいかがでしょう?本当に必要でしょうか?

外してみるとこんな感じになります

while (s[i]) {
  if (s[i] - '0' < 10) { // ←数字かどうかを判定(1)
    j = i;
    while (s[j]) {; //←前詰処理(実質的に数字を削除)(3)
      s[j] = s[j+1];
      j++;
    }
  }
  else{ //←前詰後は i の位置に入った次の文字を数字かどうか判定したいので i++ しないための else
    i++;
  }
}

いかがでしょうか

なお、もっと効率的でスマートなやり方については他の方の回答が大変参考になります
それぞれにどのような考え方で処理しているか読み解くと勉強になると思います

以上ご参考まで

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

//#include <ctype.h>を要する
void del_digit(char s[])
{
  int si, pi, len = strlen(s);
  char* p = (char*)malloc(sizeof(char) * (1 + len) );

  for(si = pi = 0; si < len; si++){
    if(!isdigit(s[si]) ){
      p[pi++] = s[si];
    }
  }
  p[pi] = '\0';
  strcpy(s, p);
  free(p);
}

私ならこう作ります。動作未確認

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

数字を削除するのであれば、(ご指摘を受け、jの初期化を入れました、ありがとうございます)

#include <stdio.h>

char *del_digit(char s[]) {
  int i,j;
  for (i=j=0; s[i]; i++) {
    if (s[i] - '0' >= 10) {
        s[j]=s[i];
        j++;
    }
  }
  s[j]=s[i];
  return s;
}

int main (void) {
  char w[50];

  printf("文字列を入力してください\n");
  scanf("%s", w);

  printf("%s\n", del_digit(w));
  return 0;
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/01/07 16:36

    jを初期化していないようです。

    キャンセル

0

void del_digit(char *s) {
    char c;
    char *p = s;

    while ((c = *s++))
        if (c < '0' || '9' < c)
            *p++ = c;
    *p = '\0';
}

動きに若干の無駄がある。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

関連した質問

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

  • C

    3805questions

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