Jak zablokować wciśnięcie klawisza?

[kurczez]

W sumie racja, można to było szybciej zapisać i równie zrozumiale. Niemniej dziękuje Baronowi za pomoc i rozumiem, że każdy ma jakieś swoje lepsze lub gorsze przyzwyczajenia w pisaniu kodu.

[Cyfrowy Baron]

Bo widzisz polymorphism sposób w jaki się wypowiadasz sprawia, że ludzie mogą przyjmować postawę agresywną wobec ciebie. Gdybyś zamiast pisać:

Panowie, litości!

To wyraźnie wskazuje, że masz o sobie wysokie mniemanie i innych traktujesz z góry, a to się akurat nie musi wszystkim podobać.

Mogłeś napisać:

Proponuję prostsze rozwiązanie.

W ten sposób wszyscy byliby ci wdzięczni.

---

Co się tyczy twoich propozycji to oczywiście jest to prostsze rozwiązanie, chociażby z tego powodu, że wyeliminowana została pętla, chociaż gdzieś tam w bibliotece funkcja strchr musi jednak przeglądać łańcuch znaków, tylko, że na pewno stosuje wydajniejszy algorytm niż pętla for.

Co do mojej propozycji, to pierwotnie testowałem to rozwiązanie w środowisku C++Builder 2010 i natknąłem się na problem z typem wchar_t, zacząłem kombinować i tak mi wyszło, szukałem analogicznej funkcji w stylu wcstrchr dla typu wchar_t ale takowej nie znalazłem, więc posłużyłem się inną znaną mi w tym środowisku funkcją. No, a potem to już jakoś tak poszło i po prostu na szybko przerobiłem kod dostosowując go do potrzeb środowiska C++Builder 2007.

Gdybym zastosował od razu rozwiązanie podobne do Twojego, problem zostałby szybciej rozwiązany, ale wtedy nie wyszłoby np. to, że funkcja strcpy w środowisku Turbo C++ nieprawidłowo przekazuje wartości, a tak kurczez już wie, że jeżeli kiedyś będzie potrzebował użyć funkcji strcpy to zamiast niej trzeba użyć strncpy i będzie już wiedział jak jej użyć, więc źle nie wyszło, a przy okazji można się czegoś nauczyć.

---

Co do C++Builder 2009 i 2010 to zajrzałem do helpa i funkcja, którą posądzałem o nazwę wcstrcht nazywa się nieco inaczej - wcschr.

Rozwiązanie problemu dla C++Builder 2009 i 2010:

Kod: Zaznacz cały

//---------------------------------------------------------------------------
void __fastcall TForm1::Memo1KeyPress(TObject *Sender, wchar_t &Key)
{
if(wcschr(L"łśćż", Key)) Key = NULL;
}
//---------------------------------------------------------------------------


[polymorphism]

To wyraźnie wskazuje, że masz o sobie wysokie mniemanie i innych traktujesz z góry,

Bez przesady. To można odwrócić w drugą stronę i powiedzieć, że jeśli ktoś ma o sobie (za)wysokie mniemanie, potraktuje moją uwagę jako atak, agresję, wszak podważam jego zdanie. Co do samej formy, miała być nieco ironiczna, no ale to nie powód do obrażania się. Inteligentny człowiek wyłapie ironię, jej intencję...

strchr musi jednak przeglądać łańcuch znaków, tylko, że na pewno stosuje wydajniejszy algorytm niż pętla for.

I tak, i nie. Tego typu proste funkcje biblioteczne implementowane są z reguły w asemblerze, więc będą (lub nie) nieznacznie szybsze od zwykłej pętli for. Jeśli chodzi o algorytm, przy tego typu zadaniu nic innego, efektywniejszego nie wymyślisz. Użycie strchr upraszcza implementację metody. A prostszy w zapisie kod jest łatwiejszy w analizie i wyłapywaniu potencjalnych błędów. Zasada KISS.

ale wtedy nie wyszłoby np. to, że funkcja strcpy w środowisku Turbo C++ nieprawidłowo przekazuje wartości, a tak kurczez już wie, że jeżeli kiedyś będzie potrzebował użyć funkcji strcpy to zamiast niej trzeba użyć strncpy

No widzisz, doszliście do złych wniosków Bo strcpy działa jak należy, tylko użyliście jej w nieodpowiedni sposób:

Kod: Zaznacz cały

char keyString[3];
...
strcpy(&keyString[2], "ś");

widzisz błąd?

Użycie strncpy do kopiowania jednego znaku jest pozbawione sensu.

[Cyfrowy Baron]

No widzisz, doszliście do złych wniosków Bo strcpy działa jak należy, tylko użyliście jej w nieodpowiedni sposób:

