мои предложения: 

• те методы, которые нужны только самому классу и все поля - в private
• те методы, которые могут понадобицца наследникам - protected
• те методы, которые нуждны другим классам - в public
• доступ к полям, которые нужны другим классам - через свойства (property)

Собственно тут сложно с чем-то не согласиться.. 

Вопросы по поводу определения какие именно методы и свойства считать нужными для самого класса, какие для наследников, а какие для других классов будем обсуждать здесь 

Это сообщение отредактировал(а) Vitalik - 9.9.2006, 18:22

нижеследующее касаецца беты5 (Виталик выложит в ближайшее время)

• SynUniFormat.pas->TSynUniFormat.(VerifyStream|VerifyFileName|VerifyEmptyFileName)
  [Pascal Hint] SynUniFormat.pas(44): H2077 Value assigned to 'TSynUniFormat.VerifyStream' never used
  т.к. вызываецца исключение, результат функции использовацца не будет:
  

  Код

  function a:Boolean begin   Result:=true;   raise Exception.Create('') end; begin   try     if a then       DoSmth;   finally   end; end.

  
  если в a будет возбуждено исключение, управление будет передано в родительский относительно if..then блок try-finally. и результат a не важен. по этому можно даже заменить функции на процедуры:
  

  Код

  class procedure TSynUniFormat.VerifyStream(AStream: TStream); begin   if not Assigned(AStream) then     raise Exception.Create(ClassName + '.ImportFromStream: AStream property can not be nil'); end; ... TSynUniFormat.VerifyStream(Stream); {дальнейшие действия} ...

• в принципе TSynUniFormat.VerifyEmptyFileName дублирует функциональность TSynUniFormat.VerifyFileName: если в VerifyFileName AFileName='', то FileExists вернёт false и будет сгенерировано исключение 
• В TSynUniFormat методы Import... и Export... сделать абстрактными?
• Методы ISynUniXmlParser и ISynUniXmlTag тоже сделать абстрактынми?
• TSynUniFormatNativeXml.ImportFromStream
  [Pascal Hint] SynUniFormatNativeXml.pas(179): H2077 Value assigned to 'TSynUniFormatNativeXml.ImportFromStream' never used
  в теле метода меняецца Result, а в конце
  

  Код

  XmlParser.Free(); Result := True;

  
  
• в том же методе конструкции вида 
  

  Код

  Result := False; raise Exception.Create(...)

  
  присвоение Result ничего не даёт (см. 1.)
• TSynUniFormatNativeXml15.(ImportToken|ExportToken) объявлены без override
  [Pascal Warning] SynUniFormatNativeXml15.pas(47): W1010 Method 'ImportToken' hides virtual method of base type 'TSynUniFormatNativeXml'
  просто описка, или это имеет какой-то глубокий смысл?
• TSynUniFormatNativeXml15.(ImportInfo|ImportAttributes|ImportKeyList|ImportSetImportToken|ImportRange) ничего не возвращают. как я понимаю, они должны возвращать true в случае успешного импорта. можно сделать как в TSynUniFormatNativeXml20 - просто возвращать true.... но тогда вопрос: а зачем вообще нужен этот результат функции?