69

일부 비동기 함수가있는 인터페이스가 있습니다. 인터페이스를 구현하는 클래스 중 일부는 기다릴 것이없고 일부는 던질 수도 있습니다. 모든 경고와 함께 약간 짜증납니다.

비동기 함수에서 기다리는 것을 사용하지 않을 때.

메시지를 억제 할 수 있습니까?

public async Task<object> test()
{
    throw new NotImplementedException();
}

경고 CS1998 :이 비동기 메서드에 'await'연산자가없고 실행됩니다.   동기식으로. 기다리기 위해 'await'연산자 사용을 고려하십시오.   논 블로킹 API 호출, 또는 'Task.Run (...)'대기열로 CPU 바인딩 작업 수행   배경 스레드.


  •   "비동기 함수에서 기다리고 있지 않을 때"라는 뜻입니까? - John Saunders
  • 비동기로 표시된 함수에서 새 await 키워드를 사용하지 않는 경우. - Simon
  • 문제를 재현하는 코드 샘플을 보여 주면 어떨까요? - John Saunders
  • 예를 추가했습니다. - Simon

13 답변


77

일부 비동기 함수가있는 인터페이스가 있습니다.

반환되는 메소드Task, 나는 믿는다.async구현 세부 사항이므로 인터페이스 메소드에 적용 할 수 없습니다.

인터페이스를 구현하는 클래스 중 일부는 기다릴 것이없고 일부는 던질 수도 있습니다.

이 경우, 귀하는async구현 세부 사항입니다.

너에게 할 말이 없다면await, 당신은 단지 돌아올 수 있습니다.Task.FromResult:

public Task<int> Success() // note: no "async"
{
  ... // non-awaiting code
  int result = ...;
  return Task.FromResult(result);
}

던지는 경우NotImplementedException, 절차는 조금 더 어리 석다 :

public Task<int> Fail() // note: no "async"
{
  var tcs = new TaskCompletionSource<int>();
  tcs.SetException(new NotImplementedException());
  return tcs.Task;
}

던지기가 많은 방법이 있다면NotImplementedException(그 자체로는 디자인 수준의 리팩토링이 좋음을 나타낼 수 있음), 단어 속성을 도우미 클래스로 마무리 할 수 있습니다.

public static class TaskConstants<TResult>
{
  static TaskConstants()
  {
    var tcs = new TaskCompletionSource<TResult>();
    tcs.SetException(new NotImplementedException());
    NotImplemented = tcs.Task;
  }

  public static Task<TResult> NotImplemented { get; private set; }
}

public Task<int> Fail() // note: no "async"
{
  return TaskConstants<int>.NotImplemented;
}

또한 헬퍼 클래스는 GC가 그렇지 않으면 수집해야하는 쓰레기를 줄입니다. 왜냐하면 동일한 반환 유형을 가진 각 메서드는TaskNotImplementedException사물.

나는 다른 여러 가지가있다.내 AsyncEx 라이브러리에 "task constant"유형의 예제를 입력하십시오..


  • 나는 그 키워드를 잃어 버리지 않을 것이라고 생각하지 않았다. Async는 인터페이스와 아무 관련이 없다고 말합니다. 나쁘다, 고마워. - Simon
  • 반환 유형이 Task 일 때 (결과가없는 경우) 접근 방식을 권장 할 수 있습니까? - Mike
  • @ 마이크 : 이후Task<T>에서 상속하다Task, 동일한 솔루션 (Task.FromResult,TaskCompletionSource) 잘 작동합니다. 일반 텍스트를 만드는 쉽고 효율적인 방법이 없습니다.Task이렇게하면 광고를 만드는 것이 가장 좋습니다.Task<T>그리고 (묵시적으로) 그것을에 던지십시오.Task. - Stephen Cleary
  • 경고:이 방법은 오류가 예상대로 전달되지 않으므로 문제를 일으킬 수 있습니다. 일반적으로 호출자는 메서드 내에서 예외가 발생할 것으로 예상합니다.Task. 대신, 메소드를 생성 할 기회가 생기기 전에 메소드가 throw됩니다.Task. 정말 가장 좋은 패턴은async방법이없는await연산자. 이렇게하면 메소드 내의 코드가 모두의 일부로 취급되도록 보장합니다.Task. - Bob Meyers
  • CS1998을 피하려면 다음을 추가하십시오.await Task.FromResult(0);당신의 방법에. Task.Yield ()와 달리 퍼펙트에 큰 영향을주지 않아야합니다. - Bob Meyers

51

또 다른 옵션은 함수의 본문을 단순하게 유지하고 코드를 작성하지 않으려는 경우 #pragma를 사용하여 경고를 생략하는 것입니다.

#pragma warning disable 1998
public async Task<object> Test()
{
    throw new NotImplementedException();
}
#pragma warning restore 1998

이것으로 충분하다면 disable 문을 파일의 맨 위에 놓고 복원을 생략 할 수 있습니다.

http://msdn.microsoft.com/en-us/library/441722ys(v=vs.110).aspx


32

async 키워드를 유지하는 또 다른 방법은 다음과 같이 사용하는 것입니다.

public async Task StartAsync()
{
    await Task.Yield();
}

메소드를 채우면 간단히 명령문을 제거 할 수 있습니다. 나는 메소드가 실제로 기다리고 있을지 모르지만 모든 구현이 실제로 그런 것은 아니기 때문에 이것을 많이 사용한다.


  • 이것은 받아 들여진 대답이어야합니다. 때로는 인터페이스 구현이 비동기 일 필요는 없으며, 이것은 모든 것을 포장하는 것보다 훨씬 깔끔합니다.Task.Run요구. - Andrew Theken
  • Task.CompletedTask를 기다립니다. // 더 나은 옵션 일 수 있습니다. - Frode Nilsen
  • 왠지 @FrodeNilsenTask.CompletedTask더 이상 존재하지 않는 것 같습니다. - Sebastián Vansteenkiste
  • @ Sebasti & TheVansteenkiste.Net Framework 4.6->, UWP 1.0-> .Net Core 1.0-> - Frode Nilsen
  • @ AndrewTheken이 답변과 귀하의 의견은 구현이 비어 있거나 예외를 던지는 경우에만 적용된다는 결론에 도달하는 데 다소 시간이 걸렸습니다 (원래 질문 에서처럼). 구현에서 값을 반환하면Task.FromResult더 나은 대답입니다. 그 점 때문에,아르예외를 던질 것 같아서 다른 대답이 생겨났다.Task.FromException이것은 결코 이상적인 솔루션이 아닙니다. 동의하니? - BlueMonkMN

7

이 스레드는 오래된 스레드이며, 모든 스레드에서 올바른 결과를 얻지는 못하지만, 다음은 메서드를 아직 구현하지 않았을 때 단순히 NotImplementedException을 throw 할 수있는 것과 비슷하지만, 메서드 서명을 변경하지 않고 그것이 문제가된다면 기꺼이 알게 될 것이지만, 나에게는 그다지 중요하지 않습니다. 어쨌든 개발 중에 만 사용하므로, 어떻게 수행하는지는 중요하지 않습니다. 그래도 나는 왜 그것이 나쁜 생각인지에 대해 듣게되어 기쁩니다.

public async Task<object> test()
{
    throw await new AwaitableNotImplementedException<object>();
}

그게 가능하도록하기 위해 내가 추가 한 타입이 있습니다.

public class AwaitableNotImplementedException<TResult> : NotImplementedException
{
    public AwaitableNotImplementedException() { }

    public AwaitableNotImplementedException(string message) : base(message) { }

    // This method makes the constructor awaitable.
    public TaskAwaiter<AwaitableNotImplementedException<TResult>> GetAwaiter()
    {
        throw this;
    }
}


7

Stephen 's Answer에 대한 업데이트와 마찬가지로 더 이상TaskConstants클래스에는 새로운 도우미 메서드가 있습니다.

return Task.FromException(new NotImplementedException());


6

답변에서 찾을 수있는 다른 솔루션에는 차이가 있습니다. 엄격히 말하면 호출자가 비동기 메서드를 호출하는 방법을 알고 있어야하지만 메서드 결과에 ".Wait ()"가 있다고 가정하는 기본 사용 패턴을 사용해야합니다.Task.CompletedTask를 반환합니다."최고의 솔루션입니다.

    BenchmarkDotNet=v0.10.11, OS=Windows 10 Redstone 3 [1709, Fall Creators Update] (10.0.16299.192)
