[c++] nie działa if else

[shreder221]

Dobry wieczór
Mógłby mi ktoś powiedzieć dlaczego w przypadku niespełnienia warunku tab>tab[i+1] else nie kończy pętli do while?

Ukryta treść:    

Kod: Zaznacz cały

do{
                for(int i=0;i<rozmiar-1;i++){

                        if(tab[i]>tab[i+1]){
                        	cout<<"pokazuje i,i+1   "<<tab[i] <<endl<<tab[i+1]<<endl;
                        		cout<<"pokazuje i,i+1   "<<tab[i] <<endl<<tab[i+1]<<endl;
                        	zamiana(i,i+1,tab);
                        	cout<<"pokazuje i,i+1 po zamianie   " <<tab[i] <<endl<<tab[i+1]<<endl;
                        	zamienione=true;
							break;
						}
                        else zamienione=false;
                }
       }while(zamienione=true);

cały program

Ukryta treść:    

Kod: Zaznacz cały

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

using namespace std;
//zamiana************************
void zamiana(int p1,int p2 ,double tablica[]){
        double k1=tablica[p1],k2=tablica[p2];
        tablica[p1]=k2;
        tablica[p2]=k1;
        return;}
//buble************************
void buble(double tab[],int rozmiar){
        bool zamienione ;
        do{


                for(int i=0;i<rozmiar-1;i++){
                		
                        if(tab[i]>tab[i+1]){
                        	cout<<"pokazuje i,i+1   "<<tab[i] <<endl<<tab[i+1]<<endl;
                        		cout<<"pokazuje i,i+1   "<<tab[i] <<endl<<tab[i+1]<<endl;
                        	zamiana(i,i+1,tab);
                        	cout<<"pokazuje i,i+1 po zamianie   " <<tab[i] <<endl<<tab[i+1]<<endl;
                        	zamienione=true;
							break;
						}
                        else zamienione=false;
                        
                }



       }while(zamienione=true);
//for(int i=0;i>20;i++){
 //               cout<<tab[i]<<endl;
 //       }
        return;}
        
    int main(){
        double t[3];
        srand(time(0));
        cout<<"wylosowane liczby  
";
        for(int i=0;i<3;i++){
                t[i]=rand()/(RAND_MAX+1.);
                cout<<t[i]<<endl;
        }

        cout<<endl<<endl<<endl;
        buble(t,3);
        cout<<"uporządkowane liczby  
";
        for(int i=0;i>3;i++){
                cout<<t[i]<<endl;
        }

}
//***************************************

//edit wersja alternatywna pytanie.2
Co mi się wydaje : Pętla do while jest wykonywana do momentu w którym na koniec pętli zamienione = false. Żeby było to możliwe na początku każdej kolejki ustawiam zamienione=false a polecenie ewentualnej zmiany jest w petli for
pętla for oznacza jeżeli choć jeden element tablicy jest większy od kolejnego zamienia je miejscami i zamienione=true co skutkuje powtórzeniem do while.

W pewnym momencie tablica jest uporządkowana zamienione ma wartość null ale pętla się nie kończy dlaczego?

Ukryta treść:    

Kod: Zaznacz cały

 do{
				 zamienione=false
                for(int i=0;i<rozmiar-1;i++){
                        if(tab[i]>tab[i+1]){
                        	zamiana(i,i+1,tab);
                        	zamienione=true;
						}  
                }

	cout<<zamienione;
       }while(zamienione=true);

[Gouranga]

klamry się nie zgadzają

[shreder221]

liczyłem kilka razy i nie widzę żadnych brakujących nawiasów

[Afish]

Kod: Zaznacz cały

while(zamienione=true);

Brakuje drugiego znaku =.

A lepiej zapisać to jako

Kod: Zaznacz cały

while(zamienione)

[shreder221]

super działa w 90% programów robię takie idiotyczne błędy których nie mogę potem znaleść przez kilka h ;(

co do wersji skróconej zawsze zapominam czy zapis ten oznacza 1 czy 0 więc go unikam

Dzięki za pomoc

[mortan517]

To dawaj stałą po lewej stronie, gdy zechcesz napisać kod typu:

Kod: Zaznacz cały

while(true=zamienione)

to kompilator zaprotestuje.

[miki999]

Wydaje mi się, że większość kompilatorów C++ wywala minimum warninga. Nie ignoruj ostrzeżeń a w przyszłości będziesz miał mniej problemów.

To dawaj stałą po lewej stronie

Polecam !

[shreder221]

oO żadnych ostrzeżeń nie było do zignorowania :p a nawet gdyby były nie potrafiłbym ich ominąć inaczej niż naprawiając błędy a używam Dev c++ A objawem błędu była nieskończona pętla po wykonaniu for

To dawaj stałą po lewej stronie, gdy zechcesz napisać kod typu:

Kod: Zaznacz cały

while(true=zamienione)

to kompilator zaprotestuje.

chyba sprawa zmęczenia ale nie rozumiem co masz na myśli. Najpierw piszesz żeby pisać żeby zamienić kolejność stała<--> zmienna a następnie żeby tego nie robić bo będzie błąd chyba że zapomniałeś o "nie" . Jeśli tak to wtedy nie trzeba pisać "=="?

[mortan517]

Usystematyzujmy, jeżeli chcesz porównać to zawsze piszesz podwójny znak "równa się". Ale załóżmy teraz, że się pomyliłeś i dajesz pojedynczy znak oznaczający przypisanie. I teraz jak po lewej stronie dawałbyś zawsze stałą, to kod się nie skompiluje, a jak dasz zmienną to niestety kompilator to puści, ale powinien zasygnalizować to jakimś ostrzeżeniem. Czyli generalnie po lewej stronie warto dawać stałe, bo jak się pomylisz to kompilacja nie przejdzie.

[Rozbitek]

Jeżeli każesz mu wyskoczyć z pętli "sztucznie", to prawdopodobnie program jest do przebudowy.