コマンドラインで無限ループするプログラムで現象をなくしたい

Question

以下の画像ですがコマンド画面上で無限ループしてキー入力などを実装しているのですがなぜ画面がチカチカするのでしょうか？画像上ではどうしても画面がチカチカするという現象を捉えずらいのです。どうすればいいのでしょうか？そういうものなのでしょうか？その場合どうすればいいのでしょうか？

#include <iostream>
#include <algorithm>
#include <vector>
#include <compare>
#include <filesystem>
#include <map>

#include "Main_Screen.hpp"
#include "SoundPlayer.hpp"
#include "Input_Key.hpp"

/*
* out_of_range の例外の出てる場所がわからない
*/
// ------------------------------------------------------------------ コンストラクタ  
Main_Screen::Main_Screen() :mAllMemory(80)//メモリの大きさ
{
    //デバッグ　関係
   mText.file("Log.txt");   //ログを書き込むテキストファイルを指定

   
    //---------------- 再生ゲージ 関係
    mPlayingMemory = 0;//時間を蓄積
 
    //----------------  曲の状態　関係
    mPlayList_Num = 0;      //曲のリスト数
    mNo_Playing = false;    //何も再生していない場合
    mEnter = false;         //選んだ曲を決定
    mEnter_push = false;    //mEnter の連打防止
    
    //---------------- ファイルパス 関係
    mRootDir_Path = "\MusicTest_wav";             //初期ディレクトリ    
    mNowDir_Path = mRootDir_Path;                   //現在いるディレクトリをルートディレクトで初期化
    mNowPath_num = 0;                               //前後のパスを調べるための値
    mIndexString.push_back(mRootDir_Path);          //mNowPath_numの値を使って前後をのパスを取得
    mCursorDir_pos[mIndexString[mNowPath_num]] = 0; //カーソルの場所(数値)をパスの値を使って調べる
    mIndexString.push_back(mRootDir_Path);          //ファイルパスの履歴
    // ファイル名と　ファイルパスを設定
    for (std::filesystem::directory_entry itr : std::filesystem::directory_iterator(mRootDir_Path))
    {   
        mPlayList_Name.push_back(itr.path().filename().string()); //ファイル名
        mPlayList_Path.push_back(itr.path().string());            //ファイルパス 
    }
    
    mNowSelect_Path = mPlayList_Path.at(0);     //適当に最初のファイルパスを設定
    mPlayList_Num = (int)mPlayList_Name.size(); //プレイリスト数を取得

    /*
    for (std::vector<std::string>::iterator itr = mPlayList_Name.begin(); itr != mPlayList_Name.end(); itr++)
    {
   
    }
    */
}
// ------------------------------------------------------------------ 



