Thread: Problem with generic function

Hello,

I am new to C++. I tried to create a function for doing the following:

1. Read a matrix from a .txt file
2. Store the retrieved data as a vector of vectors
3. Put the result in screen.

I could somehow complete parts 1 and 2. However, I am having lots of problems with the third part. I tried to use a generic function, and I am getting this complaint from my compiler:

Code:

sscr.h|9|error C2065: 'T' : undeclared identifier

T there, is the type of the generic function. It is supposed to be ambiguous.
That is the problem. Any idea is appreciated. I am learnign this by myself so any kind of modification is welcome.

My codes:

Main function:

Code:

#include <iostream>
#include <fstream>
#include <sstream>
#include <string>
#include <vector>
#include "sscr.h"

using namespace std;

int main(){

    // Variable declaration and initialization

    string x;
    typedef string::size_type string_size;
	string_size i = 0, j, sz;

	vector <vector<int> > adjm;
	vector<int> data;

	ifstream adj("adj.txt");
	istringstream sxp;

	int hi;


	// Actions

	getline(adj, x);
	sxp.str(x);
	sz = x.size() - 1;

	while (i != sz){

		for (j = 0; j != sz; ++j){
			sxp >> hi;
			data.push_back(hi);
		}
		adjm.push_back(data);
		sxp.clear();
		data.clear();

		getline(adj, x);
		sxp.str(x);

		++i;

	}

	// Closing commands
	cout << "the number of items in a row of the file is: " << sz << endl;

 int p = sscr(adjm);

 return p;

}

Generic function:

Code:

#include <iostream>
#include <vector>
#include "sscr.h"

using std::vector;   using std::iostream;

template <class T>
int sscr(vector<T> v){

	for (vector<T>::const_iterator iter = v.begin(); iter != v.end(); ++iter){

		for (vector<T>::const_iterator jter = (*iter).begin(); jter != (*iter).end(); ++jter){
			if (jter != (*iter).end() - 1)
			cout << *jter << " ";
			else
			cout << *jter << endl;
		}
	}
	return 0;
}

Generic function header:

Code:

#ifndef SSCR_H_INCLUDED
#define SSCR_H_INCLUDED

#include <iostream>
#include <vector>

using std::vector;   using std::iostream;

int sscr(std::vector<T>);

#endif // SSCR_H_INCLUDED

Thanks in advance.

Originally Posted by laserlight

The problem is that you forgot the template<class T>, but there may be a secondary problem.

It's there! right in the definition of the generic function! I will look forward to the link you mentioned. Thank you.

When I try to compile the entire project. The initial messages were these:

Code:

d:\dogma\data\research\codes\zcppzone\test01\sscr.h|9|error C2065: 'T' : undeclared identifier|
main.cpp|55|error C2664: 'sscr' : cannot convert parameter 1 from 'std::vector<_Ty>' to 'std::vector'|
||=== Build finished: 2 errors, 0 warnings ===|

Then, I read somewhere, that it was possible that the compiler was not understanding the template thing, so I should move the definition of the function itself to the header. I did so, and after that i receive the complaint message originally posted

Code:

sscr.h|9|error C2065: 'T' : undeclared identifier

Which is, I think, the same as the first complaint before the moving.
Hope I have shed some light on the facts leading to the problem.
Thank you, Laserlight.

The one posted (function code and header are separated). It produces the two error messages list.

No I haven't yet. I will post an updated version as soon as I finish reading the info in your link.

Ok. Thank you very much for your patience, Laserlight. I will update this thread soon (less than 1 hour).

Hello Laserlight,

The first alternative in the link I had already tried. It was the one that generated only one error message. The second one, including in the main file the .cpp file of the function, made things worse.
I think, I will spend some more time reading around about these functions, and will update when I find a suitable solution.
Thanks for your patience.

Thank you, MarkZWEERS

First of all, I am very new in C++. This is my first attempt to code something and I still have many doubts, even in details as simple as the use of the operators. I appreciate your comment about the de-referenciator for the iterators in the inner loop. I also had a doubt about it.
Ok. Now the core problem with the generic function. I implemented the changes you suggested, but alas, still nothing. I am using codeblocks and MS visual C++ 2005/2008 compiler and I received this complaints:

