class name student

[Mashd Taters]

Not able to enter more than one letter in for loop for class list: classList - An array of strings for the names of the classes the student is enrolled in.

  //Reads data and displays student name, class credits, and name of classes
#include <iostream>
#include <string>
#include <cstdlib>
using namespace std;

class Student
{
	public: //public class of student 
		Student(string first_name,string last_name, int num, string list);
		Student();
		
		void input();//input from user
		void output();
		
		
		
		//accessors 
	const	string get_first_name();
	const   string get_last_name();
	const	int get_numClasses();
	const	string get_classList();
		
		//mutators
		void set_first_name(string first);
		void set_last_name(string last);
		void set_numClasses(int num);
		void set_classList(string list);
		
   private: //cannot be accessed 
   	string first_name;
	string last_name; 
    int numClasses;
   	string classList;
		
	
	
};
	Student::Student()   //constructor variable 
	{
	first_name;
	last_name;
	numClasses = 0;	
	classList;	
	}

	string const Student::get_first_name() //accessor get called from class student public
	{
		return first_name;
	}
	string const Student::get_last_name()
	{
		return last_name;
	}
	int const Student::get_numClasses()
	  {
	  	return numClasses;
	  }
	  string const Student::get_classList()
	  {
	  	return classList;
	  }						//accessor get called from class student public 
	
	
	
void Student::set_first_name(string first) //mutator set called from class student public
	{
		first_name = first;
	}
	void Student::set_last_name(string last)
	{
		last_name = last; 
	}
	void Student::set_numClasses(int num)
	{
		numClasses = num;
	}
	void Student::set_classList(string list)
	{
		classList = list;
	}							  
	void Student::input() 			//accessor 
	{
		int i;
										
	 cout<<"Enter first name and last name:"<<endl;
	 cin>>first_name>>last_name;
	 cin.ignore();
         cout<< "Get number of classes taking this semester: "<<endl;
	 cin >> numClasses;
	 cin.ignore();
	    
	   ;										
			if(numClasses>	0)
    		{
    			 classList = new char[numClasses];
    			 for(i=0;i<numClasses;i++)
    		{
                  									
		cout<<"Enter name of classes " <<(i+1)<<endl;
    		cin>>classList[i];
    														
    		}
    		}
    	
    	cout<< '\n'<<endl;
	}	//accessor 
    
    void Student::output()//output what user entered
    {
    	cout<< "Student first name: " << first_name <<endl;
    	cout<< "Student last name: " << last_name <<endl;
    	cout<< "Number of classes: " << numClasses <<endl;	
    	cout<< "Class List: " << classList<<endl;
    
    }

	int main()
	{
	 char play_again = 'Y'; 
     while (play_again == 'y' || play_again == 'Y'){
     
	
	Student s1;	//student class s1
	s1.input(); //input from student 1

	s1.output(); //output from student 1
	
	cout<<endl;
	
	
	
	 cout<<"Enter the data again (press Y/N)\n "<<endl;
           cin>>play_again;
}

		
	if(play_again == 'n')
	{
		return 0;
		
	}
	system("pause");
		
	return 0;
		
	}
	

[o3hC5Di1]

Hi there,

Please consider this code:

	if(numClasses>	0)
    		{
    			 classList = new char[numClasses];
    			 for(i=0;i<numClasses;i++)
    		{
                  									
		cout<<"Enter name of classes " <<(i+1)<<endl;
    		cin>>classList[i];
    														
    		}

You declare classList as an array of char's. Char is a datatype which holds only one character. What you mean to do is classList = new std::string[numClasses];

For the record, I would advise you to use std::vector or std::array rather than "raw" arrays. They take care of a lot of possible memory pitfalls for you and are thus much safer to use.

All the best,
NwN

[Mashd Taters]

Thanks,using dev c++ so I also had to use a pointer to compile the program. I works fine, I just have to code a output for the class list.

if(numClasses>	0)
   {
    	string*	classList = new std::string[numClasses];
    	for(i=0;i<numClasses;i++)
    	{
    	cout<<"Enter name of classes " <<(i+1)<<endl;
        cin>>classList[i];
    	}
  }														
    															
    														
    														}