myHero[heroA].fightPointsTemp = -5; // zmiana z wartosci 12
myHero[heroB].fightPointsTemp = -5; // zmiana wartosci z 13
cout<<myHero[heroA].name<<" masz "<<myHero[heroA].fightPointsTemp<<" punktow walki."<<endl; // wyswietla 12 czyli tyle ile wynosi inicjalizacja przed przypisaniem
cout<<myHero[heroB].name<<" ma "<<myHero[heroB].fightPointsTemp<<" punktow walki."<<endl<<endl; // wyswietla prawidłowo -5
O co tu chodzi??
C++ jest pełne Undefined Behaviours, dużo mogłeś popsuć w kodzie sąsiednim.
Pełny kod by to pokazał.
Największe szanse stawiam, że heroA i heroB nie jest to same.
AnyKtokolwiek napisał(a):
C++ jest pełne Undefined Behaviours, dużo mogłeś popsuć w kodzie sąsiednim.
Pełny kod by to pokazał.Największe szanse stawiam, że
heroAiheroBnie jest to same.
Dopiero zaczałem pisac ten program i jest bardzo prosty. Wszystkie inne wartości wyświetlają się prawidłowo. Czyli heroA wynosi 0 i myHero[heroA].name jest prawidłowe. B wynosi 1 i name jest też prawidłowy odpowiednio do tej zmiennej. Jedynie tylko myHero[heroA].fightPointsTemp z uporem maniaka nie daje się zmienić.
czerwonymoskit napisał(a):
Dopiero zaczałem pisac ten program i jest bardzo prosty.
Tym bardziej kod
AnyKtokolwiek napisał(a):
czerwonymoskit napisał(a):
Dopiero zaczałem pisac ten program i jest bardzo prosty.
Tym bardziej kod
Znalazłem.
Po prostu zrobiłem sobie taki szkic w którym było:
if (xxxx) // zawsze prawdziwy
{
i tu wszystkie operacje
}
else if (yyyy)
{
}
else if (zzz)
{
}
else
i tu przypisania nowych wartosci
oraz wypisanie wartosci, które były błędne
I problemem były te puste miejsca w else if ' ach. Jak to skomentowałem / wywaliłem to działa dobrze. W sumie dziwne bo skoro zawsze uruchamia się if to nie logiczne że nie działało poprawnie przecież else if ' y nic tu nie wnosiły. No ale nic. Dzięki za zainteresowanie.
Pzowilisz, że nie wierzę w diagnozę i sądzę, że naprawa błędu była przypadkiem
Programowanie to nie magia. Nie mamy magicznych kul by widzieć co zrobiłeś źle, nie stoimy tez za twoimi plecami i nie widźmy twojego ekranu.
Uzupełniając braki z kodu który dałeś, u mnie działa.
Na razie za mało umiesz, żeby rozsądnie dać problematyczny fragment kodu.
Ergo najlepiej jak wkleisz wszystko, tak byćmy mogli to skopiować u siebie bez problemów.
Porzyczając PoŻyczając kryształową kulę od sąsiada przypuszczam, że name jest tablicą char i wpisałeś tam nazwę, które wykracza poza zakres tej tablicy.
MarekR22 napisał(a):
Programowanie to nie magia. Nie mamy magicznych kul by widzieć co zrobiłeś źle, nie stoimy tez za twoimi plecami i nie widźmy twojego ekranu.
Uzupełniając braki z kodu który dałeś, u mnie działa.Na razie za mało umiesz, żeby rozsądnie dać problematyczny fragment kodu.
Ergo najlepiej jak wkleisz wszystko, tak byćmy mogli to skopiować u siebie bez problemów.Porzyczając kryształową kulę od sasiada przypuszczam, że
namejest tablicąchari wpisałeś tam nazwę, które wykracza poza zakres tej tablicy.
#include <iostream>
#include <cstdio>
#include <cstdlib>
using namespace std;
int main()
{
struct Weapon
{
string name;
short damage[3] = {0, 0, 0};
short damageK;
short minAttack;
short minDefense;
short strength;
short safeZone;
};
struct Hero
{
string race;
string name;
short fightPoints;
short fightPointsTemp;
short health;
struct Weapon myWeapon;
}myHero[9];
short heroA, heroB, tempA1, tempB1, tempA2, tempB2, j = 0;
bool heroAattack, heroAdefense, heroAcounter;
bool heroBattack, heroBdefense, heroBcounter;
char wybor = 'n';
// Postac 1
myHero[0].race = "Czlowiek";
myHero[0].name = "Vigo";
myHero[0].fightPoints = 12;
myHero[0].health = 20;
myHero[0].myWeapon.name = "Miecz dlugi";
myHero[0].myWeapon.damageK = 1;
for (short i = 0 ; i < myHero[0].myWeapon.damageK ; ++i)
myHero[0].myWeapon.damage[i] = 8;
myHero[0].myWeapon.minAttack = 1;
myHero[0].myWeapon.minDefense = 1;
myHero[0].myWeapon.strength = 11;
myHero[0].myWeapon.safeZone = 3;
// Postac 2
myHero[1].race = "Elf";
myHero[1].name = "Idril";
myHero[1].fightPoints = 13;
myHero[1].health = 15;
myHero[1].myWeapon.name = "Szabla";
myHero[1].myWeapon.damageK = 1;
for (short i = 0 ; i < myHero[1].myWeapon.damageK ; ++i)
myHero[1].myWeapon.damage[i] = 6;
myHero[1].myWeapon.minAttack = 0;
myHero[1].myWeapon.minDefense = 0;
myHero[1].myWeapon.strength = 9;
myHero[1].myWeapon.safeZone = 3;
/*
// Postac 3
myHero[2].race = "pol - Ork";
myHero[2].name = "Gnod";
myHero[2].fightPoints = 8;
myHero[2].health = 25;
myHero[2].myWeapon.name = "Topor";
if (myHero[2].myWeapon.name == "Topor")
temp = 2;
for (short i = 0 ; i < temp ; ++i)
myHero[2].myWeapon.damage[i] = 6;
myHero[2].myWeapon.minAttack = 3;
myHero[2].myWeapon.minDefense = 4;
myHero[2].myWeapon.strength = 16;
myHero[2].myWeapon.safeZone = 1;
*/
// Wyswietlenie
cout<<"Bohater pierwszy:"<<endl<<endl;
cout<<"Rasa: "<<myHero[0].race<<endl;
cout<<"Imie: "<<myHero[0].name<<endl;
cout<<"Bieglosc w broni: "<<myHero[0].fightPoints<<endl;
cout<<"Zywotnosc: "<<myHero[0].health<<endl;
cout<<endl;
cout<<"Nazwa broni: "<<myHero[0].myWeapon.name<<endl;
cout<<"Obrazenia: ";
for (short i = 0 ; i < myHero[0].myWeapon.damageK ; ++i)
cout<<myHero[0].myWeapon.damage[i]<<", ";
cout<<endl;
cout<<"Minimalny atak: "<<myHero[0].myWeapon.minAttack<<endl;
cout<<"Minimalna obrona: "<<myHero[0].myWeapon.minDefense<<endl;
cout<<"Wymagana sila: "<<myHero[0].myWeapon.strength<<endl;
cout<<"Strefa bezpieczna: "<<myHero[0].myWeapon.safeZone<<endl;
cout<<endl;
cout<<"Bohater drugi:"<<endl<<endl;
cout<<"Rasa: "<<myHero[1].race<<endl;
cout<<"Imie: "<<myHero[1].name<<endl;
cout<<"Bieglosc w broni: "<<myHero[1].fightPoints<<endl;
cout<<"Zywotnosc: "<<myHero[1].health<<endl;
cout<<endl;
cout<<"Nazwa broni: "<<myHero[1].myWeapon.name<<endl;
cout<<"Obrazenia: ";
for (short i = 0 ; i < myHero[1].myWeapon.damageK ; ++i)
cout<<myHero[1].myWeapon.damage[i]<<", ";
cout<<endl;
cout<<"Minimalny atak: "<<myHero[1].myWeapon.minAttack<<endl;
cout<<"Minimalna obrona: "<<myHero[1].myWeapon.minDefense<<endl;
cout<<"Wymagana sila: "<<myHero[1].myWeapon.strength<<endl;
cout<<"Strefa bezpieczna: "<<myHero[1].myWeapon.safeZone<<endl;
cout<<endl;
/*
cout<<"Bohater trzeci:"<<endl<<endl;
cout<<"Rasa: "<<myHero[2].race<<endl;
cout<<"Imie: "<<myHero[2].name<<endl;
cout<<"Bieglosc w broni: "<<myHero[2].fightPoints<<endl;
cout<<"Zywotnosc: "<<myHero[2].health<<endl;
cout<<endl;
cout<<"Nazwa broni: "<<myHero[2].myWeapon.name<<endl;
cout<<"Obrazenia: ";
for (short i = 0 ; i < 3 ; ++i)
{
if (myHero[2].myWeapon.damage[i] != 0)
cout<<myHero[2].myWeapon.damage[i]<<", ";
}
cout<<endl;
cout<<"Minimalny atak: "<<myHero[2].myWeapon.minAttack<<endl;
cout<<"Minimalna obrona: "<<myHero[2].myWeapon.minDefense<<endl;
cout<<"Wymagana sila: "<<myHero[2].myWeapon.strength<<endl;
cout<<"Strefa bezpieczna: "<<myHero[2].myWeapon.safeZone<<endl;
cout<<endl;
*/
cout<<"Wybierz swoja postac 1 lub 2:"<<endl;
cin>>heroA;
cin.ignore();
heroA--;
cout<<"Wybierz postac przeciwnika 1 lub 2:"<<endl;
cin>>heroB;
cin.ignore();
heroB--;
cout<<endl;
cout<<"Witaj "<<myHero[heroA].name<<". Na Twojej drodze stanal "<<myHero[heroB].name<<'.'<<endl<<endl;
for (short i = 1 ; ; i++ )
{
j++;
if (j == 1) // nowa runda
{
myHero[heroA].fightPointsTemp = myHero[heroA].fightPoints;
myHero[heroB].fightPointsTemp = myHero[heroB].fightPoints;
}
cout<<"Runda: "<<j<<". Segment: "<<i<<"."<<endl<<endl;
cout<<myHero[heroA].name<<" masz "<<myHero[heroA].fightPointsTemp<<" punktow walki."<<endl;
cout<<myHero[heroB].name<<" ma "<<myHero[heroB].fightPointsTemp<<" punktow walki."<<endl<<endl;
{ // dostepne opcje
cout<<myHero[heroA].name<<endl;
if (myHero[heroA].fightPointsTemp >= myHero[heroA].myWeapon.minAttack)
{
heroAattack = true;
cout<<"Mozesz atakowac."<<endl;
}
else
{
heroAattack = false;
cout<<"Nie mozesz atakowac."<<endl;
}
if (myHero[heroA].fightPointsTemp >= myHero[heroA].myWeapon.minDefense)
{
heroAdefense = true;
cout<<"Mozesz parowac cios wlasna bronia."<<endl;
}
else
{
heroAdefense = false;
cout<<"Nie mozesz juz parowac."<<endl;
}
if (myHero[heroA].fightPointsTemp >= myHero[heroA].myWeapon.minAttack + 2)
{
heroAcounter = true;
cout<<"Jestes w stanie wykonac kontratak."<<endl;
}
else
{
heroAcounter = false;
cout<<"Nie wykonasz juz kontrataku."<<endl;
}
// przeciwnik
cout<<endl<<myHero[heroB].name<<endl;
if (myHero[heroB].fightPointsTemp >= myHero[heroB].myWeapon.minAttack)
{
heroBattack = true;
cout<<"Moze atakowac."<<endl;
}
else
{
heroBattack = false;
cout<<"Nie moze atakowac."<<endl;
}
if (myHero[heroB].fightPointsTemp >= myHero[heroB].myWeapon.minDefense)
{
heroBdefense = true;
cout<<"Moze parowac cios wlasna bronia."<<endl;
}
else
{
heroBdefense = false;
cout<<"Nie moze juz parowac."<<endl;
}
if (myHero[heroB].fightPointsTemp >= myHero[heroB].myWeapon.minAttack + 2)
{
heroBcounter = true;
cout<<"Jest w stanie wykonac kontratak."<<endl;
}
else
{
heroBcounter = false;
cout<<"Nie wykona juz kontrataku."<<endl;
}
}
if (myHero[heroA].fightPointsTemp < myHero[heroB].fightPointsTemp) // zaczynasz
{
cout<<endl<<myHero[heroA].name<<" okresl swoja akcje: "<<endl;
if (heroAattack == true)
cout<<"1 - Atak."<<endl;
if (heroAdefense == true)
cout<<"2 - Parowanie."<<endl;
if (heroAcounter == true)
cout<<"3 - Kontratak."<<endl;
cin>>tempA1;
cin.ignore();
if (heroAattack == true && tempA1 == 1) // atak
{
for (short ii = 0 ; ; )
{
cout<<"Min. atak: "<<myHero[heroA].myWeapon.minAttack<<". Podaj wartosc: "<<endl;
cin>>tempA2;
cin.ignore();
if (tempA2 < myHero[heroA].myWeapon.minAttack)
{
cout<<"Zla wartosc, sprobuj jeszcze raz."<<endl;
}
else
break;
}
if (heroBdefense == true && (rand()%(myHero[heroA].fightPointsTemp + 1) != 0 || rand()%(myHero[heroB].fightPointsTemp + 1) == 0)) // parowanie
{
cout<<endl<<myHero[heroA].name<<" atakuje za: "<<tempA2<<endl;
for (short ii = 0 ; ; )
{
tempB2 = rand()%(tempA2 + tempA2 / 2) + 1;
if (tempB2 >= myHero[heroB].myWeapon.minDefense && tempB2 <= tempA2)
break;
}
cout<<myHero[heroB].name<<" paruje swoja bronia za: "<<tempB2<<endl;
}
else if (heroBcounter == true && (rand()%(myHero[heroA].fightPointsTemp + 1) == 0 || rand()%(myHero[heroB].fightPointsTemp + 1) != 0)) // kontra
{
}
else if (heroBattack == true)
{
}
else
myHero[heroA].fightPointsTemp -= tempA2;
myHero[heroB].fightPointsTemp -= tempB2;
cout<<"wartosc tempA2:"<<tempA2<<endl;
cout<<"wartosc tempB2:"<<tempB2<<endl;
cout<<myHero[heroA].name<<" masz "<<myHero[heroA].fightPointsTemp<<" punktow walki."<<endl; // ???!!!
cout<<myHero[heroB].name<<" ma "<<myHero[heroB].fightPointsTemp<<" punktow walki."<<endl<<endl;
getchar();
}
else if (heroAdefense == true && tempA1 == 2) // parowanie
{
for (short ii = 0 ; ; )
{
cout<<"Min. parowanie: "<<myHero[heroA].myWeapon.minDefense<<". Podaj wartosc: "<<endl;
cin>>tempA2;
cin.ignore();
if (tempA2 < myHero[heroA].myWeapon.minDefense)
{
cout<<"Zla wartosc, sprobuj jeszcze raz."<<endl;
}
else
{
myHero[heroA].fightPointsTemp -= tempA2;
break;
}
}
}
else if (heroAcounter == true && tempA1 == 3) //kontra
{
for (short ii = 0 ; ; )
{
cout<<"Min. atak: "<<myHero[heroA].myWeapon.minAttack<<" + 2 pkt. Podaj wartosc: "<<endl;
cin>>tempA2;
cin.ignore();
if (tempA2 < myHero[heroA].myWeapon.minAttack)
{
cout<<"Zla wartosc, sprobuj jeszcze raz."<<endl;
}
else
{
myHero[heroA].fightPointsTemp -= tempA2 + 2;
break;
}
}
}
else
{
cout<<"Zla wartosc, sprobuj jeszcze raz."<<endl<<endl;
i--;
j--;
continue;
}
}
getchar();
} // for nieskonczony
cout<<"ENTER..."<<endl;
getchar();
return 0;
}
Moja postać 1 , przeciwnik 2, Zawsze atak czyli 1 i dalej wartosć powiedzmy 6. I te 6 powinno sie odjąć od fightPoints tego gościa. Przy B działa a dla A nie.
Już widzę bardzo dziwną pętle po wartości 1
Strasznie brzydki wielki main. Łącznie z deklaracja struktur w main, co wprawdzie kompilator C++ dopuszcza, ale jest ewenementem.
Jakbyś miał funkcje np do inicjowania byś miał zredukowaną szansę na błędy
Jakbyś miał konstruktor, byś mógł raz napisać kod inicjujący i używac go jako zaufany.
Magiczne liczby 1,2,3
Tablice C
Tak wielki i tak chaotyczny kod może działać tylko w przychylnym zbiegu okoliczności
Masz źle ustawiony kompilator.
Jeśli używasz gcc/clang dodaj takie flagi:
-std=c++20 -fsanitize=address,undefined -Wall -Wextra -Werror -pedantic
Kompilator od razu wyłapie więcej kwiatków:
https://godbolt.org/z/qE7Mj5onx
Najciekawsze:
<source>:285:47: error: 'tempB2' may be used uninitialized [-Werror=maybe-uninitialized]
285 | myHero[heroB].fightPointsTemp -= tempB2;
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~
<source>:28:41: note: 'tempB2' was declared here
28 | short heroA, heroB, tempA1, tempA2, tempB2, j = 0;
| ^~~~~~
cc1plus: all warnings being treated as errors
Nie pisz wszystkiego w main im szybciej nauczysz się dzielić kod na małe funkcje tym lepiej.
Dodatkowo sformatowanie kodu ujawnia wtopę z brakującymi kalmarami
Dzięki za info. Uczę się teraz o funkcjach więc będzie możliwość poćwiczyć ich zastosowanie. Rozumiem, że najlepiej deklarować je przed main, a definicje po main. Wydaje mi się, że przy brakujących klamrach program by się nie skomplikował. Nie bardzo wiem gdzie te flagi ustawić. O którą pętlę chodzi? Tablice C? Magiczne liczby? 
Dzięki za info. Uczę się teraz o funkcjach więc będzie możliwość poćwiczyć ich zastosowanie. Rozumiem, że najlepiej deklarować je przed main, a definicje po main.
Widzę, ze starasz się uczyć 2x szybciej
https://www.google.com/search?client=firefox-b-d&q=magiczne+liczby+programowanie
dajo napisał(a):
Dzięki za info. Uczę się teraz o funkcjach więc będzie możliwość poćwiczyć ich zastosowanie. Rozumiem, że najlepiej deklarować je przed main, a definicje po main.
Ja bym nie deklarował, a od razu definiował przed main()
Rozbicie na deklarację i defincję jest reliktem którego nie spotkasz w nowszych językach
Ale nie to jest centrum problemu. Chodzi o dobrze określone funkcje, wykonujące jedną rzecz
Tablice C?
https://www.google.com/search?client=firefox-b-d&q=c+array+vs+std%3A%3Avector+vs+std%3A%3Aarray
Elementarz C++ którego niby używasz . W gratisie bezpiecznik .
(osobiście sadzę, że korzystasz z jakiego kursu C++, autora który w tym C++ jest niewiele lepszy od ciebie)
Uzupełnię: konstruktory do tych struktur
Proste, a podstawa aby ten mega-main się trochę skrócił.
A co do rzeczywistej lub domnienanej niezgodności podstawienia i odczytu: debuger.
O wiele ważniejsza umiejętność niż trochę kodu, którego kiedyś będziesz się wstydził.
Dzięki.
Magiczne liczby opanowane. Faktycznie lepiej w warunkach podstawiać zmienne niż literały liczbowe.
Przy tak prostym kodzie to chyba myślę tablice C są ok, przeciez to nie jakaś zaawansowana gra.
Dzielenie na funkcje fajna sprawa i defniuje przed main tylko musiałem przenieść zmienne jako globalne.
Co do resztyt muszę się dokształcić.
czerwonymoskit napisał(a):
Dzielenie na funkcje fajna sprawa i defniuje przed main tylko musiałem przenieść zmienne jako globalne.
Tzn zrobiłes to źle.
Jeśli pojawiła się presja na zmienne globalne, te funkcje zrobione są źle
czerwonymoskit napisał(a):
Przy tak prostym kodzie to chyba myślę tablice C są ok, przeciez to nie jakaś zaawansowana gra.
Ale to ty masz problem, że co innego zapisujesz, co innego masz - w "tak prostym kodzie"
Właśnie chodzi o to, że mam dobrze zapisane, a ten dramat c++ robi co mu się podoba. Ja rozumiem, że są jakieś tam klasy zastępujące zwykłe tablice C bo to daje większe bezpieczeństwo ale nie oznacza że nie mogę użyć standardowych tablic c w prostym kodzie. Jeśli funkcje przeniosłem przed main to kompilator nie znał zmiennych, które były w main. Po przesunięciu ich ponad funkcje już je znał. Natomiast tu po prostu kombinuje więc nie będę się upierał. C++ to jednak nie za dobry pomysł więc nie ma się co szarpać z tym cholerstwem.
dajo napisał(a):
Właśnie chodzi o to, że mam dobrze zapisane, a ten dramat c++ robi co mu się podoba.
....
Po przesunięciu ich ponad funkcje już je znał. Natomiast tu po prostu kombinuje więc nie będę się upierał. C++ to jednak nie za dobry pomysł więc nie ma się co szarpać z tym cholerstwem.
Taaaa, tym masz dobrze napisane a wszystko wina błędów kompilatora.
To jakby oczywiste w takich sytaucjach
@dajo: dodaj ten watek do ulubionych i zajrzyj tu za powiedzmy dwa lata, jak nadal będziesz w tym zawodzie i nadal będziesz używał c++, to ciekawe co pomyślisz o samym sobie :)
Bo teraz to wiemy że że wszystkiemu winny jest kompilator
Marius.Maximus napisał(a):
@dajo: dodaj ten watek do ulubionych i zajrzyj tu za powiedzmy dwa lata, jak nadal będziesz w tym zawodzie i nadal będziesz używał c++, to ciekawe co pomyślisz o samym sobie :)
Bo teraz to wiemy że że wszystkiemu winny jest kompilator
Logika trójwartościowa (gdzies tu obok wątek mi to przypomniał)
Albo rozzłoszczony kompilatorami odpadnie.
Albo przyjmie w/s do wiadomości i sie rozwinie
Albo zostanie Zakładowym Guru Excella / Delphi / whatever, który nie przyjmuje żadnych uwag.
Po prostu jeśli zmienne mam w main jako lokalne, a definicje funkcji powyżej niego to zgłasza problem:
'heroA' was not declared in this scope|
gdy próbuje się do tych zmiennych dostać.
Co w sumie jest logiczne bo skąd ma znać tą zmienną przed main skoro jest definiowana dalej w main. Natomiast jestem zielony z funkcji.