这个解法为什么不行？

#include <iostream>
using namespace std;

int main()
{
    char* a = new char[1000];
    int j = 0;
    int Letters = 0, Digits = 0, Others = 0;
	cin.getline(a, '?');
    for(j = 0; a[j] != '?'; j++){
        if((a[j] >= 'a' && a[j] <= 'z') || (a[j] >= 'A' && a[j] <= 'Z'))
            Letters++;
        else if(a[j] >= '0' && a[j] <= '9')
            Digits++;
        else
            Others++;
    }
    cout << "Letters=" << Letters << endl;
    cout << "Digits=" << Digits << endl;
    cout << "Others=" << Others << endl;
    
    delete []a;
    return 0;
}