Документирование кода по стандарту RSDN (C#)

[McDoC]

Кто знаком со стандартами RSDN, укажите пожалуйста на ошибки оформления кода:

using System;

namespace Payroll
{
    /// <summary>
    /// Абстрактный базовый класс Payable
    /// <para> 
    /// Cвойства: <see cref="FullName"/>,  <see cref="PositionOfEmployee "/>, 
    /// <see cref="QualificationCategory"/>,  <see cref="SurchargeFactor"/>
    /// </para>
    /// <para>
    /// Метод: <see cref="GetPaymentAmount"/>
    /// </para>
    /// </summary>
    public abstract class Payable
    {
        private string _name; // Имя работника
        private string _post; // Должность, назначение, специальность работника
        private string _category; // Квалификация
        private int _factor; // Надбавка за квалификацию

        /// <summary>
        /// Свойство для получения и записи имени работника
        /// </summary>
        public string FullName
        {
            get
            {
                return _name;
            }
            set
            {
                // Если строка пуста...
                if (String.IsNullOrWhiteSpace(value))
                        // тогда создать исключение пустой строки
                        throw new Exception("Имя работника отсутствует");
                    
                else
                        _name = value;
            }
        }

        /// <summary>
        /// Свойство для получения и записи назначения работника
        /// </summary>
        public string PositionOfEmployee
        {
            get
            {
                return _post;
            }
            set
            {
                // Если строка пуста...
                if (String.IsNullOrWhiteSpace(value))
                    // тогда создать исключение пустой строки
                    throw new Exception("Назначение работника отсутствует");

                else
                    _post = value;
            }
        }

        /// <summary>
        /// Свойство для получения и записи квалификации
        /// </summary>
        public string QualificationCategory
        {
            get
            {
                return _category;
            }
            set
            {
                // Если строка пуста...
                if (String.IsNullOrWhiteSpace(value))
                    // тогда создать исключение пустой строки
                    throw new Exception("Квалификация работника отсутствует");

                else
                    _category = value;
            }
        }

        /// <summary>
        /// Свойство для получения и записи процента надбавки за квалификацию
        /// </summary>
        public int SurchargeFactor
        {
            get
            {
                return _factor;
            }
            set
            {
                // Если значение не отрицательное...
                if (value >= 0)
                    // тогда присвоить значение
                    _factor = value;
                else
                    // Иначе создать исключение ввода отрицательного числа
                    throw new Exception("Процент надбавки не может быть отрицательным.");
            }
        }

        /// <summary>
        /// Конструктор класса Payable по умолчанию
        /// </summary>
        public Payable()
        {

        }

        /// <summary>
        /// Конструктор класса Payable с четырьмя параметрами
        /// </summary>
        /// <param name="name">Имя</param>
        /// <param name="post">Назначение</param>
        /// <param name="category">Квалификация</param>
        /// <param name="factor">Надбавка</param>
        public Payable(string name, string post,
            string category, int factor)
        {
            FullName = name;
            PositionOfEmployee = post;
            QualificationCategory = category;
            SurchargeFactor = factor;
        }

        /// <summary>
        /// Конструктор класса Payable с двумя параметрами
        /// </summary>
        /// <param name="name">Имя</param>
        /// <param name="post">Назначение</param>
        public Payable(string name, string post)
        {
            FullName = name;
            PositionOfEmployee = post;
        }// Конец конструктора Payable с двумя параметрами

        /// <summary>
        /// Абстрактный метод начислениия зарплаты
        /// </summary>
        public abstract decimal GetPaymentAmount();
    }
}

[Alex11223]

Дык прочитайте и будете знакомы.

Я правда не понял как он относится к документированию, там только небольшой (6 пунктов) раздел про комментарии. И вы его явно не читали.

[McDoC]

Дык я читал и знаком с RSDN, только препода не устраивает мое оформление кода. Вот его рецензия:

Замечания:_

Оформление кода:

1. для свойств, полей, методов, классов и интерфейсов нужно ставить xml комментарии (автоматически ставятся, когда возле свойства, метода или класса пишешь тройной ///), которые нужны для автоматической подсказки при наборе названия тексте программы в Visual Studio. Поэтому нужно простые комментарии заменить на xml комментарии. Ещё есть недочёты!

2. класс оформлен не по RSDN (пункт 2 в 1.5 Задания на лабораторную работу). Ещё есть недочёты!

Что у меня оформлено не так?

[Alex11223]

Сообщение от McDoC

для свойств, полей, методов, классов и интерфейсов нужно ставить xml комментарии

И над классом фигня какая-то, вам делать нечего кроме как дублировать список всего там?

А, и еще в XML комментариях можно указывать какие исключения метод выкидывает (наверно и свойства тоже). Ну и желательно более конкретные типы использовать, а не Exception.

[McDoC]

Спасибо большое, что указали на ошибки. А можно пример, как бы вы оформили код в блоке set.

[Alex11223]

/// <summary>
/// Возвращает или изменяет процент надбавки за квалификацию
/// </summary>
public int SurchargeFactor
{
    get
    {
        return _factor;
    }
    set
    {
        if (value < 0)
        {
            throw new ArgumentOutOfRangeException("Процент надбавки не может быть отрицательным.");
        }

        _factor = value;
    }
}

Вообще комментарии обычно на английском пишут ("Gets or sets the ..."), но тут наверно сложно если это про какие-то местные правила оплаты труда.

[McDoC]

Спасибо еще раз, очень выручили.