1

こんにちは、コードでvoid initialize()関数を呼び出すのに問題があります。私はこのような配列を取得することになっています

//cccccccccccccccccccccccccc
  cxxxxxxxxxxxxxxxxxxxxxxxxc
  cxxxxxxxxxxxxxxxxxxxxxxxxc
  cxxxxxxxxxxxxxxxxxxxxxxxxc
  cxxxxxxxxxxxxxxxxxxxxxxxxc
  cxxxxhhhhhhhxxxxxxxxxxxxxc
  cxxxxhhhhhhhxxxxxxxxxxxxxc
  cxxxxhhhhhhhxxxxxxxxxxxxxc
  cxxxxhhhhhhhxxxxxxxxxxxxxc
  cxxxxxxxxxxxxxxxxxxxxxxxxc
  cccccccccccccccccccccccccc//

ここで、配列はInitialize()で定義され、変数はFIN、SteamPipe、およびGridPTクラスから取得されます。私のコードは配列の前に最初の答えのセットを与えていますが、それが配列に到達すると、記号「?」しか得られません。これが、GridPTクラスで指定されているものです。void関数を呼び出す方法が間違っているか、変数の受け渡しに何か問題があると思いますか?私はすでにconstを取り出してみましたが、それでも同じです。これが私のコード.hファイルです

#include<iostream>
#include<istream>
#include<cmath>
#include<cstdlib>

using namespace std;

const int maxFinHeight = 100;
const int maxFinWidth = 100;

class Steampipe
{
private:
  int height, width;
  double steamTemp;

public:
  Steampipe (int H = 0, int W = 0, double sT = 0.0)
  {
    height = H;
    width = W;
    steamTemp = sT;
  }

  // Access function definitions
  int getWidth()const{return width;};
  int getHeight()const{return height;};
  double getSteamTemp()const{return steamTemp;};
  void setWidth(int dummysteamwidth) {width = dummysteamwidth;};
  void setHeight(int dummysteamheight){height = dummysteamheight;};
  void setSteamTemp(double dummysteamtemp){steamTemp = dummysteamtemp;};

  // Access function definitions
  friend istream& operator>>(istream& , Steampipe& ); // YOU MUST DEFINE
  friend ostream& operator<<(ostream& , const Steampipe& ); // YOU MUST DEFINE
};

class GridPt
{
private:
  double temperature;
  char symbol;

public:
  GridPt(double t = 0.0, char s = '?')
  {
    temperature = t;
    symbol = s;
  }

  // Access function definitions
  double getTemperature()const {return temperature;};
  char getsymbol() const {return  symbol;};

  void setTemperature (double T=0) {T=temperature;}
  void setsymbol (char S='?'){S=symbol;}
};

class Fin
{
private:
  int width, height; //width and height of fin
  int pipeLocationX, pipeLocationY; //grid location of lower left corner of steampipe
  double boundaryTemp; //temperature of fin surroundings
  Steampipe Pipe; //steampipe object - COMPOSITION
  GridPt GridArray[maxFinHeight][maxFinWidth]; // array of GridPts - COMPOSITION

public:
  int getwidth() {return width;}
  int getheight() {return height;}
  int getpipeLocationX(){return pipeLocationX;}
  int getpipeLocationY(){return pipeLocationX;}
  double get_boundarytemp(){return boundaryTemp;}
  void setwidth (int w){w=width;}
  void setheight (int h){h=height;} 
  friend istream& operator>>(istream& , Fin& ); // YOU MUST DEFINE
  friend ostream& operator<<(ostream& , const Fin& ); // YOU MUST DEFINE

  void initialize(); // YOU MUST DEFINE
};

void Fin::initialize()
{
  int j(0) ,i(0);
  for ( i = 0; i < height; i++)
  {
    for (  j = 0; j < width; j++)
    {
      if (   j == pipeLocationX && i == pipeLocationY
          && i <  Pipe.getHeight() + pipeLocationY
          && j < Pipe.getWidth() + pipeLocationX)
      {
        GridArray [j][i].setTemperature(Pipe.getSteamTemp());
        GridArray [j][i].setsymbol('H');
      }
      else if (j == (width -1) || i == (height -1) || j == 0 || i == 0)
      {
        GridArray [j][i].setTemperature(boundaryTemp);
        GridArray [j][i].setsymbol('C');
      }
      else
      {
        GridArray [j][i].setTemperature((Pipe.getSteamTemp() + boundaryTemp)/2);
        GridArray [j][i].setsymbol('X');
      }
    }
  } 
}