Kod: Zaznacz cały

        char keyString[3];
        ...
        strcpy(&keyString[2], "ś");

widzisz błąd?

Użycie strncpy do kopiowania jednego znaku jest pozbawione sensu.

Nie widzę błędu. W C++Builder 2007 strcpy i wstrcpy w C++Builder 2010 działają bezbłędnie.

gdybym zapisał to tak: strcpy(keyString[2], "ś"); to byłby błąd.

---

ironia (łc. z gr. eironeía) ukryta kpina (drwina) z kogoś lub czegoś, ujęta m.in. w formę pozornego uznania, akceptacji, aprobaty. Ironia, ukryte komiczne znaczenie negatywne, różniące się od dosłownego; pogarda wyrażona przy pomocy pozornej aprobaty. Czasami przybiera postać autoironii. Rozróżnia się różne rodzaje ironii, np. sokratyczną, tragiczną, romantyczną itp. Hasło opracowano na podstawie „Słownika Wyrazów Obcych” Wydawnictwa Europa, pod redakcją naukową prof. Ireny Kamińskiej-Szmaj, autorzy: Mirosław Jarosz i zespół. ISBN 83-87977-08-X. Rok wydania 2001.

Wyłapałem ironię, ale co to zmienia. Czy napiszę, że traktujesz ludzi z góry, czy też, że kpisz z nich, czy też pogardzasz nimi - na jedno wyjdzie.

[polymorphism]

Nie widzę błędu. W C++Builder 2007 strcpy i wstrcpy w C++Builder 2010 działają bezbłędnie.

Pozwól, że przytoczę kurczeza:

strcpy(&keyString[2], "ś"); // Po wykonaniu tej linijki kodu zmienna Key zmienia wartość z "f" na '\x01'

[...] dzieje tam się coś dziwnego. Nagle zmienna sama zmienia wartość.

błąd jest ewidentny, ma nawet swoje efekty uboczne

gdybym zapisał to tak: strcpy(keyString[2], "ś"); to byłby błąd.

Owszem, byłby błąd, ale składniowy.

---

Wyłapałem ironię, ale co to zmienia. Czy napiszę, że traktujesz ludzi z góry, czy też, że kpisz z nich, czy też pogardzasz nimi - na jedno wyjdzie.

Napisałem nieco ironiczna, to po pierwsze. Po drugie, nie chodzi o ludzi, tylko o sposób rozwiązywania problemu, o podejście. Rozprawialiście nad kodem, który jest po prostu kiepski, nieprzemyślany, nadmiernie pokomplikowany. O ile kurczezowi można to wybaczyć, to Ty, Baron, powinieneś już pewne rzeczy wiedzieć, przecież programujesz nie od dziś. Cieszysz się tu pewnym autorytetem, więc nie dziw się, że moje wymagania w stosunku do Ciebie są duże. Jeśli chodzi o kurczeza, powinien być nieco bardziej uważny, analizować to, co dostaje, bo jak widać, gotowe rozwiązania nie zawsze są dobre.

[polymorphism]

Odbijając piłeczkę zakładając, że nie jestem inteligentny bo nie załapałem ironii to czy człowiek, który wyśmiewa się z kogoś inteligencji jest inteligentny?

Odpowiem na to pytanie, jeśli pokażesz mi, gdzie wyśmiewam twoją inteligencję. Ok?

[Cyfrowy Baron]

błąd jest ewidentny, ma nawet swoje efekty uboczne

Nie czytasz uważnie błąd występuje tylko w Turbo C++.

Pozwól, że przytoczę kurczeza:

strcpy(&keyString[2], "ś"); // Po wykonaniu tej linijki kodu zmienna Key zmienia wartość z "f" na '\x01'

Owszem! W Turbo C++, w C++Builder 2007 i 2010 ten błąd nie występuje, zwracane są prawidłowe wartości

[kurczez]

Odpowiem na to pytanie, jeśli pokażesz mi, gdzie wyśmiewam twoją inteligencję. Ok?

Na początku chce zaznaczyć, że rozumiem Twoje podejście, nie ma nic złego w poprawianiu kodu. Natomiast nie spodobało mi się to, że w swoich postach robisz z siebie gooru ośmieszając innych. Ja to akurat tak odebrałem, nie znam Waszych relacji.

Co do samej formy, miała być nieco ironiczna, no ale to nie powód do obrażania się. Inteligentny człowiek wyłapie ironię, jej intencję...

Zostało to tak napisane, jakoby my nie zrozumieliśmy ironii i dlatego mamy pretensje a jeśli nie zrozumieliśmy to nie jesteśmy inteligent

