Class with use of <vector>

[JoBe]

Hello ladies and gents,

Wanted to make this part of an exercise in wich I enter each time two integers into an array for wich I have to use STL container vector.

Ive got this sofar, but getting an error message when I entered two numbers and the exe shuts down? I'm almost certain it's got to do with the

while (cin>> x >>" ">> y, !cin.fail())

But, not sure about it and even if, don't know how to solve it?

What Ive got sofar in total is this:

#include <iostream>
#include <vector>

using namespace std;

class point
{
private:
	int x, y, z;

public:
	point (int xx = 0, int yy = 0, int zz = 0): x (xx), y (yy), z (zz) {}

	vector<int> s, t;

	void numbers()
	{
		while (cin>> x >>" ">> y, !cin.fail())
		{
			s.push_back(x);
			s.push_back(y);
		}

	}

	void print()
	{
		int n = s.size();

		for (z = 0; z < n; z++)
			cout<< s[z] <<endl;
	}
};

int main()
{
	point u;

	cout<<"Type twee integere getallen in per keer: "<<endl;

	u.numbers();

	u.print();

	cin.get();

	return 0;
}

Could someone please point me in the right direction, thank you ;)

[JoBe]

Found it, it's this wich was causing the problem

>>" ">>

Still, if someone might want to have a look at the part of code that I wrote allready and give some hints or suggestions to improve it, I'd very much appreciate it ;)

[Dave Sinkula]

How about this?

while ( cin >> x >> y )

What is an example of your input?

[JoBe]

Hi Dave,
Ive allready changed the cin part :)

Input for example:

10 20
30 40
50 60
stop

Output should be and is now:
10
20
30
40
50
60

This is just a small part of the exersize tough, have to include the operators == and <.

But, the exersize also talkes about two point-objects?? I'm I correct in thinking that this is when a class object has two datamembers?

Something like this,

class point
{
point (int xx = 0, int yy = 0): x (xx), y (yy){} --> two datamembers x and y.
...
};

int main()
{
point u;--> one class object 
...
}

If you don't understand what I'm trying to say, then I'll translate the entire exersize.

By the way, have you heard anything from Narue, strange for her to stay away for so long :?:

[Dave Sinkula]

Do you want a vector of points instead? That is,

#include <iostream>
#include <vector>

using namespace std;

class point
{
   int x, y, z;
public:
   point (int xx = 0, int yy = 0, int zz = 0): x (xx), y (yy), z (zz) {}
   bool get()
   {
      cout << "Type twee integere getallen in per keer: " << endl;
      return cin >> x >> y;
   }
   void print()
   {
      cout << x << "," << y << endl;
   }
};

class something
{
   vector<point> s;
public:
   something()
   {
      point u;
      while ( u.get() )
      {
         s.push_back(u);
      }
   }
   void print()
   {
      for ( int n = s.size(), z = 0; z < n; z++ )
         s[z].print();
   }
};

int main()
{
   something interesting;
   interesting.print();
   cin.get();
   return 0;
}

/* my output
Type twee integere getallen in per keer:
1 2 3 4 5 6 stop
Type twee integere getallen in per keer:
Type twee integere getallen in per keer:
Type twee integere getallen in per keer:
1,2
3,4
5,6
*/

>By the way, have you heard anything from Narue, strange for her to stay away for so long

[voice=Sgt. Shultz]I know nothing.[/voice]

[Dogtree]

>> while (cin>> x >>" ">> y, !cin.fail())
Since you already know what the problem was, I'll tell you why it's a problem, just to be thorough. cin >> is strictly for formatted input. It doesn't accept formatting strings and it already skips whitespace by default. If you want to skip precise strings then you can do this (nonportable):

#include <iostream>

namespace std {
  template <typename CharT, typename Traits>
  basic_istream<CharT, Traits>&
    operator>>(basic_istream<CharT, Traits>& in, const char *fmt)
  {
    while (in.good() && *fmt != '\0') {
      if (in.get() != *fmt++)
        in.setstate(ios::failbit);
    }

    return in;
  }
}

int main()
{
  int a, b;

  std::cout << "Enter two numbers separated by =-=: ";
  std::cin >> a >> "=-=" >> b;
  std::cout << "a: " << a << '\n' << "b: " << b << '\n';
}

It's not portable because you technically aren't allowed to add stuff to the std namespace. You can do it portably by writing a manipulator:

#include <iostream>

class skip {
  const char *_fmt;
public:
  skip(const char *fmt): _fmt(fmt) {}

