c_str(), które kiedyś działało

[mirekpl6]

Witam.
Mam problem z c_str(). Kod mojego programu, był tworzony bodajże pod C++ Builder 2007. Program działał prawidłowo. Chce ten sam kod uruchomić na obecnie najnowszej wersji środowiska: C++Builder 10.1 Berlin.

Kod: Zaznacz cały

#include <vcl.h>
#pragma hdrstop


#include "Unit3.h"
#include "Unit2.h"
#include <math.h>
#include <fstream.h>
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm3 *Form3;


Ta linijka:
1:

Kod: Zaznacz cały

MessageBox(Handle,("Twój wynik został zapisany do katalogu " + Label10->Caption).c_str(), "", MB_OK | MB_ICONINFORMATION);

Wyrzuca takie błędy:

[bcc32 Error] Unit3.cpp(144): E2034 Cannot convert 'wchar_t *' to 'const char *' [bcc32 Error] Unit3.cpp(144): E2342 Type mismatch in parameter 'lpText' (wanted 'const char *', got 'wchar_t *')

Gdy to zakomentuję to mam problem z kodem(kompilator nie wyrzuca błędów,ale kod działa niepoprawnie):

2:
Te poniższe 2 linijki działają prawidłowo...,

Kod: Zaznacz cały


ofstream outfile;
outfile.open(("Wyniki\\"+Label10->Caption+"\\"+i+".txt").c_str());


...ale kolejne nie:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

 
 outfile <<("Poziom trudności: "+Label12->Caption).c_str();
 outfile<<endl;
 outfile <<("Ilość poprawnych odpowiedzi w procentach: "+ Label4->Caption+" %").c_str();
 outfile<<endl;
 outfile <<("Ilość wykonanych działań: "+Label17->Caption).c_str() ;
 outfile<<endl;
 

Wynik powyższych linijek:

245acbc
245a15c
24eb214

Gdy w labelu jest liczba poradziłem sobie(obszedłem problem), korzystając z funkcji StrToInt, ale z napisem nie mogę sobie poradzić.


Bardzo proszę o pomoc. Kombinuje i kombinuje i nadal nie działa. Proszę o podanie kodu.

[Cyfrowy Baron]

Musisz się przestawić na UnicodeString, gdyż ten typ jest typem domyślnym. Czyli przed każdym łańcuchem znaków dajesz L np:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

MessageBox(Handle,(L"Twój wynik został zapisany do katalogu " + Label10->Caption).c_str(), L"", MB_OK | MB_ICONINFORMATION);

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

outfile.open( ( L"Wyniki\\" + Label3->Caption + L"\\" + (String)i + L".txt" ).c_str());

[mirekpl6]

Niestety poniższy kod generuje te same błędy co wcześniej

Kod: Zaznacz cały

MessageBox(Handle,(L"Twój wynik został zapisany do katalogu " + Label10->Caption).c_str(), L"", MB_OK | MB_ICONINFORMATION);

Po umieszczeniu po L spacji wyskakuje błąd:

[bcc32 Error] Unit3.cpp(159): E2451 Undefined symbol 'L' [bcc32 Error] Unit3.cpp(159): E2293 ) expected

A ten kod kompiluje się,ale nie działa prawidłowo

Kod: Zaznacz cały

outfile.open( ( L"Wyniki\\" + Label3->Caption + L"\\" + (String)i + L".txt" ).c_str());

[Cyfrowy Baron]

Najwyraźniej w C++Builder 10.1 Berlin jest trochę inaczej. Zamiast L użyj _T lub _TEXT, lub TEXT.

[mirekpl6]

Dzięki za rady, ale nie działają

Trzeba załączyć jakąś bibliotekę itd?

Może jest inny sposób,dłuższy,ale działający?

[polymorphism]

Najwyraźniej w C++Builder 10.1 Berlin jest trochę inaczej.

Nie no, to L to część języka C++, więc musi działać wszędzie tam, gdzie mowa o C++.

@mirekpl6, w treści błędu jest mowa o niemożności konwersji z wchar_t* na char*, więc pewnie chodzi o to, że c_str zwraca unikod, a MessageBox oczekuje starego ANSI/ASCII/char*/whatever. Spróbuj z unikodową wersją MessageBox:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

MessageBoxW(Handle, (L"Twój wynik został zapisany do katalogu " + Label10->Caption).c_str(), L"", MB_OK | MB_ICONINFORMATION);

Swoją drogą powinieneś używać VCL-owską implementację MessageBox, wtedy nie miałbyś takich problemów.

[Cyfrowy Baron]

