文字列比較の条件式について

Question

各行に1単語が羅列している辞書ファイルをコマンドラインから読み取り、標準入力に単語を入力すると二部探索で入力した単語の行数を探し、行数を返すコードです。
質問は、ファイルにない単語を入力した場合、-1を返すように書かなければならないのですが、60行目あたりの条件文でelseのときに-1をzに代入したらよいと思って、以下のコードのように書きました。
しかし、ファイルにない単語を入力しても何も表示されませんでした。ファイルにある単語は表示されます。何がいけないか教えていただきたいです。
また、6行目のint strCompare...から24行目の部分において、int r～の前の部分は日本語でヒントが与えられており、自分で書いたもので、int r~からの部分は資料に書かれていたコードをそのまま使うように言われました。int r～の部分が何を表しているのか教えていただきたいです。
試したこと：zの初期を-1にしておき、elseの時にzをprintするように書きましたが上手くいきませんでした。

#include <stdio.h>
#include <stdlib.h>
typedef struct {
  unsigned char s[24];
} word;
int strCompare(unsigned char *a, unsigned char *b) {
while((*a==*b)&&(*a!=0)&&(*b!=0)){
  a++;
  b++;
 }
  if (*a<*b){
    return -1;
  }else if(*a>*b){
    return 1;
  }else if(*a==*b){
    return 0;
  }
int r = *a - *b;
while((r == 0) && (*a != 0)) {
a++; b++;
r = *a - *b;
}
return r;
}
int main(int argc, char **argv) {
  int i,c,k;
  unsigned char t[24];
  word dict[110000];
  FILE *fp;
  
  if (argc < 2) {
    fprintf(stderr,"usage: ./a.out dictFile\n"); exit(1);
  }
   if ((fp=fopen(argv[1],"r")) == NULL) {
    fprintf(stderr,"cannot open file %s\n",argv[1]); exit(1);
  }
   i=0; k=0;
  while((c=fgetc(fp)) != EOF) {
    if (c == '\n') {
      dict[i].s[k] = 0;
      i++;
      k=0;
    } else {
      dict[i].s[k] = c;
      k++;
    }
  }
  fclose(fp);
  int z;
  int a, b, m;
  while(scanf("%s",t)!=EOF){
    a=0;b=110000;
    while(a<b){
      middle=(a+b)/2;
      if(strCompare(dict[m].s, t)==0){
	z=m+1;
	break;
      }else if (strCompare(dict[m].s, t)>0){
	b=m;
      }else if (strCompare(dict[m].s, t)<0){
	a=m;
      }else{
       z=-1;
      }
      }
    }
    printf("%d\n",z);
  }
  return 0;
}

Comments

[ozwk]

そもそもコンパイルエラーになり、問題を再現できません。
質問文のコードを質問文に書いてある問題が起こるコードに修正してください

[fana]

https://dixq.net/forum/viewtopic.php?f=3&t=21453

[fana]

> int r~からの部分は資料に書かれていたコードをそのまま使うように言われました。

何故言われたことすらやらないのです？？？

> 何がいけないか教えていただきたいです。

てきとーにコードを弄りまわした結果が「上手くいく」可能性に賭けていること，では？
……とか書くと言葉が悪いとか何とか言われそうだけども，要はそういう事でしょう？

> 60行目あたりの条件文でelseのときに-1をzに代入したらよいと思って

の時点で，いじくる前のコードの意味を把握してないことは明白と見える．

[int32_t]

middle は m の間違いでしょうかね。

[dodox86]

マルチポスト先で既に回答をいただいていたのに、teratailにこの質問を投稿しているようですね。どちらの回答もないがしろにする自分勝手な行為。

Answer 1

      if(strCompare(dict[m].s, t)==0){
      }else if (strCompare(dict[m].s, t)>0){
      }else if (strCompare(dict[m].s, t)<0){
      }else{
       z=-1;

strCompare() の結果が0の場合、正の場合、負の場合がそれぞれ処理されているので、それ以外のケースというのは存在しません。} else{ は無意味です。

たとえば、a=100, b=101 のとき m=100 になりますが、strCompare() の結果が0以外だったらもう探索するところがないので内側のループを抜けたいですよね。

• 結果が正の場合は b=100 になり while (a<b) が成立しなくなりループ終了します。
• 結果が負の場合は a=100 になるので無限ループしていしまいますね。

Comments

[fana]

・まず動かしているコードをそのまま貼れ
・変数の未初期化
・無限ループって怖くね？　＋　その対策法
・そのelse意味なくね？

全てマルチポスト先で既出なのに何一つ直すつもりもない模様．

Answer 2

z = -1となった時はmain関数内の変数a、bの値はどちらも変化しないのでwhile文から抜け出せず、最後のprintf関数まで行かないため、結果として何も表示されてないのではないのでしょうか？

Answer 3

ローカル変数のmですが、初期化されてませんね