Mój mały projekt

Question

Chciałbym wam przedstawić mój projekt. Jest strona podobna do https://www.strawpoll.me/ . Strona umożliwia

tworzenie dowolnych ankiet. Byłbym wdzięczny za opinie i uwagi.  Przy tworzeniu strony używałem asp.net , linq.

Link do strony: http://vvector-001-site1.btempurl.com/

Link do github: https://github.com/vvector2/ExampleOfStrawPoll

Answer 1

Jak dla mnie za dużo logiki w kontrolerach, ale na tą skale aplikacji w sumie nie ma sensu rozbudowa architektury 

na pierwszy rzut oka widze nieźle skomplikowane if() i else() 
najlepiej robić to tak, aby ich ciała jasno mówiły co się dzieje
a nie potężne wgłębienia, gdzie trzeba się zagłębić w kod aby sprawdzić co to tak właściwie robi 

przykład - jeden z wielu 

if (model.cookieProtection) {
                ///handling cookies by class Cookies
                if (Request.Cookies["identifier"] == null) {
                    ///creating new cookie
                    Response.Cookies["identifier"].Value = model.id.ToString() + ",";
                } else {
                    cookie = new Cookies(Request.Cookies["identifier"]);
                    if (!cookie.validateCookie(model.id.ToString())) {
                        ModelState.AddModelError("VotingValidation", "You have already voted on this poll !!");
                        return View("Vote", model);
                    }
                    //update cookie
                    Response.Cookies["identifier"].Value = cookie.getCookie().Value;
                }
                Response.Cookies["identifier"].Expires = DateTime.Now.Date.AddMonths(1);
            }

rozdzielając logike na metody albo najlepiej na klasy o pojedynczej odpowiedzialności, całość byłaby dużo bardziej czytelna 

Comments

rozdzielając logike na metody albo najlepiej na klasy o pojedynczej odpowiedzialności, całość byłaby dużo bardziej czytelna 

Nie masz na myśli serwisów? Lepiej odrazu podawać nazwy, bo jeszcze będzie koło na nowo wymyślał.

Answer 2

Witam,

Ja może skupię się na wyglądzie i działaniu. Mi, jako zwykłemu użytkownikowi, bardzo przeszkadza to, że aby zaznaczyć checkboxa nie wystarczy kliknąć na napis, tylko trzeba na ten mały kwadracik. Też warto popracować nad okienkiem tworzenia ankiety, bo wygląda średnio i za dużo widzę szarego tła. Poza tym ankiety chyba powinny sortować się od najnowszej do najstarszej. Za to plus za responsywność :)