[e.....o]

#programowanie #clion #c++
Cześć mirki, zadeklarowałam sobie vector w funkcji i sobie go uzupełniam w tej samej funkcji ale jak wchodzę w tryb debugu to mi się nie pokazuje zawartość vectora (tak jakby był cały czas pusty) natomiast sprawdzam sobie jego rozmiar i normalnie się zwiększa. Ktoś ma pomysł czemu tak jest?

[a231]

@emsko: Optymalizacja?

[e.....o]

Optymalizacja

@a231: Nie bardzo rozumiem co masz na myśli

[a231]

@emsko: Pokaż tę funkcję i kod bo ciężko wróżyć z fusów.

[e.....o]

@a231:
std::vector KSA(int N, int T, std::vector K)
{
std::vector S;
int L= K.size();
unsigned char c;
int z;
for(int i =0; i

[Kicer86]

@emsko: próbowaleś w innymi ide? kod na oko wygląda spoko (pomijając pare drobiazgów).

[e.....o]

@Kicer86: już w porządku, okazało się, że to problem w CLionie, tylko jakoś nie mogłam tego znaleźć wczoraj. Zgłoszono rok temu: https://youtrack.jetbrains.com/issue/CPP-7032

A możesz powiedzieć trochę o tych drobiazgach? Chętnie przyjmę konstruktywną krytykę (rzadko ostatnio mi się trafia, że ktoś mi kod reviewuje ( ͡° ͜ʖ ͡°) ). Podejrzewam, że chodzi o nieeleganckie castowanie inta?

[Kicer86]

@emsko: 1. przekazujesz wektor unsigned charow przez wartość, czyli robi się kopia. Jeśli użyjesz const referencji, z punktu widzenia funkcji nic się nie zmieni, a program będzie działał szybciej. 2. nazwy zmiennych są dośc minimalistyczne - użyłbym jakichś bardziej deskryptywnych. 3. przyjęło się stosować duże litery dla stałych, bądź makr, choć to akurat kwestia gustu. 4. rzutowanie inta może być ale, w c++ stosuje się raczej static_cast zamiast rzutowania w stylu

[e.....o]

@Kicer86:
Cenna uwaga z tymi const referencjami, muszę na to bardziej zwracać uwagę. Co do nazw zmiennych to myślę, że jeśli implementuję jakiś popularny algorytm i takie nazewnictwo jest stosowane w opisie i pseudokodzie to nie ma co szaleć ze zmianną nazw, bo można się pogubić potem. Punkt 6 zapisane do poczytania.
Dziękuję bardzo pomocny mirku