じゃんけんゲーム

[situmon]

#include "include/DxLib.h"
int pzyoutai = 0;
int azyoutai = 0;
int White   = GetColor( 255 , 255 , 255 ) ; // 白色の値を取得
void janken(){
    DrawString(  50,  100, "じゃんけんゲーム" , White);            //文字列表示
        if( CheckHitKey( KEY_INPUT_A ) == 1 ){
        DrawString(  80,  120, "パー" , White);            //文字列表示
        pzyoutai = 1;
		}
        if( CheckHitKey( KEY_INPUT_B ) == 1 ){
        DrawString(  80,  120, "グー" , White);            //文字列表示
        pzyoutai = 2;
		}
        if( CheckHitKey( KEY_INPUT_C ) == 1 ){
        DrawString(  80,  120, "チョキ" , White);            //文字列表示
        pzyoutai = 3;
		}
        	azyoutai = GetRand(3);
}

void hantei(){
	if(pzyoutai = 1 && azyoutai =2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 1 && azyoutai =3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 1 && azyoutai =1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 2 && azyoutai =2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 2 && azyoutai =3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 2 && azyoutai =1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 3 && azyoutai =2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 3 && azyoutai =3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai = 3 && azyoutai =1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
}
	


int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
        ChangeWindowMode( TRUE ) ; // ウインドウモードに変更
        if( DxLib_Init() == -1 ) return -1; // ＤＸライブラリ初期化処理 エラーが起きたら終了

janken();

hantei();

        DxLib_End() ;           // ＤＸライブラリ使用の終了処理
        return 0 ;              // ソフトの終了
}

としたのですが
test.cpp(23) : error C2106: '=' : 左のオペランドが、左辺値になっていません。
とwarning C4305: '=' : 'int' から 'bool' へ切り詰めます。
という警告がでてしまいました。
なんとなるわかるようなないようななのですが、
理由がわかりません。

[box]

＞	if(pzyoutai = 1 && azyoutai =2)

左右両辺が等しいかどうかを調べるには、等号が２個必要です。

[situmon]

またまちがえてしまいました。
これでは代入になってしまいますね。

[situmon]

#include "include/DxLib.h"
int pzyoutai = 0;
int azyoutai = 0;
int White   = GetColor( 255 , 255 , 255 ) ; // 白色の値を取得
void janken(){
    DrawString(  50,  100, "じゃんけんゲーム" , White);            //文字列表示
        if( CheckHitKey( KEY_INPUT_A ) == 1 ){
        DrawString(  80,  120, "パー" , White);            //文字列表示
        pzyoutai = 1;
		}
        if( CheckHitKey( KEY_INPUT_B ) == 1 ){
        DrawString(  80,  120, "グー" , White);            //文字列表示
        pzyoutai = 2;
		}
        if( CheckHitKey( KEY_INPUT_C ) == 1 ){
        DrawString(  80,  120, "チョキ" , White);            //文字列表示
        pzyoutai = 3;
		}
        	azyoutai = GetRand(3);
}

void hantei(){
	if(pzyoutai == 1 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 1 && azyoutai ==3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 1 && azyoutai ==1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 2 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 2 && azyoutai ==3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 2 && azyoutai ==1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 3 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 3 && azyoutai ==3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 3 && azyoutai ==1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
}
	


int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
        ChangeWindowMode( TRUE ) ; // ウインドウモードに変更
        if( DxLib_Init() == -1 ) return -1; // ＤＸライブラリ初期化処理 エラーが起きたら終了

janken();

hantei();

        DxLib_End() ;           // ＤＸライブラリ使用の終了処理
        return 0 ;              // ソフトの終了
}

ですね。
エラーがなくなりましたが
文字が表示されませんでした。
ですが処理は終わったみたいでDXライブラリが終了しました。

[ランド]

>azyoutai = GetRand(3);
たしかGetRand(3)は0,1,2,3のいずれかの値を入手だったはずです。
0が選ばれてどの条件式も真にならなかったのではないでしょうか。
azyoutai = GetRand(2)+1;にすれば正常に動作すると思います。

