Effective use of C++ iomanip library

c++, iomanip, iostream, stream

Solution

Only `std::setw()` is temporary. The other two calls, `setiosflags`, and `setprecision` have a permanent effect.

So, you could change your code to :

std::ostream& operator<<(std::ostream &output, const Vector &v){
  output<<"["
    <<std::setiosflags(std::ios::right | std::ios::scientific)
    <<std::setw(23)
    <<std::setprecision(16)
    <<v._x<<", "
    <<std::setw(23)
    <<v._y<<", "
    <<std::setw(23)
    <<v._z<<"]";
  return output;
} 

But now you've borked the flags and precision for the next guy. Try this instead:

std::ostream& operator<<(std::ostream &output, const Vector &v){
  std::ios_base::fmtflags f = output.flags(std::ios::right | std::ios::scientific);
  std::streamsize p = output.precision(16);
  output<<"["
    <<std::setw(23)
    <<v._x<<", "
    <<std::setw(23)
    <<v._y<<", "
    <<std::setw(23)
    <<v._z<<"]";
  output.flags(f);
  output.precision(p);
  return output;
} 

Finally, if you absolutely have to get rid of the duplication of the constant `23`, you could do something like this (but I wouldn't recommend it):

struct width {
  int w;
  width(int w) : w(w) {}
  friend std::ostream& operator<<(std::ostream&os, const width& w) {
    return os << std::setw(width.w);
  }
};


std::ostream& operator<<(std::ostream &output, const Vector &v){
  std::ios_base::fmtflags f = output.flags(std::ios::right | std::ios::scientific);
  std::streamsize p = output.precision(16);
  width w(23);
  output<<"["
    <<w
    <<v._x<<", "
    <<w
    <<v._y<<", "
    <<w
    <<v._z<<"]";
  output.flags(f);
  output.precision(p);
  return output;
} 

See also this other question, where they decided that you can't make width permanent.

Problem

I created a `Vector` class in C++ and it works great for my problems. I am now cleaning it up, and I ran into the following piece of code: ``` std::ostream& operator<<(std::ostream &output, const Vector &v){ output<<"[" <<std::setiosflags(std::ios::right | std::ios::scientific) <<std::setw(23) <<std::setprecision(16) <<v._x<<", " <<std::setiosflags(std::ios::right | std::ios::scientific) <<std::setw(23) <<std::setprecision(16) <<v._y<<", " <<std::setiosflags(std::ios::right | std::ios::scientific) <<std::setw(23) <<std::setprecision(16) <<v._z<<"]"; return output; } ``` The code allows to print a vector as `std::cout<<v<<std::endl;`. Each number has 23 spaces, of which 16 are the decimals. The text is right-aligned so that it will print: ``` 1.123456123456e+01 -1.123456123456e+01 ``` Instead of ``` 1.123456123456e+01 -1.123456123456e+01 ``` The code seems awfully repetitive. How can you "store" the format (all the `setiosflags`, `setw` and `setprecision` statements) such that you can say something like "print the characters in a standard way, but the numbers with this given format". Thank you! Edit As per Rob Adams' comment, I changed my ugly code (which, as pointed out by others, would mess up the precision for the "next guy") to a more succinct (and correct): ``` std::ostream& operator<<(std::ostream &output, const Vector &v){ std::ios_base::fmtflags f = output.flags(std::ios::right | std::ios::scientific); std::streamsize p = output.precision(16); output<<"[" <<std::setw(23)<<v._x<<", " <<std::setw(23)<<v._y<<", " <<std::setw(23)<<v._z <<"]"; output.flags(f); output.precision(p); return output; } ```

Original source

Related problems