C#. Тип «Settings» уже определяет член «Save» с такими же типами параметрами. Что делать?

Question

[Молзв Реках]

Всем привет
Вообщем у меня есть функция Save() которая сохраняет параметры в файл и обновляет их.
Но мне нужно её перегрузить, то есть в некоторых местах сохранить ТОЛЬКО один параметр из трёх. Или два параметра из трёх.
Код:

class Settings
    {
        public string Macros_Directory { get; set; }
        public string SteamID { get; set; }
        public string AccPath { get; set; }

        public Settings()
        {
            string stId = "";
            string macDirectory = "";
            string account_path = "";

            try
            {
                StreamReader sr = new StreamReader("**");
                stId = sr.ReadToEnd().Trim();
                sr.Close();
            }
            catch (Exception) { }

            try
            {
                StreamReader sr = new StreamReader("**");
                macDirectory = sr.ReadToEnd().Trim();
                sr.Close();
            }
            catch (Exception) { }

            try
            {
                StreamReader sr = new StreamReader("**");
                account_path = sr.ReadToEnd().Trim();
                sr.Close();
            }
            catch (Exception) { }

            if (macDirectory.Trim() != "" && stId.Trim() != "" && account_path.Trim() != "")
            {
                Macros_Directory = macDirectory;
                SteamID = stId;
                AccPath = account_path;
            }
            else
            {
                Macros_Directory = "";
                SteamID = "";
                AccPath = "";
            }
        }
        public void Save(string macDir, string steamid, string accpath)
        {
            StreamWriter stid = new StreamWriter("**");
            stid.WriteLine(steamid);
            stid.Close();

            StreamWriter macD = new StreamWriter("**");
            macD.WriteLine(macDir);
            macD.Close();

            StreamWriter accPath = new StreamWriter("**");
            accPath.WriteLine(accpath);
            accPath.Close();

            Macros_Directory = macDir;
            SteamID = steamid;
            AccPath = accpath;
        }
        public void Save(string accpath)
        {
            StreamWriter accPath = new StreamWriter("**");
            accPath.WriteLine(accpath);
            accPath.Close();

            AccPath = accpath;
        }
        public void Save(string macDir) // Тип "Settings" уже определяет член "Save" с такими же типами параметрами
        {                                  // ERROR
            StreamWriter macD = new StreamWriter("**");
            macD.WriteLine(macDir);
            macD.Close();
            
            Macros_Directory = macDir;
        }
    }

Я пометил проблему комментарием.
Как правильно перегрузить функции?
Вроде базовые C# знаю, а забыл...Думал класс создать, но что то вообще каша была бы.

Comments

[Foggy Finder]

Не совсем понятна ваша конечная цель.

Пока у вас метод довольно грязный, по сути он не делает то о чем говорит название.
Вы записываете в файл и кроме того еще устанавливаете свойство/свойства.

Я бы предложил вынеси в три отдельных метода:

public void SetAllPropertiesAndSave(string macDir, string steamid, string accpath)
{
   // Do Something
}

public void SetAccPathAndSave(string accpath)
{
   // Do Something
}

public void SetMacDirAndSave(string macDir)
{
   // Do Something
}

Кроме того, код несколько странный. Какую задачу вы пытаетесь решить?
У вас пути к файлам одинаковые или вы просто показываете упрощенную версию реального кода?

[Молзв Реках]

Foggy Finder, упрощённую версию реального кода. Это класс Settings от программы. Тут настройки всех директорий что бы пользователю не писать по сто раз, они сохраняются в файл и программа смотрит если там что то есть - показывает что уже всё написано за вас, но если хотите изменить - изменяйте. Как то так

[Молзв Реках]

Foggy Finder, Нужно как то перегрузить. Так будет немного не очень

[Молзв Реках]

Foggy Finder, Пути к файлам разные. Для каждой настройки свой.

[edward_freedom]

public void Save(string macDir) // Тип "Settings" уже определяет член "Save" с такими же типами параметрами

Ну все верно, вот эта функция

public void Save(string accpath)

задавай имена функций нормальное и не будет тебе проблем

[Молзв Реках]

edward_freedom, Так нужно вызывать ту же функцию. Я пытался перегрузить. Первый раз получилось, а второй раз выдало ошибку.

[Молзв Реках]

edward_freedom, Нужно вызывать эту функцию но иногда с отдельными параметрами. По этому требуется перегрузить их. И тут вот эта проблема. Вообще ничего не пойму.

