C++ trouble with overloaded constructor and arrays

Question

0.00/5 (No votes)

See more:

I am having trouble figuring out the syntax for how i am suppose to Call the member function to copy the values of my array into the appropriate member variables. This is specifically for the overloaded constructor because I think all the other functions are correct but they might not be.

Student.cpp

C++

#include <iostream>
#include <fstream>
#include <string>
#include "Student.h"
using namespace std;

void Student::setID(int tID)
{
	ID = tID;
}

void Student::setFName(string f)
{
	firstName = f;
}

void Student::setLName(string l)
{
	lastName = l;
}

void Student::setScores(int list[])
{
	for (int i = 0; i < 5; i++)
		scores[i] = list[i];
	//scores[5] = list[5];
}

int Student::getID()
{
	return ID;
}

string Student::getFName()
{
	return firstName;
}

string Student::getLName()
{
	return lastName;
}

int Student::getWeightedTotal()
{
	int WeightedTotal = 0;
	WeightedTotal = 25 % (scores[0]) + 25 % (scores[1]) + 30 % (scores[2]) + 10 % (scores[3]) + 10 % (scores[4]);
	return WeightedTotal;
}

char Student::getGrade() // should be working 
{
	char grade = 0;
	if (getWeightedTotal() >= 90)
		grade = 'A';
	else if (getWeightedTotal() >= 80)
		grade = 'B';
	else if (getWeightedTotal() >= 70)
		grade = 'C';
	else if (getWeightedTotal() >= 60)
		grade = 'D';
	else
		grade = 'F';
	return grade;
}

void Student::printStudent()
{
	cout << "Student ID" << "    " << "Full Name" << "         " << "Scores" << "     " << "Weighted Total" << "     " << "Grade" << endl;
	cout << ID << "       " << firstName << " " << lastName << " " <<
		scores[0] << scores[1] << scores[2] << scores[3] << scores[4] << " " << getWeightedTotal() << "   " << getGrade() << endl;
}

Student::Student()
{
	ID = 0;
	firstName = "";
	lastName = "";

	for (int i = 0; i < 5; i++) // for loop to set each element to 0
		scores[i] = 0;


	//scores[5] = { 0 };

}

Student::Student(int tID, string f, string l, int list[])
{
	setID(tID);
	setFName(f);
	setLName(l);
	for (int i = 0; i < 5; i++)
		setScores(list[i]);
	//setScores(list);


}

The problem is right above this text in the overloaded constructor and i'm not sure what to do

What I have tried:

I've tried

C++

for (int i = 0; i < 5; i++)
		setScores(list[i]);

Also tried just

C++

setScores(list);

also tried

C++

for (int i = 0; i < 5; i++)
		scores[i] = list[i];

Posted 6-Mar-17 6:52am

Member 13041778

Updated 6-Mar-17 10:41am

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Answer 1 · 2017-03-06T10:41:00

The correct syntax is

C++

Student::Student(int tID, string f, string l, int list[])
{
  setID(tID);
  setFName(f);
  setLName(l);
  setScores(list);
}

Assuming Student.h contains something like

C++

class Student
{
  int ID;
  int scores[5];
  string firstName, lastName;
public:

  void setID(int tID);
  void setFName(string f);
  void setLName(string l);
  void setScores(int list[]);
  int getID();
  string getFName();
  string getLName();
  int getWeightedTotal();
  char getGrade(); // should be working 
  void printStudent();
  Student();
  Student(int tID, string f, string l, int list[]);

};

The code compiles and runs correctly with the following main:

C++

int main()
{
  int score[] = {10,9,8,7,6}; 
  Student s(42, "Foo", "Bar", score);
  s.printStudent();
}