[p.....h]

Witam panowie i panie #programista15k
Mam zagwozdkę. Uczę się klas i metod. Stworzyłem taki o to programik do przechowywania informacji o plakacie. Problem jest taki, że gdy używam dwóch tablic char'owych to program ich w ogóle nie czyta. Nie wiem za bardzo co z tym zrobić.
Próbowałem pobrać tekst przez gets(), to nie działa. Później próbowałem przez for(...), co ciekawe jeden z dwóch się wyświetlał razem z dziwnymi znakami. Co mogę zrobić, żeby program czytał normalnie tekst do tablic charowych, a potem je normalnie wyświetlał.

Kod programu tutaj: Kod

A wynik w załączniku.


#c #cplusplus #programowanie #programowanieobiektowe #programowanieboners (xd) #kiciochpyta

[l.....e]

@pyth: zmień tag na #programista1.5k

[trojanxem]

@pyth: tak na szybko to nie korzystaj z funkcji gets, tylko fgets, chroni przed przeładowaniem buforu, a co do do Twojego pytania to jutro rano zerkne bo teraz na kom :)

[p.....h]

@trojanxem: super, byłoby fajnie bo spędza mi to sen z powiek (╯︵╰,)

[mojemirabelki]

@pyth: a nie lepiej stworzyć tablice obiektów typu plakaty i informacje przechowywać w obiektach?

[p.....h]

@mojemirabelki: W sensie każdą informację oddzielnie, czy zestaw informacji od nazwy po kategorię?
Oczywiście, że byłoby lepiej tylko chciałbym wiedzieć dlaczego to nie działa, co może być nie tak tu.

[p.....h]

Dopiszę jeszcze, że tablica nazwa[L] działa normalnie, problem jest z tablicą kategoria[L] oraz rodzaj[L].

[leoha]

@pyth: "Zostana Tobie przedstawione (...)"
To jakaś plaga na wykopie ostatnio... żadne Tobie tylko ci tam powinno być!
#grammarnazi

[p.....h]

@leoha: Tak mocno kliknąłeś enter, że dwa razy komentarz się pojawił. Spokojnie już poprawiam ( ͡° ͜ʖ ͡°)

[mojemirabelki]

@pyth nie wiem co autor miał na myśli, ale masz jeden obiekt na którym operujesz, swoja drogą nie lepiej wykorzystać string zamiast tablicy char?

problem tkwi w tym, że nie masz nic zapisane w tych tablicach rodzaj i kategoria?

[p.....h]

@mojemirabelki: Tak, problem w tym, że:
1. Na screenie widać, że nie mogę nic wpisać w kategorię.
2. W rodzaj papieru mogę wpisać, lecz nic nie wyświetla się. Tylko nazwa (pierwsza tablica charowa), ilość sztuk i cena.

Co zrobić, żebym mógł wpisać tekst, który się zapisze w tym jednym obiekcie i go potem wyświetli?

[leoha]

@pyth: no to jeszcze kilka uwag jak chcesz pisać (bardziej) "nowocześnie":

nazwy zmiennych, symboli itd nie powinno się pisać po polsku
#include => #include
#include => #include
conio.h pewnie narzucone, więc nie skomentuje ( ͡° ͜ʖ ͡°)
#define L 10 => do definiowania stałych w c++ jest const/constexpr
strcpy(nazwa, naz); to jest niebezpieczne, moze przepelnic bufor, zamień na strncpy albo snprintf albo najlpeiej użyj std::string zamiast alokować

[p.....h]

@leoha: Dzięki za rady! Co do bibliotek nie ukrywam, że opieram się na programie, na którym działaliśmy na zajęciach. Co do wszystkiego to wezmę sobie do serca. Czyli uważasz, że:

zamień na strncpy albo snprintf albo najlpeiej użyj std::string zamiast alokować recznie bufor char*

... załatwi sprawę?

[MamCieNaHita]

@pyth: Pozwolę sobie przyczepić się do twojego kodu, gdzie się tylko da :P

1. Korzystasz w swoim programie z biblioteki conio, która jest tylko na windowsie, unikaj jej.
2. Piszesz w C++, dlatego też staraj się używać std::string zamiast tablicy charów. Tak samo staraj się unikać funkcji IO z C i staraj się używać strumieni - tak jak używasz std::cout do wpisywania, możesz używać std::cin i std::getline do wczytywania danych. Mieszasz

[leoha]

... załatwi sprawę?

@pyth: poprawnie użyte na pewno przed przepełnieniem bufora - tak

[Noct]

@pyth: zrób sobie klasę produkt i druga produkty, trzymaj wszystkie dane w kontenerach i po sprawie

[p.....h]

@Noct: @leoha: @MamCieNaHita: @mojemirabelki: @trojanxem:
Dobra panowie, zrobiłem to na stringach i wszystko chula jak talala. Felerem okazał się ten gets najwyraźniej. Może awansuję na #programista2k xd
Zastosuje się do Waszych rad i jeszcze raz przerobie kod programu, tak żeby wyglądał znośnie, bardziej "nowocześnie".
Mogę wołać gdy już skończę o kolejne rady?

Mam jeszcze parę pytań:
@MamCieNaHita: Co jest złego w "using namespace std;". Nie trzeba

[MamCieNaHita]

Co jest takiego złego w define?

@pyth: define nie ma typu. Wkleja wartość na chamca. const/constexpr ma typ.

Co jest złego w "using namespace std;".

@pyth: Robi się zły nawyk. Raz wrzucisz using namespace do headera, to "zatruje" ci wszystkie pliki, gdzie załączysz ten header. W pliku .cpp można używać z głową.

[Noct]

@pyth: to raczej nie następny krok a całkowita zmiana podejścia. Rozpisz sobie na kartce co chcesz osiągnąć, a potem pisz, inaczej dużo rzeczy robi się niepotrzebnie

[trojanxem]

@MamCieNaHita: #define jest jak najbardziej dobrym nawykiem, gdyż jest preprocesorem, i kompilator przed kompilowaniem podmienia nazwy na wartości, więc warto tego używac, może to nie jest dobry przykład użycia, ale mówienie że const jest lepszy niż define to moim zdaniem duże nadużycie, gdyż wszystko zależy do czego używamy :)
@pyth wołaj kiedy tylko mogę Ci pomóc :)

[MamCieNaHita]

@trojanxem: Ja tam wolę mieć kontrolę nad typami

https://ideone.com/BAVnCp