Нужно улучшить код

#include <iostream>
 
using namespace std;
 
 
int main()
{
    char str[100];
    int alf[26];
    int j = 0;
    int probel = 0;
 
    cout<<"Vedi"<<endl;
    cin.getline(str,100);
 
    for(int i = 0;i < str[i];i++)
    {
        if(str[i] >= 'a' && str[i] <= 'z' || str[i] >= 'A' && str[i] <= 'Z')
        {
            alf[i] = 0;
            j = str[i];
 
            if(j == str[i])
            {
              alf[i]++;
            }
 
        }
 
            if(str[i] == ' ')
            {
                probel++;
            }
 
            cout<<str[i]<<" = "<<alf[i]<<endl;
 
 
        if(probel != 0)
        {
            cout<<"Probel"<<" = "<<probel<<endl;
        }
 
    }
}

#include <iostream>
#include <map>
#include <string>
#include <algorithm>
using namespace std;
int main()
{
    setlocale(LC_ALL,"Rus");
    cout<<"ведите строку:";
    string txt;
    getline(cin,txt);
    transform(txt.begin(),txt.end(),txt.begin(),toupper); // преобразование к верхнему регистру
    map<char,int> mp1; // создаем контейнер уникальных пар char->int
    for(unsigned int i=0;i<txt.size();i++)
        mp1[txt[i]]++; // считаем количество каждой букв
    multimap<int,char> mp2;  // создаем контейнер  пар (int,char) Отсортированных по int
    for(map<char,int>::iterator p=mp1.begin();p!=mp1.end();++p) // Проходим по первому контейнеру 
    {
        pair<int,char> pic;
        pic.second=p->first;
        pic.first=p->second;
        mp2.insert(pic);   // записываем во второй контейнер
    }
    string result;
    for(multimap<int,char>::reverse_iterator p=mp2.rbegin();p!=mp2.rend();++p) // Проходим по второму контейнеру  в обратном порядке
                if(p->second!=' ')
           result+=p->second; // переписываем символы в результирующую строку
 
    cout <<result<< endl;
    system("pause");
    return 0;
}

#include <iostream>
#include <string>
#include <algorithm> 
using namespace std;
 
int main()
{
    cout<<"Enter string:"<<endl;
    string str;
    getline(cin,str);
    transform(str.begin(),str.end(),str.begin(),toupper); // преобразование к верхнему регистру
    
    int alf[26]={0};
    char letters[26];
    for(int i=0;i<26;i++)
        letters[i]='A'+i; // записываем весь алфавит
    int probel=0;
    for(int i = 0;i<str.size();i++)
    {
        if(str[i] >= 'A' && str[i] <= 'Z')
        {
            alf[str[i]-'A']++; //считаем буквы
        }
        if(str[i] == ' ')
            probel++;
    }
    if(probel)
        cout<<"Probel"<<" = "<<probel<<endl;
    // сортировка
    for(int i=0;i<26;i++)
        for(int j=1;j<26;j++)
        {
            if(alf[j]>alf[j-1])
            {
                int t=alf[j]; // сортируем встречаемость
                alf[j]=alf[j-1];
                alf[j-1]=t;
                char c=letters[j]; // одновременно переставляем буквы алфавита
                letters[j]=letters[j-1];
                letters[j-1]=c;
            }
        }
        for(int i=0;alf[i]>0;i++)
            cout<<letters[i];
        cout<<endl;
        system("pause");
        return 0;
}

       for(int i=0;alf[i]>0;i++)
            cout<<letters[i]<<" - "<<alf[i]<<endl;