From 413256e850936f7812edc5bf61e46e495d1ae0d6 Mon Sep 17 00:00:00 2001
From: Ivan Gutorov <gutorov.ia1@gmail.com>
Date: Thu, 5 Dec 2024 06:23:35 +0400
Subject: [PATCH 1/2] final changes

---
 .../Forms/FormContract.Designer.cs            | 18 +++----
 .../Forms/FormCustomerExecutorReview.cs       | 20 ++++---
 .../Forms/FormService.cs                      |  4 +-
 .../Forms/FormService.resx                    | 54 +++++++++----------
 4 files changed, 52 insertions(+), 44 deletions(-)

diff --git a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormContract.Designer.cs b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormContract.Designer.cs
index d0fa481..e694e67 100644
--- a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormContract.Designer.cs
+++ b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormContract.Designer.cs
@@ -41,8 +41,8 @@
             labelContractDeadline = new Label();
             labelContractPaymentAmount = new Label();
             dataGridViewServices = new DataGridView();
-            groupBox = new GroupBox();
             ColumnServiceDescription = new DataGridViewComboBoxColumn();
+            groupBox = new GroupBox();
             ((System.ComponentModel.ISupportInitialize)numericUpDownContractPaymentAmount).BeginInit();
             ((System.ComponentModel.ISupportInitialize)dataGridViewServices).BeginInit();
             groupBox.SuspendLayout();
@@ -172,6 +172,14 @@
             dataGridViewServices.Size = new Size(352, 244);
             dataGridViewServices.TabIndex = 17;
             // 
+            // ColumnServiceDescription
+            // 
+            ColumnServiceDescription.AutoSizeMode = DataGridViewAutoSizeColumnMode.Fill;
+            ColumnServiceDescription.HeaderText = "Услуга";
+            ColumnServiceDescription.Name = "ColumnServiceDescription";
+            ColumnServiceDescription.Resizable = DataGridViewTriState.True;
+            ColumnServiceDescription.SortMode = DataGridViewColumnSortMode.Automatic;
+            // 
             // groupBox
             // 
             groupBox.Anchor = AnchorStyles.Top | AnchorStyles.Bottom | AnchorStyles.Left | AnchorStyles.Right;
@@ -183,14 +191,6 @@
             groupBox.TabStop = false;
             groupBox.Text = "Услуги:";
             // 
-            // ColumnServiceDescription
-            // 
-            ColumnServiceDescription.AutoSizeMode = DataGridViewAutoSizeColumnMode.Fill;
-            ColumnServiceDescription.HeaderText = "Услуга";
-            ColumnServiceDescription.Name = "ColumnServiceDescription";
-            ColumnServiceDescription.Resizable = DataGridViewTriState.True;
-            ColumnServiceDescription.SortMode = DataGridViewColumnSortMode.Automatic;
-            // 
             // FormContract
             // 
             AutoScaleDimensions = new SizeF(7F, 15F);
diff --git a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs
index dcf0cef..d2f67df 100644
--- a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs
+++ b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs
@@ -11,15 +11,16 @@ namespace PIbd_23_Gutorov_I.A._IT_Company.Forms
             ICustomerRepository customerRepository, IExecutorRepository executorRepository)
         {
             InitializeComponent();
-            _customerExecutorReviewRepository = customerExecutorReviewRepository ?? throw new ArgumentNullException(nameof(customerExecutorReviewRepository));
+            _customerExecutorReviewRepository = customerExecutorReviewRepository ??
+                throw new ArgumentNullException(nameof(customerExecutorReviewRepository));
 
             comboBoxCustomerExecutorReviewCustomerId.DataSource = customerRepository.ReadCustomers();
             comboBoxCustomerExecutorReviewCustomerId.DisplayMember = "Name";
-            comboBoxCustomerExecutorReviewCustomerId.DisplayMember = "Id";
+            comboBoxCustomerExecutorReviewCustomerId.ValueMember = "Id";
 
             comboBoxCustomerExecutorReviewExecutorId.DataSource = executorRepository.ReadExecutors();
             comboBoxCustomerExecutorReviewExecutorId.DisplayMember = "Name";
-            comboBoxCustomerExecutorReviewCustomerId.DisplayMember = "Id";
+            comboBoxCustomerExecutorReviewCustomerId.ValueMember = "Id";
         }
 
         private void ButtonSave_Click(object sender, EventArgs e)
@@ -33,9 +34,7 @@ namespace PIbd_23_Gutorov_I.A._IT_Company.Forms
                     throw new Exception("Имеются незаполненные поля");
                 }
 
-                _customerExecutorReviewRepository.CreateCustomerExecutorReview(CustomerExecutorReview.CreateElement(
-                    0, (int)comboBoxCustomerExecutorReviewCustomerId.SelectedValue!, (int)comboBoxCustomerExecutorReviewExecutorId.SelectedValue!,
-                    richTextBoxCustomerExecutorReviewReview.Text, Convert.ToInt32(numericUpDownCustomerExecutorReviewGrade.Value)));
+                _customerExecutorReviewRepository.CreateCustomerExecutorReview(CreateCustomerExecutorReview(0));
 
                 Close();
             }
@@ -46,5 +45,14 @@ namespace PIbd_23_Gutorov_I.A._IT_Company.Forms
         }
 
         private void ButtonCancel_Click(object sender, EventArgs e) => Close();
