K voland #48

K-Voland · 2018-11-26T10:03:17Z

No description provided.

K-Voland · 2018-12-02T11:30:35Z

Я пока плохо представляю как списки эти работают, но для одинокого кролика оно работает.

K-Voland · 2018-12-02T17:30:37Z

Вроде что-то начал понимать, теперь оно нормально работает.

jskonst · 2018-12-09T19:06:36Z

CourseApp/RabbitInfo.cs

+    public class Rabbit
+    {
+        public void RabbitInfo(string Name, string Pearent1,string Pearent2, List<string> Child) 
+        {Console.Write($"Кролик {Name}, Родители: {Pearent1} и {Pearent2}, Дети: ");foreach(var i in Child){Console.WriteLine($"{i} ");}}


Разбейте на несколько строк, читать очень сложно

jskonst · 2018-12-09T19:11:13Z

CourseApp/Menu.cs

+{
+    public class Menu:Farm
+    {
+      public  Rabbit info = new Rabbit();


имя переменной не информативно - info? но при этом класс кролика

Если это меню, то это толкьо меню!

jskonst · 2018-12-09T19:12:21Z

CourseApp/Menu.cs

+
+namespace CourseApp
+{
+    public class Menu:Farm


Тут наследование не совсем верно применено - ну или как минимум именование - мен. не является фермой

jskonst · 2018-12-09T19:14:40Z

CourseApp.Tests/UnitTest2.cs

+            var krosh = new Rabbit();
+            var N ="momo";
+            krosh.RabbitInfo(N);
+            Assert.Equal("momo",N);


этот тест не проверяет ващ класс - он только проверяет что переменная N, объявленная в тесте = "momo"

jskonst · 2018-12-09T19:15:31Z

CourseApp.Tests/UnitTest2.cs

+            var P1 = "mu";
+            var P2 = "nu";
+            krosh.RabbitInfo(N);
+            Assert.Equal("momo",N);


Та же беда - вы проверяете не свойство класса, а проверяете значение переменных, которые объявлены в тесте

jskonst · 2018-12-15T20:26:08Z

CourseApp/Zoo.cs

+{
+    public class Zoo
+    {
+        public ArrayList Klet = new ArrayList();


Используйте более понятное название - KLet? может Kletka?

jskonst · 2018-12-15T20:36:15Z

CourseApp/Zoo.cs

+
+            set
+            {
+                foreach(NewPet i in Klet)


https://metanit.com/sharp/tutorial/4.9.php - лучше использовать именно словарь

jskonst · 2018-12-15T20:47:51Z

CourseApp/Krolik.cs

+
+namespace CourseApp
+{
+   public class Krolik : Zoo


Почему кролик наследует зоопарк? Тут с именованием явная беда - при наследовании у нас отошение is-A - "является" . Кролик вряд ли является зоопарком :(

jskonst · 2018-12-15T21:32:47Z

CourseApp/Krolik.cs

+         Klet.Add(new NewPet(Name, Pearent1, Pearent2));
+         foreach(NewPet i in Klet)
+         {
+            if(Pearent1 == i.Name || Pearent2 == i.Name)


А как же первых добавить?

Покупая в магазине мы не знаем кто родители, потому иногда эти свойства могут отсутствовать.)

jskonst · 2018-12-21T21:35:55Z

CourseApp/Pet.cs

+{
+    public abstract class Pet
+    {
+        private Dictionary<string, NewPet> spisok = new Dictionary<string, NewPet>();


Думаю что это уже не список - поэтому лучше называть переменну. так, что это будет отражать сущность

Что-то совсем не приходит ничего в голову. По моему это как раз список животных с их родителями и потомками.

Не нужно вам это внутри домашнего животного

Я сейчас залью исправления и открою PR в ваш репозитарий

K-Voland · 2018-12-23T12:21:35Z

Проблем много, ещё буду править. Это не окончательная версия, просто хочется услышать комментарии.

K-Voland · 2018-12-23T12:49:55Z

Видимо я что-то не понимаю в устройстве записи памяти. Обращаясь к BuyPet из Menu я вроде должен создать новую запись в словаре, но потом когда хочу получить обратно пишет что не найден такой ключ. Что я не так делаю?

jskonst · 2018-12-27T23:11:17Z

CourseApp/Krolik.cs

+      public override void BuyPet()
+      {
+         Console.Write("Введите имя купленного кролика: ");
+         Name = Console.ReadLine();


не не - это - плохая идея - обработку пользовательского ввода вести в классахз, фактически представляюзих модели данных - уберите, а Name передавайте как параметр метода. И сразу подумайте - а как это протестировать?

jskonst · 2018-12-27T23:14:34Z

CourseApp/Krolik.cs

+
+      public override void NewPet()
+      {
+         Console.Write("Введите имя новорожденного кролика: ");


то же самое - вынесите в основную программу получение параметров, а здесь, только их проверьте

jskonst · 2018-12-27T23:17:59Z

CourseApp/Menu.cs

+         Vid = Console.ReadLine();
+         if (Vid == "rabbit")
+         {
+            krosh.BuyPet();


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

jskonst · 2018-12-27T23:35:03Z

CourseApp/Menu.cs

+         }
+
+         break;
+         case "Sel":


тут так просто не разобраться, попробую завтра переписать по другому

jskonst · 2018-12-27T23:42:34Z

CourseApp/NewPet.cs

+
+namespace CourseApp
+{
+    public class NewPet : Pet


Очень плохое название NewPet создается вот так - new Pet(), а тут все таки какая то иерархия

jskonst · 2018-12-29T07:40:01Z

CourseApp/Pet.cs

+
+        public List<string> Child { get => child; set => child = value; }
+
+        public string Vid {get; set; }


maybe Type?

jskonst · 2018-12-29T07:42:34Z

CourseApp/Cavy.cs

+{
+   public class Cavy : AbstractPet
+   {
+      public override void BuyPet()


Не дело морской свинки продавать домашние животные :)

jskonst · 2018-12-29T07:44:13Z

CourseApp/Cavy.cs

+
+      public override void NewPet()
+      {
+         Console.Write("Введите имя новорожденной морской свинки: ");


Осуществлять пользовательский ввод тоже не надо в классах, которые представляют сущности

jskonst · 2018-12-29T07:45:47Z

CourseApp/Cavy.cs

+      public override void NewPet()
+      {
+         Console.Write("Введите имя новорожденной морской свинки: ");
+         Name = Console.ReadLine();


Вот для всего этого есть порождающий паттерн фабрика. или фабричный метод - как раз для реализации вашей идеи

jskonst · 2018-12-29T07:47:21Z

CourseApp/Cavy.cs

+      {
+         Console.Write("Введите имя новорожденной морской свинки: ");
+         Name = Console.ReadLine();
+         Console.WriteLine("Введите имена родителй морской свинки: ");


А саму логику создания и внесения родителей следует унести в конструктор класса

jskonst · 2018-12-29T07:49:38Z

CourseApp/Pet.cs

+
+        public string Pearent2 {get; set; }
+    }
+}


Type - тут будет определятся типом наследника, так что тоже не надо

jskonst · 2018-12-29T07:52:54Z

CourseApp/Cavy.cs

+         }
+      }
+
+      public override string ToString()


Утаскиваю в родительский класс

jskonst · 2018-12-29T07:55:34Z

CourseApp/Pet.cs

+
+        public string Pearent2 {get; set; }
+    }
+}


Это позволяет заставить тех, кто реализует наследников поддреживать метод ToString - они просто обязаны будут реализовать GetInfo - который toString и вернет, в противном случае - никто не заставит их реализовать toString - и в какой то момент про это забудут

jskonst · 2018-12-29T08:00:46Z

CourseApp/NewPet.cs

+        this.Vid = v;
+        }
+
+        public NewPet(string child, string v)


Название было плохое :(

jskonst · 2018-12-29T08:06:31Z

CourseApp/Menu.cs

+            --(')('),,)-(')('),,)---(')_(')---(..(')(')-(..(')(')  ");
+        }
+
+      private void PetInfo(NewPet i, string name)


Это уже поддерживают сами pet - у них to string реализован

jskonst · 2018-12-29T08:09:37Z

CourseApp/Menu.cs

+         Krolik krosh = new Krolik();
+         Cavy cavy = new Cavy();
+         switch(go)
+         {


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

jskonst · 2018-12-29T08:16:56Z

CourseApp/Menu.cs

+
+         break;
+         case "Sel":
+         Console.WriteLine("Введите кого вы хоите разводить: rabbit/cavy");


Было дублирование кода

jskonst · 2018-12-29T08:18:04Z

CourseApp/Menu.cs

+         case "Inf":
+      {
+      {
+         Console.Write("Введите имя животного о котором хотите унать: ");


Это следует убрать - не дело меню про это знать

jskonst · 2018-12-29T08:19:50Z

CourseApp/Krolik.cs

+         Spisok.Add(Name, new NewPet("rabbit"));
+      }
+
+      public override void NewPet()


Это теперь делает базовый класс, только передавайте

jskonst · 2018-12-29T08:20:26Z

CourseApp/Cavy.cs

+         Spisok.Add(Name, new NewPet("cavy"));
+      }
+
+      public override void NewPet()


Опять же это теперь делает базовый класс, можно убирать

changed logic

K-Voland added 8 commits October 16, 2018 13:40

lab_2

47093d1

lab_2_2

e42b7eb

lab_2_3

88c17a6

lab_3

3686c4a

lab_3m

1057aa6

lab_4

0a6739a

lab_4_1

297b5d6

lab_4_2_ujas

addc7cc

lab_4_3

e3d4d8e

K-Voland and others added 7 commits December 2, 2018 21:56

lab_4_4

75548ba

lab_4_5

40078f8

lab_4_6

2f38d04

lab_4_7

e24d1da

lab_4_8

9e27494

lab_4_9

3390c59

Added stylecop for project

9969670

jskonst reviewed Dec 9, 2018

View reviewed changes

jskonst and others added 5 commits December 11, 2018 11:21

Added stylecop for project

c4a7f5d

lab_4_10

a863fda

lab_4_slom

2cd9e2a

lab_4_11_slom

2a6774d

lab_4_11M

69539c1

jskonst reviewed Dec 15, 2018

View reviewed changes

lab_5_1

09cd8ff

jskonst reviewed Dec 21, 2018

View reviewed changes

lab_5_2

2056920

jskonst reviewed Dec 27, 2018

View reviewed changes

jskonst reviewed Dec 29, 2018

View reviewed changes

CourseApp/Cavy.cs Outdated

}

}

public override string ToString()

Copy link

Contributor

jskonst Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Утаскиваю в родительский класс

jskonst reviewed Dec 29, 2018

View reviewed changes

CourseApp/NewPet.cs Outdated

this.Vid = v;

}

public NewPet(string child, string v)

Copy link

Contributor

jskonst Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Название было плохое :(

jskonst reviewed Dec 29, 2018

View reviewed changes

jskonst and others added 3 commits December 29, 2018 11:21

changed logic

76833e2

Merge pull request #1 from jskonst/K-Voland

2d2bd84

changed logic

lab_6_No_work

90f087e


		public List<string> Child { get => child; set => child = value; }

		public string Vid {get; set; }

K voland #48

Are you sure you want to change the base?

K voland #48

Uh oh!

Conversation

K-Voland commented Nov 26, 2018

Uh oh!

K-Voland commented Dec 2, 2018

Uh oh!

K-Voland commented Dec 2, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

K-Voland commented Dec 23, 2018

Uh oh!

K-Voland commented Dec 23, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment