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

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

ただいまの
回答率

90.52%

  • C

    3685questions

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

文字列をうまい位置で区切って、共通に含まれる文字の種類数を最大化したい

解決済

回答 1

投稿

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

BONERer

score 11

 前提・実現したいこと

https://beta.atcoder.jp/contests/abc098/tasks/abc098_b
この問題です。
方針は、入力した文字列sを、区切り文字の前と後でそれぞれsa,sbに
分けて、myJudge関数に渡して、それぞれにaからzまでの各文字が
含まれるかを判定して、どちらにもそのアルファベットが含まれた場合のみ
共通の文字があるとしてカウントしていって〜というものです。

 発生している問題・エラーメッセージ

発生している問題:aab|bcaの時にa,bが共通で、myJudge関数の
戻り値が2のはずなのに、printfすると1になっていること。
エラーメッセージはありません。

 該当のソースコード

#include<stdio.h>

int myJudge(char *sa, char *sb);

int main(void)
{
  char s[100+1]={0};
  char sa[100+1]={0};
  char sb[100+1]={0};
  int n=0;
  int ans=0;
  int i=0;
  int k=0;
  int j=0;
  int max=0;

  scanf("%d", &n);
  scanf("%s", s);

  //kは仕切りの位置を表していて
  //[i]と[i+1]の間にあるときk=iとしています
  for(k=0; k<=n-2; k++){

    i=0;
    j=0;

    //saに入れる
    while(i!=k+1){
      sa[i] = s[i];
      i++;
    }
    sa[i] = '\0';
    printf("sa:%s\n", sa);
    //sbに入れる
    while(i<=n-1){
      sb[j] = s[i];
      i++;
      j++;
    }
    sb[j] = '\0';
    printf("sb:%s\n", sb);

    ans = myJudge(sa, sb);
    printf("ans:%d\n", ans);

    if(ans > max){
      max = ans;
    }


  }
  printf("%d\n", max);

  return 0;
}

//aからzの文字を一つずつ、saとsbの両方に含まれるか判定する関数
int myJudge(char *sa, char *sb)
{
  char c;
  int cnt=0;
  int ans1=0,ans2=0;

  for(c='a'; c<='z'; c++){
    ans1=0,ans2=0;

    while(*sa != '\0'){
      if(*sa == c){
    ans1 = 1;
    break;
      }
      sa = sa + 1;
    }
    while(*sb != '\0'){
      if(*sb == c){
    ans2 = 1;
    break;
      }
      sb = sb + 1;
    }
    if(ans1==1 && ans2==1)
      cnt++;

  }

  return cnt;
}

 試したこと

配列をsa,sbに分けれてるか、関数の戻り値の確認等

 補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+2

myJudge関数で、saとsbを直接インクリメントしているためですね。
forループの2周目は常にNULL文字から始まるので、while文が直ぐ抜けてしまうのでしょう。
インクリメント用の変数を用意すれば解決かと思います。

int myJudge(char *sa, char *sb)
{
    char c;
    int cnt=0;
    int ans1=0,ans2=0;
    char *p;

    for(c='a'; c<='z'; c++){
        ans1=0,ans2=0;

        p = sa;
        while(*p != '\0'){
            if(*p == c){
                ans1 = 1;
                break;
            }
            p = p + 1;
        }
        p = sb;
        while(*p != '\0'){
            if(*p == c){
                ans2 = 1;
                break;
            }
            p = p + 1;
        }
        if(ans1==1 && ans2==1)
            cnt++;

    }

    return cnt;
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/29 08:34

    回答ありがとうございます。
    forループの2周目は常にNULLから始まる、というところが理解できませんでした。saのみに注目して書かせていただきますが、
    例えば、1周目でsa"a,\0"のアドレスが60のとき、forループを終えた後ではsaは61、つまりNULL文字を指していることはわかります。
    しかし、2周目ではsaは新しい文字列"a,a,\0"の先頭のaを指していると思うのですが違いますか?
    根拠は、myJudge関数のforループの直前にsaのアドレスを表示させると
    何周目でも、saは60(1周目の文字列の先頭アドレス)を指していることです。
    2周目でsaがNULL文字を指している⇔saの先頭アドレスが61で固定される
    と考えての意見なんですが、僕の意見はどこが勘違いしているか
    教えていただけると幸いです。
    よろしくお願いします。

    キャンセル

  • 2018/05/29 08:48

    二周目と言っているのはforループの二周目です。
    なのでforループの直前(ループ外)でアドレスを出力してもmainから渡されたsa配列の先頭アドレスを表示するだけですので、毎回同じ値になります。
    簡単に説明するとこんな感じでしょうか。
    main(){
     char sa[] = "a,"; // 仮にアドレス0x100とする
     myJudge(sa,sb);
    }
    myJudge(char *sa, char *sb){
     printf("%p\n", sa); // ここでは0x100
     for(省略){
      printf("%p\n", sa); // 1周目は0x100、2周目は0x102
      while(*sa != '\0'){
       sa = sa + 1; // ここでアドレスを加算
      }
      printf("%p\n", sa); // 0x102 NULL文字の位置 このアドレスをforの2周目は引き継ぐ
     }
    }
    実際に動かして確認しているわけではないですし、mainの方は全然見ていないのでこちらも何か勘違いしているところがあるかもしれません。
    不明点疑問点があればツッコミお願いします。

    キャンセル

  • 2018/05/29 13:22

    回答ありがとうございます。
    完全に理解しました。
    毎回sa,sbをmain関数から渡したとしても、aで一致してbreakして
    '\0'までインクリメントして、そのままb以降の判定をするも、'\0'で
    止まっててwhileを抜けて、結局どの組み合わせの文字列を渡しても
    実質aの判定しかできない、ということですね!
    for文のところ勘違いしてて申し訳ありませんでした。
    ありがとうございました。

    キャンセル

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

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

関連した質問

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

  • C

    3685questions

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