プログラム

質問

以前ファイルからXとYの値だけ取り出し図形を作る質問をした者ですが
void Ctest::OnOK()
{
FILE *fp;
char *str,buf[1000];
　 　　char buf3[100],*s3;
char buf4[100],*s4;
if ((fp = fopen("test.txt", "r")) == NULL) {
printf("ファイルが開けません\n");
return EXIT_SUCCESS;
}
while (fgets(buf, 1000, fp) != NULL) {
str=buf;
　　　　　　　while((*str!='\0'){
if(*str!='\0' && *str=='X'){
*str++;
if((*str!='\0' &&(isdigit(*str) || *str=='-' || *str=='.'))
s3=buf3;
if((*str!='\0' &&(isdigit(*str) || *str=='-' || *str=='.'))
while(*str!='\0' &&(isdigit(*str) || *str=='-' || *str=='.')) *s3++=*str++;
*s3='\0';
}
if(*str!='\0' && *str=='Y'){
*str++;
if((*str!='\0' &&(isdigit(*str) || *str=='-' || *str=='.'))
s4=buf4;
if((*str!='\0' &&(isdigit(*str) || *str=='-' || *str=='.'))
while(*str!='\0' &&(isdigit(*str) |*str=='||*str=='.')) *s4++=*str++;
*s4='\0';

CDC *v;
GetDC();
MoveTo(0,0);
LineTo(atof(buf3),atof(buf4));
ReleaseDC(v);
}
else
str++;
}
}
fclose(fp);
return 0;
}

でコンパイルはできるのですが直線の作図がされません。
どこが間違ってるのでしょうか。VC+6.0のMFC使ってます

回答No.6

redfox63

データの値域と描画する範囲はどの程度なのでしょう？

1000x1000画素の中に ±10の値域を持つデータを小数点以下2位までを使って描画したいってことなら
LineToの引数を (int)atof(buf3)*100 などといった具合に下駄を履かせてやれば良いでしょう

回答No.5

D-Matsu

あー、なるほど、座標に小数以下も使いたいって話ですか。
であればそれは#3でも言われるとおり「無理」ですね。

まず、MFCのLineTo()のプロトタイプは
BOOL CDC::LineTo(int x, int y)
もしくは
BOOL CDC::LineTo(POINT point)
です。
http://msdn.microsoft.com/ja-jp/library/wzc1344s.aspx

で、POINTは
struct POINT
{
LONG x;
LONG y;
}
です。
http://msdn.microsoft.com/ja-jp/library/8kk2sy33.aspx

見ての通りすべて整数型であり、小数を扱うことはできません。
#2でも言いましたが、小数を渡そうとしても勝手に整数(int)に変換され、小数以下は切り捨てられます。

従って、関数へ渡す前段階で小数値が問題なく取れていたとしても無意味な訳です。

ただ、線描が出来ない理由は別なところにあるようです。

> buf3の中身は"-12.04"
> buf4の中身は"-7.01"

最初のv->MoveTo(0, 0)は描画領域の左上に移動しますが、ここからマイナスの座標となると、さらに左上で描画領域の外になります。
buf3、buf4(終点座標)を正の値になるようにして試してください。

回答No.4

arain

http://okwave.jp/qa4511580.html
から移動したようなので、こっちに。
なぜ数値の取り出しにピリオドも含めていたのかの理由については、
No.1の
>この時点では文字列として扱われるのでatofで数字に変換する
>XとY座標は浮動少数も扱うのでdouble型にしたい。
で納得しました。

で問題のほう。
No.3より
>ファイルからの読み込みじゃなく
>入力打ちなら浮動少数でも表示できるので
>そんなことは無いと思いますが・・・
ファイルだろうが手入力だろうが、全く一緒です。
LineTo()はどのAPIを使用していますか？
No.3氏の推測のとおり
>BOOL LineTo(int nXEnd , int nYEnd);
であれば、絶対にできていません。

>入力打ちなら浮動少数でも表示できるので
この「表示」とは、まさかprintf("%f")ではないですよね？
変数の型が片方は「実数」でもう一方は「整数」です。
この状態では実数の少数部位は桁落ちしてしまうため意味を持ちません。
実数で「1.0」でも「1.9」でも整数では「１」として扱われます。
逆に、intの範囲を超えた場合もオーバーフローとしておかしな値を示すことになります。
（本当に引数と使用する型が違っていたら、コンパイル時にワーニングエラーが出てるはずだけど）

少数位まで扱うのであれば、No.3氏が回答しているとおり、一旦整数に変換する必要があります。
また、単純に整数に変換しただけでは座標は描画面のサイズをオーバーする可能性があるので、描画面の範囲も同様に等比倍して、
その上で実描画のための相対的な座標点を算出する必要があります。

それと、起点は左上が(0,0)等になっていませんか？
であれば、そのままではマイナス座標は使用できません(使用できる環境でも論理座標扱いなら表示されない)
この場合も同様に、起点を論理的に変更する処理を作成するか、開発環境が対応していれば、起点を動かす処理が必要になります。

補足

CEdit *p,*q,*r,*s,*t;
CDC *v;
CString a,b,c,d,z;

p=(CEdit *)GetDlgItem(IDC_EDIT1);
q=(CEdit *)GetDlgItem(IDC_EDIT2);
r=(CEdit *)GetDlgItem(IDC_EDIT3);
s=(CEdit *)GetDlgItem(IDC_EDIT4);
t=(CEdit *)GetDlgItem(IDC_EDIT5);

p->GetWindowText(a);
q->GetWindowText(b);
r->GetWindowText(c);
s->GetWindowText(d);
t->GetWindowText(z);

int Z=atoi(z);
double A=atof(a);
double B=atof(b);
double C=atof(c);
double D=atof(d);

switch(Z){
case 92:
v=GetDC();
v->MoveTo((int)(A),(int)(B));
bx=A;
by=B;
ReleaseDC(v);
break;
case 01:
v=GetDC();
v->MoveTo((int)(bx),(int)(by));
v->LineTo((int)(A),(int)(B));
bx=A;
by=B;
ReleaseDC(v);
break;
}
}というVisualC++6.0のMFCダイアログベースで書かれたプログラムで
Editボックス1にGの数字を
Editボックス2にXの数字を
Editボックス3にYの数字を入力して
OKを押すと図形が作成される。
このプログラムをファイルから一行ずつ読み込んで
図形を書く、のが求めるプログラムです。

上記のプログラムなら小数点もいけました。
ただ小さくて線がみえないなんてこともしばしばですので５倍にしたりすれば普通に見えます。

原点は上記の手入力のプログラムは、ですが入力する全ての座標を
ずらすことでなんとかできました。オフセットというのかな

回答No.3

chie65536

＞XとY座標は浮動少数も扱うのでdouble型にしたい。
＞でこういう風にしたのですがそこも間違ってるのかな

貴方が「こうしたい」と期待しても、API関数は「貴方の期待通り」の引数は期待していない。

たぶん、LineToは
BOOL LineTo(int nXEnd , int nYEnd);
とかって定義されてて「引き数はintが２個だ」って決まってる。

なので、貴方がatofを呼んだり、floatでどうこうしようと思っても、すべて無駄。

最後の最後は「intの値を２つ渡す」のだから、座標が浮動少数になってようが整数になってようが、そんな事はすべて忘れ去られ「整数部分だけが使われてしまう」のはどうしようもない。

座標を浮動少数で扱う意味があるとしたら「座標値を１００倍して、０．００は０の位置、１．００は１００の位置、０．５０は５０の位置に拡大して描画する」と言う場合のみ。

等倍で描画している限りは、小数点はまったく無意味。デバイス座標系では「点の座標は整数」なので「１．５とか、小数点の付く位置」には点は存在しない。

補足

ファイルからの読み込みじゃなく
入力打ちなら浮動少数でも表示できるので
そんなことは無いと思いますが・・・

回答No.2

D-Matsu

double値を扱ってもどうせLineTo()に渡す際にはintにキャストされます。なのでまぁオーバーフロー注意、以外は問題ないといえばないんですが気になったので。

で、ブレークポイントとウォッチ式、デバッグ用変数等を使って「別環境ではなくこのコード内で期待通りになっているか」を確認してください。
追加でもう一つですが、
・atof()の結果をintにキャストしたときに期待通り(小数点以下が落ちるだけ)の結果になっているか
も確認をお願いします。

補足

buf3の中身は"-12.04"
buf4の中身は"-7.01"
になってました

回答No.1

D-Matsu

数値切り出しならsscanf()とかfscanf()とか使った方が楽だと思いますがそれはさておき。
あとデバイスコンテキストまわりのコードが微妙におかしいようです。

> CDC *v;
> GetDC();
> MoveTo(0,0);
> LineTo(atof(buf3),atof(buf4));
> ReleaseDC(v);

CDC *v = GetDC();
v->MoveTo(0, 0);
v->LineTo(atof(buf3), atof(buf4));
ReleaseDC(v);

でないとコンパイルできない気が。
GetDC()も描画対象のCWnd *を使う必要がありますし。
とは言うものの、前回の質問によるとこの辺は出来てるんですよね？

てな訳で確認事項。
・buf3、buf4の値はatof()の結果も含めて正常というか期待通りですか？
・LineTo()の引数x, yはint型ですが、なぜfloatで渡していますか？

補足

コピペできないものでミスが少しあったようで申し訳ない。
確認事項なんですが
>buf3、buf4の値はatof()の結果も含めて正常というか期待通りですか？
試しに
タイプが違うのですがコンソールで試した所buf3,buf4には
期待通りの値でした。

２個目の方がよくわからないんですが
文字列を一行よんで
XとYの座標を取り出してbuf3,buf4に格納する
この時点では文字列として扱われるのでatofで数字に変換する
XとY座標は浮動少数も扱うのでdouble型にしたい。

でこういう風にしたのですがそこも間違ってるのかな