#include <iostream>
#include <conio.h>
using namespace std;

int main(void){

char * str="ABC-129";
int i = 0;

while(*(str+i)){
    if((*(str+i))>='A' && (*(str+i))<='Z')
    *(str+i)='X';
     i++;
}
cout << str;
getch();
return 0;
}

ich hebe die Klammerausdrücke immer durch Leerzeichen deutlich in ihrer Funktion hervor,
so

if(  (*(str+i))>='A' && (*(str+i))<='Z'  )

kommt man nicht so leicht durcheinander.

Wenn du Stringliterale im Quelltext erzeugst,
also

char * str="ABC-129";

Dann ist das per C Standard so, dass man den String nicht verändern (beschreiben) darf.
Deshalb schreibt man besser immer

const char * str="ABC-129";

Und wenn er verändert werden soll, dann macht man das als Array.

char str[] = {"ABC-129"};

//char* str = "ABC-129";
  char str[] = "ABC-129";

In deinem Beispiel versuchst du ein konstantes String-Literal zu ändern.
Das darfst du nicht. Du musst, und willst sicher auch, einen String bearbeiten.

du versuchst in eine string-konstante zu schreiben. wenn du willst, daß str nicht auf eine konstante zeigt, dann mach es so:

char str[]="ABC-129";

dann legt der compiler speicher fur ein char-array an und initialisiert ihn mit der stringkonstante.

EDIT: mist! schon wieder letzter

*(str+i)

kann man auch übersichtlicher

str[i]

schreiben.

danke

Falls du auch in der Definition auf [] verzichtest, kannst du es auch
über malloc/free machen.
(Normalerweise nimmt man in C++ new/delete, aber dann brauchst du ja wieder das verbotene [] bei delete)
Wo da der Sinn liegen soll, naja

const char * tmp="ABC-129";
char* str = (char*) malloc(strlen(tmp)+1); //+1 wegen /0
strcpy(str, tmp);
... 
free(str);

grüßle und danke