[Justy]

＞文字が表示されませんでした

　ソースを見る限り、文字の表示はされてるはずですよ。
　・・・多分黒色で。


# GetColor()は DxLib_Init()の後に使わないと正しく取得できません。

[situmon]

#include "include/DxLib.h"
int pzyoutai = 0;
int azyoutai = 0;
void janken(int White){
    DrawString(  50,  100, "じゃんけんゲーム" , White);            //文字列表示
        if( CheckHitKey( KEY_INPUT_A ) == 1 ){
        DrawString(  80,  120, "パー" , White);            //文字列表示
        pzyoutai = 1;
		}
        if( CheckHitKey( KEY_INPUT_B ) == 1 ){
        DrawString(  80,  120, "グー" , White);            //文字列表示
        pzyoutai = 2;
		}
        if( CheckHitKey( KEY_INPUT_C ) == 1 ){
        DrawString(  80,  120, "チョキ" , White);            //文字列表示
        pzyoutai = 3;
		}
        	azyoutai = GetRand(2)+1;
}

void hantei(int White){
	if(pzyoutai == 1 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 1 && azyoutai ==3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 1 && azyoutai ==1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 2 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 2 && azyoutai ==3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 2 && azyoutai ==1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 3 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 3 && azyoutai ==3)
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	if(pzyoutai == 3 && azyoutai ==1)
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
}
	


int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
        ChangeWindowMode( TRUE ) ; // ウインドウモードに変更
        if( DxLib_Init() == -1 ) return -1; // ＤＸライブラリ初期化処理 エラーが起きたら終了

int White;
    White = GetColor( 255 , 255 , 255 ) ;                    //色の取得



janken(White);

hantei(White);

ScreenFlip();//裏画面反映

        DxLib_End() ;           // ＤＸライブラリ使用の終了処理
        return 0 ;              // ソフトの終了
}

と書き換えることによりなんとか文字を表示することができましたが、
最初から　あなたがかちです。　と表示されてしまいます。
それにあいてはパーですなどが表示されません。

[non]

> if(pzyoutai == 1 && azyoutai ==2)
> DrawString( 80, 150, "あいてはグーです。" , White); //文字列表示
> DrawString( 80, 180, "あなたのかちです。" , White); //文字列表示

DXライブラリは勉強したことがないので、わかりませんが、文法の話として
ifが成立したときに実行する文は１行だけですか、それとも２行ですか？

[木霊]

　boxさんの指摘で既に直ってるんですが、代入（＝）と比較（＝＝）を良く間違えるのであれば、
if文の中だけ数字と変数の場所を逆にするようにしてみてください。

　こんな感じです。

if ( 1 == pzyoutai && 2 == azyoutai ) {・・・}

　こうしておけば＝を一つしか書いてなかった時にはコンパイル時にエラーが出るので間違いに気付けますよ。

　1 == pzyoutai・・・○　１とpzyoutai　が同じ値かを比較する事はできます。
　pzyoutai = 1・・・○　pzyoutaiに１を代入する事もできます。
　1 = pzyoutai・・・×　でも１にpzyoutaiを代入する事はできません。
　だからエラーが出ます。

　一度自分で確かめてみてください。

[situmon]

< ifが成立したときに実行する文は１行だけですか、それとも２行ですか？
２行です。

[box]

慣れないうちは、

if (...) {
	// if文が成立したとき
}
else {
	// if文が成立しなかったとき
}

のように、実行する文の数に関係なく、必ず { と } で囲う方がよいと思います。

[conio]

●situmonさんへ
-----------------------------------------------------------------------------------
>> if(pzyoutai == 1 && azyoutai ==2) 
>> DrawString( 80, 150, "あいてはグーです。" , White); //文字列表示 
>> DrawString( 80, 180, "あなたのかちです。" , White); //文字列表示 
>>DXライブラリは勉強したことがないので、わかりませんが、文法の話として 
>>ifが成立したときに実行する文は１行だけですか、それとも２行ですか？
 