Szykuje się Panowie temat rodem z forum gospodyń domowych

[Witold]

Owszem! W Turbo C++, w C++Builder 2007 i 2010 ten błąd nie występuje, zwracane są prawidłowe wartości

Spróbuj włączyć CodeGuard'a. Próbowałeś przekopiować znak (jako cstring) do tablicy, począwszy od ostatniego elementu tablicy, ale strcpy kopiuje jeszcze znak 0 (znacznik końca cstring’u), i w ten sposób te zero zostało skopiowane za tablicą, możliwe że nadpisało przy okazji zmienną Key (u mnie tak wyszło). Tylko czemu u kurczez'a jest tam 1?. „// Po wykonaniu tej linijki kodu zmienna Key zmienia wartość z "f" na '\x01' ” ?

[polymorphism]

Natomiast nie spodobało mi się to, że w swoich postach robisz z siebie gooru ośmieszając innych.

Heh, zacytuj mnie

Zostało to tak napisane, jakoby my nie zrozumieliśmy ironii i dlatego mamy pretensje a jeśli nie zrozumieliśmy to nie jesteśmy inteligent

Zostało to napisane na zachętę. Po to, żeby przemyśleć sprawę ponownie.

---

Nie czytasz uważnie błąd występuje tylko w Turbo C++.

Nie tylko, ten błąd będzie występował wszędzie. Oczywiście konsekwencje mogą być różne, np. żadne, jak u Ciebie, lub takie jak u kurczeza - psucie stosu, czyli poważna sprawa. Witold wyjaśnił gdzie leży błąd.

[Cyfrowy Baron]

Spróbuj włączyć CodeGuard'a

A oto log:

► 

---

Functions called: fflush (4 times) delete (21 times) SysReallocMem (2 times) SysFreeMem (362 times) SysGetMem (362 times) lstrlenA (1 times) lstrcpynA (4 times) free (20 times) _lsetlocale (6 times) new (30 times) calloc (9 times) strlen (85 times) realloc (1 times) strdup (5 times) malloc (5 times) memcpy (10 times) Resource types used: object (30 allocs, 30 max) memory block (382 allocs, 274 max) Modules used: 00400000 01/19/2010 20:44:46 D:\Programy_testowe\MemoTest\Project1.exe ==========================================

---

Nie dostrzegam błędu. Sprawdzałem na różne sposoby, nie ma w tej zmiennej nic ponad to co tam wprowadziłem, więc nie rozumiem gdzie mam doszukiwać sie tego błędu.

[Cyfrowy Baron]

Help dla C++Builder 2007:

► 

---

RAD Studio strcpyHeader File string.h, wchar.h, mbstring.h Category Memory and String Manipulation Routines, Inline Routines Prototype char *strcpy(char *dest, const char *src); wchar_t *wcscpy(wchar_t *dest, const wchar_t *src); unsigned char *_mbscpy(unsigned char *dest, const unsigned char *src); Description    Copies one string into another.    Copies string src to dest, stopping after the terminating null character has been moved. Return Value strcpy returns dest. Example Kod: Zaznacz cały #include <stdio.h> #include <string.h> int main(void) {    char string[10];    char *str1 = "abcdefghi";    strcpy(string, str1);    printf("%s\n", string);    return 0; } Portability strcpy [+] POSIX Win32 ANSI C ANSI C++ _mbscpy [+] Win32 wcscpy [+] Win32 ANSI C ANSI C++ Copyright© 2007 CodeGear™. All Rights Reserved. What do you think about this topic? Send feedback!

---

Dinkumware

► 

---

<string.h> strcpy char *strcpy(char *s1, const char *s2); The function copies the string s2, including its terminating null character, to successive elements of the array of char whose first element has the address s1. It returns s1.

---

gdzie tu jest jakaś informacja o możliwości wystąpienia błędów o których piszecie.

[Cyfrowy Baron]

Mam jeszcze jedno pytanie. Czy błąd o którym piszecie odnośnie użycia funkcji strcpy dotyczy również funkcji wcscpy?

[polymorphism]

W zasadzie tak.

[Witold]

Nie dostrzegam błędu. Sprawdzałem na różne sposoby, nie ma w tej zmiennej nic ponad to co tam wprowadziłem, więc nie rozumiem gdzie mam doszukiwać sie tego błędu.

char keyString[3];

tablica keyString:
[0][1][2]

po:
strcpy(&keyString[2], "ś");

'ś' będzie w komórce [2] , 0 (znacznik końca cstring’u) poza tablicą.
gdyby było "char keyString[4];" byłoby OK.

u mnie CodeGuard wywala: