wskazywanie najmniejszej lub najwiekszej liczby c++ .oscar's grind, casino.

[11100001 0]

Czołem.

 

Mam pewien problem. kod jest napisany "bezblednie". Przy algorytmie napisanym na podstawie tablic, gdzie pewna zmienna przyrownujemy do tablicy 0, a tablice 0 z kolei do tablicy [x], z czego wybieramy najmniejsza lub najwieksza wartosc.

 

Problem w tym ze wyswietlaja sie numerki (2686916)zamiast wlasciwego wyniku. Jest to niezrozumiałe. Czasem dopisze do kodu w petli czy w innym miejscu jakas zmienna i juz wskazywanie najm i najw wartosci nie dziala, tj. wyswietlaja sie numerki.

 

czasem z kolei jest tak ze buduje program i uruchamiam, wszystko dziala, za nastepnym razem przy zmianie wartosci, lub i nawet bez zmieniania czegokolwiek nie dziala.

 

Beznadzieja, tym bardziej że część działa poprawnie a część niepoprawnie.

 

Nie wiem dlaczego te numerki wyskakują???

 

------------------------------------------------------------------------

data: 20/01/2018,

Mam nadzieję, że pomysł nie zostanie wykorzystany przez próżne osoby chcące żyć kosztem innych, w najgorszym wypadku pomoc w odegraniu i wyjściu na prostą drogę w życiu.

 

Oczywiście widać że ten system nie działa.

 

Niech was Bóg zachowa od kasyn... oraz nieuczciwego, niemoralnego, niewolniczego, życia.

-----------------------------------------------

data:25/01/2018.

 

Oczywiście, nie jest to właściwy kod dla idei grindu oscara, jest on nieco zmieniony...

(dodaje sumę rozegranych losowań...)

[11100001 0]

Teraz nie mam błędu ale nie wyświetla właściwych danych. Już nie wiem.

 

#include <iostream>
#include <ctime>
#include <cstdlib>

using namespace std;



int main()
{
   srand(time(0));
   int n=10;
   float wynik=0;
   float ile1=0;
   float ile0=0;

   int tabNajmBilans[n];
   int NajmBilans;
   int tabNajwZaklad[n];
   int NajwZaklad;


   for(int x=1;x<=n;x++){


   int zaklad=0;
   int wygrana=0;
   int tabwygr[n];




   for(int i=0;i<=n;i++){
       int r=rand()%2;

       zaklad++;
       tabwygr[i]=wygrana;
       if(r==0) {ile0++; wygrana-=zaklad;}
       if(r==1) {ile1++; wygrana+=zaklad;}

       cout<<i<<" "<<r<<" "<<zaklad<<" "<<wygrana<<endl;
       if((tabwygr[i]>tabwygr[i-1])||(r==1))break;



   }cout<<endl<<"Wygrana: "<<wygrana<<" zak: "<<zaklad<<endl;
   cout<<"----------------------------------------------------------"<<endl;
   wynik+=wygrana;

   tabNajmBilans[x]=wygrana;
   tabNajmBilans[0]=NajmBilans;
   if(tabNajmBilans[x]<NajmBilans) {NajmBilans=tabNajmBilans[x];}

   tabNajwZaklad[x]=zaklad;
   NajwZaklad=tabNajwZaklad[0];
   if(tabNajwZaklad[x]>NajwZaklad) NajwZaklad=tabNajwZaklad[x];

}
   cout<<endl<<"============================Statystyki:=============================="<<endl;
   cout<<"Wynik: "<<wynik<<endl;
   cout<<"Razem 0+1: "<<ile0+ile1<<endl;
   cout<<"%: "<<wynik/(ile0+ile1)<<endl;
   cout<<"1: "<<ile1<<endl;
   cout<<"0: "<<ile0<<endl<<endl;

    cout<<"Najnizszy bilans:"<<NajmBilans<<endl;
    cout<<"Najwyzszy zaklad:"<<NajwZaklad<<endl;

   return 0;
}

 

NajwZaklad przyjmuje jako stałą wartość, wartość zakladu z ostatniej pętli x.

 

Nie potrafię rozszyfrować o co chodzi.

Ten algorytm do sprawdzania powyżej jest identyczny i działa, a właściwie jest jeszcze gorzej, ponieważ kiedy napiszę poniższy algorytm do typowania najm/najw, w taki sam sposób jak powyzszy, wyskakuja numery zamiast wyniku.

 