Processor=Intel Core i5-2500K CPU 3.30GHz (Sandy Bridge), ProcessorCount=4
Frequency=3233537 Hz, Resolution=309.2589 ns, Timer=TSC
.NET Core SDK=2.1.2
  [Host] : .NET Core 2.0.3 (Framework 4.6.25815.02), 64bit RyuJIT
  Clr    : .NET Framework 4.7 (CLR 4.0.30319.42000), 64bit RyuJIT-v4.7.2600.0
  Core   : .NET Core 2.0.3 (Framework 4.6.25815.02), 64bit RyuJIT


         Method |  Job | Runtime |         Mean |       Error |      StdDev |       Median |          Min |          Max | Rank |  Gen 0 |  Gen 1 |  Gen 2 | Allocated |
--------------- |----- |-------- |-------------:|------------:|------------:|-------------:|-------------:|-------------:|-----:|-------:|-------:|-------:|----------:|
 CompletedAwait |  Clr |     Clr |    95.253 ns |   0.7491 ns |   0.6641 ns |    95.100 ns |    94.461 ns |    96.557 ns |    7 | 0.0075 |      - |      - |      24 B |
      Completed |  Clr |     Clr |    12.036 ns |   0.0659 ns |   0.0617 ns |    12.026 ns |    11.931 ns |    12.154 ns |    2 | 0.0076 |      - |      - |      24 B |
         Pragma |  Clr |     Clr |    87.868 ns |   0.3923 ns |   0.3670 ns |    87.789 ns |    87.336 ns |    88.683 ns |    6 | 0.0075 |      - |      - |      24 B |
     FromResult |  Clr |     Clr |   107.009 ns |   0.6671 ns |   0.6240 ns |   107.009 ns |   106.204 ns |   108.247 ns |    8 | 0.0584 |      - |      - |     184 B |
          Yield |  Clr |     Clr | 1,766.843 ns |  26.5216 ns |  24.8083 ns | 1,770.383 ns | 1,705.386 ns | 1,800.653 ns |    9 | 0.0877 | 0.0038 | 0.0019 |     320 B |
 CompletedAwait | Core |    Core |    37.201 ns |   0.1961 ns |   0.1739 ns |    37.227 ns |    36.970 ns |    37.559 ns |    4 | 0.0076 |      - |      - |      24 B |
      Completed | Core |    Core |     9.017 ns |   0.0690 ns |   0.0577 ns |     9.010 ns |     8.925 ns |     9.128 ns |    1 | 0.0076 |      - |      - |      24 B |
         Pragma | Core |    Core |    34.118 ns |   0.4576 ns |   0.4281 ns |    34.259 ns |    33.437 ns |    34.792 ns |    3 | 0.0076 |      - |      - |      24 B |
     FromResult | Core |    Core |    46.953 ns |   1.2728 ns |   1.1905 ns |    46.467 ns |    45.674 ns |    49.868 ns |    5 | 0.0533 |      - |      - |     168 B |
          Yield | Core |    Core | 2,480.980 ns | 199.4416 ns | 575.4347 ns | 2,291.978 ns | 1,810.644 ns | 4,085.196 ns |   10 | 0.0916 |      - |      - |     296 B |

노트 :FromResult직접적으로 비교할 수는 없다.