  template <typename CharT, typename Traits>
  friend std::basic_istream<CharT, Traits>&
    operator>>(std::basic_istream<CharT, Traits>& in, const skip& sk)
  {
    const char *tmp = sk._fmt;

    while (in.good() && *tmp != '\0') {
      if (in.get() != *tmp++)
        in.setstate(std::ios::failbit);
    }

    return in;
  }
};

int main()
{
  int a, b;

  std::cout << "Enter two numbers separated by =-=: ";
  std::cin >> a >> skip("=-=") >> b;
  std::cout << "a: " << a << '\n' << "b: " << b << '\n';
}

But that's not as clean, so unless somebody is looking, go for the first option. ;)

[JoBe]

>Do you want a vector of points instead?
Euh, no, don't think I need that, but thanks for the example, why would you use this?

The translation of the exersize is:
Define a class "point" with two datamembers, x and y. Use the STL-container vector to stack a row of point-objects. Enter them by using the keyboard by two(pairs), x and y. It's not known how many (pairs) there will be entered. Use a non numeric character to stop the input of the (pairs) x and y.(<-- so far so good, no problem)

But this part of the exersize, I don't really understand what I'm supposed to do:Define the operators == and < for two point-objects in wich p

I understand that those operators "== and <" are linked towards this: "p.x < q.x || (p.x == q.x && p.y < q.y)" But, don't know how this should get incorporated into a piece of code?

>[voice=Sgt. Shultz]I know nothing.[/voice]
LOL :lol:

[voice=Homer].....DOH,.. ssstupid policeman[/voice]

[JoBe]

Hey Dogtree,

Thanks for the explanation man ;)

But euh, ... if you don't mind, I'm going to skip it in this exersize since it's not being asked to implement it, I'm having a hard time allready getting it to do what it should without incorporating some more code :cheesy:

But, greatly appreciated man :!:

[Dogtree]

>> Define a class "point" with two datamembers, x and y.

struct point {
  int x, y;
};

>> Use the STL-container vector to stack a row of point-objects.

#include <vector>

std::vector<point> points;

>> Enter them by using the keyboard by two(pairs), x and y.
>> Use a non numeric character to stop the input of the (pairs) x and y.

while (std::cin.peek() != 'q') {
  point item;
  int x, y;

  if (!(std::cin >> x >> y))
    break;

  item.x = x;
  item.y = y;

  points.push_back(item);
}

>> Define the operators == and < for two point-objects

bool operator==(const point& a, const point& b)
{
  return a.x == b.x && a.y == b.y;
}

bool operator<(const point& a, const point& b)
{
  return a.x < b.x || (a.x == b.x && a.y < b.y);
}

>> Use the STL-algorithm 'sort' to sort the previous entered (pairs) x and y
>> by using the defined order operator <.
std::sort uses operator< by default, so you can simply do this:

std::sort(points.begin(), points.end());

[JoBe]

Why do people like you, Narue, Dave,... make it all look so simple, pfff, life isn't fair you know LOL :lol:

Thanks Dogtree, though, have one favor to ask you, if possible, could you give me some hints and clues in the future or even examples wich are related to the exersize instead of the solution because, that way, I'll learn much more then by simply getting the solution on a silver platter.

Please, don't missunderstand me, I really appreciate your help, but, I'd like to try the exersize first, if I really am not able to solve it, after getting tips, hints, examples, from you, Dave, Narue, Vegaseat, Asif, 1OOOBHP,... I'll ask you to help me out with the solution ;)

Anyway, thanks for the solution, this way, I'll get to the next chapter much faster :cheesy:

[Dogtree]

There's really no way I could have hinted how to define the operators and call sort that would be useful without giving code. Don't worry, if possible I try not to solve other people's problems. ;) I also didn't give it to you on a silver platter. There are multiple ways to do something, some better than others, and you would still have to piece it all together even if you did use the exact code I gave.

>> Why do people like you, Narue, Dave,... make it all look so simple
We make it look simple because we worked really hard to learn it, just like you're doing now.

[Dave Sinkula]

Why do people like you, Narue, Dave,... make it all look so simple, pfff, life isn't fair you know LOL :lol:

