dcsimg
Login | Register   
RSS Feed
Download our iPhone app
TODAY'S HEADLINES  |   ARTICLE ARCHIVE  |   FORUMS  |   TIP BANK
Browse DevX
Sign up for e-mail newsletters from DevX

By submitting your information, you agree that devx.com may send you DevX offers via email, phone and text message, as well as email offers about other products and services that DevX believes may be of interest to you. DevX will process your information in accordance with the Quinstreet Privacy Policy.


Tip of the Day
Home » Tip Bank » C++
Language: C++
Expertise: Intermediate
Apr 7, 1999

WEBINAR:

On-Demand

Application Security Testing: An Integral Part of DevOps


Always Use the Container's Iterators Rather Than Pointers

Bare pointers can sometimes be used to iterate through the elements of a container:
 
#include <vector>
#include <iostream>
using namespace std;

void hack(){
  vector<int> vi;
  vi.push_back(5);
  int *p = vi.begin();//bad programming practice, although it may work
  *p = 6; //assign vi[0]
  cout<<vi[0]; //output 6
}
However, using bare pointers instead of iterators is poor programming practice and should be avoided. It undermines the very idea of information hiding and encapsulation. An iterator may not be represented as a pointer, and even if it is, this implementation detail can change in future versions of the library. The correct form of the previous code excerpt is:
 
void nohack()
{
  vector<int> vi;
  vi.push_back(5);
  vector<int>::iterator p = vi.begin(); //correct
  *p = 6; //assign vi[0]
  cout<<vi[0]; //output 6
}
Danny Kalev
 
Comment and Contribute

 

 

 

 

 


(Maximum characters: 1200). You have 1200 characters left.

 

 

Sitemap
×
We have made updates to our Privacy Policy to reflect the implementation of the General Data Protection Regulation.
Thanks for your registration, follow us on our social networks to keep up-to-date