How is the vector<hit>::iterator itr; line used in this code?

  • Context: C/C++ 
  • Thread starter Thread starter WarPhalange
  • Start date Start date
  • Tags Tags
    C++ Vectors
Join the discussion
Ask a follow-up here, or get your own question answered by working scientists, mathematicians and engineers — people, not an autocomplete.
Real named experts · corrections over time · the nuance an AI answer skips
4 replies · 3K views
WarPhalange
I came across a snippet of code like this:

Code:
  protonPropagator pp;
  vector<hit> hits;

  vector<hit>::iterator itr;
  for(int i=0;i<5;i++){
    hits.clear();
    //x,y,z,E,theta,phi
    hits = pp.compute(startX,startY,startZ,startE,theta,phi);

    if(hits.size()>0){
    fout<<"startEvent"<<endl;
    fout<<i<<" "<<hits.size()<<" "<<startX<<" "<<startY<<" "<<startZ<<" "<<startE<<" "<<theta<<" "<<phi<<endl;
    copy(hits.begin(),hits.end(),ostream_iterator<hit>(fout,"\n"));

    }else{
      cerr<<"No hits in this event, skipping"<<endl;
    }
  }

And I was told it will iterate through the vector "hits" 5 times. But by the looks of it, nothing is really being iterated. You have an initialization of "itr", but then it's never used.

So it looks like you get some value for hits and overwrite it on each successful iteration of the for() loop. Since it's printed out, you store it in a file so it doesn't matter if it's written over.

Am I correct here? That the line "vector<hit>::iterator itr;" is useless? Or is it used to print to file, i.e. "copy(hits.begin(),hits.end(),ostream_iterator<hit>(fout,"\n"));"?
 
Physics news on Phys.org
You're right, the iterator itr is irrelevant to this section of code. Is it used anywhere else?

It certainly wouldn't be the first time that comments or documentation don't match the actual code. :rolleyes:

There is a for-loop that iterates (cycles) five times. Maybe that's what they're referring to with the verb "iterates", not the iterator itr.
 
Okay, I think I see what is going on now. The vector "hits" has some number of values in it like an array. I had thought that each value of hits carried its own array, i.e. making it a 2d array where each value of hits had its own array associated with it.

That's why it's being cleared and reused every time.

I still can't figure out what this line does, though:

copy(hits.begin(),hits.end(),ostream_iterator<hit>(fout,"\n"));

I know it copies the entire vector hits into the file, but how does ostream_iterator work?