C コンパイルエラー

Question

C言語でキーボードから２０個の点のｘ、ｙ座標を配列ｐに入力させるプログラム
のコンパイルがエラーになってしまいます。色々な書き方をしてみたのですが点の入力のコードが間違っているのかその他でエラーになっているのかが分からななく手詰まり状態という感じです。エラーの書き忘れ申し訳ないです。

Main.c:14:8: error: assigning to 'int' from incompatible type 'struct point'
pivot =p[somewhere];
^~~~~~~~~~~~~
Main.c:16:16: error: invalid operands to binary expression ('struct point' and 'int')
do{ while(p[i]<pivot)i++;
~~~~^~~~~~
Main.c:17:16: error: invalid operands to binary expression ('struct point' and 'int')
while(p[j]>pivot)j--;
~~~~^~~~~~
Main.c:19:13: error: assigning to 'int' from incompatible type 'struct point'
{ temp = p[i];p[i] =p[j];p[j] =temp;
^ ~~~~
Main.c:19:36: error: assigning to 'struct point' from incompatible type 'int'
{ temp = p[i];p[i] =p[j];p[j] =temp;
^~~~~
Main.c:24:21: error: member reference base type 'int' is not a structure or union
quicksort(first.j);/pivotより小さなデータについてquicksort()を行う/
~~~~~^~
Main.c:39:17: error: use of undeclared identifier 'n'
for (i =1;i<n;i+ +)
^
Main.c:39:23: error: expected expression
for (i =1;i<n;i+ +)
^
Main.c:40:46: warning: '&&' within '||' [-Wlogical-op-parentheses]
if(p[i].y<p[min].y ||p[i].y==p[min].y&&p[i].x>p[min].x)
~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~
Main.c:40:46: note: place parentheses around the '&&' expression to silence this warning
if(p[i].y<p[min].y ||p[i].y==p[min].y&&p[i].x>p[min].x)
^
( )
Main.c:42:22: error: expected expression
temp =p[0];p[0] =[min];p[min] =temp;/関数area()を使って1番目以降の点を偏角順にソート/
^
Main.c:44:16: error: use of undeclared identifier 'n'
for(i =2;i<n;i++)
^
Main.c:45:27: error: no member named 'p' in 'struct point'
{while (area(p[top-1].p[top].p[i])<= 0)
~~~~~~~~ ^
Main.c:47:11: warning: misleading indentation; statement is not part of the previous 'while' [-Wmisleading-indentation]
temp [top];p[top] =p[i];p[i]=temp;
^
Main.c:45:6: note: previous statement is here
{while (area(p[top-1].p[top].p[i])<= 0)
^
Main.c:47:16: error: subscripted value is not an array, pointer, or vector
temp [top];p[top] =p[i];p[i]=temp;
~~~~ ^~~~
Main.c:54:1: error: unknown type name 'Void'; did you mean 'void'?
Void main(void)
^~~~
void
Main.c:54:1: error: 'main' must return 'int'
Void main(void)
^~~~
int
Main.c:58:47: error: expected expression
printf("p[%d].y=",i); scanf("%d",&p[i].y);)
^
Main.c:62:2: error: expected '}'
}
^
Main.c:55:1: note: to match this '{'
{
^
2 warnings and 16 errors generated.

#include<stdio.h>
#include<math.h>
#define N 20
struct point{int x,y;};
struct point p[N];
int area(struct point q1,struct point q2,struct point q3)
{
    return((q1.x-q3.x)*(q2.y-q3.y)+(q2.x-q3.x)*(q3.y-q1.y));
}
void quicksort(int first,int last)/*quicksort()*/
{int i,j,pivot,somewhere,temp;
 if(first<last)
 {somewhere=(first+last)/2;
 pivot =p[somewhere];
 i = first;j =last;
 do{ while(p[i]<pivot)i++;
     while(p[j]>pivot)j--;
     if(i<=j)
     { temp = p[i];p[i] =p[j];p[j] =temp;
     /*pivotより大きなデータp[i]とpivotより小さいデータo[j]を交換する*/
     i++;j--;
      }
     }while(i<=j);
     quicksort(first.j);/*pivotより小さなデータについてquicksort()を行う*/
     quicksort(i,last);/*pivotより大きなデータについてquicksortを行う*/
     
 
 }
    
}
int GrahamScan()
{
    int i,top,min;
    struct point temp;
    int area();
    void quicksort();

    min =0;
    for (i =1;i<n;i+ +)
        if(p[i].y<p[min].y ||p[i].y==p[min].y&&p[i].x>p[min].x)
        min =i;/*最も下の番号を求める*/
    temp =p[0];p[0] =[min];p[min] =temp;/*関数area()を使って1番目以降の点を偏角順にソート*/
    top =1;
    for(i =2;i<n;i++)
    {while (area(p[top-1].p[top].p[i])<= 0)
          top --;/*3点が時計回りなら中央の点を除去*/
          temp [top];p[top] =p[i];p[i]=temp;
          /*i番目の点を凸包の点として登録（元のデータを破壊しないため、topとi番目の点を交換する）*/
    }
    return(top+1);/*凸包の頂点を報告*/


}
Void main(void)
{
for(int i =0; i < N; ++i){
        printf("p[%d].x=",i); scanf("%d",&p[i].x);
    printf("p[%d].y=",i); scanf("%d",&p[i].y);)

 

}

Comments

[int32_t]

エラーメッセージを記入してください。
コードを人に見せる前にちゃんと整形しましょう。インデントがメチャクチャでとても読みにくいです。

[neconekocat]

コンパイルエラーが出るならその行数なりエラー内容なりが表示されるはずですが何と表示されていますか？
大方未定義のnが使われているとかだと思いますが。

[1T2R3M4]

{}があってないとかもありそうです。

[nob.]

Main.c:14:8: error: assigning to 'int' from incompatible type 'struct point'
pivot =p[somewhere];

に対して、どういう対応をしましたか？
それとも、メッセージの意味が分からないのでしょうか？
そこをはっきりさせて下さい。そうでないと、どう答えていいか分かりません。
因みに、このメッセージの意味は、
型が異なる物を代入しようとした、という意味です。

[int32_t]

エラーメッセージが追記されましたね。
2警告16エラーそれぞれにたいして、メッセージの意味がわからないのか対処方法がわからないのか書いてください。

あと、struct point の値をどういう順番でソートしたいのか不明確ですね。x=3 y=2 と x=1 y=5 はどちらが大きいのでしょうか?

[chums]

色々なご指摘ありがとうございます。
2警告16エラーメッセージに対しての対処意味が分かりません。

[int32_t]

「Main.c:54:1: error: unknown type name 'Void'; did you mean 'void'?」に対して意味も対処もわからないとなると英語は苦手なのですね。苦手でも翻訳ソフトを使うなりして頑張って読んでくださいね。

Answer 1

エラーメッセージぐらいは読みましょう。

エラーメッセージは怒声でも暴言でも罵倒でもハラスメントでもなく、
コンパイラ等からのメッセージです。

つまり、今の状態は『相手の話をまったく聞かずに逆ギレしている人』です。

そんな人はコミュニケーションなんて取れませんよね。

まずは相手の話を聞くことから始まります。

---

[追記1]

英語が苦手なら『Google翻訳』とかを使えばいいでしょ。……と思うのですが。

プログラマは『考える事』と『調べる事』が仕事です。
趣味であっても。

仕事をするときもおんぶに抱っこでやるのでしょうか。違いますよね。

なので調べることを学びましょう。
そうすれば上達が早くなります。

Main.c:14:8: error: assigning to 'int' from incompatible type 'struct point'
pivot =p[somewhere];

[訳]
Main.cの14行目、エラーっす。なんかぁ、struct pointをintとして当ててるっすよ?
pivot = p[somewhere]; のところっす。

その14行目の周辺を見ると、pivot = p[somewhere]; ですが、

p[N] は「point構造体」です。
でも pivot は int。

それでは出来るはずがありません。

フライパンを食い物と見立てるようなものです。え？ 人間が作ったものでしょ? って感じで。

Main.c:62:2: error: expected '}'

[訳]
Main.cの62行目、エラーっす。なんかぁ、終了ブロック(?) のやつ、足りないっすよ?

プログラムは上から下へ流れるように処理していきます。

コンパイラは左上から右に向かっていき、さらに下にもぐっていきます。

Answer 2

最終行の
2 warnings and 16 errors generated
は、警告（コンパイル自体は可能だが、バグの原因になりそうな『通常の書き方とは違う書き方をしている』個所）が2個、エラー（コンパイル自体が不可能な文法上の誤り）が16個発見されたことが書かれています。
それぞれの警告、エラーについて、あなた自身どのように解釈して、どのように対処しようとして、最終的にどう失敗したのかがわかると、アドヴァイスのしようもあるかと思います。
xxx.c:ll:cc: tttt: ～～内容～～
xxx: ファイル名
ll: 行数
cc: 何文字目か（多分）
tttt: タイプ（警告 or エラー）
の形で警告とエラーが表示されています。一つずつ切り分けて解決していきましょう

Answer 3

なんか、文法がむちゃぐちゃですねー
基本的なところをもちょっとしっかり学びましょう。

ってところで、ひとつだけ。
main関数ですが、カッコの数があってませんよ

Answer 4

ご指摘を受けて自分でもう一度調べて直せるところは直してみました。
色々なご指摘感謝します。これからは自分で調べて理解ができない部分を切り取って質問していきます。

#include<stdio.h>
#include<math.h>
#define N 20
struct point{ int x, y; };
struct point p[N];
struct point pivot, temp;
int area(struct point q1, struct point q2, struct point q3)
{
	return((q1.x - q3.x)*(q2.y - q3.y) + (q2.x - q3.x)*(q3.y - q1.y));
}
void quicksort(int first, int last)
{
	int i, j, somewhere;

	if (first<last)
	{
		somewhere = (first + last) / 2;
		pivot = p[somewhere];
		i = first; j = last;
		do{
			while (area(p[0], pivot, p[i])<0)i++;
			while (area(p[0], pivot, p[j])>0)j--;
			if (i <= j)
			{
				temp = p[i]; p[j] = temp;

				i++; j--;
			}
		} while (i <= j);

		quicksort(first, j);
		quicksort(i, last);
	}
}
int GrahamScan()
{
	int i, top, min;
	struct point temp;
	//int area();
	//void quicksort();
	min = 0;
	for (i = 1; i < N; i++)
	if (p[i].y < p[min].y )
		min = i;
	temp = p[0]; p[0] = p[min]; p[min] = temp;

	quicksort(1, N - 1);
	top = 1;
	for (i = 2; i<N; i++)
	{
		while (area(p[top - 1], p[top], p[i]) <= 0) top--;
		top++;
		temp = p[top]; p[top] = p[i]; p[i] = temp;
	}
	return(top + 1);
}
void main(void)
{
	int i,k;
	for (i = 0; i < N; i++){
		printf("p[%d].x=", i);
		scanf("%d", &p[i].x);
		printf("p[%d].y=", i);
		scanf("%d", &p[i].y);
	}
		k = GrahamScan();


		printf("%d", k);
		for (i = 0; i < k; i++){
			printf("(%d,%d)", p[i].x, p[i].y);
		}
		
	
	


}

Comments

[thkana]

> これからは自分で調べて理解ができない部分を切り取って質問していきます。
それはどうでしょう...自分で調べて理解できなかった部分は実は正しくてそこに接続する「理解しているつもり」の部分との整合が間違っている、という可能性もあります。

今回見る限りだれも「余計なことを書くな」とは言っていないでしょう。そもそも、あなたが問題を解決できていないのであれば、どこが間違いなのかわかっていないわけです。そのあなたに、どこを切り取ったら適切なのかの判断ができるでしょうか? もちろん、症状が変わらない範囲で情報を絞り込んでいく分には構いませんというか歓迎すべき行動ですけれど「関係ないと思った」だけでその部分を切り捨ててしまうことには慎重になるべきと思います。多すぎる情報は読み手側で無視することはできますが、書いていない情報は読み手は知ることができないのです。であれば、質問には多少冗長になる可能性はあっても多くの情報を盛り込んだほうがよい(=解決につながる)とは思えませんか?　(まぁ、たまには「そこは関係ない」という指摘を受けることはあるでしょうが)

他の方の質問でも、「関係ないと思った」「大丈夫と思った」で質問に書かれなかった部分にうまくいかない原因があった、ということはしばしばあることです。情報を削りすぎないようにしてください。