Having trouble with my output.

[monopoman]

The text file this file is reading is as such...

CS162;Finish Lab 2;09/26/2009
CS161;Finish Lab 3;09/26/2009
CS201;Finish Finals;05/23/2010
CS211;Take Quiz 1;04/22/1999

It outputs this though when I ask it to show me the data.

CS162;Finish Lab 2;09/26/2009
CS161
CS201;Finish Finals;05/23/2010
CS211

I have no clue why it is removing half of my data when I ask it to show me the data in the program.

I'm also having trouble with the find part of the program trying to figure out how I can search for the course name and then output just that.

#include <iostream>
#include <iomanip>
#include <fstream>

using namespace std;

const int MAXCHAR = 101;
const int CAP = 100;

struct Task
{
	char courseName[MAXCHAR];
	char taskDescription[MAXCHAR];
	char dueDate[MAXCHAR];
}; 

void menu();
char getCommand();
void exeCommand(Task [], int &, char);
void addItem(Task [], int &);
void showList(Task [], int &);
void findTask(Task [], int &);
void loadData(Task [], int &);
void addItem(ifstream &, Task [], int &);
void saveData(Task [], int &);

int main(void)
{
	Task courses[CAP];
	int size = 0;
	char option;
	loadData(courses, size);
	do
	{
		menu();
		option = getCommand();
		exeCommand(courses, size, option);
	}while(tolower(option) != 'q');
}

void menu()
{
	cout << "Welcome to my Task List:\n"
		<< "(a) to add a task\n"
		<< "(s) to show the task list\n"
		<< "(f) to find a task by course name\n"
		<< "(q) to quit\n";
	cout << "Please enter an option: ";
}

char getCommand()
{
	char ans;
	cin.get(ans); 
	switch (tolower(ans))
	{	
	case 'a': 
	case 's':
	case 'f':
	case 'q': 
		cin.ignore(100, '\n');
		return ans;
	default: 
		cout << "Illegal input.";
		cin.clear();
		cin.ignore(100, '\n');
		system("cls");
		menu();
		getCommand();
	}
}

void exeCommand(Task list[], int &size, char ans)
{
	switch(tolower(ans))
	{
	case 'a':
		addItem(list, size);
		saveData(list, size);
		break;
	case 's':
		showList(list, size);
		break;
	case 'f':
		findTask(list, size);
		break;
	case 'q':
		cout << "Thanks for using this program goodbye!" << endl;
		return;
	default:
		return;
	}
}

void loadData(Task list[], int &size)
{
	ifstream inFile;
	inFile.open("task.txt");
	if(!inFile)
	{
		cout << "File did not open! Program terminating!" << endl;
		exit(1);
	}
	while(!inFile.eof())
	{
		addItem(inFile, list, size);
	}
}

void addItem(Task list [], int &size)
{
	char course[MAXCHAR];
	char taskDesc[MAXCHAR];
	char date[MAXCHAR];
	char junk;
	cout << "Enter Course Name (less than 101 characters.): ";
	cin.get(course, MAXCHAR, '\n');
	cin.get(junk);
	cout << "Enter Task Description (less than 101 characters.): ";
	cin.get(taskDesc, MAXCHAR, '\n');
	cin.get(junk);
	cout << "Enter Due Date (mm/dd/yyyy): ";
	cin.get(date, MAXCHAR, '\n');
	cin.get(junk);
	strcpy_s(list[size].courseName, course);
	strcpy_s(list[size].taskDescription, taskDesc);
	strcpy_s(list[size].dueDate, date);
	size++;
}

void addItem(ifstream &inFile, Task list[], int &size)
{
	inFile.get(list[size].courseName, MAXCHAR, ';');
	inFile.ignore(50, ';');
	inFile.get(list[size].taskDescription, MAXCHAR, ';');
	inFile.ignore(50, ';');
	inFile.get(list[size].dueDate, MAXCHAR, ';');
	inFile.ignore(50, '\n');
	size++;
}

void showList(Task list[], int &size)
{
	int i;
	for(i = 0; i < size; i++)
	{
		cout << left << list[i].courseName << ";"
			<< left << list[i].taskDescription << ";"
			<< left << list[i].dueDate << endl;
	}
}

void saveData(Task list[], int &size)
{
	int i = 0;
	ofstream outFile;
	outFile.open("task.txt");
	if(!outFile)
	{
		cout << "File did not open! Program terminating!" << endl;
		exit(1);
	}

	for(i = 0; i < size; i++)
	{
		outFile << list[i].courseName << ';'
			<< list[i].taskDescription << ';'
			<< list[i].dueDate << endl;
	}
}

void findTask(Task list[], int &size)

{
	int i = 0;
	char junk;
	char course[MAXCHAR];
	cout << "This will allow you to search for a task by it's course name." << endl;
	cout << "Enter Course Name (less than 101 characters.): ";
	cin.get(course, MAXCHAR, '\n');
	cin.get(junk);
	if (strcmp(list[i].courseName, course) == 0)
		cout << "found it";
}

[Khaltazar]

Your code won't compile correctly. You must be missing a header or something.

strcpy_s - Undeclared... first use in this function.

Either means you need a header or you are trying to use a function that doesn't exist.

[monopoman]

The program runs I have ran it like 10 times in Microsoft Visual C++ 2010 Express.

Not sure why you say it won't run...

if the strcpy_s is giving ya prob's can use strcpy.

Figured it out myself anyways though...

[Khaltazar]

Oh, I use Dev-C++ compiler. It doesn't have a built in library to access strcpy_s. I switched it to strcpy only and it worked. I'm glad you found the answer, sorry I wasn't quicker. I'm a beginner too, but helping others helps me become a better programmer. Debugging is the main part of the job after all.