Odnośnie tego co napisał polymorphism, to domyślnie wszystkie funkcje począwszy od wersji XE oczekują argumentów typu Unicode, więc dodawanie tego W nie jest konieczne.

Swoją drogą powinieneś używać VCL-owską implementację MessageBox, wtedy nie miałbyś takich problemów.

To nie rozwiąże problemu całkowicie, gdyż to co u mnie działa, u niego już nie.

---

Jedno ważne pytanie: czy ty otworzyłeś jakiś projekt stworzony w starszej wersji środowiska w nowym, czy też stworzyłeś nowy projekt od podstaw w nowym środowisku?

[mirekpl6]

Problem rozwiązany

@Cyfrowy Baronie, tak jak pisałem w pierwszym poście tego wątku: otworzyłem projekt stworzony w starszej wersji środowiska w nowym.
Wspomniałeś o UnicodeString co doprowadziło mnie do AnsiString. Dzięki.

@polymorphism MessageBoxW nie sprawdzałem czy działa, bo wpadłem na pomysł z poniższym kodem.

O to kod,który napisałem, a który działa:

Kod: Zaznacz cały

AnsiString a;

Kod: Zaznacz cały


outfile <<("Poziom trudności: ");
a= AnsiString (Label12->Caption);
outfile <<a


a z MessageBox:

Kod: Zaznacz cały

a=AnsiString(Label10->Caption);
MessageBox(Handle,("Twój wynik został zapisany do katalogu " + a).c_str(), "", MB_OK | MB_ICONINFORMATION);


[Cyfrowy Baron]

Problem związany jest z tym, że stary projekt działający w oparciu o AnsiString zaimportowałeś do nowego środowiska, gdzie głównym typem jest UnicodeString. Występują różnice w pliku projektu np. Project1.cpp:

Starsza wersja środowiska ma:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop
//---------------------------------------------------------------------------
USEFORM("Unit1.cpp", Form1);
//---------------------------------------------------------------------------
WINAPI WinMain(HINSTANCE, HINSTANCE, LPSTR, int)
{
        try
        {
                Application->Initialize();
                Application->CreateForm(__classid(TForm1), &Form1);
                Application->Run();
        }
        catch (Exception &exception)
        {
                Application->ShowException(&exception);
        }
        catch (...)
        {
                try
                {
                        throw Exception("");
                }
                catch (Exception &exception)
                {
                        Application->ShowException(&exception);
                }
        }
        return 0;
}
//---------------------------------------------------------------------------

zwróć uwagę na WinMain i LPSTR.

Nowa wersja wygląda tak:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop
#include <tchar.h>
//---------------------------------------------------------------------------
USEFORM("Unit1.cpp", Form1);
//---------------------------------------------------------------------------
WINAPI _tWinMain(HINSTANCE, HINSTANCE, LPTSTR, int)
{
        try
        {
                Application->Initialize();
                Application->MainFormOnTaskBar = true;
                Application->CreateForm(__classid(TForm1), &Form1);
                Application->Run();
        }
        catch (Exception &exception)
        {
                Application->ShowException(&exception);
        }
        catch (...)
        {
                try
                {
                        throw Exception("");
                }
                catch (Exception &exception)
                {
                        Application->ShowException(&exception);
                }
        }
        return 0;
}
//---------------------------------------------------------------------------

Zwróć uwagę na _tWinMain, #include <tchar.h> i LPTSTR.

Jeżeli projekt nie jest nadmiernie rozbudowany, to wystarczy dokonać tych drobnych korekt w pliku projektu, żeby go przestawić na Unicode.

Najlepiej jest jednak przepisać cały projekt i używać nowego typu. Ja kiedyś przenosiłem projekt modyfikując pliki projektu. Zajęło mi to wiele godzin ale się udało.

[mirekpl6]

Dokonałem zmian w pliku projektu i niestety nadal,gdy nie użyje AnsiString,występują problemy,te co w moim pierwszym poście tego wątku. Nie pomaga dodanie L lub _T lub _TEXT lub TEXT.

[Cyfrowy Baron]

Stwórz nowy projekt w nowym środowisku. Umieść na nim button i Label1, utwórz zdarzenie OnClick dla Buton1, wprowadź w nim ten kod:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

MessageBox(Handle,(L"Twój wynik został zapisany do katalogu " + Label1->Caption).c_str(), L"", MB_OK | MB_ICONINFORMATION);

Powinien zadziałać. Jeżeli nie zadziała, powinieneś przejrzeć przykładowe projekty dołączone do środowiska, by sprawdzić jak to zostało tam rozwiązane.

Prawdopodobnie są różnice w pliku projektu. Ja podałem przykład dla starszej wersji środowiska niż twoje.