0

プログラムは、入力ファイルからテキストを読み取ります。私の出力は次のようになります。

Level         Score          Stars
----------------------------------
1              3840           **

2              5940           **

3             11560           **

4             18140           **

5             18780           **

星はレベルに一致するはずです。

プログラムは最初の行のみを表示しますが、プログラムはループが合計出力を表示できるように記述されています。何らかの理由で、プログラム ループはコードの最初の行までしか実行されません。

プログラムが複数回ループしない理由がわかりません。どんなアドバイスも役に立ちます。

#include <iostream>
#include <cstring>
#include <cstdlib>
#include<fstream>

using namespace std;

int buildArrays(int A[],int B[],int C[])
{
    int i=0,num;
    ifstream inFile;
    inFile.open("candycrush.txt");

    if(inFile.fail())
    {
        cout<<"The candycrush.txt input file did not open"<<endl;
        exit(-1);
    }
    while(inFile)
    {
        inFile>>num;
        A[i]=num;

        inFile>>num;
        B[i]=num;

        inFile>>num;
        C[i]=num;

        i++;
    }
    inFile.close();

    return i;
}

void printArrays( string reportTitle, int levelsArray[], int scoresArray[], int starsArray[], int numberOfLevels )
{
    cout<<endl;
    cout<<reportTitle<<endl;
    cout<<"Levels\tScores\tStars"<<endl;
    cout<<"---------------------"<<endl;

    for(int i=0;i<numberOfLevels;i++)
    {
        cout<<levelsArray[i]<<"\t"<<scoresArray[i]<<"\t";
        for(int j=0;j<starsArray[j];j++)
        {
            cout<<"*";
        }
        cout<<endl;
    }
}

void sortArrays( int levelsArray[], int scoresArray[], int starsArray[], int numberOfLevels )
{
    for(int i=0;i<numberOfLevels;i++)
    {
        for(int j=0;j<numberOfLevels;j++)
        {
            if(levelsArray[i]<levelsArray[j])
            {
                int temp1=levelsArray[i];
                int temp2=scoresArray[i];
                int temp3=starsArray[i];

                levelsArray[i]=levelsArray[j];
                scoresArray[i]=scoresArray[j];
                starsArray[i]=starsArray[j];

                levelsArray[j]=temp1;
                scoresArray[j]=temp2;
                starsArray[j]=temp3;
            }
        }
    }
}


int main()
{
    const int MAX=400;
    int levelsArray[MAX];
    int scoresArray[MAX];
    int starsArray[MAX];

    int numberOfLevels=buildArrays(levelsArray,scoresArray,starsArray);

    printArrays( "Candy Crush UNSORTED Report", levelsArray, scoresArray, starsArray, numberOfLevels );
    sortArrays( levelsArray, scoresArray, starsArray, numberOfLevels);
    printArrays( "Candy Crush SORTED Report", levelsArray, scoresArray, starsArray, numberOfLevels );

    system("pause");
}
4

2 に答える 2

0

「candycrush.txt」ファイル*の 3 列目に 1 つ以上の文字が含まれていますが、それを整数として読み取ろうとしています。これはもちろんエラーなので、入力ストリームのエラー状態が設定されます。この状態の間、このストリームからの後続の読み取りは失敗します。

これを修正するのは非常に簡単です。最後の列を完全な文字列として読み取るだけです。これにより、もうprintArraysカウントする必要がないため、単純化されます。*そのまま印刷するだけです。

buildArraysオーバーランしないように、実際のサイズを少し修正したものを次に示します。

template <size_t maxsize>
int buildArrays(istream &is, int (&A)[maxsize], 
                             int (&B)[maxsize], 
                             std::string (&C)[maxsize])
{
  assert(is);

  size_t i;
  for(i = 0; i < maxsize && is; ++i)
  {
    is >> A[i] >> B[i] >> C[i];
  }

  return i;
}

メインは入力ファイルを開き、に渡すだけifstreamですbuildArrays

int main()
{
  const int MAX = 400;
  int levelsArray[MAX];
  int scoresArray[MAX];
  string starsArray[MAX];

  ifstream infile("candycrush.txt");
  int numberOfLevels = buildArrays(infile, levelsArray, scoresArray, starsArray);
  // and other stuff
  // ...
}
于 2013-11-11T10:29:20.150 に答える