Obiekty i thread safe :: 4programmers.net

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

Cześć, mam pewien dylemat i chciałem się upewnić, czy dobrze myślę. Załóżmy, że mam metodę w klasie:

Kopiuj

 
function TMyClass.Sum(a, b: integer): integer;
begin
  cs.Enter;
  result:=a+b;
  cs.Leave;
end;

cs to oczywiście TCriticalSection.

Mam też utworzony jeden obiekt tej klasy. Jest on używany w kilku wątkach. I teraz pytanie, czy taki zapis klasy jak wyżej wystarczy, żeby uczynić ją thread safe, czy powinienem raczej stosować sekcje na obiekt, tzn:

Kopiuj

 
cs.Enter;
a:=ObjOfMyClass.Sum(1, 2);
cs.Leave;

Delphi

_13th_Dragon

2013-04-24 09:41

_13th_Dragon2013-04-24 09:41

Rejestracja:ponad 19 lat
Ostatnio:3 miesiące

To co pokazałeś jest thread safe nawet bez TCriticalSection:

Kopiuj

function TMyClass.Sum(a, b: integer): integer;
begin
  result:=a+b;
end;

i już.

Juhas

2013-04-24 09:52

Juhas2013-04-24 09:52

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

A możesz wyjaśnić dlaczego to jest thread safe?

vpiotr

2013-04-24 09:58

~~vpiotr~~2013-04-24 09:58

Rejestracja:ponad 13 lat
Ostatnio:prawie 3 lata

Juhas napisał(a):

A możesz wyjaśnić dlaczego to jest thread safe?

Bo operujesz tylko na argumentach funkcji. Nie czytasz ani nie zmieniasz obiektu.

Juhas

2013-04-24 10:24

Juhas2013-04-24 10:24

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

Aha, to teraz bardziej szczegółowo. Jest taka klasa(wersja bardzo uproszczona):

Kopiuj

type
  TMyClass = class
  private
    FADO: TADOQuery;
  public
    function ExecuteQuery(sql: string): boolean;
end;

//i ciało
constructor Create();
  FADO:=TADOQuery.Create(nil);
end;

function ExecuteQuery(sql: string): boolean;
begin
  FADO.SQL.Text:=sql;
  try
    FADO.Execute;
    result:=true;
  except
    result:=false;
  end;
end;

Jak sprawa ma się teraz?

vpiotr

2013-04-24 10:51

~~vpiotr~~2013-04-24 10:51

Rejestracja:ponad 13 lat
Ostatnio:prawie 3 lata

Zrób tak:

Kopiuj

function ExecuteQuery(sql: string): boolean;
begin
          EnterCriticalSection(CritSect);
          try
            // przetwarzanie
          finally
            LeaveCriticalSection(CritSect);
          end;
end;

Przykład bez try/finally:
http://www.delphicorner.f9.co.uk/articles/op4.htm
http://edn.embarcadero.com/article/22411

Przykład dla nowszego Delphi:
http://docwiki.embarcadero.com/RADStudio/XE3/en/Using_Critical_Sections

edytowany 1x, ostatnio: ~~vpiotr~~ 2013-04-24 10:52

_13th_Dragon

2013-04-24 13:05

_13th_Dragon2013-04-24 13:05

Rejestracja:ponad 19 lat
Ostatnio:3 miesiące

Znowu nie potrzebujesz CS:

Kopiuj

type
  TMyClass = class
  private
  public
    function ExecuteQuery(sql:String):Boolean;
end;
 
function TMyClass.ExecuteQuery(sql:String):Boolean;
var FADO:TADOQuery;
begin
  Result:=false;
  FADO:=TADOQuery.Create(nil);
  try
    FADO.SQL.Text:=sql;
    try
      FADO.Execute;
      result:=true;
    except
    end;
  finally
    FADO.Free;
  end;
end;

I niech ADO się martwi o synchronizacje.

Juhas

2013-04-24 13:13

Juhas2013-04-24 13:13

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

Chodzi o to, żeby nie tworzyć ADO w executeQuery, tylko w konstruktorze. Mam np. takie pętle, w których wykonuję kilka nawet tysięcy insertów i muszę znać rezultat każdego z nich. I teraz tworzyć i niszczyć kilka tysięcy razy obiekt podczas jednej pętli, to... ;)

_13th_Dragon

2013-04-24 13:34

_13th_Dragon2013-04-24 13:34

