C++ - Konwersja tablicy char do string w Visual Studio 2010 - śmieci w pamięci

Witam serdecznie. Mam problem z konwersją tablic znaków do zmiennej klasy string. Mam powiedzmy zmienną char imie_nazwisko[40]; oraz zmienną klasy string imie_naz;

Próbowałem następującej konwersji:



Konwersja niby działa, ale w zmiennej pojawiają się jakieś śmieci z pamięci.
Ktoś wie jak się tego pozbyć?

Piszę w Microsoft Visual Studio 2010[/code]

W jaki sposób wpisałeś do zmiennej imie_nazwisko?

Użyłem konwersji w drugą stronę. Mam bowiem strukturę bazującą na zmiennych klasy string i drugą strukturę na tablicach znaków. Konwersję na char'y wykonuję by zapisać dane do pliku binarnego i móc je później odczytać. Po odczytaniu muszę te dane znów przekonwertować na stringi i zapisać je w liście jednokierunkowej, która operuje na stringach.

Dane do stringa zapisuję przy użyciu getline, a konwersja na tablice char polega na przepisywaniu znak po znaku.

Dodałeś na koncu 0 (Zero)?

W którym miejscu mam to zero uwzględnić? Wiem że tablica char na ostatnim miejscu przyjmuje '\0', ale jak to się ma do mojego problemu?

Jeśli wiesz ile bajtów wczytałeś, to nie musisz dodawać 0, ale zamiast tego użyj:



Zakładam, że imie_nazwisko jest typu char*.

Tak czy owak, jeśli używasz .c_str() przed zapisem, to na końcu jest znak \0 już dodany. Więc gdy kopiujesz dane z .c_str() do tablicy, to skopiuj o 1 znak więcej, żeby do pliku dopisało się również 0.

mcvsama napisał:

Jeśli wiesz ile bajtów wczytałeś, to nie musisz dodawać 0, ale zamiast tego użyj:

Kod: text

Zaloguj się, aby zobaczyć kod

Zakładam, że imie_nazwisko jest typu char*.

Tak czy owak, jeśli używasz .c_str() przed zapisem, to na końcu jest znak \0 już dodany. Więc gdy kopiujesz dane z .c_str() do tablicy, to skopiuj o 1 znak więcej, żeby do pliku dopisało się również 0.

Wydaje mi się, że zrobiłem tak jak mówiłeś. Poniżej podaję kod mojej funkcji konwertującej:


Efekt działania tego jest taki, że wprowadzam następujące dane:


Przy powyższym kodzie i zastosowaniu polecenia sizeof, konwersja daje taki efekt:


A bez zastosowania polecenia sizeof wychodzi coś takiego



Dodam tylko że typ "dane" operuje na stringach, a "dane_conv" na tablicach znaków.

Ma ktoś jakiś pomysł na to?


Pozdrawiam

Prawdopodobnie chciałeś stworzyć nowe stringi, ale tego tak się nie robi. Użyłeś właśnie operatora „przecinka” (nawet mimo tego, że są tam nawiasy). To co jest powyżej ma taki sens:
do element.imie_nazwisko przypisz rozmiar wskaźnika (czyli 4 w przypadku 32-bitowego adresu, 8 w przypadku 64-bitów).

Jeśli element.imie_nazwisko jest typu std::string, to został wywołany std::string::operator=(const char*) i zrobiona była niejawna konwersja z size_t (zwracanego przez sizeof()) do wskaźnika. (To jest jedna z wad C++, takie niejawne konwersje.)

Zarówno w C jak i w C++ operator , (przecinek) wykonuje to co jest po lewej stronie, potem po prawej stronie i zwraca to co zwróciło wyrażenie po prawej stronie.
Napis:

wywoła funkcję f(), następnie g() i przypisze do "a" to co zwróciła funkcja g. (Oczywiście zależy od kontekstu - przy wywołaniu funkcji przecinek rozdzieli argumenty, ale jeśli całą listę umieścisz w dodatkowych nawiasach, to wtedy właśnie zadziała operator przecinka). Podobny właśnie jest efekt w Twoim kodzie.

Masz następujące opcje:
1. skonstruować i skopiować obiekt std::string():


2. Użyć metody assign() na stringu, który już istnieje w Twojej strukturze:


3. Użyć inicjalizatora z C++11 - nawiasów klamrowych:


-----

Edit: acha, jeszcze jeden błąd: sizeof() zwróci rozmiar wskaźnika, nie długość C-łańcucha (to jest taka pseudofunkcja wykonywana w czasie kompilacji programu, nie w czasie jego działania). sizeof() może przyjąć różne wyrażenia, ale one nigdy nie są wykonywane. To czego potrzebujesz, to funkcja strlen().

Edit2: Jeszcze taka ciekawostka - w C łańcuchy nie mogą zawierać znaków NUL (\0), bo on oznacza ich koniec. Natomiast std::string() może zawierać dowolną ich liczbę, dlatego łańcuchy z C można trzymać w std::string(), ale nie wszystkie łańcuchy z std::string() dadzą się przedstawić w postaci C-stringów.

Wykorzystałem zarówno metodę assign klasy string jak i tworzenie obiektu klasy string. Jest lepiej, ale nadal diabli wiedzą skąd, pojawiają się krzaki.



Sprawdzałem już na każdym etapie zapisu do pliku i jestem pewien że do pliku binarnego są przesyłane właściwe dane bez żadnych niechcianych "atrakcji".


Mam jeszcze jedno pytanie. Otóż odczyt z pliku wykonuję w pętli while aż do spełnienia warunku feof(*FILE). Teoretycznie gdy zapiszę jeden element do pliku, jeden element powinien zostać odczytany. Natomiast u mnie, przy zapisaniu w pliku jednego elementu typu "dane", do listy jednokierunkowej na której operuje program są zapisywane te dane z krzakami (na zrzucie powyżej) oraz drugi element który zawiera dokładnie te same krzaki co na zrzucie bez właściwych danych.

Teoretycznie, jeśli zapiszesz jedną strukturę, to powinieneś też odczytać jedną strukturę. Pytanie czy zapisujesz poprawnie.

Dane w trakcie wykonywania operacji na nich są przechowywane w liście jednokierunkowej opartej na STL'u (plik nagłówkowy "list"). Lista operuje na typie strukturalnym "dane". Ciągi znakowe zapisuje przy użyciu polecenia



i przekazuję dane do listy poleceniem nazwa_listy.push_back(element).

Gdy już chcę zapisać dane do pliku, tworzę sobie zmienną typu dane_conv, "zeruję" ją żeby mieć pewność, że nie ma tam żadnych śmieci. Następnie przechodzę przez całą listę z wykorzystaniem iteratora i przesyłam do funkcji konwertującej stringi na tablice znaków element listy na który wskazuje iterator oraz stworzoną wcześniej zmienną typu dane_conv i ostatecznie używając polecenia

przesyłam dane do pliku.