初学者不知道哪里出错了，输入三个整数按照从大到小顺序排列总是不对

菜鸟不知道哪里出错了，输入三个整数按照从大到小顺序排列总是不对
#include <iostream>
using namespace std;
int main()
{
void sort(int x,int y,int z);
int x,y,z;
cout<<"输入三个整数按照从大到小顺序排列"<<endl;
cin>>x>>y>>z;
sort(x,y,z);
system("pause");
}

void sort(int x,int y,int z)
{
int temp;
if(x>y) {temp=x;x=y;temp=y;}
if(z>y) cout <<z<<','<<y<<','<<x<<endl;
else if(z>x) cout<<y<<','<<z<<','<<x<<endl;                                                                    
else cout<<y<<','<<x<<','<<z<<endl;
}
------解决思路----------------------
if(x>y) {temp=x;x=y;temp=y;}
红色的写错了。

这种问题，自己断点调试就能发现了。
------解决思路----------------------
if(x>y) {temp=x;x=y;temp=y;}
改成
if(x>y) {temp=x;x=y;y=temp;}

------解决思路----------------------
要自己学会如何检查错误
------解决思路----------------------
代码功能归根结底不是别人帮自己看或讲解或注释出来的；而是被自己静下心来花足够长的时间和精力亲自动手单步或设断点或对执行到某步获得的中间结果显示或写到日志文件中一步一步分析出来的。
提醒：再牛×的老师也无法代替学生自己领悟和上厕所！
单步调试和设断点调试（VS IDE中编译连接通过以后，按F10或F11键单步执行，按Shift+F11退出当前函数；在某行按F9设断点后按F5执行停在该断点处。）是程序员必须掌握的技能之一。