这是一段有问题的代码，从运行上来看没什么问题。这里的vecotr 容器明显的被滥用了。26行声明了新的容器变量 tiles并把cells的数组变量复制给了 tiles; 这里就发生了很多不必要的浪费！vector是可扩容容器，会根据元素数量的大小，自动的进行扩容！这里在内存上占用的空间起码是原来空间的1.5倍。

解决方法：声明 &tiles 引用tiles 前缀const， 加const 只是让你引用的更加正确一点。

 const std::vector<Position>& tiles = cells[rotationState];

 27行，又创建了一个新的容器，没有给出具体大小。所以容器初始化会有0个元素。后续操作又会触发2次扩容的机制。

解决方法：指定容器具体的大小。

movedTiles.reserve(tiles.size());

28行，泛型for 在C++11之后有了泛型，这里又触发了，重复复制元素的问题。泛型给了很多的方便之处，但，还是没明白其中的所以然。如果只是遍历其中的元素，并没有修改可以用const &。

解决方法：const & 

 for(const Position& item: tiles){}

30,31行 新建一个变量用来存放新的位置，push_back(newPos); 简单一点这里完全没必须来这样写，直接放进去就可以；push_back是放到元素的最后一个位置，使用push_back()远没emplace_back()来的高效！

解决方法：

 for(const Position& item: tiles){movedTiles.emplace_back(item.row + rowOffset, item.column+columnOffset);}

这也算是比较常见的错误！