このプログラムについて。

質問

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <time.h>
#include <string.h>

#define NVALUE 30

#define MAXSIZE NVALUE

struct City{
float x;
float y;
};

struct Country{
struct City cities[MAXSIZE];
int size;
};

struct Itine{
float quality;

int route[MAXSIZE];
int noc;
};

float plength(struct Itine *tour,struct Country *pcountry)
{
int i,j;
double dy;
double dx;

float length=0.0;
for(i=0;i<tour->noc;i++){
j=i+1;
if(j==tour->noc)
j=0;
struct City &c1=pcountry->cities[tour->route[i]-1];
struct City &c2=pcountry->cities[tour->route[j]-1];
double dx = c1.x-c2.x;
double dy = c1.y-c2.y;
length+=(float)sqrt(dx*dx+dy*dy);
}
return length;
}

なんか間違っていますか？
エラーメッセージは、この部分
struct City &c1=pcountry->cities[tour->route[i]-1];
struct City &c2=pcountry->cities[tour->route[j]-1];
double dx = c1.x-c2.x;
double dy = c1.y-c2.y;

が、
';'が'型'の前にありません

と出ています。Visual Express 2005です。

回答No.1

ddnp009 · ベストアンサー

試していないのですが、

>';'が'型'の前にありません

このエラーは、型名がコンパイラに解決できないとき
よく見ます。

提示されたソースでは、City型がコンパイラに見えていないのでは。

もしstruct City{...};が別ファイル(なんちゃら.hなど)に書かれているなら
それをインクルードする。
または(C++では冗長になった)structを取っ払うかしてみると
幸せになれるかもしれません。

お礼

struct City{...};
は質問のプログラムどおり、書いています。

おかしいです。

回答No.6

bnosuke-x

昔よくあったのが、
「全角スペースが入っていた」
という、正に目で見えないものが原因でした。
今は全角スペースがエディタ上では見えるようになってると思いますが、これもちょっと疑ってみて下さい。

回答No.5

麻野 なぎ（@AsanoNagi）

既に回答があるとおりですが、Ｃ＋＋としてみれば、文法的にはＯＫのようです。

Ｃとしてみれば

１．変数の宣言は、ブロック中の式や文より前にある必要があります。このために、c1, c2 の宣言はエラーになります。

２．さらに、&c1 は、参照型であり、Ｃにはありません。

３．というわけで、c1, c2 の宣言に失敗しています。故に、c1, c2 は、「未定義の変数」となって、これでまた、エラーになります。

ということでしょうか。

回答No.4

noocyte

これは C++ のソースを C コンパイラにかけてますね．
ソースファイルの拡張子が .cpp ではなく .c になってませんか？
.cpp に修正してください．

回答No.3

FAY

#2です。ごめんなさい、ちょっと紛らわしかったかも。
>まず、c1,c2,dx,dyをforループ内に定義するのをやめて、ループの外に出しましょう。
を
c1,c2,dx,dyの宣言をforループの前にしましょう
に訂正させてください。

回答No.2

FAY

ああ、突っ込みどころが満載・・・
Ｃ言語ですよね？
まず、c1,c2,dx,dyをforループ内に定義するのをやめて、ループの外に出しましょう。
でも、それだけじゃコンパイル通らないのでプログラムを修正してください。
あと、int main(void)が無いとダメです。

お礼

ご指摘ありがとうございます。

main関数は下のほうにあります。