Rodzaj String/AnsiString w pętli

[lukagrom]

Robiłem sobie prostą szachownicę w komponencie StringGrid, czyli 64 pól o indeksach od A1 do H8. Sprawa trywialna. Założyłem, że zrobię tablicę podwójną tablicę stringów 8 na 8, i wartość będzie generowana z połączenia tablicy char o wartościach od 'A' do 'Z', lub String o wartościach od "A" do "Z" do tego połączenie wartości int z zakresu: 1-8, połączone funkcją IntToStr(tab[8 elementow])

Czyli wyszło coś takiego:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

int main_Int[8],i,k;
 AnsiString main_Char[8]={"A","B","C","D","E","F","G","H"},j;
 AnsiString chess_Table[8][8];
 for(i=0;i<8;i++)
    main_Int [ i] = i+1;
 for(i=0;i<8;i++)
 {
  for(k=0;k<8;k++)
  {
   chess_Table[ i ][ k ]=(main_Char[ i ]+" "+IntToStr(main_Int[ k ]));
   StringGrid1->Cells[ k ] [ i ]=chess_Table[ i ][ k ];
  }
 }

 

Chodzi i kompiluję się jak najbardziej. Tyle, że zapis:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

AnsiString main_Char[8]={"A","B","C","D","E","F","G","H"},j;

został wymuszony, pierwotnie chciałem wykorzystywać pętle dla Stringa, czyli coś takiego:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

 String main_St[8],k;
  char main_Ch[8],j;
  int main_Int[8];
   for(k="A",i=0;k<"I";k++,i++)
  {
          main_St[ i ]=k;
          main_Int[ i ]=i+1;
  }
 

Jednak kompilator pokazuje błąd jako - bcc32 Error mCh2.cpp(29): E2096 Illegal structure operation

zrobienie analogicznej operacji, dla typu char

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

 for(j='A',i=0;j<'I';j++,i++)
  {
          main_Ch[ i ]=j;
          main_Int[ i ]=i+1;
  }
 

już błędem nie jest. Dlatego mam pytanie, czy istnieje jakiś mechanizm dla typów String/AnsiString/UnicodeString, który pozwoli zliczać wartości "a..z A..Z" np za pomocą pętli for?
Spację przy tablicach, robiłem celowo z powodu formatowania (znaczniki html się włączały)

[Mironas]

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

  String znaki = "ABCDEFGH";
  char z;
  for (int i=1 ; i<=znaki.Length() ; i++)
  {
    z = znaki[i];
    /* ... */
  }
 

[lukagrom]

Dzięki, zawsze to jakieś ułatwienie. Próbowałem, to jeszcze zrobić tak dla treningu za pomocą funkcji w C z użyciem odrzucanej itoa, ale tutaj już stanąłem w miejscu (co Stringi to Stringi)

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

  char chess_Table[64][64];
    int tab_Int[8];
    char tab_Char[8];
    char buf_Itoa[8][8];
    int i,k,l;
    char j;
    for(j='A',i=0;j<'I';j++,i++)
    {
        tab_Char[i]=j;
        tab_Int[i]=i+1;
        itoa(tab_Int[i],buf_Itoa[i],10);
    }
    l=0;
    for(i=0;i<8;i++)
    {
        for(k=0;k<8;k++)
        {
         chess_Table[l][0]=tab_Char[i];
         strcat(chess_Table[l],buf_Itoa[k]);
         l++;
        }
    }
 

[Mironas]

A co tutaj chcesz zrobić i co Ci nie działa?

[lukagrom]

chess_Table ma dostać 64 zmiennych char[] od A1 do H8. Dostaje coś innego, tj same cyfry, które po pzekształceniu itoa są naturalnie łańcuchami char. Czyli zamiast A1,A2 etc... 1,2,3.. W sumie nie wiem, gdzie leży błąd.

[Mironas]

Nie analizowałem całości, ale taka pętla:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

for(j='A',i=0;j<'I';j++,i++)

nie wykona się dla wszystkich pól.
Zrób dwie oddzielne pętla dla i oraz j, jedna zagnieżdżona w drugiej.

[polymorphism]

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

        tab_Int[i]=i+1;
        itoa(tab_Int[i],buf_Itoa[i],10);

Po co tablica tab_Int? Nie wystarczy tak:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

itoa(i + 1, buf_Itoa[i], 10);

?

Chociaż całość można uprościć do:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

for (int i = 0, j = 0; j < 8; ++j) {
    for (int k = 0; k < 8; ++k, ++i) {
        chess_Table[i][0] = "ABCDEFGH"[j];
        chess_Table[i][1] = "12345678"[k];
        chess_Table[i][2] = 0;
    }
}

[lukagrom]

Tu jeszcze wariant, gdzie String zlicza sobie wartosci dzieki petli for, a nie przypisaniu szeregow wartosci. W sumie najprostrzy i najwygodniejszy.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

UnicodeString string_Board[8],chess_Table[8][8];
 int int_Board[8];
 wchar_t f_Char='A';
 for(int i=0;i<8;i++)
 {
         string_Board[i]=UnicodeString(f_Char);
         int_Board[i]=i+1;
         ++f_Char;
 }
 for(int i=0;i<8;i++)
 {
         for(int j=0;j<8;j++)
         {
                 chess_Table[i][j]=string_Board[i]+" "+IntToStr(int_Board[j]);
                 StringGrid1->Cells[j][i]=chess_Table[i][j];

         }
 }
 

[polymorphism]

W sumie najprostrzy i najwygodniejszy.

Polemizowałbym mocno.

a nie przypisaniu szeregow wartosci.

Jeśli wartości jest niewiele, to tablice z gotowymi wartościami są lepsze, bo zwiększają czytelność kodu (redukując go do niezbędnego minimum).

Jeszcze raz zadam to pytanie: po co ta tablica intów int_Board?

[lukagrom]

Rzeczywiscie tablica int-ow w tej sytuacji calkowicie zbedna. No to jeszcze mozna to zmniejszyc, wszystko "tablicujac" i przypisujac cyfry do string-ow.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

 UnicodeString chess_Table[8][8],string_Board[8]={"A","B","C","D",
  "E","F","G","H"},int_Board[8]={"1","2","3","5","5","6","7","8"};
  for(int i=0;i<8;i++)
  {
          for(int j=0;j<8;j++)
          {
                  chess_Table[i][j]=string_Board[i]+int_Board[j];
                  StringGrid1->Cells[j][i]=chess_Table[i][j];

          }
  }
 

[polymorphism]

chess_Table też wygląda na zbędną

[lukagrom]

Przy okazji prosta funkcja zwracajaca ciag String na podstawie zakresu ograniczonego dwoma wartosciami char/wchar_t. (przewaga nad wypisywaniem danych z deklarowanych tablic)

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

UnicodeString __fastcall TForm1::write_Text(wchar_t a, wchar_t b)
{
        UnicodeString result;
        result="";
        if(a<b)
         for(char i=a;i<b+1;i++)
          result+=UnicodeString(i);
        else
        for(char i=b;i<a+1;i++)
         result+=UnicodeString(i);
        return result;
}
 

[polymorphism]

Dlaczego w pętli masz char, skoro parametry są wchar_t?

Trzymaj się zasady DRY:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

UnicodeString __fastcall TForm1::write_Text(wchar_t a, wchar_t b)
{
    UnicodeString result;

    if (a > b) std::swap(a, b);
    for (wchar_t i = a; i <= b; ++i) result += i;
    return result;
}

przewaga nad wypisywaniem danych z deklarowanych tablic

IMO ta funkcja może mieć praktyczne zastosowanie jedynie w przedziałach ASCII [a-z], [A-Z] i [0-9], ale czy do tego trzeba aż oddzielnej funkcji..?

[lukagrom]

ale czy do tego trzeba aż oddzielnej funkcji..?

Zawsze tam jakaś funkcja ze skromnego "dorobku" w tych czasach końca na chwilę pozostanie. Akurat mój stary podręcznik miał zgięcie kartki na rozdziale z tablicami "Variantowymi", to takową do tego wątku postanowiłem dołączyć.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

 String St[8]={"A","B","C","D","E","F","G","H"};
 String In[8]={"1","2","3","4","5","6","7","8"};
 typedef String sum_Array[8][8];
 int min,max;
 Variant SUM=VarArrayCreate(OPENARRAY(int,(1,8,1,8)),varString);
 sum_Array *chess_Board=(sum_Array *)VarArrayLock(SUM);
 try
 {
         min=VarArrayLowBound(SUM,VarArrayDimCount(SUM));
         max=VarArrayHighBound(SUM,VarArrayDimCount(SUM));
         for(int i=min;i<max;i++)
         {
                 for(int j=min;j<max;j++)
                 {
                         (*chess_Board)  [i ] [j]=St[ i] +In[j];
                         StringGrid1->Cells[ i ] [j ]=VarToStr((*chess_Board)[ j ][ i ]);
                 }
         }
 }
 catch(...)
 {
         VarArrayUnlock(SUM);
         throw;
 }
 VarArrayUnlock(SUM);
 

Chyba jest dobrze, bo to przerobiony programik książkowy z typu obliczania macierzy. Kompilacja bez warningów, przy czym jako, że testowane na dwóch kompilatorach, to na jednym wysypuje się z powodu błędów z .dll, na drugim dostaje zawiechy i pojawia się komunikat - Error creating variant array.
Prawdopodobnie nie chce pracować ze "Stringami" (z każdym innym typem jak varByte,varWord,varDate etc współpracuje bez zawiechy).
W podręczniku jest nawet adnotacja - varString - niekompatybilny z COM

[polymorphism]

Dajże spokój, komplikujesz prostą rzecz.