Weird logic error... I need another set of eyes.

[blackmars0]

Can anyone see a logic error in the init (char [],char [], int) function?? This is in C++.

I've tried setting the null byte earlier in the string, and using diagnostic printf's, and nothing's worked. Using the early null byte, I still get the error at the bottom, and using the printf's, it prints two instances of the string, the second is one x shorter than the first.

I get compile warnings as follows

ld: 0711-224 WARNING: Duplicate symbol: .strcpyFPcPCc
ld: 0711-345 Use the -bloadmap or -bnoquiet option to obtain more information.

I'll post everything.

#include <stdio.h>
#include <string.h>
#include "Account.h"


//initialize any blank account objects
void Account::init() {
	this->balance = 0;
	strcpy(this->accountNumber, "000000000000000");
	strcpy(this->customer, ",,,;");
}

//intialize objects with specified values
void Account::init(char c[], char num[], int b) {
	int i=0,q,p;
	q=strlen(c);
	//clean c
	//convert tab and newline to whitespace
	for (i=0;i<=q;i++) {
		if (c[i] == '\t' || c[i] == '\n') {
			c[i] = ' ';
		}
	}
	//remove leading whitespace
	while (c[0]==' ') {	
		for (i=0; i<=q; i++) {
			c[i] = c[i+1];
		}
	}
	//remove double whitespace
	i=0;
	while (c[i]!='\0') {
		q=i;
		q=strlen(c);
		if(c[i] == ' ' && c[i+1] == ' ' || c[i+1] == '\0') { 
			for (p=i;p<=q;p++) {
				c[p] = c[p+1];
			}
		i--;
		}	
		i++;		
	}
	
	//cleaning finished
	
	q=strlen(c);
	if (q > 250) {
		strncpy(this->customer,c,251);
		this->customer[250] = '\0';
		}
	
	
	else if (q <=250);
		strcpy(this->customer,c);
	
	this->balance = b;
	strcpy(this->accountNumber,num);
}


void Account::changeAccountNumber(char num[]) {
	strcpy(this->accountNumber, num);
}

void Account::changeBalance(int b) {
	this->balance = b;
}

void Account::getLastName(char st[]) {
	int n,t,i=0;
	
	while (this->customer[i] != ',') {
		i++;
	}
	st[i]='\0';
	i--;
	
	for (t=0;t<=i;t++) {
		st[t] = this->customer[t];
	}
}

void Account::getFirstName(char st[]) {
	int n,t,i,p,c=0;
	p=strlen(this->customer);
	for (i=0;i<=p;i++) {
		if (this->customer[i] == ',' && c == 1 && c != 2) {
			t = i;
			c++;
		}
		else if (this->customer[i] == ',' && c != 2) {
			p = i;
			c = 2;
		}
		else if (this->customer[i] == ',');
			c++;
		
	}
	t++;
	
	for (i=t;i<p;i++) {
		st[i] = this->customer[i];
	}
}

void Account::getCity(char st[]) {
	int n,t,i,p,c=0;
	p=strlen(this->customer);
	for (i=0;i<=p;i++) {

		if (this->customer[i] == ',' && c == 2) {
			t = i;
			c++;
		}
		else if (this->customer[i] == ',' && c == 3) {
			p = i;
			c++;
		}
		else if (this->customer[i] == ',');
			c++;
	}
	t++;
	
	for (i=t;i<p;i++) {
		st[i] = this->customer[i];
	}
}

void Account::getPhoneNumber(char st[]) {
	int n,t,i,p,c=0;
	p=strlen(this->customer);
	for (i=0;i<=p;i++) {

		if (this->customer[i] == ',' && c == 3) {
			t = i;
			c++;
		}
		else if (this->customer[i] == ';' ) {
			p = i;
			c++;
		}
		else if (this->customer[i] == ',');
			c++;
	}
	t++;	
		
	for (i=t;i<p;i++) {
		st[i] = this->customer[i];
	}
}

const char* Account::getCustomer() const {
	return this->customer;
}



const char* Account::getAccountNumber() const {
	return this->accountNumber;
}

int Account::getBalance() const {
	return this->balance;
}

Account::Account(char c[], char num[], int b) { init(c, num, b); }		
Account::Account() {init();}

This is the code that passes stuff in.

