Witam,
programowania uczę się dość krótko. W zasadzie przerobiłem tylko internetowe kursy c++ a teraz wziąłem się za książkę c# rusz głową. Jest tam sporo o obiektowości, ale często mam jakieś wątpliwości. Chodzi mi tu o taki dryg, takie przeczucie co powinno być metodą, co klasą, jak bardzo rozdzielać ten cały kod. Domyślam się, że jest to dość typowe dla początkujących, gdy zastanawiają się czy lepiej rozbić ten blok kodu na metody, czy nie. Chciałbym jednak, żebyście mi odpowiedzieli tak bardziej praktycznie.
Załóżmy, ze piszę aplikację, która działa jak bot do np. youtube. Bot powinien posiadać możliwości zakładania kont, łapkowania filmów, oglądania ich, pamiętania które filmy odwiedził. No i tu mam pewne wątpliwości - nie tyle jak to okodować, ale w jaki sposób zaprojektować tego bota. Wymyśliłem sobie taki scenariusz:
Obiekt formularza Form1
Będzie on posiadał metody zwracające i ustawiające wartość kontrolek, np. textboxów (chyba to jest jedyna opcja, bo chciałbym, by program był wielowątkowy. Wtedy nie mógłbym z innego wątku zmienić właściwości danego textboxa. Musiałbym to zrobić to wewnątrz metody, gdzie sprawdzałbym właściwość invoke required danego obiektu kontrolki). W taki sposób mógłbym w textboxie wpisać np. login i hasło a bot dodałby nowe konto do listy.
No właśnie: konta. Wydaje mi się, że najlepiej byłoby zrobić klasę Account:
class Account {
public string Login {get; private set;}
public string Password {get; private set;}
public CookieContainer Cookie {get; private set;}
public Account (string login, string password, CookieContainer cookie) {
this.Login = login;
this.Password = password;
this.Cookie = cookie;
}
}
Dzięki temu wszystko miałbym spójne. Za każdym razem tworząc obiekt konta miałbym i ciasteczka i wszystkie dane wewnątrz instancji klasy.
Ale co z akcjami np. łapkowania filmów? Mógłbym zrobić klasę link:
class Link {
public string Url {get; private set;}
public List<Comment> Comments = new List<Comment>();
public int Upvotes;
public int Downvotes;
}
oraz klasę. Bot, która posiadałaby:
class Bot {
public RestClient Client = new RestClient("http://youtube.com"); //obiekt klasy RestClient
public List<Account> Accounts = new List<Account>();
public bool voteVideo(Account account, Video video) {
//
}
}
No właście, ale czy oddanie łapki na film powinno być zadaniem klasy Account, czy Bot? To samo z logowaniem na konto. Powinien robić to bot:
public bool Login(Account account) {
return //sprawdzamy czy konto moze sie zalogowac
}
czy może:
public bool Login(Account account) {
return account.CanLogin();
}
czy może w klasie Account:
public bool Login() {
return //sprawdzamy czy mozna sie zalogowac
}
Jak myslicie, ktora opcja jest najlepsza?
Reszta pytan:
- Ktore stworzenie listy jest lepsze:
public List<Account> Accounts;
public Bot(Form1 form) {
this.Accounts = new List<Account>();
}
czy:
public List<Account> Accounts = new List<Account>();
public Bot(Form1 form) {
}
- Chciałbym też np. zrobić możliwość dodania historii tego co się dzieje. Takie
Console.WriteLineale np. w textboxie. Nie ma problemu by tę metodę, czy metodęWriteAllTextwykonać w każdej klasie, która posiada odpowiedniusing. Ale gdybym chciał zrobić własne zapisywanie tekstu, które byłoby dostępne w każdej klasie i umożliwiałoby zapisanie tekstu do textboxa? Na ten moment wpadłem na to, by tak zmodyfikować klaseBot:
class Bot {
Form1 form;
public Bot(Form1 form) {
this.form = form
}
}
Dzięki temu w klasie bot uzyskuję dostęp do referencji formularza, a co za tym idzie dostep do metody np.
public bool AddLog(string message) {
return this.SetTextboxText(this.historyTextBox);
}
- No właśnie, czy funkcja
AddLogpowinna zwracaćbool? Chodzi tu o to, że miałbym możliwość sprawdzenia, czy funkcja się udała modyfikując ją tak:
public bool AddLog(string message) {
try {
return this.SetTextboxText(this.historyTextBox);
} catch {
return false;
}
}
Ale czy może należałoby tu użyć zwykłego void? Co o tym myślicie?
- Powiedzmy, że w moim bocie za wszystkie akcje będzie odpowiadała klasa
Bot. Metody będą miały nagłówek taki:
public bool VoteVideo(Account account, Video video)
public bool VoteComment(Account account, Comment comment)
public bool AddVideo(Account account, File video)
Teraz np. okazuje się, że przed każdą akcją potrzebuję pobrać token z kodu źródłowego. Powinienem zrobić osobną klasę Token i tam umożliwić zwrócenie tokenu po przesłaniu kodu źródłowego:
public string GetToken(Account account, string siteHtml)
czy może całą operację pobierania tokenu powinienem zawrzeć w metodzie tej klasy:
public string GetToken(Account account, Link link)
Druga opcja powodowałaby, że musiałbym dodać usingi do klasy Token tylko zeby zwrocic wartosc.
Czy może pobieranie tokenu powinno być zadaniem klasyBot, a sam token powinien być stringiem, a nie klasą (token to w zasadzie tylko string, ale jako klasa moglby miec wlasciwosc string i metode zwracajaca ten string z kodu zrodlowego strony)
-
Kolejna sprawa. W funkcji
GetTokenprzekazuję cały obiekt, ale głównie chodzi mi tylko o właściwośćCookieContainertego obiektu. Czy w takim razie powinienem stworzyć funkcje, która przyjmuje za argument obiektCookieContainer, czy możeAccount, który ma referencję do obiektuCookieContainer? Co prawda mógłbym zrobić oba sposoby przesłaniając metodę, ale wtedy dochodzą do takiego problemu, że mam pełno przesłoniętych metod, a korzystam albo z jednego, albo z drugiego sposobu. -
Publiczne metody i właściwości pisać tak:
GetToken,Login, czy tak:getToken,login? -
Czy taka struktura programu, który opisałem byłaby dobra:
formzawiera referencję do obiektubot.
botjako "mózg" całego bota zawiera referencję do obiektuformw celu pisania logów. Posiada on rówież listę obiektów klasyAccountoraz listę obiektów klasyLink. Bot posiada wszystkie akcje łapkowania, oglądania, dodawania filmów itd. Za argumenty przyjmuje konta, linki itd.
Każdy link posiada listę obiektów klasyComment, która zaś posiada właściwość account, które jest obiektem klasyAccount. No właśnie. account powinno być polem czy właściwością? Chodzi o to, by każdy komentarz był przypisany do konkretnego obiektu konta.