Nie potrafię tego rozpracować.

 

Za wszelkie uwagi dziękuję.

[11100001 0]

   tabNajmBilans[x]=zaklad;
   tabNajmBilans[0]=NajmBilans;
   if(tabNajmBilans[x]<NajmBilans) {NajmBilans=tabNajmBilans[x];}

   tabNajwZaklad[x]=wygrana;
   tabNajwZaklad[0]=NajwZaklad;
   if(tabNajwZaklad[x]<NajwZaklad) {NajwZaklad=tabNajwZaklad[x];}

 

 

zamienilem miejscami zaklad z wygrana.

 

wygrana pokazuje wlasciwie i zaklad tez, ale kiedy zmieniam znak na przeciwna strone

 if(tabNajmBilans[x]<NajmBilans)

wywala mi dziwny blad typu (1990954197).

 

Jestem bardzo rozczarowany tym programem, jednym slowem w c++ wszystkie chwyty dozwolone, i kompilator odmowi posluszenstwa kiedy mu pasuje, tak jak gdyby nie bylo zasad dzialania tego czegos.

 

No jestem poczatkujacym i kod moze nie najlepszej jakosci, ale uwazam ze to przesada, takie zdarzenia juz miewalem w innych programach, wystarczylo ze dodalem o 1 zmienna wiecej w petli i juz blad wywalal.

[MitycznyJeż 478]

Jak każdy szanujący się początkujący programista zwalasz winę na kompilator

Problem jest na linii

to, co myślisz że napisałeś <-> to, co zostało napisane

 

Czytaj (a jeżeli ich nie ma - włącz) warningi:

main2.cpp(52): warning C4700: uninitialized local variable 'NajmBilans' used
main2.cpp(56): warning C4700: uninitialized local variable 'NajwZaklad' used

 

Btw, to nie jest poprawny c++:

int n=10;
///...
int tabNajmBilans[n];

Ale nie zawracaj sobie tym głowy póki wystarczy że u Ciebie działa

 

Bonus:

Przypomnij sobie jak indeksujemy kolejne wartości w tablicach.

[11100001 0]

Nie szkoda Panu swojego czasu? Mi Pan nie pomaga a czas Pan marnuje...

 

Zatem...

 

dla wygrana dziala a dla zaklad nie, chodzi o inkrementacje, ...

 

dziwne,

 

---------------------------

Re:Bonus,

 

Jezeli się dobrze rozumiemy to nie zmienia niczego, n-zamienie na manualny wpis 10 000 czy 100 000, i wywali tylko inny blad.

[MitycznyJeż 478]

Co więc niby ma być pomocą? Podanie rozwiązania na tacy?

Jak masz cokolwiek programować, to musisz nauczyć się sam znajdować błędy we własnym kodzie. Dlatego nie oczekuj, że dostaniesz odpowiedź wprost.

A jeżeli dostaniesz, to będzie ona antyrozwojowa - bo zamiast wiedzieć jak sobie samemu pomóc następnym razem, to znowu zrobisz temat "znajdźcie mi błąd"...

 

Podanie treści warningów, które pokazują wprost gdzie i jaki jest błąd nie jest pomocą?

Zasugerowanie, że błędnie iterujesz po tablicy nie jest pomocą?

 

Bonus2:

Zastanów się, co się dzieje w tej linijce gdy i==0:

if ((tabwygr[i]>tabwygr[i - 1]) || (r == 1))break;

[11100001 0]

dziękuję za poświęcony mi czas ale obawiam się że z edukacją ma Pan niewiele wspólnego.

 

Sprawa jest prosta,

 

Tu nie ma co myśleć, są to gotowce już przez kogoś stworzone i nie ma sensu marnować czasu na rozmyślania a na jak najszybsze poznanie.

 

Kiedy mechanik nie może znaleźć przyczyny awari, dzwoni do starszego doświadczeniem kolegi, podaje objawy i już wie.

Jaki sens logiczny aby rozebrać cały samochód na części pierwsze, zmarnować tydzień, jak można dowiedzieć się w ciągu kilku minut a awarie usunąć w przeciągu kilku godzin i posiąść już wiedzę w tym temacie.

 

