c言語 関数の使い方

Question

１つの画像を値として考えたものと表しています。

このようにラベリング処理されたものがあるとします。
それをラベルごとの座標の重心を求めたいです。

例えば　１の場合（５、０）（６、０）（７、０）（４、１）（５、１）（６、１）（４、２）（５、２）（６、２）（５、３）（６、３）で

(５＋６＋７＋４＋５＋６＋４＋５＋６＋５＋６）/11=5 小数点はなし
(０＋０＋０＋１＋１＋１＋２＋２＋２＋３＋３）/11=1 小数点はなし

（５、１)

という風なプログラムを作っています

void zyusin(Img img){

Img limg;
int sx,sy;
 int x,y,label1x=0,label1y=0,label2x=0,label2y=0;
 int label1=0,label2=0;

  limg.sx=sx; limg.sy=sy;

for(y=0;y<sy;y++){
   for(x=0;x<sx;x++){
     if(limg.d[0][y][x]==1){
       label1++;
     }else if(limg.d[0][y][x]==2){
       label2++;
     }
   }
 }


    for(y=0;y<sy;y++){
      for(x=0;x<sx;x++){
if(limg.d[0][y][x]==1){
label1x=label1x+x;
label1y=label1y+y;

 }if(limg.d[0][y][x]==2){

  label2x=label2x+x;
  label2y=label2y+y;
}

      }

    }


    label1x=label1x/label1;
    label1y=label1y/label1;

    label2x=label2x/label2;
    label2y=label2y/label2;


   printf("1 %d,%d\n2 %d,%d",label1x,label1y,label2x,label2y);




    }

Comments

[cateye]

個別にソースを提示しないで（最低限コンパイルが通る形で）まとめて下さい。
・・・インデントの整合性確認もコンパイルも出来ない・・・

Answer 1

void zyusin(Img limg){

Img limg;

引数と同じ名前のローカル変数を定義してますが、これはダメです

Comments

[0126tami]

コンパイルできましたありがとうございます！

しかし関数が動かず　関数内のprintfが表示されない場合は何がいけないのでしょう泣

[y_waiwai]

どこかで無限ループになってるんでしょうね。
sx、sy の変数にはどういう数字が入ってるんでしょうか。

もちょっとC言語の基礎から学んでいきましょうよ。。

[y_waiwai]

C言語を勉強したいなら、VisualStudioやEclipseなど、Cをデバッグできる環境を整えましょう。
任意の場所でコードの実行を止め、変数のナカミを見ることができます。
こうすれば、当てずっぽでコードを書かなくて済みますぜ

[0126tami]

sxとsyには８＊１０の配列なので８と１０が入っています！
この関数で無限ループになっていそうなところがわかりません

[y_waiwai]

sxとsy に８と１０入れてるところはどこでしょうか。
提示のコードには見当たりませんが。

[0126tami]

投稿にそれが入っている　ソースコードを追加しました！

[0126tami]

８＊１０の上の　０と１と２でできているテキストファイルを読み込んでいます

[y_waiwai]

void zyusin(Img img){
Img limg;
int sx,sy; <<<<こいつに値が入ってない
int x,y,label1x=0,label1y=0,label2x=0,label2y=0;
int label1=0,label2=0;
limg.sx=sx; limg.sy=sy;