Code:

||=== test01, Debug ===|
d:\dogma\data\research\codes\zcppzone\test01\sscr.h|19|error C2039: 'begin' : is not a member of 'std::_Vector_const_iterator<_Ty,_Alloc>'|
d:\dogma\data\research\codes\zcppzone\test01\sscr.h|19|error C2039: 'end' : is not a member of 'std::_Vector_const_iterator<_Ty,_Alloc>'|
d:\dogma\data\research\codes\zcppzone\test01\sscr.h|20|error C2039: 'end' : is not a member of 'std::_Vector_const_iterator<_Ty,_Alloc>'|
d:\dogma\data\research\codes\zcppzone\test01\sscr.h|21|error C2679: binary '<<' : no operator found which takes a right-hand operand of type 'const std::vector<_Ty>' (or there is no acceptable conversion)|
d:\dogma\data\research\codes\zcppzone\test01\sscr.h|23|error C2679: binary '<<' : no operator found which takes a right-hand operand of type 'const std::vector<_Ty>' (or there is no acceptable conversion)|
||=== Build finished: 5 errors, 0 warnings ===|

Now, the updated code is this:

Code:

// Attempt to READ a matrix from a file: Successful

#include <iostream>
#include <fstream>
#include <sstream>
#include <string>
#include <vector>
#include "sscr.h"

using namespace std;

int main(){

    // Variable declaration and initialization

    string x;
    typedef string::size_type string_size;
	string_size i = 0, j, sz;

	vector <vector<int> > adjm;
	vector<int> data;

	ifstream adj("adj.txt");
	istringstream sxp;

	int hi;


	// Actions

	getline(adj, x);
	sxp.str(x);
	sz = x.size() - 1;

	while (i != sz){

		for (j = 0; j != sz; ++j){
			sxp >> hi;
			data.push_back(hi);
		}
		adjm.push_back(data);
		sxp.clear();
		data.clear();

		getline(adj, x);
		sxp.str(x);

		++i;

	}

	// Closing commands
	cout << "the number of items in a row of the file is: " << sz << endl;

 int p = sscr(adjm);

 return p;

}

The only change here was to the line now depicting #include "sscr.h". The header sscr.h is this:

Code:

#ifndef SSCR_H_INCLUDED
#define SSCR_H_INCLUDED

#include <iostream>
#include <vector>

using std::vector;   using std::iostream;

template<typename T>
int sscr(vector<T> v);

using std::vector;   using std::iostream;

template <typename T>
int sscr(vector<T> v){

	for (vector<T>::const_iterator iter = v.begin(); iter != v.end(); iter++){

		for (vector<T>::const_iterator jter = iter.begin(); jter != iter.end(); jter++){
			if (jter != iter.end() - 1)
			cout << *jter << " ";
			else
			cout << *jter << endl;
		}
	}
	return 0;
}

#endif // SSCR_H_INCLUDED

The changes here were the one that you pointed out in your comment.
Thanks in advance for your help.

Hey fellas,

I just did something interesting (at last!).
I thought the problem could be not in the definition on the layout of the generic function or the way that the compiler was interpreting the thing but in the special type I am using in the input vector<vector<int> >. So I modify the code to just send to screen a message instead of the elements of the matrix. I rewrote the body like this:

header sscr.h

Code:

#ifndef SSCR_H_INCLUDED
#define SSCR_H_INCLUDED

#include <iostream>
#include <vector>

//using std::vector;   using std::iostream;

//template<class T>
//int sscr(T v);

using std::vector;   using std::iostream;

template <class T>
int sscr(T v){

	for (T::const_iterator iter = v.begin(); iter != v.end(); iter++){

		//for (T::const_iterator jter = iter.begin(); jter != iter.end(); jter++){
			//if (jter != iter.end() - 1)
			cout << "Hello" << " ";
			//else
			//cout << *iter << endl;
		}
		return 0;
	}

#endif // SSCR_H_INCLUDED

The code went smoothly. Could anybody tell me what is wrong with the way I am sending out the information stores in the vector<vector<int> >?

Thank you.