エラーメッセージの意味が分からない

Question

###発生している問題・エラーメッセージ
文字列配列を辞書順にソートしてから表示するプログラムです。エラーメッセージは恐らく、無効な引数(型がint) のような内容が書いてあると思うのですが、何故intが出てくるのかが特に分かりません。

invalid type argument of unary ‘*’ (have ‘int’)

###該当のソースコード

#include<stdio.h>

void dictsort(char **,int);
int strcmp(char *,char *);

int main(void){
  int i;
  char *colors[]={"white","black","red","pink","orange","blue","yellow","green","purple","gray","brown","gold","silver"};

  dictsort(*colors[13][10],13); //ここでエラーメッセージ
  for(i=0;i<13;i++){
    printf("%s,",colors[i]);
  }
  printf("\n");
  return 0;
}


void dictsort(char **ptr,int y){

  int i=0,count=0,hako;

  while(count < y){
    while(i < y){
      if(strcmp(*(ptr+i),*(ptr+i+1)) == 1){
	**ptr=hako;
	**(ptr+1)=**ptr;
	hako=**(ptr+1);
      }
      i++;
    }
  }
}
int strcmp(char *str1,char *str2){
    int result;
    
    for(; *str1 == *str2; str1++,str2++){
      if(*str1 == '\0')return 0;
    }
    if(*str1-*str2 < 0){result=-1;}
    if(*str1-*str2 == 0){result=0;}
    if(*str1-*str2 > 0){result=1;}
    
    return result;
  }

Answer 1

invalid type argument of unary ‘*’

は、直訳すると「単一''の無効な引数」となり、
意訳すると「''を単一では用いることはできない」となります。
'*'は、文字リテラル以外は、掛け算としての演算子もしくはポインター演算子のどちらかしか無いので

*colors[13][10]

はどちらでもない単一の使い方と解釈されてエラーになっています。

Answer 2

char *colors[]={"white","black","red",... };

であるから、
colors[i] の型は char*
colors[i][j] の型は char
*colors[i][j] の型は...ありえへん! と言うてはります。

Comments

[退会済みユーザー]

大変明快で分かりやすいです。回答いただきありがとうございます。

Answer 3

質問文のコードを元に書きなおしてみました。

#include <stdio.h>

void dictsort(char **, int);
int xstrcmp(const char *, const char *);

int main(void) {
  char *colors[] = {
    "white", "black",  "red",   "pink",   "orange",
    "blue",  "yellow", "green", "purple", "gray",
    "brown", "gold",   "silver"};
  int num = sizeof(colors) / sizeof(char**);

  dictsort(colors, num); //ここでエラーメッセージ
  for (int i = 0; i < num; i++) {
    printf("%s, ", colors[i]);
  }
  printf("\n");
  return 0;
}

void dictsort(char **ptr, int num) {
  int count = 0;

  while (count < num) {
    int i = 0;
    while (i < num - 1) {
      if (xstrcmp(ptr[i], ptr[i + 1]) == 1) {
        // ptr[i] と ptr[i+1] を交換する
        char * tmp = ptr[i];
        ptr[i]= ptr[i + 1];
        ptr[i + 1] = tmp;
      }
      i++;
    }
    count++;
  }
}

// See http://chausson.eng.kagawa-u.ac.jp/Comp/Prog/DosC/Sec6/Note627.html
int xstrcmp(const char *str1, const char *str2) {
  //--  列長が短い方まで、文字同士の比較
  for (; *str1 != '\0' && *str2 != '\0'; str1++,str2++ ) {
    if (*str1 < *str2) { return -1; }
    if (*str1 > *str2) { return  1; }
  }
  //--  残った文字による判定
  if (*str2 != '\0' ) { return -1; }    // str1 が短いとき
  if (*str1 != '\0' ) { return +1; }    // str2 が短いとき
  return 0;                             // 一致するとき
}

実行結果例

$ gcc x.c

$ ./a.out
black, blue, brown, gold, gray, green, orange, pink, purple, red, silver, white, yellow, 

Comments

[退会済みユーザー]

回答頂きありがとうございます。初歩的に過ぎるかもしれませんが、コードを拝見していて理解できなかった部分について質問させてください。

1. 11行目
'int num = sizeof(colors) / sizeof(char**);'
の部分なのですが、sizeof演算子というものを初めて見ました。調べたところ、「指定した型や変数名で利用されるメモリのサイズ」とありました。"sizeof(colors)"は104となるようですが、これは一体どう数えて104になっているのでしょうか。'char **'のメモリサイズについてもイメージが掴めません。
2. 13行目
dictsort関数の第一引数は'char **'型なのに、'colors'という表記でコンパイルが通る理由が分かりません。
3. 13行目
私が書いたコードの該当部分を'colors'表記に直したのですが、コンパイルは通るようになったものの実行結果がコアダンプとなってしまいます。これはおそらく、'dictsort'関数を呼び出した際に文字列で確保されていないメモリを参照する箇所があるからだと思うのですが、この認識は正しいでしょうか。

[katoy]

sizeof(colors) / sizeof(char**);'は sizeof(colors) / sizeof(char*);' の間違いです。
たまたま sizeof(char**) = 8, sizeof(char*) = 8 なにで、 colors の配列数を 13 ともとめられています。
1. sizeof(xxx) は xxx のバイト数を求めるものです。
参考：
http://www.kis-lab.com/serikashiki/C/C03.html
> sizeof演算子
http://www.bohyoh.com/CandCPP/FAQ/FAQ00012.html
＞ 配列の要素数を取得するにはどうすればよいでしょうか。

2. color にデータ型は cjar** と同じなのです。
char *foo[] と char ** foo　はデータ型とは同じなのです...
ポインターについて理解をしているひつゆがあります。わからなければ、書籍でしらべるか、別途 質問を投稿してください。

3 質問にあるコードでは
ptr+i+1 にアクセスしています。そして、i は最大で 12 まで変化します。
i が 12 になったときに ptr + 13 となり、color の領域を超えてしまいます。
私の回答のコードでは i は最大で 11 までしか変化しないようにしています。

[退会済みユーザー]

丁寧にご回答頂き感謝致します。

Answer 4

単項演算子*はポインタに付けるものですが、colors[13][10]というint型の値に付いているので、エラーになっています。
（注：char型変数の値を参照すると値はint型になります）

Comments

[退会済みユーザー]

回答頂きありがとうございます。
であれば、char **型の引数としては '** colors'のように書くのが正しいのでしょうか。

[otn]

そもそも、*colors[13][10] という記述は何を意味していると理解していますか？
何のつもりでそう書いたのでしょうか？

[退会済みユーザー]

「文字列配列colorsの13番目の文字列の10番目の文字」ですか...?
確かに何故こんなことを書いているのか分かりませんね...

[otn]

> 確かに何故こんなことを書いているのか分かりませんね...
自分で書いたのに忘れたと言うことですか？
あるいは、自分で書いたのでないのなら、プログラムごと捨てるのか良いかと思います。

[退会済みユーザー]

もちろん自分で書いたものです。恥ずかしながらコンパイラのエラーを減らそうと躍起になって的外れな「修正」を行ってしまったようです。ところで↑の私の認識は正しいのでしょうか。

[otn]

「文字列配列colorsの12番目の文字列の9番目の文字」ですね。先頭が1番目としての話ですが。
先頭を0番目と呼ぶのならお書きの通りです。
> コンパイラのエラーを減らそうと躍起になって的外れな「修正」を行ってしまったようです。
プログラムは「考えて」書きましょう。

[退会済みユーザー]

肝に銘じておきます。貴重なお時間割いていただきありがとうございました。