Пару вопросов про синхронизацию\критическую секцию

[bakanaev]

1)

function TNewThread.Load(img:TIdMultiPartFormDataStream):bool;
begin 
img.AddFile( 'photo', 'C:\\1.jpg', 'image/jpg' );
end;

К данной функции могут обратиться несколько потоков сразу. Так что есть шанс что файл C:\\1.jpg будет считываться одновременно в несколько потоков.

Нужна ли тут синхронизация или крит. секция ?

2)

function TNewThread.GRT():string;
begin
Result:=list[RandomRange(0,list.Count)];
end;

list типа TStringList, объявлен глобально.
К данной функции могут обратиться несколько потоков сразу. Нужна ли тут крит. секция ?

3)

procedure TNewThread.Execute;
begin
if Form1.CheckBox1.Checked = true then ........
end;

Создается несколько потоков.Мы ни чего не меняем, только сверяем, но все же.Нужна ли тут синхронизация?

Спасибо за ответы

[Slym]

1. на чтение не нужна... только правильно флаги открытия файла выстави
если будет запись то нужна синхронизация через крит сек
2. если значение list.Count не меняется то можно и так, если меняется размер и содержимое то критической секцией
3. обязателен Sync

[Kix.IV]

После ответа Slym'a сомневаюсь, что я правильно понимаю, как всё это работает. Но всё же напишу, что думаю:
1 - Обязательно синхронизация или крит.секция.
2 - Не уверен. Возможно лучше будет, если будет синхронизация или крит.секция.
3 - Синхронизация или крит.секция не нужны.

[Slym]

Сообщение от Kix.IV

После ответа Slym'a сомневаюсь, что я правильно понимаю, как всё это работает. Но всё же напишу, что думаю:
1 - Обязательно синхронизация или крит.секция.
2 - Не уверен. Возможно лучше будет, если будет синхронизация или крит.секция.
3 - Синхронизация или крит.секция не нужны.

твой совет полностью противоположен
в общем случае криты нужны там где что-то меняется..
а синхрониз только для общение с формами из потока

[bakanaev]

Так, и кому верить ?:D

только правильно флаги открытия файла выстави

Как в данном случаи они выставляются)?

img.AddFile( 'photo', 'C:\\1.jpg', 'image/jpg' );

[Kix.IV]

в общем случае криты нужны там где что-то меняется..
а синхрониз только для общение с формами из потока

Частично согласен. Предлагаю разобраться.
1 - Я не знаю, как устроена эта функция, но явно что-то изменяет. Если не ставить синхронизацию, то последствия не предсказуемы.
2 - Тут я с тобой согласен.
3 - Зачем здесь синхронизация? Это ведь логиеская переменная, а не свойство или функция. Конфликта здесь не должно быть.

Так, и кому верить ?:D

Себе. Попробуй разные варианты и посмотри, что будет.

[evg_m]

К данной функции могут обратиться несколько потоков сразу

Главное в синхронизации потоков не функции, а данные с которыми она (функция) работает.
- локальные (доступные только одному конкретному экземпляру потока) никакой синхронизации не требуют (каждый может читать свою книгу у себя под одеялом);
- общие данные используемые всеми только для чтения тоже могут работать без синхронизации. (один и тот же текст на странице могут читать несколько человек одновременно);
- если кто-то может эти данные менять (делать запись) лучше бы синхронизироваться (как читать если кто-то другой листает страницы книги при одновременном чтении);
- если мы пишем (тоже самое)

Исключение атомарные(неделимые с точки зрения процессора|памяти) данные (байт,...). На этом собственно и основаны все системы синхронизации (атомарная операция фиксации захвата и ...вперед при удаче до освобождения)

1)чтение общих данных(файла) в локальный(надеюсь) объект. (п2) общий доступ к файлу давно существует.
2)чтение общих данных (п2 если все читают)(п3 если другие там копаются и правят)
3)а здесь мы прежде всего используем очень большой объект (Form1)(и еще объект поменьше checkbox1) c которыми неизвестно кто и неизвестно что делает), а не маленькую логическую переменную (Form1.checkBox1.Checked).
гарантии только чтения нет =>синхронизировать добычу данной переменной в локальное пространство потока.

3 - Зачем здесь синхронизация? Это ведь логиеская переменная, а не свойство или функция. Конфликта здесь не должно быть.

именно что свойство и даже функция. Чтение свойства у заданного объекта. (а другие потоки могут его и менять).

[Kix.IV]

TCheckBox = class(TCustomCheckBox)
  strict private
    class constructor Create;
    class destructor Destroy;
  published
    property Checked;

Действительно свойство.

[bakanaev]

Ребята спасибо, особенно evg_m. Можно закрывать тему)