Practice, practice, [failure, failure, failure << me at least>> ], practice...Please, don't missunderstand me, I really appreciate your help, but, I'd like to try the exersize first, if I really am not able to solve it, after getting tips, hints, examples, from you, Dave, Narue, Vegaseat, Asif, 1OOOBHP,... I'll ask you to help me out with the solution ;)I generally don't try to provide complete solutions to exact questions, especially homework. But sometimes it's just easier for me to try to explain what I'm thinking with code. Or I feel that posting a correct method ofwhatever is much better than "STFW for a (likely bad) example". And since the web is awash with bad examples, I try to make sure I'm posting only a correct bit of compilable code that demonstrates some portion of an answer; or a different approach to doing the same thing.

It's sort of a "the more good code that people can see, the more likely they will be to become good coders" type of thing -- and in the process of trying to tell someone else how to do something, you often end up learning a heckuva lot more than than you expect to.

Lather, rinse, repeat... and keep trying to get better.

[JoBe]

Hi guys,

Why am I able to write this:

#include <iostream>
#include <vector>
#include <algorithm>

class point
{
...
public:
	...
	void sort()
	{
		std::sort(s.begin(), s.end());
	}
	
	...
};

int main()
{
	point u;

	...
	u.sort();

	...
}

But not this:

class point
{
...
};

int main()
{
	...
             vector<point>points; 

	std::sort(points.begin(), points.end());

	...
}

[Dave Sinkula]

It would be much easier to answer if you didn't snip it to ... all over the place. I don't really care to guess at what goes into all the ...'s and attempt to get something to compile.

[JoBe]

Sorry Dave,

Tought you wouldn't need the total code and therefore posted only those parts. Don' worry about the operators '==' '<', haven't gotten them in working order yet ;)

#include <iostream>
#include <vector>
#include <algorithm>

using namespace std;

class point
{
private:
	int x, y, z;

public:
	point (int xx = 0, int yy = 0): x (xx), y (yy){}

	vector<int> s;

	void numbers()
	{
		while (cin>> x >> y, !cin.fail())
		{
			s.push_back(x);
			s.push_back(y);
		}
	}

	/*void sort()
	{
		std::sort(s.begin(), s.end());
	}*/
	
	/*bool operator==(const point &a, const point &b)
	{
		return a.x == b.x && a.y == b.y;
	}

	bool operator< (const point &a, const point &b)
	{
		return a.x < b.x || (a.x == b.x && a.y < b.y);
	}*/

	void print()
	{
		int n = s.size();

		for (z = 0; z < n; z++)
			cout<< s[z] <<endl;
	}
};

int main()
{
	point u;

	vector<point>points; 

	cout<<"Type twee integere getallen in per keer: "<<endl;

	u.numbers();

	u.print();

	//u.sort();

	std::sort(points.begin(), points.end());

	u.print();

	cin.get();

	return 0;
}

[Dave Sinkula]

I get an error message (trimmed):
'operator<' not implemented in type 'point'With that and your previous reply, in the first case you have a vector ofints? And the second you have a vector of points? I believe the reason might be that operator< is implemented for ints.

[edit]Tought you wouldn't need the total code and therefore posted only those parts.Little things likevector vs vector often lurk in ..., but many times that is where the answer is.

[JoBe]

Wait Dave,

What's that got to do with

std::sort(points.begin(), points.end());

As I said in the previous post, I'm not working with the operator functions just yet.

[Dave Sinkula]

What's that got to do with

std::sort(points.begin(), points.end());

As I said in the previous post, I'm not working with the operator functions just yet.

I believestd::sort uses operator< to do the sorting.

class point
{
   // ... (sorry I mean these are just the couple changes to what you have)
   bool operator< (const point &b)
   {
      return x < b.x || (x == b.x && y < b.y);
   }

   friend bool operator< (const point &a, const point &b);
};

bool operator< (const point &a, const point &b)
{
   return a.x < b.x || (a.x == b.x && a.y < b.y);
}

Suddenly it compiles.

[Reminder: C++ is not my native language.]

[JoBe]

Sorry DAve,

But let me get this straight, you're saying that

std::sort(points.begin(), points.end());

Is related to this part

class point
{
   bool operator< (const point &b)
   {
      return x < b.x || (x == b.x && y < b.y);
   }

   friend bool operator< (const point &a, const point &b);
};

bool operator< (const point &a, const point &b)
{
   return a.x < b.x || (a.x == b.x && a.y < b.y);
}

Now I'm really confused :confused:

Is this like you are determinating one bool operator first and that solution is given to the friend operator as one of the parameters??

[Dave Sinkula]

How would you perform a sort if you didn't have a comparison function?
How would the compiler know if widget a is supposed to be before widget b?