Dobrze że nie bierze się Pan za assembler. Właściwie to dobre pytanie, po co nauczył się Pan c++ - sa skoro Pan od tych myślących, wszystkowiedzących.

 

?

 

Rozmyślać można nad problemami nierozwikłanymi, natury czysto matematycznej, fizycznej, chemicznej itp.

 

To ludzki twór i nie ma nad czym myśleć...

 

 

-------------------------------------------------

 

if ((tabwygr[i]>tabwygr[i - 1]) || (r == 1))break;

 

Mój kompilator to chyba ignoruje, nic nie wywala, zlicza właściwie. Nie wydaje mi się aby miało to powiązanie z moim problemem. Swoją drogą dobre spostrzeżenie.

 

Właśnie stąd ten zapis że inaczej nie umiałem wykazać o co mi chodzi...

 

----------------------------------------------------

 

Proszę nie odpisywać.

 

Niepotrzebnie będzie marnował Pan czas, pomimo Pana dobrych chęci.

Nie chce mieć Pana na sumieniu , i nie mówię tego złośliwie.

 

Dziękuję za pomoc.

Temat do zamknięcia.

[_Piotr_Z_ 0]

dziękuję za poświęcony mi czas ale obawiam się że z edukacją ma Pan niewiele wspólnego.

 

Ktoś usiłuję Ci pomóc, a Ty się na niego rzucasz, nieładnie tak.

Sądząc też po Twoim podejściu, programowanie może nie być dla Ciebie...

[11100001 0]

Dlaczego? Bądź bardziej bezpośredni ,proszę. Możesz wyeksponować swoje ego, nie będę zgłaszał do modernizacji. Powiedz nam jaki to prawdziwy programista, jednym słowem człowiek sukcesu w każdym calu, czyli ty.

 

Już i tak zaszczycił nas Pan swoją wypowiedzią..., śmiało!...

[Skullek 2]

Tu nie chodzi o to, czy ktokolwiek jest cudownym programistą tylko o metodologie rozwiązywania trudniejszych problemów na które kiedyś się natkniesz, których nikt nie rozwiązał, bądź na danym etapie nie ujawni odpowiedzi.

 

Samo "nie kodź czegoś co ktoś już napisał" odnosi się do momentu kiedy wiesz co chcesz osiągnąć, czego szukasz i dlaczego tak działa, Ty masz podstawowe braki.

 

Jeśli kiedyś wymyślisz jakiś algorytm sam - bądź jego pochodną, bądź będziesz robił code review koledze z pracy sam musisz działać jak kompilator, nie ważne czy to Assembler czy Java - czy jeszcze coś wyżej w abstrakcji.

 

Twój kod nie jest napisany bezbłędnie, nie ważne ile razy wykorzystasz cudzysłów. Nie inicjalizujesz zmiennych, nie przydzielasz dynamicznie pamięci. Skup się na zmiennej najmBilans - jeśli to ogarniesz, to resztę po sznurku również.

[11100001 0]

Tu nie chodzi o to, czy ktokolwiek jest cudownym programistą tylko o metodologie rozwiązywania trudniejszych problemów na które kiedyś się natkniesz, których nikt nie rozwiązał, bądź na danym etapie nie ujawni odpowiedzi.

Dlaczego mówimy o metodologii?

 

Samo "nie kodź czegoś co ktoś już napisał" odnosi się do momentu kiedy wiesz co chcesz osiągnąć, czego szukasz i dlaczego tak działa, Ty masz podstawowe braki.

 

Pan również może być bardziej bezpośrednii. W czym zatem problem? W zasadzie jakiej natury jest Pański problem?

 

Jeśli kiedyś wymyślisz jakiś algorytm sam - bądź jego pochodną, bądź będziesz robił code review koledze z pracy sam musisz działać jak kompilator, nie ważne czy to Assembler czy Java - czy jeszcze coś wyżej w abstrakcji.

 

Dlaczego mówimy o pracy programisty?

[Karister 1595]

Skoro Pan taki kulturalny i obyty, pora już dojść do odpowiednich wniosków, że gotowca nikt najwyraźniej nie zapoda i pora zmienić taktykę. Pozostaje więc albo zapoznać się z poradami innych i zrozumieć podstawy, aby swój problem rozwiązać, albo przeszukać forum za odpowiednim działem: Dam Pracę, jeśli nie chce się włożyć minimum trudu w naukę.