// ------------------------------------------------------------------ キー入力受付
void Main_Screen::Input()
{

    KeyCode Key = mInput.InputDown();  //キーコード入力

    if (Key == KeyCode::Space)//再生と一時停止
    {
        mText.Write("Log.txt", "KeyCode::Space\n");

        printf("SPACE \n");

    // --------------------------------------------------- ディレクトリ選択　Left key ←
    }else if (Key == KeyCode::Left)    
    {
        mText.Write("Log.txt", "KeyCode::Left\n");



        


        //曲リストと曲パス、局の数を　再設定
        mPlayList_Name.clear();//曲リストを初期化
        mPlayList_Path.clear();//曲パスリストを初期化 

        for (std::filesystem::directory_entry itr : std::filesystem::directory_iterator(mNowDir_Path))
        {
            mPlayList_Name.push_back(itr.path().filename().string()); //ファイル名
            mPlayList_Path.push_back(itr.path().string());            //ファイルパス 
        }

        mPlayList_Num = (int)mPlayList_Name.size(); //プレイリスト数を取得

    }
    // --------------------------------------------------- 



    // --------------------------------------------------- ディレクトリ選択　Right key →
    else if (Key == KeyCode::Right) 
    {
        mText.Write("Log.txt", "KeyCode::Right\n");





                        
        //曲リストと曲パス、局の数を　再設定
        mPlayList_Name.clear();//曲リストを初期化
        mPlayList_Path.clear();//曲パスリストを初期化 

        for (std::filesystem::directory_entry itr : std::filesystem::directory_iterator(mNowDir_Path))
        {
            mPlayList_Name.push_back(itr.path().filename().string()); //ファイル名
            mPlayList_Path.push_back(itr.path().string());            //ファイルパス 
        } 

        mPlayList_Num = (int)mPlayList_Name.size(); //プレイリスト数を取得    
    }
    // --------------------------------------------------- 

    else if (Key  == KeyCode::Up)//曲セレクト　↓
    {
        mText.Write("Log.txt", "KeyCode::Up\n");
        printf("UP \n");
        if (mPos_y > 0) 
        {
            mPos_y--;
        }
    }
    else if (Key == KeyCode::Down)//曲セレクト　↑
    {
        mText.Write("Log.txt", "KeyCode::Down\n");
        printf("DOWN \n");
        if (mPos_y < mPlayList_Num - 1) 
        {
            mPos_y++;
        }
    }


    // 決定キー
    else if (Key == KeyCode::Enter)//Enter
    {
        printf("ENTER \n");
        if (mEnter == false) 
        {          
            mEnter = true;
        }    
    }
    
   
}
// ------------------------------------------------------------------ 


// ------------------------------------------------------------------ 
void Main_Screen::Update()
{
    Input();    //キー入力


    if (mNo_Playing == false)
    {
        mPlayer.InputFile((const char*)"Music.wav");  //ファイル読み込み

          //再生時間のゲージを作成
        mMemsec = (float)mPlayer.mAllsec / mAllMemory; //１秒のメモリの大きさ
        mPlayingMemory = mMemsec;
        mPlayer.Play();
        mNo_Playing = true;
    }

    //選択中の曲にパスを変更
    if (mEnter == true)
    {
        try {
            mPlayer.mFilePath = mPlayList_Path.at(mPos_y);  //曲のパスを設定
        }
        catch (std::exception e)
        {
            printf("mEtner  Error: %s", e.what());
        }
        mPlayer.Stop();//一旦再生終了


        mPlayPath = mPlayList_Path.at(mPos_y);  //現在再生中の曲にパスを設定
        mPlayer.Play(); //再生 ！
        


        mEnter = false;

    }
}
// ------------------------------------------------------------------ 


// -------------------------- ループ ----------------------------------
void Main_Screen::Loop()
{
    // --------------------------------------
    while (true) 
    {
        GenerateOutput();   //描画
        Update();           //計算

      //  Sleep(16);
        system("cls");      //画面クリア
    }
    // --------------------------------------
}
// --------------------------     ------------------------------------