>>  < ifが成立したときに実行する文は１行だけですか、それとも２行ですか？ 
>>２行です。  
------------------------------------------------------------------------------------


	if(pzyoutai == 1 && azyoutai ==2)
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示

ifが成立した時に、２行分の文を実行したいのであれば、｛｝で閉じないと駄目です。
上の場合、if文の成立によって判定されるのは"あいてはグーです"と書いてある行だけです。
"あなたのかちです"という行はif分の判定に含まれません。

2行ともif分の判定により実行させたいのであれば、

	if(pzyoutai == 1 && azyoutai ==2)
{
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
}
と書く必要があります。
他の部分も2行ずつ実行させたいのであれば、同じように｛｝で囲めばいいと思います。

[situmon]

みなさまどうもありがとうございました。
成功しました！
今度は１０回勝負ができるようにしたり
switch文を使ったりしてコードをより簡潔にしたいです。
最初は{}を付け忘れないように、と==をしっかりと覚えて今後、
このようなミスが内容に努めたいです。
成功したコードを表示しておきます。
本当にどうもありがとうございました。

#include "include/DxLib.h"
int pzyoutai = 0;
int azyoutai = 0;
void janken(int White){
    DrawString(  50,  20, "じゃんけんゲーム" , White);            //文字列表示
    DrawString(  50,  40, "パーならA、グーならB、チョキならCキーを押してください。" , White);            //文字列表示
    DrawString(  50,  60, "A、B、C以外のキーを押すと終了します。" , White);            //文字列表示

WaitKey() ; 

        if( CheckHitKey( KEY_INPUT_A ) == 1 ){
        DrawString(  80,  120, "パー" , White);            //文字列表示
        pzyoutai = 1;
		}
        if( CheckHitKey( KEY_INPUT_B ) == 1 ){
        DrawString(  80,  120, "グー" , White);            //文字列表示
        pzyoutai = 2;
		}
        if( CheckHitKey( KEY_INPUT_C ) == 1 ){
        DrawString(  80,  120, "チョキ" , White);            //文字列表示
        pzyoutai = 3;
		}
        	azyoutai = GetRand(2)+1;
}

void hantei(int White){
	if(pzyoutai == 1 && azyoutai ==2){
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 1 && azyoutai ==3){
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 1 && azyoutai ==1){
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 2 && azyoutai ==2){
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのあいこです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 2 && azyoutai ==3){
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 2 && azyoutai ==1){
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ;
	}
	if(pzyoutai == 3 && azyoutai ==2){
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 3 && azyoutai ==3){
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 3 && azyoutai ==1){
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	}
}
	


int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
        ChangeWindowMode( TRUE ) ; // ウインドウモードに変更
        if( DxLib_Init() == -1 ) return -1; // ＤＸライブラリ初期化処理 エラーが起きたら終了

int White;
    White = GetColor( 255 , 255 , 255 ) ;                    //色の取得

janken(White);

hantei(White);

ScreenFlip();//裏画面反映

        DxLib_End() ;           // ＤＸライブラリ使用の終了処理
        return 0 ;              // ソフトの終了
}

[situmon]

みなさまどうもありがとうございました。
成功しました！
今度は１０回勝負ができるようにしたり
switch文を使ったりしてコードをより簡潔にしたいです。
最初は{}を付け忘れないように、と==をしっかりと覚えて今後、
このようなミスが内容に努めたいです。
成功したコードを表示しておきます。
本当にどうもありがとうございました。

#include "include/DxLib.h"
int pzyoutai = 0;
int azyoutai = 0;
void janken(int White){
    DrawString(  50,  20, "じゃんけんゲーム" , White);            //文字列表示
    DrawString(  50,  40, "パーならA、グーならB、チョキならCキーを押してください。" , White);            //文字列表示
    DrawString(  50,  60, "A、B、C以外のキーを押すと終了します。" , White);            //文字列表示

WaitKey() ; 

        if( CheckHitKey( KEY_INPUT_A ) == 1 ){
        DrawString(  80,  120, "パー" , White);            //文字列表示
        pzyoutai = 1;
		}
        if( CheckHitKey( KEY_INPUT_B ) == 1 ){
        DrawString(  80,  120, "グー" , White);            //文字列表示
        pzyoutai = 2;
		}
        if( CheckHitKey( KEY_INPUT_C ) == 1 ){
        DrawString(  80,  120, "チョキ" , White);            //文字列表示
        pzyoutai = 3;
		}
        	azyoutai = GetRand(2)+1;
}