int main()
{
	Account one;
	if (test1(one))
	{
		char d[] = "   xxxxx  xxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx"
				   "xxxxxxxxxxxxxxxxxxxxxxxxx";
		char num[] = "aw0000000000000";
		Account two(d, num, 5347);
		Account three;
		three.init(d, num, 5347);
		if (initTest(two,three))
		{
			 printf("Failed init test\n"
					"fix your init function\n");
		}
		else
		{
			test2(two);
		}
	}
	return 0;
}
int initTest(Account& a, Account& b)
{
	int result = 1;
	if(!strcmp(a.getCustomer(), b.getCustomer()))
	{
		if (!strcmp(a.getAccountNumber(), b.getAccountNumber()))
		{
			if (a.getBalance() == b.getBalance())
			{
				result = 0;
			}
		}
	}
	return result;
}

int test1(Account& in)
{
	int result = 1;
	if(strcmp(in.getCustomer(), ",,,;"))
	{
		printf("Failed test one-a, your getCustomer returned %s\n"
				"it should have returned ,,,;\n"
				"fix your default constructor\n", in.getCustomer());
		result = 0;
	}
	else if (strcmp(in.getAccountNumber(),"000000000000000"))
	{
		printf("Failed test one-b, your getAccountNumber returned %s\n"
				"it should have returned 000000000000000\n"
				"fix your default constructor\n",in.getAccountNumber());
		result = 0;
	}
	else if (in.getBalance() != 0)
	{
		printf("Failed test one-d, your getBalance returned %d\n"
				"it should have returned 0\n"
				"fix your default constructor\n", in.getBalance());
		result = 0;
	}
	return result;
}

void test2(Account& in)
{
	char d[] = "xxxxx xxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx"
			   "xxxxxxxxxxxxxxxxxxxxxxxxx";
	char num[] = "aw0000000000000";
	if (strcmp(in.getCustomer(), d))
	{
		printf("Failed test two-a, your getCustomer returned %s\n"
				"it should have returned %s\n"
				"fix your Account(char c[], char num[], int b) constructor"
				, in.getCustomer(), d);
	}
	else if (strcmp(in.getAccountNumber(), num))
	{
		printf("Failed test two-b, your getAccountNumber returned %s\n"
				"it should have returned %s\n"
				"fix your Account(char c[], char num[], int b) constructor"
				, in.getAccountNumber(), num);
	}
	else if (in.getBalance() != 5347)
	{
		printf("Failed test two-d, your getBalance returned %d\n"
				"it should have returned 5347\n"
				"fix your Account(char c[], char num[], int b) constructor"
				, in.getBalance());
	}
	else
	{
		strcpy(d, "James,Harry,St.   Louis,416-555-5555;");
		Account j(d, num, 1234);
		strcpy(d, "James,Harry,St. Louis,416-555-5555;");
		if (strcmp(j.getCustomer(), d))
		{
		printf("Failed test two-e, your getCustomer returned %s\n"
				"it should have returned %s\n"
				"fix your Account(char c[], char num[], int b) constructor"
				, j.getCustomer(), d);
		}
		else
		{
			test3(in);
		}
	}
}

It fails at test 2-a. Here's the error:

Failed test two-a, your getCustomer returned xxxxx xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxaw0000000000000
it should have returned xxxxx xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
fix your Account(char c[], char num[], int b) constructor

Anyone?? I've been working on this way too long...

[redhead]

I dont have the time to read through it all, but one thing stood out:

    q=strlen(c);
    //clean c
    //convert tab and newline to whitespace
    for (i=0;i<=q;i++) {

Shouldn't that be:

    q=strlen(c);
    //clean c
    //convert tab and newline to whitespace
    for (i=0;i< q;i++) {

Since strlen() returns the length of the string hence char tmp[] = "ab" has length 2, but at index tmp[2] you will have a NULL placed instead of 'b', and I suspect similar off by one errors might be hidden...

[blackmars0]

Hmm, I'll give that a look, thanks.

[blackmars0]

Originally Posted by redhead

Since strlen() returns the length of the string hence char tmp[] = "ab" has length 2, but at index tmp[2] you will have a NULL placed instead of 'b', and I suspect similar off by one errors might be hidden...

Actually, wouldn't tmp[2] = '\0'? because arrays start at 0, not 1. so tmp[0]='a', tmp[1]='b', and tmp[2]='\0', does it not??

[redhead]

NULL is '\0', but that dosn't change the fact that a part like:

for (p=i;p<=q;p++) {
    c[p] = c[p+1];
}

Will attempt to copy past the NULL (or '\0') index of the array...