Code reviewHardcr-g586

Subject Concurrency await holding lockLevel Senior–Staff~31 minCommon in Concurrency interviewsIndustries Software development, Technology

Question

Review this C# cache that serializes refreshes with a lock.

What a strong answer looks like

Separate real bugs from style. Rank issues by severity, point at the root cause rather than the symptom, and suggest a concrete fix — specific and kind.

Learn the concepts

Talk through your review

Code to reviewcsharp

1private readonly object _gate = new object();2private Data _data;3 4public async Task<Data> GetAsync() {5    lock (_gate) {6        if (_data != null) return _data;7        _data = await FetchFromNetworkAsync(); // refresh8        return _data;9    }10}

Run or narrate your approach, then ask the coach.