Naprawa ostrzeżeń

[RaQezZ]

Witam.

Ostatnio usunąłem sobie wszystkie ostrzeżenia z game za pomocą tego poradnika: 

 

Kiedy wg. tego poradnika: 

Zniosłem limit stack wyskoczyły nowe warningi np:

shop_manager.cpp:299: warning: comparison between signed and unsigned integepressions

Dotyczy on tego fragmentu kodu:

if (bCount == 0 || bCount > item->GetCount())

Chciałbym się dowiedzieć czy lepiej likwidować tego typu warningi czy je ignorować? Jak naprawić ten fragment kodu by nie sypał warningiem. Z góry dziękuję ++

[NoName#2]

zobacz jaki typ ma zmienna bCount i co jakiego typu zwraca metoda GetCount(), jedno jest ze znakiem a drugie bez znaku. Masz 2 drogi, albo w tym ifie zrzutujesz wartość np do bez znaku, albo przy zdefiniowaniu zmiennej bCount zmienisz na tak jak jest w GetCount().

[RaQezZ]

bCount mam w short... Mógłyś mi pokazać przykład jak to zrobic? Resztę dam sobie radę

 

[NoName#2]

a GetCount w czym?

[Lovable]

1 godzinę temu, NoName#2 napisał:

a GetCount w czym?

To jest niepotrzebne. Jedna zmienna jest w unsigned, a druga w signed. Najlepiej zrzutować obie na signed, bo wątpię by ilość była ujemna. Z resztą, jak ja bym miał się takimi pierdołami przejmować....

[NoName#2]

49 minut temu, Lovable napisał:

To jest niepotrzebne. Jedna zmienna jest w unsigned, a druga w signed. Najlepiej zrzutować obie na signed, bo wątpię by ilość była ujemna. Z resztą, jak ja bym miał się takimi pierdołami przejmować....

Jak nie potrzebne, to co obie będziesz rzutował, jak już jedna jest? Po co robić coś nadmiarowo. Z resztą takie pierdoły Cie w końcu zaczną obchodzić.

[RaQezZ]

bCount jest w short a GetCount()... No właśnie nie wiem... Ale wydaje mi się że w DWORD... To jest chyba ten wycinek z item.cpp

DWORD CItem::GetCount()
{
    if (GetType() == ITEM_ELK) return MIN(m_dwCount, INT_MAX);
    else
    {
        return MIN(m_dwCount, ITEM_MAX_COUNT);
    }
}

 

[Lovable]

23 godziny temu, NoName#2 napisał:

Jak nie potrzebne, to co obie będziesz rzutował, jak już jedna jest? Po co robić coś nadmiarowo. Z resztą takie pierdoły Cie w końcu zaczną obchodzić.

Nadmiarowo to jest pytanie o typy zmiennych, gdy chodzi o rezerwację bitu dla znaku. Dla mnie logiczne jest to, że jedną ze zmiennych należy przerzucić do signed skoro druga już jest. 

Zaraz również użyjesz innego kompilatora nie gcc, a clang i już będziesz miał inne warningi. Dlatego uważam, że przejmowanie się takimi rzeczami - okej, ale zbędne.