#include <iostream>
#include <conio.h>
using namespace std;
int main()
{
int x;
char A,B,C,D;
cout<<"please input an integer between 0 and 35"<<endl;
for(x=0;x<=9;)
{
cin>>x;
cout<<x;
if(x>=10)
{
cout<<static_cast<char>('A'+ (x-10));
Is it good? Well, it's had to say, is it isn't clear what it is supposed to do.
There are unused variables char A,B,C,D;
The for loop looks a bit odd. I think in this case a while or do-while would be more legible.
What happens if the user inputs something outside the requested range?
I won't say this is "better" as I'm not sure how the program was intended to work.