void hantei(int White){
	if(pzyoutai == 1 && azyoutai ==2){
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 1 && azyoutai ==3){
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 1 && azyoutai ==1){
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 2 && azyoutai ==2){
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのあいこです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 2 && azyoutai ==3){
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 2 && azyoutai ==1){
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ;
	}
	if(pzyoutai == 3 && azyoutai ==2){
        DrawString(  80,  150, "あいてはグーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのまけです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 3 && azyoutai ==3){
        DrawString(  80,  150, "あいてはチョキです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたとあいこです。" , White);            //文字列表示
WaitKey() ; 
	}
	if(pzyoutai == 3 && azyoutai ==1){
        DrawString(  80,  150, "あいてはパーです。" , White);            //文字列表示
        DrawString(  80,  180, "あなたのかちです。" , White);            //文字列表示
WaitKey() ; 
	}
}
	


int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
        ChangeWindowMode( TRUE ) ; // ウインドウモードに変更
        if( DxLib_Init() == -1 ) return -1; // ＤＸライブラリ初期化処理 エラーが起きたら終了

int White;
    White = GetColor( 255 , 255 , 255 ) ;                    //色の取得

janken(White);

hantei(White);

ScreenFlip();//裏画面反映

        DxLib_End() ;           // ＤＸライブラリ使用の終了処理
        return 0 ;              // ソフトの終了
}

[組木紙織]

＞みなさまどうもありがとうございました。
＞成功しました！

完成したようでよかったです。


私からvoid hantei(int White)の部分で二つアドバイスを

1:長さが少し長いと感じませんか？
もう少し短く出来るので考えてみたらいかがでしょうか？

2:表示の色を変えることが出来ますよね。でも変数名がWhiteだと読んでる人が勘違いします。
もっと適切な変数名を考えてみたらどうでしょうか？

[kazuoni]

自分からも・・・おめでとうございます＾＾
進歩してる感があっていいですねｗ
そうですねー。確かに
相手の出したもの、勝負を全とおり書くのは長いですね。
あとWaitKey()も。
作るゲームが大きくなっていくと、このままだとものすごいことになって、
自分でもどこがどうなってるのかわからなくなってしまう気がします。
早い段階からコードを簡潔に書くように工夫することはとても大切かと＾＾

がんばってください！

[木霊]

　とりあえず完成したみたいでなによりです。

　上の御二方も今後へのアドバイスをされているので自分からも・・・

１・変数のスコープ（使える範囲）を理解して、グローバル変数を控える（極力使わない）
　　グローバル変数はどこからでも使えるので便利ですが、同時にどこからでも変更できるので
　　思わぬバグの温床になり、しかも駆除しにくいのでしっかり管理できないなら使わない方がいいです。

２・（１とは逆に）whiteのように値を変更する事のない変数はconstをつけて定数にするか、
　　#defineでマクロ化しておく。（個人的には定数化を勧めます）

３・変数の横にコメントで「何に使うか」と「取る値の範囲」を書いておく。（フラグなら説明も）
　　（前トピまでのsitumonさんの質問内容をみると、この部分をほとんど把握してない様子なので。）
　　他人のコードを改変するならば、これをおざなりにすると何をやってるか全く分からないので
　　改変する前に「何に使うか」ぐらいは把握するようにしないと同じことの繰り返しになります。

４・インデント（字下げ）はきっちりと。
　　ところどころインデントがおかしいので少し見難いです。
　　処理がif文の{}の中か外かが分かりくいので、それぐらいは分かりやすくするようにしてください。

　それでは本命のＳＴＧ作成、頑張ってください。

Ｐ．Ｓ．
　中学生という事は来月には期末試験ですよね。
　そっちの方も頑張ってくださいね。