Инициализация массива с использованием указателя, а затем чтение данных из него.

Question

Это программа, написанная на C ++.Он использует указатель для инициализации массива int и затем считывает данные обратно.Но вместо значений int он дает дерьмо букв и цифр.В чем может быть проблема?Я не могу решить это.это файл cpp:

#include<string>
#include<cmath>
#include<sstream>
#include "TwoSeries.hpp"

using namespace std;

TwoSeries::TwoSeries() {
}

TwoSeries::TwoSeries(const int& size, const int& a0, const int& b0) 
{
    int arrayA [size];
    int arrayB [size];

    arrayA[0] = a0;
    arrayB[0] = b0;

    int *aArray = getArrayA();
    int *bArray = getArrayB();

    for(int x = 1; x < 200; x++)
    {       
        *(aArray + x) = -1;
        *(bArray + x) = -1;   

    }

}

TwoSeries::~TwoSeries() {
}

int TwoSeries::getA(int& index)
{

    return 0;
}
int TwoSeries::getB(int& index)
{

    return 0;
}

string TwoSeries::getArrays()
{
    ostringstream outString;
    string stA;
    string stB;
    string valueA;
    for(int x = 0; x < 200; x++)        
    {
        outString << x;
        string index = outString.str();

        int *arrayA = getArrayA();
        outString << *(arrayA + 1);
         valueA = outString.str();


        int * arrayB = getArrayB();
        outString << getArrayB() + x;
        string valueB = outString.str();


    //    stA += index + ":" + valueA + " ";
    //    stB += index + ":" + valueB + " ";
    }

  //  return "Series A: \n"+stA+ "/n"+"Series B: \n"+ stB;   
    return valueA;
}

int* TwoSeries::getArrayA()
{
    int * pointer = arrayA;

    return pointer;
}
int* TwoSeries::getArrayB()
{
   int * pointer = arrayB;

    return pointer;
}

Это файл hpp:

#include<string>
#include<cmath>
#include<sstream>

using namespace std;

#ifndef TWOSERIES_HPP
#define TWOSERIES_HPP

class TwoSeries {
public : 
  TwoSeries();
  TwoSeries(const int&, const int& , const int&);
  int getA(int&);
  int getB(int&);
  string getArrays();
  int* getArrayA();
  int* getArrayB();

    virtual ~TwoSeries();
private:
    int arrayA[200];
    int arrayB[200];
};

#endif  /* TWOSERIES_HPP */

Это основной метод:

#include<iostream>
#include<string>
#include "TwoSeries.hpp"

using namespace std;

/*
 * 
 */
int main() {
    const int A0 = 1; // the value at index 0 at array A
    const int B0 = 1; // the value at index 0 at array B
    const int SIZE = 200; // the size of array A and B
    const int EXIT = 4; // the digit on which the loop stops    
    int option; // stores the value entered by the user i.e. selected option from the list
    int index;  // stands for the index in either A or B array
    TwoSeries two = TwoSeries(SIZE, A0, B0);

    do
      {
        cout << "You have the following options: \n 1 - Display the current arrays\n";
        cout << " 2 - request the values of A \n 3 - request the values of B\n 4 - exit \n";
        cin >> option;
        if(option == 1)
        {
          cout << two.getArrays();  
        }
        if ((option == 2) || (option == 3))
        {
            do
            {
               cout << "Please enter the position: ";
               cin >> index;
               if((index >= 0) && (index <= (SIZE - 1)))
               {
                  if(option == 2){}
                     // getA(index);                 
                  //else
                   //   getB(index);
               }
            }
            while((index >= 0) && (index <= (SIZE - 1)));

        }
      }
      while((option != EXIT) || option < 1 || option > 4 );

    return 0;
}

Answer 1

Здесь так много неправильных вещей, поэтому давайте сделаем это медленно:

TwoSeries::TwoSeries(const int& size, const int& a0, const int& b0) 
{
    int arrayA [size]; //no way this compiles without new [] operator
    int arrayB [size]; //also why you name identical local variable with the same name as your private data members?

    arrayA[0] = a0; //so you initialize the first member with some value
    arrayB[0] = b0; //and then all the other values to -1 is this intended?

    int *aArray = getArrayA(); //why are you doing this?
    int *bArray = getArrayB(); // you can access the data member by using this->arrayA; even without the this, but since you have identical name with locals you need the "this"

    for(int x = 1; x < 200; x++)
    {       
        *(aArray + x) = -1; //avoid pointer arithmetic, it's error prone
        *(bArray + x) = -1; //do this instead bArray[x] = -1;   

    }

}

Далее:

int TwoSeries :: getA (int & index) {// Я думаю, вы хотитевернуть элемент из массива?вернуть массивA [index];// подвержен ошибкам, проверяем, находится ли индекс между границами и положительным. return 0;}

Далее:

int* TwoSeries::getArrayA()
{
    //int * pointer = arrayA; //not needed

    return arrayA;
}
int* TwoSeries::getArrayB()
{
   //int * pointer = arrayB; //same as above

    return arrayB;
}

Далее:

string TwoSeries::getArrays()
{
    ostringstream outString;
    string stA;
    string stB;
    string valueA;
    for(int x = 0; x < 200; x++)        
    {
        outString << x;
        string index = outString.str();

        //int *arrayA = getArrayA(); //same mistake as before 
        //outString << *(arrayA + 1); //you realize you always access the same element right?
        outString << arrayA[x]; //I guess this is what you wanted?
         valueA = outString.str();


        int * arrayB = getArrayB(); // see above
        outString << getArrayB() + x;
        string valueB = outString.str();


    //    stA += index + ":" + valueA + " ";
    //    stB += index + ":" + valueB + " ";
    }

  //  return "Series A: \n"+stA+ "/n"+"Series B: \n"+ stB;   
    return valueA;
}

Возможно, здесь скрыто больше ошибок.Но вы опубликовали что-то, что даже не компилируется, поэтому мы не можем действительно помочь вам.

Answer 2

Это

TwoSeries::TwoSeries(const int& size, const int& a0, const int& b0) 
{
    int arrayA [size];
    int arrayB [size];

должно быть просто

TwoSeries::TwoSeries(const int& size, const int& a0, const int& b0) 
{

, так как в противном случае остальная часть этой функции будет предполагать использование локальных переменных вместо закрытых членов классов.

Также я не вижу смысла аргумента размера, так как вы жестко запрограммировали размер массива в файле hpp?

Последнее замечание:

int * arrayB = getArrayB();
outString << getArrayB() + x;

просто будь

int * arrayB = getArrayB();
outString << arrayB[x];