Thread: Declaring Pointers/Passing Data

Code:

Stock* newStock;
newStock= new Stock(SN,Dn,CS,US,UP,MS);
newStock->Print();

Here im declaring a pointer of type Stock called newStock. Then sending the data to the stock class and saving it to my pointer.
I run the print method here as a way of checking if all has gone well.

I have used similiar ways of doing this in past and never had any problems. This seems strange to me.

I get these errors...

Error: mainapp.cpp(189,17):Undefined symbol 'newStock'
Error: mainapp.cpp(190,21):Type name expected
Error: mainapp.cpp(190,21):Statement missing ;


So...

1. Is what im doing a standard way of doing things? or am i doing something inherantly wrong?

2. Why is it telling me its undefined when i am defining it in that very piece of code.

I think its something regarding the 'newStock= new Stock(SN,Dn,CS,US,UP,MS);' that is causing the problems, especially the new. But im really stumped. Any ideas?

The stock.h and stock.cpp can be supplied if needed but i doubt its that.

Rest of the program works fine without this function but when its included it throws a fit.

If its of any benefit here is the whole function.

Code:

void Application::DoAddStock()
{
eMenu.getOption();
string SN;
string Dn;
double CS;
float US;
float UP;
double MS;

	cout<<"\nEnter Stock Number : ";
	Get(SN, 8);
	cout<<"\nEnter a Description of the Product : ";
	Get(Dn, 30);
	cout<<"\nEnter Initial Stock Level : ";
	Get(CS, 6);
	cout<<"\nEnter Unit Size : ";
	Get(US, 6);
	cout<<"\nEnter Unit Price : ";
	Get(UP, 6);
	cout<<"\nEnter Minimum Stock Level :";
	Get(MS, 6);


Stock* newStock;
newStock =new Stock(SN,Dn,CS,US,UP,MS);
newStock->Print();
}

stock.h is included, the memory leak isnt really an issue atm, but ill add a destructor later.

Get is part of the <iostream.h> header. Not my work.

Should my stock class have some reference to this 'new new Stock(SN,Dn,CS,US,UP,MS);' bit, apart from the obvious constructor, which btw doesnt mention 'new' anywhere.

Well Get() is working, get() however does not work.

This isnt the problem im quite sure.

Stock files below.

Code:

/////////////////////////
//Ryan Pincher
//Stock.cpp
//
//
//////////////////////////

#include <defs.h>
#include <iostream.h>
#include <conio.h>
#include <cstring.h>
#include "stock.h"

Stock::Stock()
{
StockNumber=" ";
Description=" ";
CurrentStock=0;
UnitSize=0;
UnitPrice=0;
MinStock=0;
};

Stock::Stock(string SN, string Dn, double CS, float US, float UP, double MS)
{
StockNumber=SN;
Description=Dn;
CurrentStock=CS;
UnitSize=US;
UnitPrice=UP;
MinStock=MS;
};

void Stock::Print(ostream& strm)
{

	strm << "\nStock Number : " << StockNumber<< endl;
	strm << "Description : " << Description << endl;
	strm << "Current Stock : " << CurrentStock << endl;
	strm << "Unit Size : " << UnitSize << endl;
	strm << "Unit Price : " << UnitPrice << endl;
   strm << "Minimum Stock : " << MinStock << endl;
}; // end Print()


ostream& operator<<(ostream& strm, Stock& S)
{
	S.Print(strm);
	return strm;
};

Code:

////////////////////////////////////////
// Ryan Pincher
// Stock.h
//
//////////////////////////////////////////

#include <defs.h>
#include <iostream.h>
#include <conio.h>
#include <cstring.h>

#ifndef STOCK_H
#define STOCK_H


class Stock
{

		private:
      //Attributes
      string	StockNumber;
      string	Description;
      double	CurrentStock;
      float		UnitSize;
      float		UnitPrice;
      double	MinStock;

      public:
      //Methods
      Stock();
      Stock(string SN, string Dn, double CS, float US, float UP, double MS);
      virtual void Print(ostream& strm = cout);
      string	getStockNo();
      string	getDescription();
      double   getCurrStock();
      float    getUnitSize();
      float    getUnitPrice();
      double   getMinStock();


      friend ostream& operator<<(ostream& strm, Stock& S);

};

#endif

Im using borland 5.01, Get() works so im not concerned with the case of the text.

Just getting this data to go back to the class.

Check istream.h its included in iostream

I have never seen Get() in iostream or iostream.h either. Have seen get(). get() shouldn't be used to get numerical values, use >> instead.

in Stock.h you use string variables but never #include the header file for string class which is

#include <string>

Unfortunately,

#include <cstring.h>

wrong on two accounts. First cstring is the file for functions that manipulate CStyle strings (null terminated char arrays) not objects of the STL string class as you try to do. Second, you don't use the .h extension with the preprocessor directive, just cstring.