找不出哪错了，求解按1的个数排序

输入样例:
3
10011111
00001101
1010101
输出样例:
00001101
1010101
10011111

#include<iostream>   
//#include<fstream>   
#include<string>   
#include<vector>   
#include<algorithm>   
  
using namespace std;   
  
bool cmp(const string &s1,const string &s2)   
{   
    //彻底修改排序规则，只按1的数量排序   
    //若1的数量相等，则按照出现顺序排序   
    //否者，按ASCII排序   
    //只能用“<”或“>"   
    int c1= count(s1.begin(),s1.end(),'1');   
    int c2= count(s2.begin(),s2.end(),'1');   
    return c1!=c2?c1<c2:c1<c2; //Thinking  
}   
  
int main(int argc,char *argv[])   
{   
    //ifstream cin("aaa.txt");   
    vector<string>vstr;   
    string str;
    int n;
    cin>>n;
    for(int i=1;i<=n;i++)
   // while(cin>>str)   
    {   
        cin>>str;
        vstr.push_back(str);   
    }   
    sort(vstr.begin(),vstr.end(),cmp);   
    vector<string>::iterator it;   
    for(it= vstr.begin();it<vstr.end();it++)   
      cout<<*it<<endl;   
    return 0;   
}  

17行条件表达式写得不对，1的个数一样的时候按照字符串大小，所以应该是：

return c1!=c2?c1<c2:s1<s2;