+
+        private CustomerExecutorReview CreateCustomerExecutorReview(int id)
+        {
+            return CustomerExecutorReview.CreateElement(
+                    id, (int)comboBoxCustomerExecutorReviewCustomerId.SelectedValue!,
+                    (int)comboBoxCustomerExecutorReviewExecutorId.SelectedValue!,
+                    richTextBoxCustomerExecutorReviewReview.Text,
+                    Convert.ToInt32(numericUpDownCustomerExecutorReviewGrade.Value));
+        }
     }
 }
diff --git a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.cs b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.cs
index c11301c..2d41b29 100644
--- a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.cs
+++ b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.cs
@@ -42,8 +42,8 @@ namespace PIbd_23_Gutorov_I.A._IT_Company.Forms
                 if (string.IsNullOrEmpty(richTextBoxServiceDescription.Text))
                     throw new Exception("Имеются незаполненные поля");
 
-                _serviceRepository.CreateService(Service.CreateEntity(0,
-                    richTextBoxServiceDescription.Text));
+                if (_serviceId.HasValue) _serviceRepository.UpdateService(Service.CreateEntity(_serviceId.Value, richTextBoxServiceDescription.Text));
+                else _serviceRepository.CreateService(Service.CreateEntity(0, richTextBoxServiceDescription.Text));
 
                 Close();
             }
diff --git a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.resx b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.resx
index 8b2ff64..1af7de1 100644
--- a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.resx
+++ b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormService.resx
@@ -1,17 +1,17 @@
 <?xml version="1.0" encoding="utf-8"?>
 <root>
-  <!--
-    Microsoft ResX Schema
-
+  <!-- 
+    Microsoft ResX Schema 
+    
     Version 2.0
-
-    The primary goals of this format is to allow a simple XML format
-    that is mostly human readable. The generation and parsing of the
-    various data types are done through the TypeConverter classes
+    
+    The primary goals of this format is to allow a simple XML format 
+    that is mostly human readable. The generation and parsing of the 
+    various data types are done through the TypeConverter classes 
     associated with the data types.
-
+    
     Example:
-
+    
     ... ado.net/XML headers & schema ...
     <resheader name="resmimetype">text/microsoft-resx</resheader>
     <resheader name="version">2.0</resheader>
@@ -26,36 +26,36 @@
         <value>[base64 mime encoded string representing a byte array form of the .NET Framework object]</value>
         <comment>This is a comment</comment>
     </data>
-
-    There are any number of "resheader" rows that contain simple
+                
+    There are any number of "resheader" rows that contain simple 
     name/value pairs.
-
-    Each data row contains a name, and value. The row also contains a
-    type or mimetype. Type corresponds to a .NET class that support
-    text/value conversion through the TypeConverter architecture.
-    Classes that don't support this are serialized and stored with the
+    
+    Each data row contains a name, and value. The row also contains a 
+    type or mimetype. Type corresponds to a .NET class that support 
+    text/value conversion through the TypeConverter architecture. 
+    Classes that don't support this are serialized and stored with the 
     mimetype set.
-
-    The mimetype is used for serialized objects, and tells the
-    ResXResourceReader how to depersist the object. This is currently not
+    
+    The mimetype is used for serialized objects, and tells the 
+    ResXResourceReader how to depersist the object. This is currently not 
     extensible. For a given mimetype the value must be set accordingly:
-
-    Note - application/x-microsoft.net.object.binary.base64 is the format
-    that the ResXResourceWriter will generate, however the reader can
+    
+    Note - application/x-microsoft.net.object.binary.base64 is the format 
+    that the ResXResourceWriter will generate, however the reader can 
     read any of the formats listed below.
-
+    
     mimetype: application/x-microsoft.net.object.binary.base64
-    value   : The object must be serialized with
+    value   : The object must be serialized with 
             : System.Runtime.Serialization.Formatters.Binary.BinaryFormatter
             : and then encoded with base64 encoding.
-
+    
     mimetype: application/x-microsoft.net.object.soap.base64
-    value   : The object must be serialized with
+    value   : The object must be serialized with 
             : System.Runtime.Serialization.Formatters.Soap.SoapFormatter
             : and then encoded with base64 encoding.
 
     mimetype: application/x-microsoft.net.object.bytearray.base64
-    value   : The object must be serialized into a byte array
+    value   : The object must be serialized into a byte array 
             : using a System.ComponentModel.TypeConverter
             : and then encoded with base64 encoding.
     -->

From a82cf75e14dfdca7f1f7d0751259ff4aa4168027 Mon Sep 17 00:00:00 2001
From: Ivan Gutorov <gutorov.ia1@gmail.com>
Date: Thu, 5 Dec 2024 06:27:56 +0400
Subject: [PATCH 2/2] last little fix (maybe not last)

---
 .../Forms/FormCustomerExecutorReview.cs                         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs
index d2f67df..a12334a 100644
--- a/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs
+++ b/PIbd-23_Gutorov_I.A._IT-Company/Forms/FormCustomerExecutorReview.cs
@@ -20,7 +20,7 @@ namespace PIbd_23_Gutorov_I.A._IT_Company.Forms
 
             comboBoxCustomerExecutorReviewExecutorId.DataSource = executorRepository.ReadExecutors();
             comboBoxCustomerExecutorReviewExecutorId.DisplayMember = "Name";
-            comboBoxCustomerExecutorReviewCustomerId.ValueMember = "Id";
+            comboBoxCustomerExecutorReviewExecutorId.ValueMember = "Id";
         }
 
         private void ButtonSave_Click(object sender, EventArgs e)