// ------------------------------------------------------------------- 
void Main_Screen::GenerateOutput()
{

    printf("Sound Player\n\n"); //ソフトタイトル

    printf("FilePath: %s\n", mPlayer.mFilePath.c_str()); //ファイルパス
    
    //チャンネル数
    if (mPlayer.mChannels == AL_FORMAT_STEREO16) 
    {
        printf("Channels: Stereo 16 bit\n");    
    }
    else 
    {  
        printf("Channels: Monaural 16 bit\n");
    }
    
    printf("BisPerSample: %llu bit\n", mPlayer.mBisPerSample);	//ビットパーフレーム
    printf("SamleRate %llu Khz\n\n", mPlayer.mSampleRate);     //サンプルレート

    printf("Music Status: ");
    //再生か一時停止中
    if (mPlayer.mPause == false)
    {
        printf("NOW PLAYING ♪"); 
        printf("\n");
    }
    else
    {
        if (mNo_Playing == true)
        {
            printf("STOP\n");
        }
        else 
        {
            printf(" PAUSE\n");
        }
    }

    alGetSourcef(mPlayer.mSource, AL_SEC_OFFSET, &mPlayer.mTime); 
    printf("Time: %.2f / %llu\n", mPlayer.mTime, mPlayer.mAllsec);
    printf("\n\n\n\n\n\n\n");//改行


    printf("mPos_y: %d",mPos_y);

    // ------  再生時間ゲージ
    if (mPlayingMemory <= mPlayer.mTime) {
        mPlayingMemory += mMemsec;
        mNowMemory++;
    }

   
    printf("\n");

    
    printf("PLAY: [ ");    
    for (int i = 0; i < mNowMemory; i++) 
    {
        mPrint.RenderText(Color_Code::Black, Color_Code::Green, "#");
    }

    for (int i = mAllMemory; i > mNowMemory; i--)      
    {
        printf(" ");
    }

    printf(" ]");


    printf("\n\n\n");
    // -------------------   曲リスト
    int i = 0;
   // printf("mPos_y: %d",mPos_y);
    mPrint.RenderText(Color_Code::Green,Color_Code::Black,"----------------------- PLAY LIST -----------------------\n\n");
    const std::string str = mPlayList_Path.at(mPos_y);
    for (std::vector<std::string>::const_iterator itr = mPlayList_Name.begin(); itr != mPlayList_Name.end(); itr++)
    {

        if (mPos_y == i)
        {
            printf("  >  ");//カーソル 
            mPrint.RenderText(Color_Code::Green, Color_Code::White, "%s", itr->c_str());

            //ディレクトリの場合　＜ DIR ＞
            if (std::filesystem::directory_entry(mPlayList_Path.at(i)).is_directory() == true)
            {
                printf(" < DIR > \n");
            }
            else
            {
                printf("\n");
            }
        }
        else 
        {
            printf("     %s", itr->c_str());//曲

            //ディレクトリの場合　＜ DIR ＞
            if (std::filesystem::directory_entry(mPlayList_Path.at(i)).is_directory() == true)
            {
                printf(" < DIR > \n");
            }
            else
            {
                printf("\n");
            }


        }
        i++;
    }

    
}
// ------------------------------------------------------------------- 

// ------------------------------------------------------------------- 
Main_Screen::~Main_Screen()
{

}
// ------------------------------------------------------------------- 

Comments

[dodox86]

int main()が無いので実際にどう動き始めているか分かりません。Main_Screen::Loop()が実質、本体だとして、
> void Main_Screen::Loop()
> {
> // --------------------------------------
> while (true)
> {
> GenerateOutput(); //描画
> Update(); //計算
>
> // Sleep(16);
> system("cls"); //画面クリア
> }
> // --------------------------------------
> }

であれば、超高速で手動で描画、clsコマンド実行、を繰り返しているかたちですから、チラついて当然です。

Answer 1

KeyCode Key = mInput.InputDown();

が，キー入力待ちではなくて，入力状態に関わらず処理が進むような物であるならば，
無限ループ内で問答無用で

• 消去 : system("cls");
• 表示 : (printfかな?)

を繰り返しているのでしょうから，まぁ，ちらつくでしょう．

表示の更新が必要なときだけ更新するならば軽減できるのではないでしょうか．
（結局消去したタイミングというのは見えるでしょうが）

---

Windowsのコンソールアプリなら WriteConsoleOutput() を使って一気に書き換えるとか．

Comments

[退会済みユーザー]

WriteConsoleOutput() をやりましたがやはり同じ現象が置きます。

[fana]

system("cls"); は取っ払ってますか？

[退会済みユーザー]

取っ払うと今度は画面が消えず下に続いてしまいます。両方試しました。

[fana]

？
WriteConsoleOutput()　って，Writeする範囲を引数で指定するのでは？

Answer 2

clsコマンドで毎回全画面クリアして全描画しているからでしょう。
コマンドラインで描画制御するのは厳しいので、ひとつの画面で何でもやろうとせず、再生中はせいぜい時間表示の更新にとどめるなどがよいかと。
その場合、\nではなく\rとすると、その行の先頭にカーソル位置が戻るのでチラつかずに済むかと思います。