테스트 코드 :

   [RankColumn, MinColumn, MaxColumn, StdDevColumn, MedianColumn]
   [ClrJob, CoreJob]
   [HtmlExporter, MarkdownExporter]
   [MemoryDiagnoser]
 public class BenchmarkAsyncNotAwaitInterface
 {
string context = "text context";
[Benchmark]
public int CompletedAwait()
{
    var t = new CompletedAwaitTest();
    var a = t.DoAsync(context);
    a.Wait();
    return t.Length;
}

[Benchmark]
public int Completed()
{
    var t = new CompletedTest();
    var a = t.DoAsync(context);
    a.Wait();
    return t.Length;
}

[Benchmark]
public int Pragma()
{
    var t = new PragmaTest();
    var a = t.DoAsync(context);
    a.Wait();
    return t.Length;
}

[Benchmark]
public int Yield()
{
    var t = new YieldTest();
    var a = t.DoAsync(context);
    a.Wait();
    return t.Length;
}

    [Benchmark]
    public int FromResult()
    {
        var t = new FromResultTest();
        var t2 = t.DoAsync(context);
        return t2.Result;
    }

public interface ITestInterface
{
    int Length { get; }
    Task DoAsync(string context);
}

class CompletedAwaitTest : ITestInterface
{
    public int Length { get; private set; }
    public async Task DoAsync(string context)
    {
        Length = context.Length;
        await Task.CompletedTask;
    }
}

class CompletedTest : ITestInterface
{
    public int Length { get; private set; }
    public Task DoAsync(string context)
    {
        Length = context.Length;
        return Task.CompletedTask;
    }
}

class PragmaTest : ITestInterface
{
    public int Length { get; private set; }
    #pragma warning disable 1998
    public async Task DoAsync(string context)
    {
        Length = context.Length;
        return;
    }
    #pragma warning restore 1998
}

class YieldTest : ITestInterface
{
    public int Length { get; private set; }
    public async Task DoAsync(string context)
    {
        Length = context.Length;
        await Task.Yield();
    }
}

    public interface ITestInterface2
    {
        Task<int> DoAsync(string context);
    }

    class FromResultTest : ITestInterface2
    {
        public async Task<int> DoAsync(string context)
        {
            var i = context.Length;
            return await Task.FromResult(i);
        }
    }

}


  • 불행한 점은#pragma하나는 오버 헤드가 발생하는 것 같습니다. 아마도 반환하는 대신 오버 헤드만큼의 오버 헤드가있을 것입니다.CompletedTask당신은AsyncOperation. 컴파일러에게 메서드가 동 기적으로 실행될 때 건너 뛰어도 무방하다는 것을 알릴 수 있으면 좋을 것입니다. - binki
  • 당신은 얼마나 비슷한가요?Task.CompletedTask~와 비슷하다Task.FromResult? 흥미로운 점은 FromResult가 가장 유사하고 값을 반환해야하는 경우에도 여전히 최고의 연기자라고 생각합니다. - BlueMonkMN
  • 나는 그것을 추가 할 것이다. 나는이 경우에 상태 머신 코드가 더 장황 해지고 CompletedTask가 이길 것이라고 생각한다. - Roman Pokrovskij

3

Reactive Extension에 이미 링크되어있는 경우 다음 작업을 수행 할 수 있습니다.

public async Task<object> NotImplemented()
{
    await Observable.Throw(new NotImplementedException(), null as object).ToTask();
}

public async Task<object> SimpleResult()
{
    await Observable.Return(myvalue).ToTask();
}

리 액티브 및 비동기 / 대기는 모두 놀랍지 만 잘 작동합니다.

포함 된 항목 :

using System.Reactive.Linq;
using System.Reactive.Threading.Tasks;


2

아래 cs1998이 발생할 수 있습니다.

public async Task<object> Foo()
{
    return object;
}

그런 다음 아래에서 개혁 할 수 있습니다.

public async Task<object> Foo()
{
    var result = await Task.Run(() =>
    {
        return object;
    });
    return result;
}


1

기다릴 것이 아무것도 없다면 Task.FromResult를 반환하십시오.

public Task<int> Success() // note: no "async"
{
  ... // Do not have await code
  var result = ...;
  return Task.FromResult(result);
}


1

메소드 서명에 따라 몇 가지 대안이 있습니다.

    public async Task Test1()
    {
        await Task.CompletedTask;
    }

    public async Task<object> Test2()
    {
        return await Task.FromResult<object>(null);
    }

    public async Task<object> Test3()
    {
        return await Task.FromException<object>(new NotImplementedException());
    }


-1

// This is to get rid of warning CS1998, please remove when implementing this method.
await new Task(() => { }).ConfigureAwait(false);
throw new NotImplementedException();


0

이 시도:

[System.Diagnostics.CodeAnalysis.SuppressMessage("Await.Warning", "CS1998:Await.Warning")]

만나다:https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.codeanalysis.suppressmessageattribute?view=netframework-4.7.2


-1

메서드에서 async 키워드를 삭제하고 Task를 반환하도록 할 수 있습니다.

    public async Task DoTask()
    {
        State = TaskStates.InProgress;
        await RunTimer();
    }

    public Task RunTimer()
    {
        return new Task(new Action(() =>
        {
            using (var t = new time.Timer(RequiredTime.Milliseconds))
            {
                t.Elapsed += ((x, y) => State = TaskStates.Completed);
                t.Start();
            }
        }));
    }

연결된 질문


관련된 질문

최근 질문