Essential C++中的一个问题

今天在看Essential C++的时候，发现书中有个不太明智的地方，当然我所说的这个问题并非是非常的不明智，只是个人感觉而已，书还是好书，只是有时候我们不应该完全依赖书，完全用书中所提到的做法，学习应该懂得什么可以拿来用，什么不应该拿来用，什么样的做法是明智的，什么样的做法是不明智的。
如果只是一味的效仿书中的做法，那我们又能真正的学到些什么呢？
这个问题其实很小，在书中42页定义
void bubble_sort(vector<int> vec)
{
for( int ix=0;ix<vec.size();ix++)
for( int jx=ix+1;jx<vec.size();jx++)
if(vec[ix]>vec[jx])
swap(vec[ix],vec[jx]);
}
在这个函数中，关于外循环中的ix多循环了一次（关于书中的传值和传址暂不考虑，只考虑这个函数），其实改做
for(int ix=0;ix<vec.size()-1;ix++)
最终的效果是一样的，书中的做法使得外循环多循环了一次，书中的vec.size()的值是8，当ix==7的时候,相应的内循环中的jx==ix+1==8，这时内循环终止，相比让ix<vec.size()-1要多去判断一次。
个人认为修改为如下会更好一些
void bubble_sort(vector<int> vec)
{
for(unsigned int ix=0;ix<vec.size();ix++)
for(unsigned int jx=ix+1;jx<vec.size();jx++)
if(vec[ix]>vec[jx])
swap(vec[ix],vec[jx]);
}
其中在定义ix和jx时，用到unsigned，是因为直接用定义为int，在VS2005中会给出一个警告。
本人菜鸟一只，若有什么错误之处，希望各位老鸟能给出一些指点^_^