11 odpowiedzi w tym temacie

[Alelluja]

Cześć, mam problem z wyciągnięciem danych z tablicy 2d, która jest stringiem. Próbowałem wyciągać te dane sposobem z PAWN'a używając sizeof w pętli ale kończy się to segmentation fault. Przeszukałem chyba wszystkie tematy związane z tablicami 2d i wyciąganiem z nich danych ale nie było tego co potrzebuję. 

 

Podaję link do kodu. 

 

https://pastebin.com/G2aP9s6K

[Rivit]

Okej, to nie będę Ci dawał pełnego rozwiązania na tacy, ale zrób to:

cout << sizeof gamesData << endl;

w jakimś case w switchu i zobacz ile Ci wypisze

[Alelluja]

Po zrobieniu tak wypisało:

 

Dostępne gry (Gry Akcji):

1824

[Rivit]

No dokładnie, i twoja pętla jedzie od i = 0 aż do tej wartości, co oczywiście dla pierwszych wartości zadziała, a dla reszty dostaniesz seg faulta, bo odwołujesz się poza tablice

 

EDIT:

Jeszcze dodam, że jeśli robisz sizeof na jakimś obiekcie, to dostaniesz rozmiar tego obiektu. W sensie ile fizycznie ta klasa okupuje bajtów. sizeof dla stringa nie zwraca jego długości, tylko zwraca ile bajtów zajmuje sam w sobie obiekt (jego zmienne prywatne, publiczne etc)

Użytkownik TibacK edytował ten post 06.04.2020 13:50

[Alelluja]

Dobra nie spodziewałem się, że to będzie takie proste ale wystarczyło zamiast 

for(int i = 0; i < sizeof gamesData; i++)

Dać to:

for(int i = 0; i < 3; i++)

Ale nie wiem czy jest to dobry sposób bo dla reszty zrobiłem tak że powiększałem pułap startowy o 3 i dodawałem 3 do limitu aby wczytać tylko te dane które potrzebuje.

[Rivit]

Ja bym zrobił coś takiego:

struct gameData{
    string id;
    string name;
    string number; //lepiej zeby bylo np intem, unsigned intem
};

gameData gamesData[] = {
    {"1", "Battlefield 1", "100"},
    {"2", "WatchDogs 2", "120"},
    {"3", "Grand Theft Auto 5", "80"},
    {"4", "Black Desert Online", "50"},
    {"5", "World of Warcraft", "50"},
    {"6", "Eternal Magic", "50"},
    {"7", "Ori and the Blind Forest", "170"},
    {"8", "Resident Evil 3", "50"},
    {"9", "Firewatch", "40"},
    {"10", "The Wither 3", "50"},
    {"11", "Cyberpunk 2077 (Preorder)", "199"},
    {"12", "Disco Elysium", "150"},
    {"13", "Fifa 2020", "160"},
    {"14", "Rocket League", "50"},
    {"15", "NBA 2k19", "40"},
    {"16", "Car Mechanic Simulator", "50"},
    {"17", "Farming Simulator 2020", "100"},
    {"18", "Tank Mechanic Simulator", "30"},
};

            for(int i = 0; i < sizeof(gamesData) / sizeof(gamesData); i++)
            {
                if(typeGamesId[i] == choice)
                {
                    cout << gamesData[i].id << ". " << gamesData[i].name << " - Cena: " << gamesData[i].number << " PLN (Wpisz " << i + 1 << ")" << endl;
                }
            }

Chociaż na to:

sizeof(gamesData) / sizeof(gamesData)

trzeba uważać czasem

[Alelluja]

Ten sposób z 

sizeof(gamesData) / sizeof(gamesData)

 

Nie działa poprawnie, dokładniej to pokazuje tylko 1 pozycję z 3 możliwych. Ale te struktury to sobie zapożyczę bo nawet spoko sprawa. Dzięki za pomoc.

 

[Rivit]

A to dziwne, bo jak sprawdzałem u siebie to wypisało wszystkie kategorie.

Uzywałem g++9 na ubuntu. 

[Alelluja]

Ja robię to na stronie ale zostałem przy tym co miałem czyli zwiększałem o 3 i git jest.

 

Dzięki za pomoc można zamknąć.

[Amaroq]

Ten sposób z 

sizeof(gamesData) / sizeof(gamesData)

Nie działa poprawnie, dokładniej to pokazuje tylko 1 pozycję z 3 możliwych. Ale te struktury to sobie zapożyczę bo nawet spoko sprawa. Dzięki za pomoc.

 

Powinno być

sizeof(gamesData) / sizeof(gameData)

Wtedy zadziała poprawnie

[Rivit]

Tak, masz rację, wredna literówka

I to tłumaczy dlaczego tylko jeden wypisywało

[DarkGL]

Wiadomość wygenerowana automatycznie


Ten temat został zamknięty przez moderatora.

Powód: Pomoc udzielona

Jeśli się z tym nie zgadzasz, raportuj ten post, a moderator lub administrator rozpatrzy go ponownie.


Z pozdrowieniami,
Zespół AMXX.PL