Zmienna się nie wyświetla c++

Question

​void zakoduj(string haslo)
{
    string zakodowane_haslo;
    
    for (int i = 0; i < haslo.length(); i++)
    {
        if(haslo[i] != ' ')
        {
            zakodowane_haslo[i] = '_';
        }
        else
        {
            zakodowane_haslo[i] = ' ';
        }
    }
    cout << zakodowane_haslo;
}

Witam!
Mam problem z voidem zakoduj(). Chodzi o linie 16. wszystko się kompiluje, ale nie wyświetla się zmienna zakodowane_haslo.

Comments

Treści pytania nie wrzucaj do ramki przeznaczonej na kod. :)

---

Czasami edytor żyje własnym życiem, każdemu może się zdarzyć. :) Poprawiłem troszkę. ;)

---

No kurde teraz to jest picuś glancuś. :D

---

dziękuje

Answer 1

Zmienna zakodowane_haslo nie ma żadniej długości. Radzę użyć funkcji push_back:

if(haslo[i] != ' ') {
  zakodowane_haslo.push_back('_');
} else {
  zakodowane_haslo.push_back(' ');
}

 

Comments

Albo ewentualnie użyć konkatenacji:

zakodowane_haslo += '_';

 

---

Ewentualnie x2 funkcji append:

zakodowane_haslo.append(1, '_');

---

Przy korzystaniu ze zmiennych string wygodniej jest moim zdaniem używać operatora +=. Specjalnie dla napisów został on tak przeładowany.

Ja mam też taką swoją osobistą zasadę, że jeśli operuję na tablicy, liście czy innej strukturze, to rozmiar sprawdzam tablica.size(). Natomiast dla napisów zawsze wywołuję: napis.length(). W tej kwestii jest oczywiście dowolność. Nikogo nie przekonuję, że to słuszne. Taka tylko ciekawostka ;-)

---

Zdaje mi się, że dawniej std::string::size() i std::string::length() mogły zwracać różne wyniki - metoda size() zwracała faktyczny rozmiar bufora w bajtach, length() natomiast zwracała ilość elementów (znaków) w buforze. Nie jestem tego w 100% pewny, zdaje się że kiedyś o tym czytałem.
Taka niepewna ciekawostka, fajnie by było gdyby ktoś ja potwierdził / zanegował.

W tej chwili na 101% są to synonimy. :)

---

GCC 4.9.2

size_type size() const _GLIBCXX_NOEXCEPT
{ return _M_rep()->_M_length; }

size_type length() const _GLIBCXX_NOEXCEPT
{ return _M_rep()->_M_length; }

W starszej implementacji z tego co kojarzę length() zwracała po prostu size(). Ogólnie jest to kwestia wyboru i przyzwyczajenia, lecz ja preferuję size(). W przypadku tworzenia szablonu funkcji z przesyłaniem kontenerów, size() zapewnia działanie dla wszystkich (string, vector, list).

---

MSVC

size_type length() const _NOEXCEPT
	{	// return length of sequence
	return (this->_Mysize());
	}

size_type size() const _NOEXCEPT
	{	// return length of sequence
	return (this->_Mysize());
	}

To ja dopowiem, że nie preferuję ani size ani length. Wolę globalne std::size (C++17, jednak wszystkie popularniejsze kompilatory już to mają), głównie ze względu na łatwość użycia w szablonach i możliwość pracy ze zwykłymi tablicami.

Answer 2

Zawartość poszczególnych komórek tablicy zakodowane hasło dodawaj do innej zmiennej string używając konkatenacji w każdej iteracji pętli. Na końcu wyświetl tą zmienną string.