[HQC-Part1] Homeworks


0

Понеже гледам, че все още няма тема, пускам тази. Пускайте вашите решения на домашните тук.

Аз все тамън пиша второто за Naming-Identifiers и не е готово още, но пускам написаното до сега

02.Code-Formatting

03.Naming-Identifiers

Със сигурност имам още доста да им оправям, давайте насоки ако се сещате.




Отговори



0
Условието да оправим форматирането включва ли и създаването на отделни папки с отделни класове?



0
На лекцията Стивън каза да. 

от hankrum (71 точки)

0
Дайте идея за разделяне по папки за миничките :D

от olebg (598 точки)



1

Здравей колега,

 На първо четене само в кода Bunny се виждат доста грешки: при поставянето на запетаи, space-инга, два стейтмънта на един ред , след затварянето с къдрава скоба на if, for, foreach трябва да има празен ред и т.н. Задължително трябва да си свалиш Style cop и да го използваш. Доста ще улесни работата ти.

Поздрави!




0
Благодаря колега, съвсем бях забравил, че мога да ползвам StyleCop. А отностно hungarian notaion счита ли се за грешка или трябва да си добавим в StyleCop да не пищи за префикси to и by?

от olebg (598 точки)

0
Не се счита според мен и можеш да добавиш изключения за "by" и "to" и няма да прави проблем StyleCop-a

от qwerty123 (157 точки)


1

здравей,

използването в JS на evil() се приема за лоша практика,

тоест редът

theLayer = eval('document.layers[\'menu1\']'); theLayer.visibility = 'show';

има 3 неща за пооправяне от гледна точка на добра практика

1 theLayer трябва да се декларира (може би като глобална променлива някъде в началото),

2 eval() не е добре да бъде ползвано

3 втората част е уместно да я разделиш на друг ред

тоест този ред е хубаво да изглежда така:

// някъде в началото на кода определяш типа променлива // - от самото начало var theLayer = Object; // а може theLayer да се декларира във всяка подфункция // където се ползва - това много зависи от целия контекс theLayer = document.layers.menu1; theLayer.visibility = 'show';

в C# class Startup, иницилизирането на пропъртитата

new Bunny { FurType = FurType.ALittleFluffy, Name = "Tiberii" , Age= 3, },

според мен трябва да става в еднакъв ред, може би определен от важността на информацията

например

new Bunny { Age = 1, Name = "Leonid", FurType = FurType.NotFluffy },

new Bunny { Age = 2, Name = "Rasputin", FurType = FurType.ALittleFluffy },

new Bunny { Age = 3, Name = "Tiberii", FurType = FurType.ALittleFluffy },


от mitkop (578 точки)


0

Едно допълнение и от мен - document.all (IE) и document.layers (Netscape) не се използват вече (obsolete), препоръчва се използването на getElementById('menu1').

Предвид че HideMenu1/ShowMenu1 правят същото като HideMenu2/ShowMenu2, може да се направи по една функция за двете с аргумент menuId (например), за да се избегне повтаряне на код.


от Daniela_Popova (1125 точки)

0

Идеята за Show и Hide Menu е абсолютна правилна, но може би вече не би трябвало да се нарича рефакториране на кода. По правилвия въпрос е дали function, които не се ползват трябва да съществуват в кода?

Все пак си мисля, че между document.all и getElementById има разлика. Не се сещам как с една команда да заместя document.all.

Друг въпрос е че това рядко би се наложило да се ползва - винаги има филтър при избора и по-удачен би бил getElementById.


от mitkop (578 точки)



0
Благодаря за помощта, давайте и вашите домашни в темата, да е полезно на всички.

от olebg (598 точки)


0

здрасти,

ето и моето домшно за CodeFormating


от mitkop (578 точки)