２次元配列の文字列をポインタで扱う問題です

Question

Shiftleftという関数を用いて、各文字列を左に一文字ずらし、先頭の文字を一番後ろにくっつけるという課題なんですが、

#include<stdio.h>

void shiftleft(char (*str)[16] , int n){
  
  int a , b = 0 , c ;
  char tmp;
  for(a = 0 ; a < n ; a++){
    while((*str[b]) != '\0'){
      b++;
    }
    tmp = (*str[0]); 
    for(c = 0 ; c < b ; c++){
      (*str[c]) = (*str[c]) + 1 ;
    }
    (*str[b-1]) = tmp;
    b = 0;
    str++;
  }
}

int main(){
  int i ;
  int j;
  char str2[4][16] = {"Tokyo" , "Osaka" , "Fukuoka" , "Nagoya" };
  
  printf("文字列を出力します\n") ;    
  for(i = 0 ; i < 4 ; i++){
    printf("%s\n" , str2[i] );
  }
  
  j = sizeof(str2) ;
  printf("配列の大きさは%dバイトです。\n" , j);
  
  
  shiftleft(str2 , 4);
  
  
  printf("文字列を出力します\n") ;    
  for(i = 0 ; i < 4 ; i++){
    printf("%s\n" , str2[i] );
  }
  
  return 0;
  
}

出力結果を見ると、そもそも文字列をずらすことすらできていません。
２次元配列をポインタで渡した時の操作があまり理解できてないのでご協力をお願いしたいです。

Answer 1

こんにちは。

char (*str)[16]は、char型16個の配列へのポインタです。
i番目の文字は、(*str[i])ではなく(*str)[i]です。

*演算子と[]演算子では[]演算子の方が優先順位が高いため、(*str[i])と記述すると、*(str[i])と解釈されます。

例えば、i=1の時、strはchar型16個の配列へのポインタですから、str[1]は次のchar型16個の配列へのポインタとなります。つまり、str[0]が指すところの16バイト先を指してしまいます。

この考え方で数カ所修正するば仕様通り動作するようです。
あともう少しのようですので頑張って下さい。

Comments

[TK0107]

ありがとうございました！！

Answer 2

コードを見ると混乱しているように見えるので、こんな感じにすると分かり易いかなと思って。参考までに。

void shiftleft_2(char str[][16], int n) {
	int i = 0;
	printf("str[][16]と書いても\n");
	printf("引数で渡すとこんな感じ^^ --> %d\n", sizeof str);
	// (*str)[16]と同じですが、分かり易いかなと^^、

	printf("%c \n", str[0][0]);
	printf("%c \n", str[1][0]);
	printf("%c \n", str[2][0]);
	printf("%c \n", str[3][0]);

	// mainの方で使っているコードですが
	for (i = 0; i < n; ++i) {
		// str[0]からstr[3]はchar[16]の変数名、配列の変数名は。。。
		printf("%s \n", str[i]);
	}
}

Comments

[TK0107]

回答ありがとうございます。
今回は課題が文字列をポインタとして扱うことが条件なので、文字列を直接触るのはダメみたいです・・・
すみません

Answer 3

b番目の文字は

(*str[b]) → (*str)[b]