Problem ze sprawdzeniem danych

[Adanos]

Pytanie czysto techniczne : Czy mogę się w jaki kolwiek sposób odwołać do rzucone poza pętlą ??

Tak, zależy to tylko od tego w jakim bloku zadeklarujesz zmienną, tablicę.

Jak zrobić aby oba losowania były równe . No bo w kości gra się w 2 osoby , a nie pojedynczo . Gdy stworzyłem taką samą funkcje ( nie wiem czy źle , czy dobrze , ale myśle że potem mi to się przyda ) i wywołałem ją z funkcji main to oba losowania są identyczne . Myśle że jest to problem z ziarnem , bo jako parametr jest czas (który jest taki sam w obu przypadkach .

Wystarczy ci jedna funkcja, która losuje. Najwyżej zrobisz, aby funkcja zwracała tablicę.

[DominikT]

Teraz takie pytanie : Często widuje taki błąd : http://ifotos.pl/zobacz/Untitledj_xaenhrh.jpg/ (na dole ) czego to jest oznaką ??, napewno coś z indeksem tablicy , ale chciałbym nie popełniać tego błędu i być swiadomym i unikać tego . Jak widać zrobiłem funkcje losującą ale jako trzeci parametr dałem jakąś zmienną do moich wcześniejszych pomysłów i o dziwo losuje liczby , no ale nie wbija się wartość tablicy i nie mówi czy dwójka trójka ... kod (jeżeli link nie działa )

Kod: [Zaznacz]

int wartoscWygranej ;
using namespace std;
void losuj (int kosci , int wszystkie_kosci , int rzucone [] )
{
   for (int i = 0 ; i < kosci ; i++)
   {  
      rzucone [i] = rand () % (wszystkie_kosci) + 1;

      cout << "Twoje kosci to: " << rzucone[i] << endl;
   }
}
void rzucKosci (int kosci , int wszystkie_kosci)

{
   char znak = 'T' ;
   srand (time(NULL));

   int rozmiar = wszystkie_kosci + 1;
   int rzucone[kosci];
   int tablica[rozmiar];

   for (int i = 0; i < rozmiar; i++)
      tablica[i] = 0;

losuj (5,6,&wartoscWygranej) ;
tablica[rzucone[i]]++;
   
   int wartosc = 0 ;
   
   for (int i = 0; i < rozmiar; i++)
   
   {
cout << tablica[i] << endl ;

    if (tablica[i] == 5)
    {
rozmiar += 5;
    cout << "Piatka !!! " << endl ;
  wartoscWygranej = 5;
break ;

}
else if (tablica[i] == 4)
    {
rozmiar += 4 ;
    cout << "Kareta !!! " << endl ;
    wartoscWygranej = 4;
break ;
}


else if (tablica[i] == 2)
    {
    wartosc += 2;
   
}

  else if (tablica[i] == 3)
    {
    wartosc += 3;
   
}
  else if (tablica[i] == 1)
    {
    wartosc += 0;
   
}


  }  
 
  if (wartosc == 4 )
  {

  cout << " Para !!! " << endl ;
  wartoscWygranej = 2 ;
}
else if (wartosc == 5 )
  {
  cout << " FULL !!! " << endl ;
  wartoscWygranej = 6 ;
}
else if (wartosc == 2 )
  {
  cout << " Dwojka !!! " << endl ;
  wartoscWygranej = 1 ;
}
else if (wartosc == 3 )
  {
  cout << " Trojka !!! " << endl ;
  wartoscWygranej = 3 ;
}
else  if (wartosc == 0 )
  {
  cout << "Nic" << endl ;
  wartoscWygranej = 0 ;
}

Dodałem wartoscWygranej do dalszego programu aby można było liczyć które losowanie lepsze . I Jak moge się zabrać za JEDNĄ ZMIANE Z WYLOSOWANYCH JUŻ LICZB . Czy musze gruntownie zmieniać kod ?? , troche nie ogarniam funkcji (dzisiaj sobie o nic poczytam w nocy w symfoni c++) i chyba dlatego tak trudno mi jest to zrozumieć . Adanos naprawde mi pomagasz

[Adanos]

Kod: [Zaznacz]

tablica[rzucone[i]]++;
Przecież to jest poza pętlą for.

Nie wiem, czy to przez wklejenie na forum kodu, ale formatowanie jest fatalne.

I Jak moge się zabrać za JEDNĄ ZMIANE Z WYLOSOWANYCH JUŻ LICZB .

Nie bardzo rozumiem.

[DominikT]

Chodzi dokładnie o to że w tej grze Po rzucie moim i przeciwnika można zamienić kilka kości i ich poprostu przerzucić , aby uzyskać lepsze wyniki . U mnie formatowanie w Dev c++ jest normalne .

[Adanos]

Zamień:

Kod: [Zaznacz]

void losuj (int kosci , int wszystkie_kosci , int rzucone [] )
na:

Kod: [Zaznacz]

int* losuj (int kosci , int wszystkie_kosci , int rzucone [] )
i na końcu funkcji daj

Kod: [Zaznacz]

return rzucone;Jak będziesz wywoływał funkcję losuj, to przekażesz do jakiejś tablicy, np.:

Kod: [Zaznacz]

kosci_przeciwnika = losuj(int kosci , int wszystkie_kosci , int rzucone [] )W zasadzie ostatni parametr jest zbędny int rzucone [], bo nie ma sensu przekazywać tego, tylko w funkcji zadeklarujesz to.

Co do formatowania, zamień tabulatory na spacje i nie używaj Dev C++. Użyj np. Code::Blocks.

[DominikT]

znowu poległem ... chyba jestem jeszcze za słaby na pisanie takich rozbudowanych programów , ale Zrobiłem tak jak mi powiedziałeś , troche zmieniłem (np przed kosci_przeciwnika dałem int ) , ale mam błąd "expected primary-expression before 'int' , i expected ',' or ';' before int (wstawiłem tam przecinek to nie wiem o co chodzi :(  ) Tutaj kod  , a jeżeli dam int przed argumentami funkcji to też mam expected primary-expression :(

Kod: [Zaznacz]

#include <iostream>
#include <stdlib.h>
#include <time.h>
int wartosc = 0 ;
int wartoscWygranej ;
using namespace std;
int* losuj (int kosci , int wszystkie_kosci , int rzucone [] )
{  
   for (int i = 0 ; i < kosci ; i++)
   {  
      rzucone [i] = rand () % (wszystkie_kosci) + 1;

      cout << "Twoje kosci to: " << rzucone[i] << endl;
   }
   return rzucone;
}
void rzucKosci (int kosci , int wszystkie_kosci)

{
   char znak = 'T' ;
   srand (time(NULL));

   int rozmiar = wszystkie_kosci + 1;
   int rzucone[kosci];
   int tablica[rozmiar];

   for (int i = 0; i < rozmiar; i++)
   {
      tablica[i] = 0;
   }

   int kosci_przeciwnika = losuj( kosci ,  wszystkie_kosci , int rzucone [] )
   
   int wartosc = 0 ;
   
   for (int i = 0; i < rozmiar; i++)
   
   {
cout << tablica[i] << endl ;

    if (tablica[i] == 5)
    {
rozmiar += 5;
    cout << "Piatka !!! " << endl ;
  wartoscWygranej = 5;
break ;

}

[Adanos]

Popraw to:

Kod: [Zaznacz]

int kosci_przeciwnika = losuj( kosci ,  wszystkie_kosci , int rzucone [] )
na:

Kod: [Zaznacz]

int* kosci_przeciwnika = losuj( kosci ,  wszystkie_kosci , rzucone [] )


[DominikT]

Nadal błąd expected primary ... before ']' , pomijając braak średnika na końcu ( poprawiłem ) .
Co wogóle oznacza taki błąd ???

[Adanos]

Kod: [Zaznacz]

int* kosci_przeciwnika = losuj( kosci ,  wszystkie_kosci , rzucone);

Nadal błąd expected primary ... before ']'

Że oczekuje czegoś przed ']'. Ogólnie jest to błąd składniowy.

[DominikT]

Wymodziłem coś takiego , jednak informuje mnie o tym : [Warning] address of local variable 'rzucone' returned [enabled by default] . Co to oznacza? I wogóle gdy ten program uruchomie to mi się zawiesza konsola .

Kod: [Zaznacz]

#include <iostream>
#include <stdlib.h>
#include <time.h>
int wartosc = 0 ;
int wartoscWygranej ;
using namespace std;
int* losuj (int kosci , int wszystkie_kosci )
{ int  rozmiar = wszystkie_kosci +1 ;
int rzucone [rozmiar];
   for (int i = 0 ; i < kosci ; i++)
   {    
       rzucone [i] = rand () % (wszystkie_kosci) + 1;

      cout << "Twoje kosci to: " << rzucone[i] << endl;
   }    
   return rzucone;
}
void rzucKosci (int kosci , int wszystkie_kosci)

{
   char znak = 'T' ;
   srand (time(NULL));

   int rozmiar = wszystkie_kosci + 1;
   int rzucone[kosci];
   int tablica[rozmiar];

   for (int i = 0; i < rozmiar; i++)
   {
      tablica[i] = 0;
   }

   int * kosci_przeciwnika = losuj( kosci ,  wszystkie_kosci  );
   
   for (int i = 0; i < rozmiar; i++)
   {
   
   tablica[rzucone[i]] ++;
   cout << tablica [i];
  }

[DominikT]

Napisałem kod od nowa , w tamtym się już pogubiłem :(  . Użyłem funkcji , jest bardziej przyjrzysty . W kompilacji nie ma żadnego błedu , lecz konsola się poprostu zawiesza . Dlaczego ?

Kod: [Zaznacz]

#include <iostream>
#include <stdlib.h>
#include <time.h>

using namespace std;
int * losuj (int , int) ;
void rzucKosci (int kosci , int wszystkie_kosci) ;

int main()
{


   string START;

   cout << "**************************"<<endl;
   cout << "|      GRA W KOSCI       |"<<endl;
   cout << "|    DOMINIK TAMIOLLO    |"<< endl;
   cout << "**************************"<<endl;

   rzucKosci (5, 6);

   return 0;
}

void rzucKosci (int kosci , int wszystkie_kosci)
{
losuj (6 , 5  ) ;
int rozmiar = wszystkie_kosci +1 ;
int rzucone [kosci]  ;
int tablica [rozmiar] ;

for (int i = 0 ; i < rozmiar ; i++)
{
tablica [i] = 0 ;  
}







}
int * losuj (int kosci , int wszystkie_kosci  )
{  


int rozmiar = wszystkie_kosci  ;
int * tablica = new int [rozmiar] ;
int * rzucone = new int [rozmiar]  ;
    srand (time(NULL));

for (int i = 0 ; i <wszystkie_kosci  ; i++)
{
 *rzucone = rand () % kosci + 1  ;
 tablica [rzucone[i]] ++ ;
 cout << *rzucone << endl ;  


}

 return rzucone  ;





}

[Adanos]

Kod: [Zaznacz]

*rzucone = rand () % kosci + 1  ;
Co ty niby chcesz tu zrobić? Do adresu pamięci chcesz zapisać jakąś liczbę. Tego raczej nie chcesz. Chcesz zrobić coś takiego

Kod: [Zaznacz]

rzucone[i] = rand () % kosci + 1  ;
Poza tym to wypieprzyłeś połowę programu (sprawdzanie dwójek itp.)...

[DominikT]

No tak  Dzięki , wywaliłem bo byłem w trakcie pracy zaraz to dodam pamiętam o tym

Wszystko jest dobrze , kompiluje się , pokazuje mi liczby , jednak czasami to się myli , czasami wypisuje tylko jedną dwójke kiedy jest para . Czasami wypisuje dwie dwójki , nie moge się dopatrzeć logiki w działaniu tego sprawdzacza . Kod

Kod: [Zaznacz]

#include <iostream>
#include <stdlib.h>
#include <time.h>

using namespace std;
int * losuj (int , int) ;
void rzucKosci (int kosci , int wszystkie_kosci) ;

int main()
{


   string START;

   cout << "**************************"<<endl;
   cout << "|      GRA W KOSCI       |"<<endl;
   cout << "|    DOMINIK TAMIOLLO    |"<< endl;
   cout << "**************************"<<endl;

   rzucKosci (5, 6);

   return 0;
}

void rzucKosci (int kosci , int wszystkie_kosci)
{
losuj (6 , 5  ) ;
int rozmiar = kosci +1 ;
int rzucone [kosci]  ;
int tablica [rozmiar] ;

for (int i = 0 ; i < rozmiar ; i++)
{
tablica [i] = 0 ;  
}












}
int * losuj (int kosci , int wszystkie_kosci  )
{  


int rozmiar = wszystkie_kosci  ;
int * tablica = new int [rozmiar] ;
int * rzucone = new int [rozmiar]  ;
    srand (time(NULL));
   
for (int i = 0 ; i < rozmiar ; i++)
{
tablica [i] = 0 ;
}

for (int i = 0 ; i < wszystkie_kosci   ; ++i)
{
 rzucone[i] = rand () % (wszystkie_kosci) + 1  ;
 tablica [rzucone[i]] ++ ;
 cout << rzucone[i] << endl ;

 
 


}
cout << endl ;
for (int i = 0 ; i < rozmiar ; i++)
{
cout << tablica [i] << endl ;
}

 return rzucone  ;





}

[DominikT]

Z tym problemem sobie już poradziłem , jednak teraz nie wiem z jakiej przyczyny nie uwzględnia mi liczby 6 w sprawdzaniu do dwójek trójek itp . Naprawde nie moge znaleźć błędu wszędzie już próbowałem .

Kod :

Kod: [Zaznacz]

 rzucKosci (6, 5);

system ("pause");
   return 0;
}

void rzucKosci (int kosci , int wszystkie_kosci)
{
losuj (6 , 5  ) ;
int rozmiar = kosci +1 ;
int rzucone [kosci]  ;
int tablica [rozmiar] ;

}





int * losuj (int kosci , int wszystkie_kosci  )
{  
int strit1 ;
    int maly_strit = 1;
    int duzy_strit = 0;
int wartosc_kosci  = 0 ;
int rozmiar = wszystkie_kosci  ;
int * tablica = new int [kosci] ;
int * rzucone = new int [kosci]  ;
    srand (time(NULL));
   
for (int i = 0 ; i < rozmiar ; i++)
{
tablica [i] = 0 ;
}

for (int i = 0 ; i <  ; i++)
{
 rzucone[i] = rand () % (kosci ) + 1   ;
 tablica [rzucone[i]] ++ ;
 cout << rzucone[i] << endl ;

 
 


}
cout << endl ;
for (int i = 0 ; i < kosci   ; i++)
{
cout << tablica [i] << endl ;
}

for (int i = 0 ; i < rozmiar   ; i++)
{
if (tablica[i] == 5)
{
cout << " Piatka !!! " << endl ;
break ;
}

else if (tablica[i] == 4)
{
cout << " Kareta !!! " << endl ;
break ;
}
   else if (tablica[i] == 2)
{
wartosc_kosci += 2 ;

}
   else if (tablica[i] == 3)
{
wartosc_kosci += 3 ;

}

    }


if (wartosc_kosci == 5)
{
cout << "FULL !!! " << endl ;

}
   else if (wartosc_kosci == 4)
{
cout << "Para !!!" << endl ;

}
else if (wartosc_kosci == 2)
{
cout << "Dwojka !!!" << endl ;
   }
else if (wartosc_kosci == 3)
{
cout << " Trojka !!!" << endl ;
}
 

cout << endl ;

[DominikT]

Problem rozwiązany w pętlach sprawdzających dałem 7 obiegów pętli ,  bo pierwszy element sprawdzenia jest zawsze zerem .

Jest !!! Udało mi się zrobić dużego i małego strita !!! , będzie to działało również z moimi wartościami_wygranej , bo jeżeli to się spełni to wartość_wygranej = np.5 . Oto jak się z tym uporałem :

Kod: [Zaznacz]

if  (tablica [2] == 1 && tablica [3] == 1 && tablica [4] == 1 && tablica [5] == 1 && tablica [6] == 1)
{

cout << "Duzy Strit !!!" << endl ;  
   }

[DominikT]

Teraz żeby potem sobie ułatwić zmiane jakiejś kości chciałem zrobić to w funkcjach , lecz mam błąd linkera "undefined refference to 'sprawdz' , nawet nie chodzi o poprawność kodu co do jego działania , bo sam wiem że jest tam kilka błędów , ale mi chodzi żeby chociaż to przeszło i się skompilowało a potem sobie to poprawie , jednak no nie wiem dlaczego tak się dzieje ?
Kod :
Tam wyżej mam deklaracje funkcji :

Kod: [Zaznacz]

sprawdz (int , int , int ) ;

Kod: [Zaznacz]



int * losuj (int kosci , int wszystkie_kosci  )
{





int rozmiar = wszystkie_kosci + 2 ;
int * tablica = new int [rozmiar] ;
int * rzucone = new int [kosci]  ;
int strit [kosci] ;
int wartosc_strita = 0  ;
    srand (time(NULL));
   
for (int i = 0 ; i < rozmiar ; i++)
{
tablica = 0 ;
}

for (int i = 0 ; i < wszystkie_kosci  ; i++)
{
  rzucone = rand () % (kosci) + 1    ;
  tablica [rzucone] ++ ;
  cout << endl ;
  cout << rzucone << endl ;
 

cout << endl ;

sprawdz (rzucone , tablica , rozmiar  );
}








}

int sprawdz (int rzuconekosci [] , int tablica [] , int rozmiar )
{

int punkty = 0 ;
int wartosc_kosci  = 0 ;

for (int i = 0 ; i < rozmiar   ; i++)
{
if (tablica == 5)
{
wartosc_kosci += 10;

}

else if (tablica == 4)
{
  wartosc_kosci +=20 ;
}
    else if (tablica == 2)
{
wartosc_kosci += 2 ;

}
    else if (tablica == 3)
{
wartosc_kosci += 3 ;

}

    }

if (wartosc_kosci == 5)
{
cout << "FULL !!! " << endl ;
punkty += (rzuconekosci [0] + rzuconekosci [1] + rzuconekosci [2] + rzuconekosci [3] + rzuconekosci [4] ) + 10  ;

}
    else if (wartosc_kosci == 10)
{
cout << "Poker !!!" << endl ;
punkty += (rzuconekosci [0] + rzuconekosci [1] + rzuconekosci [2] + rzuconekosci [3] + rzuconekosci [4] ) +50 ;

}
    else if (wartosc_kosci == 20)
{
cout << "Kareta !!!" << endl ;
punkty += (rzuconekosci [0] + rzuconekosci [1] + rzuconekosci [2] + rzuconekosci [3] + rzuconekosci [4] ) + 20 ;

}
else if (wartosc_kosci == 2)
{
cout << "Dwojka !!!" << endl ;
punkty += 5 ;
    }
else if (wartosc_kosci == 3)
{
cout <<  "Trojka !!!" << endl ;
punkty += 15 ;
}
    else if (wartosc_kosci == 4)
{
cout << "Para !!!" << endl ;
punkty += 10 ;

}

cout << endl ;


if  (tablica [2] == 1 && tablica [3] == 1 && tablica [4] == 1 && tablica [5] == 1 && tablica [6] == 1)
{
punkty += 20 ;
cout << "Duzy Strit !!!" << endl ;  
    }
if  (tablica [1] == 1 && tablica [2] == 1 && tablica [3] == 1 && tablica [4] == 1 && tablica [5] == 1)
{
punkty += 15 ;
cout << "Maly Strit  !!!" << endl ;  
    }
cout << endl ;



cout << endl ;

return wartosc_kosci ;
return punkty ;

}

[Adanos]

Popełniasz ciągle te same błędy. Nie możesz tablicy przypisać wartości! Możesz jedynie jakiemuś elementowi przypisać wartość. Popraw to.

Kod: [Zaznacz]

return wartosc_kosci ;
return punkty ; W C++ funkcja nie może zwracać dwóch wartości, więc tylko jeden return się wykona.

[DominikT]

No tak , wiem że nie można tylko że jak wpisywałem tablica , rzucone to był ten sam błąd , wsumie zapomniałem o tym ale to nie jest wina tego tylko czegoś innego . Więc co mam wpisać tam ?