Rejestracja:ponad 19 lat
Ostatnio:3 miesiące

I myślisz że za pomocą synchronizacji przyspieszysz? - grubo się mylisz, wręcz odwrotnie.

vpiotr

2013-04-25 07:05

~~vpiotr~~2013-04-25 07:05

Rejestracja:ponad 13 lat
Ostatnio:prawie 3 lata

Juhas napisał(a):

Chodzi o to, żeby nie tworzyć ADO w executeQuery, tylko w konstruktorze. Mam np. takie pętle, w których wykonuję kilka nawet tysięcy insertów i muszę znać rezultat każdego z nich. I teraz tworzyć i niszczyć kilka tysięcy razy obiekt podczas jednej pętli, to... ;)

To zupełnie nie tędy droga.
Jaką masz bazę danych?

Poszukaj czegoś o bulk load dla swojej bazy
Zrób CSV z danymi które chcesz zaimportować
Odpal narzędzie masowego ładowania i załaduj CSV-a do tabeli importowej - pola np. VARCHAR(255)
W SQL-u popraw dane zaimportowane lub przekonwertuj je do tabel docelowych

Odpalanie po jednym SQL-u na rekord - w każdym środowisku - będzie wolne.

Inne rozwiązanie - jeśli to MySQL: masz np. składnie pozwalającą wpisać np. 500 rekordów jedną komendą SQL:

Kopiuj

INSERT INTO tbl_name (a,b,c) VALUES(1,2,3),(4,5,6),(7,8,9);

http://dev.mysql.com/doc/refman/5.5/en/insert.html

Ale to raczej tylko w MySQL-u. W każdej bazie jest inaczej.

Juhas

2013-04-29 11:00

Juhas2013-04-29 11:00

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

_13th Dragon, wydaje mi się, że będzie bardziej stabilne. Poza tym sekcje krytyczne chyba nie są aż tak powolne?
vpiotr - ale ja używam tej klasy w kilku wątkach. Stąd sekcje. Moja baza to MSSQL. Raz mam jeden insert, raz 1000. Ale poczytam o tym bulk load.

vpiotr

2013-04-29 11:06

~~vpiotr~~2013-04-29 11:06

Rejestracja:ponad 13 lat
Ostatnio:prawie 3 lata

Juhas napisał(a):

vpiotr - ale ja używam tej klasy w kilku wątkach. Stąd sekcje. Moja baza to MSSQL. Raz mam jeden insert, raz 1000. Ale poczytam o tym bulk load.

Jeśli nie potrzebujesz wyniku wykonania to pomyśl nad skorzystaniem ze zsynchronizowanej kolejki (struktury danych).

wiele wątków piszących do zsynchronizowanej kolejki -
jeden wątek czytający z kolejki, który wykonuje wszystkie zadania z kolejki

W ten sposób zadania piszące szybko będą mogły kontynuować pracę, a wątek zapisujący do bazy będzie równo pracował.

Kolejka może przechowywać zapytania w postaci rekordów: kod SQL (string), lista parametrów i ich wartości (Variant).

edytowany 1x, ostatnio: ~~vpiotr~~ 2013-04-29 11:08

_13th_Dragon

2013-04-29 11:15

_13th_Dragon2013-04-29 11:15

Rejestracja:ponad 19 lat
Ostatnio:3 miesiące

Tam już jest synchronizacja (wewnątrz ADO) , dodając kolejny poziom synchronizacji - bardzo spowalniasz.
Przy wywołaniu metody: function ExecuteQuery(sql:String):Boolean; (w tym wierszu: FADO.SQL.Text:=sql;) i tak tworzysz kopie napisu sql - więc przydzielasz znacznie więcej pamięci niż potrzebuje egzemplarz TADOQuery.

edytowany 1x, ostatnio: _13th_Dragon 2013-04-29 12:20

Azarien 2013-04-29 12:16

Delphi domyślnie operuje na ansistringach (albo unicodestringach w nowszych wersjach), który jest przekazywany do funkcji jako wskaźnik, nie jest kopiowany cały napis.

_13th_Dragon 2013-04-29 12:19

Źle się wyraziłem, poprawiam.

crowa

2013-04-30 17:29

crowa2013-04-30 17:29

Rejestracja:ponad 18 lat
Ostatnio:około 8 lat
Lokalizacja:Poznań
Postów:295

btw dla nowych wersji delphi jest FireDac.
Poczytaj o firedac i DMLArray.

http://docs.embarcadero.com/products/rad_studio/firedac/frames.html?frmname=topic&frmfile=uADStanOption_TADResourceOptions_ArrayDMLSize.html

A tak swoja droga taka liczba insertow to nie przesada jakas? Czy to naprawde ma uzasadnienie?

_13th_Dragon 2013-04-30 17:33

Prawdopodobnie schrzanione w jakimś wcześniejszym stadium, ale pytający nie chce naprawiać, chce załatać.

OMG

2013-05-03 23:01

OMG2013-05-03 23:01

Ludzie, po co jakieś CS, synchronizacja, skoro i tak każdy wątek tworzy osobny obiekt i połączenie z db do której wali query, o ile się nie mylę to tak masz zrobione, bo utworzenie głównego obiektu połączenia z db i podpięcie do niego wątków, które tylko wywołują zapytanie (przez obiekt bazodanowy) jest głupotą.
Jedyne co można by zrobić to try except by logować nieudane/błędne zapytania. Pamiętaj, żaden uproszczony schemat kodu nie odda prawdziwego stanu tak jak to zrobi oryginał.

Juhas

2013-05-10 13:20

Juhas2013-05-10 13:20

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

OMG - Wszystkie wątki korzystają z tego samego, jednego obiektu globalnego. Pisałem o tym w pierwszym poście.
Crowa - tak, te inserty mają uzasadnienie.

OMG

2013-05-10 20:19

OMG2013-05-10 20:19

Wybacz, zasugerowałem się drugim postem kolegi @_13th_Dragon
To powiem ci że popełniasz błąd waląc query w kilku wątkach na jednym kontrolerze, tak się nie powinno robić, bezpieczniej tworzyć kolejne połączenia na wątki.

crowa

2013-05-11 07:46

crowa2013-05-11 07:46

Rejestracja:ponad 18 lat
Ostatnio:około 8 lat
Lokalizacja:Poznań
Postów:295

dokladnie osobne polaczenia na watek lub praca asynchroniczna (mozna to ustawic w connection string)

Juhas

2013-05-11 11:27

Juhas2013-05-11 11:27

Rejestracja:około 22 lata
Ostatnio:około 2 miesiące
Postów:5042

Kwestia jest taka, że nie tworzę wszystkich wątków sam. Tzn. upraszczając. Główna aplikacja wywołuje funkcję DBProc, w której jest kilka zapytań SQL. To jest wykonywane pod przyciskiem. Ale aplikacja pracuje z SDKami, które mają swoje callbacki. I teraz, jeśli SDK odpala jakiegoś swojego callbacka, to wtedy jest wywoływana też funkcja DBProc. Czyli wg Was najlepiej byłoby wyposażyć aplikację powiedzmy w 2 obiekty TADOConnection, dodać parametr do DBProc typu TADOConnection i po prostu odpalać DBProc z odpowiednim połączeniem?

crowa

2013-05-11 12:45

crowa2013-05-11 12:45

Rejestracja:ponad 18 lat
Ostatnio:około 8 lat
Lokalizacja:Poznań
Postów:295

a connection pooling ? moze cos takiego Ci pomoze

http://dn.embarcadero.com/article/30027

Liczba odpowiedzi na stronę

Treść *

Treść

Podgląd

Kliknij, aby dodać treść...

Kliknij, aby dodać załącznik lub wklej ze schowka.

Instrukcja obsługi Markdown

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj underline.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, , , <kbd>, <del>, , <dfn>, <pre>, <blockquote>, <hr/>, ,  oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H2O i m2 otrzymasz: H₂O i m².

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript document.write('Hello World'); ```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

Alt+L - zaznaczenie całej linii
Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

Ctrl+Enter - dodaj post
⌘+Enter - dodaj post (MacOS)

Obiekty i thread safe

vpiotr

Juhas napisał(a):

vpiotr

vpiotr

Juhas napisał(a):

vpiotr

Juhas napisał(a):

OMG

OMG

Smileys & People

Animals & Nature

Food & Drink

Travel & Places

Activities

Objects

Symbols

Typografia

Linki

Wewnętrzne odnośniki

Wspomnienia użytkowników

Znaczniki HTML

Skróty klawiszowe

Indeks górny oraz dolny

Składnia Tex

Kod źródłowy

Krótkie fragmenty kodu

Kod wielolinijkowy

Tabelki

Lista uporządkowana i nieuporządkowana

Składnia Markdown

Skróty klawiszowe