c++ uruchamianie programu z linii poleceń z przełącznikami

Question

Program, który mam do napisania jest uruchamiany z linii poleceń z czterema przełącznikami -i -o -k -d

Utworzyłam taką strukturę do przechowywania tych wszystkich przełączników

struct daneWejsciowe
{
	std::string i{};
	std::string o{};
	int k{};
	int d{};
};

następnie mam funkcje która ma przypisać mi te wartości podane w linii poleceń do zmiennej o nazwie przelaczniki i type daneWejsciowe jednak coś mi nie chce tutaj działać i nie do końca wiem co robię źle

daneWejsciowe wczytywaniePrzelacznikow(int argc, char* argv[])
{
	daneWejsciowe przelaczniki;
	if (argc == 9)
	{
		for (int i = 1; i < argc; i++)
		{
			if (argv[i] == "-i")
			{
				przelaczniki.i = std::string(argv[i++]);
			}
			if (argv[i] == "-o")
			{
				przelaczniki.o = argv[i + 1];
			}
			if (argv[i] == "-k")
			{
				przelaczniki.k = int(argv[i + 1]);
			}
			if (argv[i] == "-d")
			{
				przelaczniki.d = int(argv[i + 1]);
			}
		}
	}
	else
	{
		std::cerr << "Nieprawidlowa ilosc przelacznikow" << std::endl;
	}


	return przelaczniki;
}

tak wyglada naglowek funkcji main

int main(int argc, char* argv[])

 

Answer 1

Linijka 8 wygląda podejrzanie

przelaczniki.i = std::string(argv[i++]);

Dlaczego nie zrobisz jak przy innych przełącznikach?

Dodatkowo iterację skrócił bym o jeden element. Żeby nie przekroczyć zakresu tablicy argv w przypadku błędnego wpisania parametrów.

Comments

błąd w linijce 8 wynika z nieuwagi, więc już go zmieniłam, ale nadal po przypisaniu tego co zwraca funkcja do zmiennej typu daneWejsciowe i przy próbie wypisania tej zmiennej coś nie działa.

daneWejsciowe przelaczniki;
	przelaczniki = wczytywaniePrzelacznikow(argc,argv);
	std::cout << przelaczniki.i << std::endl;

 

---

Musisz zainwestować w lepszy kompilator (lub zacząć czytać ostrzeżenia kompilatora)!

argv[i] jest typu char*

Porównywanie

if (argv[i] == "napis")

sprowadza się do porównywania adresów a nie o to chodziło.

Kolejnym błędem jest próba inicjalizacji zmiennej int adresem napisu

int(argv[i + 1]);

Aby przekonwertować napis na liczbę musisz użyć odpowiedniej funkcji.

Wersja poprawiona:

#include <iostream>

struct daneWejsciowe
{
    std::string i{};
    std::string o{};
    int k{};
    int d{};
};

daneWejsciowe wczytywaniePrzelacznikow(int argc, char* argv[])
{
    daneWejsciowe przelaczniki;
    if (argc == 9)
    {
        for (int i = 1; i < argc; i++)
        {
            auto tmp = std::string(argv[i]);
            if (tmp == "-i")
            {
                przelaczniki.i = argv[i + 1];
            }
            if (tmp == "-o")
            {
                przelaczniki.o = argv[i + 1];
            }
            if (tmp == "-k")
            {
                przelaczniki.k = std::atoi(argv[i + 1]);
            }
            if (tmp == "-d")
            {
                przelaczniki.d = std::atoi(argv[i + 1]);
            }
        }
    }
    else
    {
        std::cerr << "Nieprawidlowa ilosc przelacznikow" << std::endl;
    }

    return przelaczniki;
}

int main(int argc, char* argv[]) {
    daneWejsciowe przelaczniki;
    przelaczniki = wczytywaniePrzelacznikow(argc,argv);
    std::cout << przelaczniki.i << std::endl;
    std::cout << przelaczniki.o << std::endl;
    std::cout << przelaczniki.k << std::endl;
    std::cout << przelaczniki.d << std::endl;
}

 

---

Dziękuję bardzo, faktycznie kilka głupich błędów, ale teraz już wszystko działa.