[11100001 0]

Do kogo Pan pisze?

[209458 867]

Tu nie ma co myśleć, są to gotowce już przez kogoś stworzone i nie ma sensu marnować czasu na rozmyślania a na jak najszybsze poznanie.

W takim razie dobra rada - zmień źródło gotowców, bo te twoje są pośledniej jakości.

 

Kiedy mechanik nie może znaleźć przyczyny awari, dzwoni do starszego doświadczeniem kolegi, podaje objawy i już wie.

Za pierwszym razem pewnie tak, ale jak za którymś kolejnym będzie dzwonił z duperelą, to w końcu usłyszy "weź sp...", ew. "jak sobie nie radzisz, to przyprowadź do naszego zakładu, zapłać a naprawimy".

[11100001 0]

#include <iostream>
#include <ctime>
#include <cstdlib>

using namespace std;



int main()
{
   srand(time(0));
   int n=10;
   float wynik=0;
   float ile1=0;
   float ile0=0;


   int NajmBilans;
   int NajwZaklad;


   for(int x=1;x<=n;x++){


   int zaklad=0;
   int wygrana=0;

   int tabNajwZaklad[n];
   int NajwZaklad2;
   int tabNajmBilans[n];
   int NajmBilans2;



   for(int i=0;i<=n;i++){
       int r=rand()%2;
       int tabwygr[n];
       zaklad++;
       tabwygr[i]=wygrana;
       if(r==0) {ile0++; wygrana-=zaklad;}
       if(r==1) {ile1++; wygrana+=zaklad;}

       cout<<i<<" "<<r<<" "<<zaklad<<" "<<wygrana<<endl;
       if((tabwygr[i]>tabwygr[i-1])||(r==1))break;



   }cout<<endl<<"Wygrana: "<<wygrana<<" zak: "<<zaklad<<endl;
   cout<<"----------------------------------------------------------"<<endl;
   wynik+=wygrana;

   tabNajmBilans[x]=wygrana;
   tabNajmBilans[0]=NajmBilans2;
   if(tabNajmBilans[x]<NajmBilans2) {NajmBilans2=tabNajmBilans[x]; NajmBilans=NajmBilans2;}

   tabNajwZaklad[x]=zaklad;
   tabNajwZaklad[0]=NajwZaklad2;
   if(tabNajwZaklad[x]>NajwZaklad2) NajwZaklad2=tabNajwZaklad[x]; NajwZaklad=NajwZaklad2;

}
   cout<<endl<<"============================Statystyki:=============================="<<endl;
   cout<<"Wynik: "<<wynik<<endl;
   cout<<"Razem 0+1: "<<ile0+ile1<<endl;
   cout<<"%: "<<wynik/(ile0+ile1)<<endl;
   cout<<"1: "<<ile1<<endl;
   cout<<"0: "<<ile0<<endl<<endl;

    cout<<"Najnizszy bilans:"<<NajmBilans<<endl;
    cout<<"Najwyzszy zaklad:"<<NajwZaklad<<endl;

   return 0;
}

 

Kończąc temat, dodaję odpowiedź dla przyszłego poszukiwacza...

 

rozmieszczenie zmiennych i tablic w odpowiednich petlach pomoglo. chociaz program w dalszym ciagu wydaje sie byc nieco kruchy.

 

Nie ma żadnej filozofii, ani nad czym myśleć. Typowa monotonia i mechaniczność. Kwestia poświęcenia t czasu na zapoznanie się z odpowiednią literaturą opisującą budowę tego programu oraz działania podzespołów.

 

 

Do zamknięcia.

[11100001 0]

W takim razie dobra rada - zmień źródło gotowców, bo te twoje są pośledniej jakości.

 

 

Za pierwszym razem pewnie tak, ale jak za którymś kolejnym będzie dzwonił z duperelą, to w końcu usłyszy "weź sp...", ew. "jak sobie nie radzisz, to przyprowadź do naszego zakładu, zapłać a naprawimy".

 

Każdy jest inny. Po co takie wstawki? Jaki był ich cel?

 

------------------------------------------

 

Słyszął Pan że równe przyciąga równe? Złodziej boi się że zostanie okradziony. Oszust, że oszukany. Niedowartościowany, stara się ująć innym.

 

