Правильность и грамотность кода (опытный взгляд на код новичка)

#include <iostream>
#include <fstream>
#include <regex>
 
using namespace std;
 
int main(int argc, char* argv[]){
    
    if(argc>1){
        ifstream f; //Создаем объект потока чтения из файла 
        f.open(argv[1]); //Открываем файл на чтение
        // Проверяем открылся ли файл на чтение
        if(f.is_open())
        {
            string out_s;  //Временная переменная для чтения в нее из файла
            regex Reg("^[0-9]{6}.*"); //Регулярное выражение, которому соответствует строка, начинающаяся с 6-и цифер
            while(true){
                getline(f, out_s);
                if(!f.eof())
                {
                    i++;
                    if(regex_match(out_s, Reg)) //Проверка соответствия строки
                    {
                        cout << out_s;  //Вывод резултата в консоль
                    }
                }
                else
                {
                    break;
                }
            }
            //Закрываем файл
            f.close();
        }
        else
        {
            cout << "Неверное имя файла";
        }
    }
    else
    {
        cout << "Не задано имя файла";
    }
    return 0;
};

while(!f.eof()){...}

int main(int argc, char* argv[])
{    
    if (argc != 2) {
        cout << "Usage: progname filename" << endl;
        return 1;
    }
 
    string filename = argv[1];
    ifstream f (filename); //Создаем объект потока чтения из файла 
    if (!f) {
        cout << "Cannot open file " << filename;
        return 1;
    }
 
    string out_s;  //Временная переменная для чтения в нее из файла
    regex reg("^[0-9]{6}.*"); //Регулярное выражение, которому соответствует строка, начинающаяся с 6-и цифер
    while (getline(f, out_s)) {
        if (regex_match(out_s, reg)) { //Проверка соответствия строки
            cout << out_s;  //Вывод резултата в консоль
        }
    }
    //Закрываем файл
    f.close();
}

if(argc < 2)
{
cout << "";
return 0;
}

//Закрываем файл
f.close();

while(true){
if(getline(f, out_s))
{

#include <string>
#include <iostream>
#include <fstream>
#include <sstream>
#include <regex>
 
void filtering(std::istream& input, std::ostream& output) {
    std::string line;
    std::regex reg("^[0-9]{6}.*"); //Регулярное выражение, которому соответствует строка, начинающаяся с 6-и цифер
 
    while(std::getline(input, line)) {
        if (std::regex_match(line, reg)) { //Проверка соответствия строки
            output << line << std::endl;  //Вывод резултата в консоль
        }
    }
}
 
int main(int argc, char* argv[]){
    if (argc != 2) {
#ifdef DEBUG
        std::istringstream ss(
          "123\n"
          "123456 abc\n"
          "1234567 abcd\n"
        );
 
        filtering(ss, std::cout);
 
        return 0;
#else
        std::cerr << "Usage: progname filename\n";
        return 1;
#endif
    }
 
    std::string fileName(argv[1]);
    std::ifstream file(fileName); //Создаем объект потока чтения из файла
    if (!file) {
        std::cerr << "Cannot open file: " << fileName << std::endl;
        return 1;
    }
 
    filtering(file, std::cout);
 
    file.close();
    return 0;
}

#include <iostream>
#include <fstream>
#include <cassert>
#include <string>
#include <regex>
 
// ======================================================================
// ======================================================================
 
bool match(const std::string& value)
{
    // срабатывает на строки, который начинаются с 6ти цыфр
    // которые не имеют в конце белых символов (\n)
    // static std::regex reg("^[0-9]{6}.*");
    
    // срабатывает на строки, который начинаются с 6ти цыфр
    static std::regex filter("^\\d{6}.*\\s*"); 
    return std::regex_match(value, filter);
}
 
void filtering(std::istream& input, std::ostream& output) 
{
    std::string line; 
    while(std::getline(input, line)) 
        if(match(line)) 
            output << line << '\n';
}
 
// ======================================================================
// ======================================================================
 
#ifndef NDEBUG
 
bool test(const char* value, const bool expected) { 
    return match(value) == expected; 
}
 
#define assert_match(v, b) \
    assert(test(v,b));
 
#endif
 
// ======================================================================
// ======================================================================
 
int main(int argc, char* argv[])
{
    assert_match("123"           , false);
    assert_match("123456 abc"    , true );
    assert_match("1234567 abcd"  , true );
    
    assert_match("123456 abc\n"  , true );
    assert_match("1234567 abcd\n", true );
    
    assert_match("123456 abc\0"  , true );
    assert_match("1234567 abcd\0", true );
    
    if (argc != 2) 
        return std::cerr << "usage: progname filename\n", 1;
    
    std::ifstream file(argv[1]); 
    if (!file)
        return std::cerr << "cannot open file: " << argv[1] << '\n', 1;
 
    filtering(file, std::cout);
}