Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

CharInSet doesn't work with non English letters?

I have updated an application from Delphi 2007 to Delphi 2010, everything went fine, except one statement that compiled fine but not working which is:

If Edit1.Text[1] in ['S','س'] then 
  ShowMessage('Found')
else
  ShowMessage('Not Found')

However, I knew that in will not, so I changed to CharInSet

If CharinSet(Edit1.Text[1],['S','س']) then
  ShowMessage('Found')
else
  ShowMessage('Not Found')

but it never worked when the string is س, but always work with S, even I cast the edt1.Text1 with AnsiChar it always not work Arabic letters.

Am doing anything wrong, or it's not the way CharInSet works?, or that's a bug in CharinSet?

UPDATE:

My Great friend Issam Ali has suggested another solution which's worked fine as it :

  If CharinSet(AnsiString(edt1.Text)[1],['S','س']) then
like image 775
Mohammed Nasman Avatar asked Nov 21 '10 10:11

Mohammed Nasman


4 Answers

CharInSet is useless for the characters above 255. In your case you should use

  case C of
    'S','س' : ShowMessage('Found');
  end;
like image 199
kludg Avatar answered Oct 03 '22 18:10

kludg


This happens because set of char structured type (limited to 256 elements maximum) doesn't support Unicode at all. That is, any characters Ord(ch) > High(AnsiChar) being truncated in the set constructor and warning W1061 about narrowing WideChar to AnsiChar is being emitted. Look at the following testcase:

  { naturally, fails, emits CharInSet() suggestion }
  Result := 'س' in ['S','س'];

  { fails because second argument is set of AnsiChar }
  Result := CharInSet(
    'س',
    ['S','س']
  );

  { workaround for WideChar in AnsiCharSet, fails }
  Result := WideStrUtils.InOpSet(
    'س',
    ['S','س']
  );

  { a syntactical workaround, which finally works }
  Result := WideStrUtils.InOpArray(
    'س',
    ['S','س']
  );

  if Result then
    ShowMessage('PASS')
  else
    ShowMessage('FAIL');
like image 22
Free Consulting Avatar answered Oct 03 '22 17:10

Free Consulting


In addition.

sets are limited to ordinal values of 256 elements. So AnsiChar fits and (Unicode)Char does not fit. You can use CharInSet to port pre unicode versions of Delphi to the unicode versions. Because of the set limitation, sets are not extremely usefull anymore with Chars.

The reason behind this, is that sets are implemented as bitmasks. You are free to implement your own version of a set. For example:

type
  TSet<T> = class 
  public
    procedure Add(const AElem: T);
    function InSet(const AElem: T): Boolean;
  end;
like image 34
Toon Krijthe Avatar answered Oct 03 '22 18:10

Toon Krijthe


Use TCharHelper.IsInArray as follows:

if Edit1.Text[1].IsInArray(['S','س']) then 
  ShowMessage('Found')
else
  ShowMessage('Not Found');
like image 34
mas_oz2k1 Avatar answered Oct 03 '22 17:10

mas_oz2k1