// Return your data with the return value of the function.// Modern compilers have a pretty good idea how to optimize this for vectors.// Use std::string to represent text data, such as file names.// Be explicit with variable names: "name" doesn't say what is named.// Think about the order of default arguments: if the user passes 3 arguments,// it makes more sense to interpret the 3rd as the number of values, rather than// the offset, agreed?// We can generalize the approach for any primitive data type.template <typename T>std::vector<T> get_values_from_file(const std::string& filename, int how_many = 256, int offset = 0) {	std::vector<T> result;	T value;	// Specifying ios::in is redundant for an *i*fstream.	ifstream myfile(filename.c_str(), ios::binary);	myfile.seekg(offset);	// As mentioned, use a for loop.	for (int i = 0; i < how_many; ++i) {		// Use C++-style casts. Don't make assumptions about type sizes		// if you don't have to.		myfile.read(reinterpret_cast<char*>(&value), sizeof(T));		vec.push_back(value);	}	// There is no need to .close() explictly; the destructor does that.	return result;}