#elif WINDOWS doesn't work

[nk54]

Description

I want to create a service and use dependency injection.

It works on each platform except windows.
First, #elif __WINDOWS__ wasn't taken into account by Visual Studio (see screenshot below)

Next, #elif WINDOWS was recognized by Visual Studio but not by the compiler as it tells me it can't find the implementation of my service.

It should work and the documentation MUST (just my opinion, you're the boss haha) be updated to tell to developpers what's the difference between #if __ANDROID__ and #if ANDROID
On the Microsoft documentation I saw both, it's quite confusing. Even more when #if __WINDOWS__ doesn't work while #if WINDOWS does.
Edit: answer given here don't use the double underscore.

Steps to Reproduce

• create an interface inside the MAUI project
• create an implementation of that interface inside each platforms folder
• try to register the service inside MauiProgram.cs

Version with bug

Release Candidate 2 (current)

Last version that worked well

Unknown/Other

Affected platforms

Windows (I'm using Visual Studio 17.2.0 Preview 6.0)

Affected platform versions

net6.0-windows10.0.19041

Did you find any workaround?

Yes !

I handle each platform and handle WINDOWS in the #else

But it's a dirty hack because if the problem occurs on Tizen too, it cannot work

Relevant log output

No response

[Eilon]

In my experience, the underscore versions of the define's are never used. Only the "text" defines, such as #if WINDOWS, etc. There are also version-specific defines, such as WINDOWS10_0_19041_0, but those are rarely used, because most code differs only by platform, and not specific version of the platform.

Can you show a code sample that doesn't work if you use only the "text" defines?

[Eilon]

For example, with this code:

#if WINDOWS
// Windows
#elif ANDROID
// Android
#elif IOS
// iOS
#elif MACCATALYST
// MacCatalyst
#endif

I get this result, which is what I expected (note that the IOS platform is selected, and the iOS comment is "green", meaning it is included in the compilation, and all others are "grayed out"):

[jfversluis]

We're using this in various places in our own code and is described in the docs here as being #if WINDOWS.

I don't think there is an issue here. Have a look in our codebase here to see how we're using it to guide you as an example.

[nk54]

Hi @Eilon thank you for your time.

In this thread on the maui documentation, it seems we have to use #if PLATFORMNAME instead of #if __PLATFORMNAME__

If I set #if WINDOWS the compiler doesn't find my class whereas the overview of the type in Visual Studio indicates it's found on each platform

[nk54]

@jfversluis you may missed the following part

Next, #elif WINDOWS is recognized by Visual Studio but not by the compiler as it tells me it can't find the implementation of my service.

There is a bug. Don't close this yet please

[jfversluis]

As I said: we have this in multiple places in our codebase, it works fine there.

If it says that there is still something wrong, I would think it is something else. It would greatly help if you provide a reproduction sample that shows this behavior or at the very least specify what "tells me it can't find the implementation of my service." is exactly. What is the exact error message and the code that triggers it?

[nk54]

[nk54]

@jfversluis

I've written the steps : go to my message.

Add in MauiProgram this

#if ANDROID
using SimpleApp.Platforms.Android;
#elif IOS
using SimpleApp.Platforms.iOS;
#elif WINDOWS
using SimpleApp.Platforms.Windows;
#endif

Register the service

THE BUG

With #elif WINDOWS the compiler doesn't find TestService.
Replace #elif WINDOWS by #else the compiler finds the class.

It is cleaarly a bug.

The code of service if you want to repro.
// Path SimpleApp/ITestService.cs public interface ITestService { void PrintPlatform(); } // Path SimpleApp/Platforms/Windows/TestService.cs namespace SimpleApp.Windows { public class TestService : ITestService { public void PrintPlatform() => Debug.WriteLine("Windows"); } } // Path SimpleApp/Platforms/Android/TestService.cs namespace SimpleApp.Android { public class TestService : ITestService { public void PrintPlatform() => Debug.WriteLine("Android"); } } // Path SimpleApp/Platforms/iOS/TestService.cs namespace SimpleApp.iOS { public class TestService : ITestService { public void PrintPlatform() => Debug.WriteLine("iOS"); } }

[jfversluis]

This seems to work fine for me: https://github.com/jfversluis/Repro6815

Just have to make sure that either all platforms have the TestService or exclude the registration for the platforms that are not supported or remove the TargetFrameworks from the csproj that you're not going to use

[nk54]

It builds on your repro but I can't launch it.

YES ! I found the bug. @jfversluis

In your repo, create a folder Services inside Repro6815\Platforms\Windows

Move your windows implementation inside : Repro6815\Platforms\Windows\Services\SomeService.cs

Now it doesn't build anymore even if you change the namespace