• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Mój mały projekt

0 głosów
185 wizyt
pytanie zadane 30 maja 2018 w Nasze projekty przez Wunsz Użytkownik (640 p.)
zmienione kategorie 31 maja 2018 przez draghan
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

2 odpowiedzi

+1 głos
odpowiedź 30 maja 2018 przez marcin99b Maniak (64,090 p.)

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 

komentarz 30 maja 2018 przez HaKIM Szeryf (83,060 p.)

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ł.

+1 głos
odpowiedź 30 maja 2018 przez OdsetekGlupoty Stary wyjadacz (13,300 p.)
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ść :)

Podobne pytania

+6 głosów
2 odpowiedzi 457 wizyt
pytanie zadane 26 listopada 2015 w Nasze projekty przez Sprytnykrzys Gaduła (3,920 p.)
+8 głosów
4 odpowiedzi 271 wizyt
pytanie zadane 26 września 2018 w Rozwój zawodowy, nauka, szkoła, praca przez Paweł Romanoski Początkujący (330 p.)
0 głosów
1 odpowiedź 110 wizyt
Porady nie od parady
Zadając pytanie postaraj się o odpowiedni tytuł, kategorię oraz tagi.Tagi

64,062 zapytań

110,456 odpowiedzi

231,324 komentarzy

47,827 pasjonatów

Przeglądających: 216
Pasjonatów: 19 Gości: 197

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...