Czy można w ten sposób skrócić kod?

Question

Hej,

zaczęłam dziś przerabiać książkę C++ Primer, 5th ed. i jest tam zadanie 1.19, w którym chodzi o to, żeby użytkownik wpisał dwie liczby różniące się od siebie i następnie program ma wypisać uszeregowane od najmniejszej liczby, które znajdują się między tymi wypisanymi liczbami.

Stworzyłam taki kod jak poniżej i zastanawiam się, czy można, tak jak to zrobiłam, pozostawić "body" pętli if puste? Inaczej musiałabym wpisywać w nie tą pętlę while, co dałoby aż 2 jej wywołania w paru linijkach kodu. Program działa, przynajmniej u mnie. 

#include <iostream>

int main()
{
	int small = 0, big = 0;
	std::cout << "Please input 2 integers: ";
	std::cin >> small >> big;

	if (small < big) {
	
	}
	else {
		int temp = small;
		small = big;
		big = temp;
	}
		
	while (small <= big)
	{
		std::cout << small << std::endl;
		++small;
	}

		return 0;
}

 

Comments

a po co ten pusty blok if? Tego if można zapisac jako:

if (small >= big) {
    int temp = small;
    small = big;
    big = temp;
}

 

---

@Tomek Sochacki, faktycznie, wtedy cały kod diametralnie się upraszcza 

---

btw możesz tutaj użyć std::swap, lub z racji, że to liczby xor swap

---

Jeszcze nie znam tego sposobu. Pokombinuję.

---

[ edit ]

#include <iostream>

int main() {
   ...
   if (small > big)
       std::swap(small, big);
   ...
}

 

---

@rain.deer, pamiętaj, że if to nie pętla ;) Możesz powiedzieć "warunek" lub wyrażenie warunkowe, ale nie pętla

---

@VBService,

a te <bits/stdc++.h> to za jakie grzechy?

---

@NewEraOfPeace, ach słusznie, pisałem coś innego w między czasie i zostało w edytorze i bezmyślnie zaznaczyłem przy kopiowaniu. 

---

@Milesq, to był taki niefortunny skrót myślowy. Faktycznie przecież w "if" nic się nie zapętla.

Answer 1

A nie lepiej zapytać się odwrotnie?

if (small > big) {
    int temp = small;
    small = big;
    big = temp;
 }

 

Comments

@SzkolnyAdmin, faktycznie lepiej.

Answer 2

 [ edit ]

@mokrowski

a ^= b ^= a ^= b;

...

    if (small > big) //swapping using bitwise operator
        small ^= big ^= small ^= big;

    while (small < big - 1)
        std::cout << ++small << std::endl;

...

...

    if (small > big)
        std::swap(small, big);

    while (small < big - 1)
        std::cout << ++small << std::endl;

...

 

Comments

Twoje rozwiązanie rewelacyjnie skraca kod. Dziękuję 

Answer 3

A żeś się zafiksował na tym swapowaniu... Po co?

#include <iostream>
#include <algorithm>
 
int main()
{
    int a = 0, b = 0;
    std::cout << "Please input 2 integers: ";
    std::cin >> a >> b;
    for(auto i = std::min(a, b) + 1; i < std::max(a, b); ++i) {
	    std::cout << i << '\n';
    }
}

Poza tym...

a ^= b ^= a ^= b;

.. ma jeszcze krótszy kod zmiany wartości zmiennych. Niemniej jednak nie generuje to wydajnego kodu asemblera na większości platform.