プログラマ専用SNS ミクプラ

史上最悪のデスペナ さんが書きました:しかし、自作ゲームにおいて以下のようなプログラムでは変数「OpacityRate」が0.01fづつ増えません
やりたいことは
「DeathFlag」が０なら普通に敵を描画
　　　　　　　　　１なら敵モデルを徐々に透明に、ラグドール（死体のモデル）を徐々に不透明にする（OpacityRateが1.0になったら敵モデル破棄してDeathFlagを２にする）
　　　　　　　　　２ならラグドールを徐々に透明にする
OpacityRateを敵モデルの保持する構造体の中に入れてa->OpacityRateで処理すればちゃんと動きます
どこがいけないのでしょうか？

bitter_fox さんが書きました:このシチュエーションにおいてOpacityRateというのはどのような変数なのでしょうか？
関数内変数？クラス内変数？また、それらにおいてどのような修飾がされているのでしょうか？

//test.h
 
#include "DxLib.h"
 
class C_Test
{
 
    int count;
    int OpacityRate;

public:
 
    void Count(void);
    void Draw(void);
};

softya（ソフト屋) さんが書きました:ちなみに、この方法を使って狙った動作をしているか確認できますよ。お試し下さい。
「簡単RPG講座　番外編。　デバッグ入門 • C言語交流フォーラム　～ mixC++ ～」
blog.php?u=114&b=982

softya（ソフト屋) さんが書きました:これだけのコードだと何とも言えませんね。変数の定義、インスタンスの生成、呼び出し部分も見てみないことには。

史上最悪のデスペナ さんが書きました:

softya（ソフト屋) さんが書きました:ちなみに、この方法を使って狙った動作をしているか確認できますよ。お試し下さい。
「簡単RPG講座　番外編。　デバッグ入門 • C言語交流フォーラム　～ mixC++ ～」
blog.php?u=114&b=982

一応、似たような（？）方法として
ゲーム画面上に自分が今追跡しようとしてる変数を表示したり無理やりキーで値を変更するようにしてifやswitch内を通るかVC++のブレークポイントとかいうのを作成してやっていますがどうでしょう？

史上最悪のデスペナ さんが書きました:プリントデバッグなるものが分かりませんがゲーム画面上に表示すると
HPが0より大きい時は
OpacityRate = 0.0f
DeathFlag = 0
ですが0以下になると
OpacityRate = 0.01f
DeathFlag = 1

のまま変化しません

史上最悪のデスペナ さんが書きました: あ、ごめんなさい。ここでの表記忘れです。
正しくは

コード:

//test.h
 
#include "DxLib.h"
 
class C_Test
{
 
    int count;
    int OpacityRate;

public:
 
    void Count(void);
    void Draw(void);
};

です

修飾なるものが分かりません。

bitter_fox さんが書きました:修飾というのはprivateやstaticと言った変数等の性質を決定することです。
C++だとprivateとかはアクセス指定とかって言うのかな？

史上最悪のデスペナ さんが書きました:OpacityRateの値に関してですが、プロジェクト内の全てのファイルを走査しましたがここに貼り付けたコード以外の部分にはなかったので
多分常に０にするようなとこはないんじゃないかと思います。

史上最悪のデスペナ さんが書きました:

史上最悪のデスペナ さんが書きました:OpacityRateの値に関してですが、プロジェクト内の全てのファイルを走査しましたがここに貼り付けたコード以外の部分にはなかったので
多分常に０にするようなとこはないんじゃないかと思います。

（switch前 - case 1:内のbreak前 - switch後）という形でOpacityRateの値をコンソール出力したところ
（0.00000 - 0.01000 - 0.01000）の繰り返しになったのでどうやらどこかで常に０にされてるようです。引き続き追跡します

bitter_fox さんが書きました:C_Testのインスタンスが毎回作成されているという可能性はありませんか？
C_Testのコンストラクタでデバッグプリントをしてみてください。

beatle さんが書きました:int型変数に0.01fを加算しても意味ありませんよ。

float OpacityRate;

beatle さんが書きました:全体のソースをお示しになられると混乱もなくなり、みなさんハッピーかと思います。
長いならspoilタグで囲むか、zipして添付ですかね。

史上最悪のデスペナ さんが書きました: thisの値ですが全て009469C8という値でしたので同じインスタンスという解釈でいいですよね？

史上最悪のデスペナ さんが書きました:なんででしょう？
古いPC（XPでWiMAX使用＆Chrome）でも、新しいPC（７でstarcat＆Chrome）でも、学校のPC（Vistaで回線不明＆IE）でも出来ないのですが・・・・・
前は出来たのに。前の分を削除すればいいのですかね・・・・・？

softya（ソフト屋) さんが書きました:VC++のプロジェクトは公開するには余分なものを含んでいるのでzipから削除して下さいね。
xxx.ncbとかxxxx.ipchとかxxxx.sdfとかです。消しても自動的に復元されます。

史上最悪のデスペナ さんが書きました:何故メモリ破壊が起きたのでしょう・・・・・・
今日はもうインできませんが、教えていただいた方法でデバッグしてみます。

申し訳ありませんが、また明日もお付き合いください。

ふりかけ さんが書きました:OpacityRate が float型なのは理由があるのですか？

softya（ソフト屋) さんが書きました:ヒント：メモリ破壊の原因は配列の添字かポインタの操作ミスです。

史上最悪のデスペナ さんが書きました:当たり判定での配列の添字が１つ足りなかったからでした。
OpacityRateを使ってるところだけを見てても分からないはずです＾＾；

softya（ソフト屋) さんが書きました:プロでもその手のバグが一番多いのです。

softya（ソフト屋) さんが書きました:enum {Ground, Build, enemy};
があるので
enum {Ground, Build, enemy, MAP_HIT_MAX};
としておいて
MV1_COLL_RESULT_POLY MapHit[MAP_HIT_MAX]; //Map当たり判定
とすれば防げたでしょう。

softya（ソフト屋) さんが書きました:あとミスをすること想定して
assert(Flag<MAP_HIT_MAX);
assert(Flag>=0);
も書いておくと完璧です。

softya（ソフト屋) さんが書きました:解決マークをお願いしますね。

史上最悪のデスペナ さんが書きました: そういうテクニックって独学だと思いつきませんよね。なかなか。頭のいい人は思いつくんだろうけど・・・・