[Foggy Finder]

Молзв Реках, это делается не так.
Все конечно зависит от требований к настройкам, но если брать в расчет ваш код, то, подозреваю, вам подойдет такой способ:

При запуске вы открываете файл настроек, считываете данные (или десериализуете) в объект Settings, отображаете информацию, в ходе работы пользователь может менять их (путем установки значения свойств (Macros_Directory, ..) и работать уже с ними. При выходе (в том числе аварийном) происходит перезапись файла. Также можно перезаписывать всякий раз при закрытии окна/страницы настроек.

[Владимир Коротенко]

Вот вполне рабочий код, и как по мне он немного проще для понимания. Обратите внимание на аттрибуты

/// <summary>
        /// Восстановление состояния из файла.
        /// </summary>
        /// <param name="s"></param>
        public void Restore(Store s)
        {
            s.FillModes();
            s.CurrentPasport = Passports.First(x => x.Id == CurrentPassportId);
            s.SelectedCountry = Countryes.First(x => x.Id == CurrentCountryId);
        }
        private static DataCotext Cotext => DataCotext.Instance;
        /// <summary>
        /// Сериализуем настройки в файл. 
        /// <see cref="SettingsConst"/>
        /// </summary>
        private static void Save(Store store)
        {
            var data = JsonConvert.SerializeObject(store);
            File.WriteAllText(StorageFullPath, data);
        }
        /// <summary>
        /// Сериализуем настройки в файл. 
        /// <see cref="SettingsConst"/>
        /// </summary>
        public void Save()
        {
            Save(this);
        }

#region Сериализуемые данные


        public bool IsInitialized { get; set; }
        public int CurrentCountryId { get; set; }
       

        public bool AutoBackupOn { get; set; }

        #endregion

        /// <summary>
        /// Список языков в приложении.
        /// </summary>
        [JsonIgnore]
 public IEnumerable<LanguageItem> Languages { get; } = new List<LanguageItem>()
        {
            new LanguageItem(Resource.LanguageRU,"ru",""),
            new LanguageItem(Resource.LanguageCH,"zh",""),
            new LanguageItem(Resource.LanguageUA,"ua",""),
            new LanguageItem(Resource.LanguageEN,"en",""),
        };

Answer 1

[eRKa]

При перегрузке методы должны отличаться сигнатурой - отличаться типом или кол-вом входящих параметров. Если при вызове метода Save Вы всегда знаете какой параметр или параметры хотите сохранить, то в Вашем случае я вижу два варианта:
первый, если таких перегрузок требуется два-три, то каждый метод можно назвать по другому, например SaveMacDir, SaveAccpath и т.д.
второй, если этих парметров три, то два из них сделать необязательными

public void Save(string accpath, string macDir = null, string stId = null)
{
	try
	{
		using(var sw = new StreamWriter("**"))
		{
			if(!string.IsNullOrEmpty(accpath))
			{
				sw.WriteLine(accpath);
				AccPath = accpath
			}
			if(!string.IsNullOrEmpty(macDir))
			{
				sw.WriteLine(macDir);
				Macros_Directory  = macDir;
			}
			if(!string.IsNullOrEmpty(stId))
			{
				sw.WriteLine = stId;
				SteamID = stId;
			}
		}
	}
	catch(Exception ex) { }
}

И вызывать их как

Save(accpath)
Save(accpath, macDir)
Save(accpath, macDir, stId)
Save(null, macDir, stId)
Save(accpath, null, stId)

Comments

[Молзв Реках]

АААА!!! Стандартные значения в параметрах функции. Вот оно что. А я думаю, думаю, вроде базовые знаю а вот не могу сделать :\
Спасибо!

Answer 2

[Артур Мустафин]

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

public interface IStreamSourceWriter: IDisposable {
  WriteLine(string);  
}
public void SaveFromSource(IStreamSourceWriter streamSourceWriter) {
  // код ниже уходит в реализацию паттерна интерфейс, то есть в IStreamSourceWriter)
  // управление временем жизни уходит в реализацию паттерна
  // using(StreamWriter stid = new StreamWriter("**");
  // stid.WriteLine(steamid);
  // stid.Close();   
  streamSourceWriter.WriteLine(steamid);
}
public void SaveFromSources(IEnumerable<IStreamSourceWriter> streamSourceWriters) { 
  foreach(source in streamSourceWriters) {
    SaveFromSource(source);
  }
}