istream &operator >> (istream & in, Fin& f)
{
  int   ws, hs; 
  double  ts;
  char comma;

  cout << "Enter the height of the fin (integer) >>> ";
  in >> f.height;

  cout << "Enter the width of the fin (integer) >>> " ;
  in >> f.width;

  cout << "Enter the height of the streampipe (integer) >>> ";
  in >>hs;
  f.Pipe.setHeight(hs);
  cout << "Enter the width of the steampipe (integer) >>> ";
  in >> ws;
  f.Pipe.setWidth(ws);
  cout << "Enter coordinates of lower left corner of steampipe (X,Y) >>> ";
  in >> f.pipeLocationX >> comma >> f.pipeLocationY;

  cout << "Enter the steam temperature (floating point) >>> ";
  in >> ts;
  f.Pipe.setSteamTemp(ts);
  cout << "Enter the temperature around the fin (floating point) >>> "; 
  in >> f.boundaryTemp;
  return in;
}

ostream &operator << (ostream &stream, const Fin& t)
{
  int i = 0;
  int j = 0;

  stream << "The width of the fin is " << t.width << endl;

  stream << "The height of the fin is " << t.height << endl;

  stream << "The outside temperature is " << t.boundaryTemp << endl;

  stream << "The lower left corner of the steam pipe is at " 
         << t.pipeLocationX <<      "," << t.pipeLocationY << endl;

  stream << "The steampipe width is " << t.Pipe.getWidth() << endl;

  stream << "the steampipe height is " << t.Pipe.getHeight() << endl;

  stream << "The temperature of the steam is " << t.Pipe.getSteamTemp() << endl;

  for ( i = 0; i < t.height; i++)
{
    for (  j = 0; j < t.width; j++)
    {
        if (j == t.pipeLocationX && i == t.pipeLocationY && i < t.Pipe.getHeight() + t.pipeLocationY && j < t.Pipe.getWidth() + t.pipeLocationX)
        {
            t.GridArray [j][i].getTemperature();
            stream<<t.GridArray [j][i].getsymbol();

        }
        else if (j == (t.width -1) || i == (t.height -1) || j == 0 || i == 0)
        {
            t.GridArray [j][i].getTemperature();
            stream<<t.GridArray [j][i].getsymbol();
        }
        else
        {
            t.GridArray [j][i].getTemperature();
            stream<<t.GridArray [j][i].getsymbol();
        }

    }stream<<endl;
}   

  int coorx(0),coory(0);char comma;

  stream << "Enter the coordinates of the gridpoint of interest (X,Y) >>> "
         << endl;
  cin>>coorx>>comma>>coory;
  stream<<"The temperature at location ("<<coorx<<","<<coory<<") is "
         << t.GridArray << endl;

  return stream;
}

そして私のドライバーファイルcppは:

#include "pipe.h"

int main()
{
  Fin one; 
  cin >> one;
  one.initialize();
  cout << one;

  return 0;
}

任意の提案は非常に高く評価されます。ありがとうございました

4

1 に答える 1

0

あなたのコードでは、以下が間違っています:

void setTemperature (double T=0) {T=temperature;}
void setsymbol (char S='?'){S=symbol;}

それは違いない:

void setTemperature (double T=0) {temperature = T;}
void setsymbol (char S='?') { symbol = S;}

余談ですが、関数定義をヘッダーに配置するのは良い習慣ではありません (テンプレートでない限り)。むしろ、ヘッダーに宣言だけを含むソース ファイルに配置します。

コードt.GridArrayでは、出力ストリームでも直接使用します
stream<<t.GridArray;
。これは機能しません。これは関数ではなく、ostream が理解できる型でもありません。宣言した i 変数と j 変数を使用し、配列を初期化するときと同じように、配列の各要素をステップ実行して出力します。

于 2012-06-27T16:18:42.630 に答える