restrorest / DZ1

0 stars 0 forks source link

Ненужное усложнение #1

Open HukuTa45 opened 2 weeks ago

HukuTa45 commented 2 weeks ago

https://github.com/restrorest/DZ1/blob/b7b37965540b1e638ddc1d9f100335b106561a66/DZ1/MainWindow.xaml.cs#L31

Зачем нам городить такие сложные конструкции, добавлять ненужные циклы? Можно намного проще сделать и более читабельным


 public MainWindow()
 {
     InitializeComponent();
 }

 int counter = 0;

 private void Button_Click(object sender, RoutedEventArgs e)
 {
     MyListView.Items.Add(counter);
     counter++;
 }

 private void Button_Click_1(object sender, RoutedEventArgs e)
 {
     MyListView.Items.Remove(MyListView.SelectedItem);
 }

Не надо изобретать велосипед) Лучше всё максимально упрощенно продумывать

HukuTa45 commented 2 weeks ago

Я раньше также делал на самом деле) Не надо самому себе проблем добавлять, поддерживать такой код сложнее будет

HukuTa45 commented 2 weeks ago

Сейчас еще раз пересмотрел https://github.com/restrorest/DZ1/blob/b7b37965540b1e638ddc1d9f100335b106561a66/DZ1/MainWindow.xaml.cs#L36

Да действительно ты прав, оценку поправил Но счетчик можно было попроще сделать