Niezależna telewizja na youtube polecam, wydawnictwo limbus oraz wywiady z Mariuszem Piotrowskim.

 

Pozdrawiam cieplutko.

 

Pa Pa

[209458 867]

Każdy jest inny. Po co takie wstawki? Jaki był ich cel?

Każdy jest inny, każdemu cierpliwość skończy się w innym momencie

 

A po co były te wstawki? To chyba oczywiste. Ta o źródle gotowców ma oszczędzić ci czasu w przyszłości, gdybyś przypadkiem zechciał użyć innego gotowca z tego źródła.

Z kolei odnośnie twojej anegdotki o mechaniku to tylko tyle, że była nie adekwatna do sytuacji. Ciężko się żyje wiedząc, że ktoś tam w internecie nie ma racji, więc człowiek poświęca swój czas, żeby taką osobę przekonać i sprowadzić na dobrą drogę. A głębsze zrozumienie mojej wersji anegdoty pomaga zrozumieć dlaczego ludzie odpowiadający w wątkach o programowaniu traktują początkujących tak, a nie inaczej.

[11100001 0]

.................................................................................

 

powód:wpis nic nie wnosi do tematu. Przepraszam

[tomangelo 0]

Program "wydaje się" kruchy bo jest sporo błędów przez które program nadal będzie w losowych momentach wyświetlać "numerki" czy nawet się wywalać, chociażby niezainicjalizowane zmienne o których powiedziano wcześniej. Na przykład:

tabNajmBilans[0]=NajmBilans2;

jaką wartość będzie mieć tabNajmBilans[0]? Czy inaczej, jaka liczba kryje się w tym momencie pod zmienną NajmBilans2? Mamy deklarację zmiennej NajmBilans2, potem nic do niej nie przypisujemy, jednak wartość zmiennej (do której nic nie zapisaliśmy) przypisujemy do innej zmiennej. A takich przypadków jest tu więcej. To czy program zachowa się zgodnie z oczekiwaniami zależy tylko od tego czy kompilator domyśli się o co nam chodziło. Jeśli takie "tajemnicza zmienne" dostaną inne wartości niż przy poprzednim uruchomieniu (bo np. zmienna przy uruchamianiu dostała obszar pamięci na którym już były jakieś dane), to program będzie działać inaczej za każdym razem. A oprócz tego są inne błędy, które mogą spowodować przerwanie pracy programu (linia 44, patrz #4).

Właśnie na to kompilator chce zwrócić uwagę wyświetlając ostrzeżenia (jeśli ma je włączone), zachęcając programistę do analizy kodu pod kątem błędów w zachowaniu. Jeśli jednak zamiast przyjąć radę ktoś woli obrażać się na cały świat to niestety, ale kariery w tym kierunku nie zrobi.

[barneyPL 0]

for(int i=0;i<=n;i++){

...

if((tabwygr>tabwygr[i-1])||(r==1))break;

 

Podobuje mnie się to niezwykle

"Niewinne programowanie cz. 1. Dostęp do pamięci innych aplikacji dla opornych."

[Karister 1595]

słowo gotowiec w tamtym wpisie jest równoważne ze słowem kompilator

Poczytaj sobie, co to jest kompilator, bo nie rozumiesz słów, których używasz i nie ma opcji, by ktokolwiek domyślił się, o co chodzi. Kompilator nie ma nic wspólnego z gotowcem kodu i sam też nie jest żadnym gotowcem. To tak, jakby statek miał być równoważny z wodą albo był gotowcem, bo pływa.. .

[misiutychy 4]

Ciekawe że to się w ogóle kompiluje

Inicjujesz tablicę n-elementową a potem iterujesz jakby po (n+1) elementach Nie wspomnę już o tym co już Ci wcześniej koledzy wytłuścili.

Masakra ten kod

[209458 867]

"Niewinne programowanie cz. 1. Dostęp do pamięci innych aplikacji dla opornych."

No właśnie jak sięgnie to innych to pół biedy, system operacyjny uwali segmentation fault. Ale przed czytaniem własnych śmieci nic nie uchroni i będzie się zastanawiał, co to za cuda.

BTW., niezawodny "Geek & poke" ma obrazek na wszystko:

http://geek-and-poke.com/geekandpoke/2017/8/13/just-happened