2

Estoy trabajando en un desarrollo en C#/.net8.0/winform

Tengo el siguiente codigo:

incluyendo algunas formas de validacion/control de datos:

internal static string GetWorkDirectory()
{
    string root;
    string path = Environment.SystemDirectory;
    if (!string.IsNullOrEmpty(path) || Path.IsPathRooted(path) )
    {
        root = Path.GetPathRoot(path);
    }
    else
    {
        root = @"C:/";
    }

    return Path.Combine(root, @"Test");
}

segun lo que estoy intentando he guardado Environment.SystemDirectory dentro de la variable string path esto lo he hecho para poder luego hacer una validacion/control usando if (!string.IsNullOrEmpty(path) || Path.IsPathRooted(path) )

porque?

no quiero aceptar cualquier string o null en la variable path, quiero que este sea un string rooted valido.

que es lo que sucede o no funciona como esperas?

a pesar de hacer las validaciones al usar la variable path dentro de otro metodo obtengo un warning persistente:

Lo que entiendo de este warning es: que la variable path aun puede ser null o es un tipo null-able, lo cual es incorrecto si previamente hice las validaciones para evitarlo.

introducir la descripción de la imagen aquí

Odio los Warning ... en el codigo ... es como si el IDE me dijero esta esta mal hecho o mal controlado ...

Por favor enfoquence en la variable path ... NO lo hagan en el metodo Path.GetPathRoot este es para otra pregunta ...

11
  • Si te fijas la "firma" de la función: string? Path.GetPathRoot(string? path) es decir, la función retorna una cadena eventualmente nula, y con root = Path.GetPathRoot(path); estas haciendo string = string?, en todo caso deberías definir la variable root así: string? root;, esto modo de spoiler, si consigo tiempo me gustaría darte una respuesta más abarcativa. Commented el 3 may. a las 19:40
  • @PatricioMoracho he actualizado el post para que se entienda y se enfoquen en la variable path Commented el 3 may. a las 19:44
  • 1
    Creo que estas confundido path no es null aquí no es el warning, es una anotación informativa, el warning en sí lo genera root Commented el 3 may. a las 20:12
  • @PatricioMoracho es una cadena de suceciones ... Commented el 3 may. a las 20:15
  • 1
    Aunque no tenga sentido, el compilador es bastante tonto todavía. Aguante Kotlin
    – Mateo
    Commented el 4 may. a las 3:25

4 respuestas 4

3

El Warning CS8600 - Converting null literal or possible null value to non-nullable type es bastante claro, estas queriendo convertir un valor o literal posiblemente nulo a un valor no "nulleable". El problema es básicamente esto:

string root;
root = Path.GetPathRoot(path);

Si revisamos la firma de la función string? Path.GetPathRoot(string? path) podemos ver que se espera un valor "nulleable" path y retorna un valor también "nulleable", el "problema" es que justamente root es "no nulleable" y no es consistente con el retorno de l función, esto es potencialmente peligroso, y digo potencialmente, puede que jamás ocurra, pero la interacción de los tipos de variables hace que ocurra este warning. Hay varias alternativas:

1. Hacer root "nulleable"

Es decir, definir la variable como una cadena "nulleable" string?:

string? root;

Esto en tu caso tiene un problema, que luego usas la variable en un contexto dónde se espera que sea "no nulleable": Path.Combine(root, @"Test");

2. Hacer un null checking

Usando la anotación ??, conocida como operador de coalescencia nula, se puede establecer el valor por defecto en caso de null:

root = Path.GetPathRoot(path) ?? "";

Con esto en el caso que Path.GetPathRoot(path) sea null el valor será "" o cualquier otro que definas.

3. Ignorar el caso en particular

root = Path.GetPathRoot(path)

esto simplemente elimina el warning, pero no impide asignar un valor null a una cadena no nulleble, ¿por que pasa esto? bueno, eso es definitivamente material para una nueva pregunta.

7
  • a pesar que todo lo que has dicho es cierto, ninguna de las 3 opciones es lo que prentendia implementar ... por decirlo asi fue la parte que forzosamente tuve que aprender ... pero no cumple... Commented el 3 may. a las 21:46
  • opcion 1 ya lo digiste: necesito un dato no nulleable para ser usado despues ... Commented el 3 may. a las 21:47
  • opcion 2: no puedo retornar un ?? "" esto remperia el programa por que se supone que estoy trabajando con directorios ... en donde quiero guardar unos settings personalizados, esto quiere decir que en lugar de aceptar un null o vacio deberia de emitir un throw exception ya que no seria compatible con el programa... y cerrar el programa... Commented el 3 may. a las 21:48
  • opcion 3: jamas ignoro el caso en particular, trato de darle una respuesta a todo aquello que afecta el correcto funcionamiento. Commented el 3 may. a las 21:48
  • mi respuesta seria la opcion4: hacer algo custom que cumpla con las necesidades. Commented el 3 may. a las 21:49
1

Tu problema es que definiste "root" como una cadena simple, pero deseas asignarle una cadena que acepte valores null.

string root;
...
root = Path.GetPathRoot(path);

Pruebas que el "path" no sea null, pero la función "GetPathRoot()" devuelve una cadena que acepta valores null ("string?").

No puedes asignar un "string?" a un "string" en c#

Necesitas cambiar la definición de "root" para que sea "string?", o convertir el resultado de Path.GetPathRoot() de "string?" a "string".

9
  • Tus controles están en la variable "path", no en el valor que se devuelve de GetPathRoot(). Son muy diferentes
    – Peter M
    Commented el 3 may. a las 19:33
  • 1
    Pero "root" es "string", y GetPathRoot() se deveulve "string?"
    – Peter M
    Commented el 3 may. a las 19:36
  • No, NO sale que path puede ser null, sale que Path.GetPathRoot puede ser null
    – Yussef
    Commented el 3 may. a las 19:36
  • 1
    por eso, esto root = Path.GetPathRoot(path) ?? "C:/"; soluciona el problema
    – Yussef
    Commented el 3 may. a las 19:36
  • he actualizado el post para que se entienda y se enfoquen en la variable path Commented el 3 may. a las 19:44
0

Esto es un falso warning, es una deficiencia del Visual Studio, ya que no es capaz de resolver que el nulleable está validado

Por ejemplo, esto arroja un warning:

introducir la descripción de la imagen aquí

Al validarlo desaparece:

introducir la descripción de la imagen aquí

Esto es similar al caso:

if (Path.IsPathRooted(path) )
{
   root = Path.GetPathRoot(path);
}

Si IsPathRooted == true NUNCA Path.GetPathRoot devolverá un nulo, pero está vez Visual Studio no es capaz de validar esa situación, y es por eso que arroja un falso warning.

Esto se resuelve simplemente agregando el operador !

if (Path.IsPathRooted(path) )
{
   root = Path.GetPathRoot(path)!; //notar el "!" al final
}

Por otro lado, Environment.SystemDirectory en windows siempre retornará una carpeta con root (nunca entregará una ruta relativa). En cambio, en Linux no retornará valor alguno. Y en cuyo caso la raíz es siempre /, por lo que simplemente se puede hacer:

private const string FOLDER = "Test";
internal static string GetWorkDirectory()
{
    string path = Environment.SystemDirectory;
    if (string.IsNullOrEmpty(path)) //True para linux
    {
        return "/" + FOLDER;
    }
    return Path.Combine(Path.GetPathRoot(path)!, FOLDER);
}

EDIT

Otra manera sin hacer uso del !

private const string FOLDER = "TEST";
internal static string GetWorkDirectory()
{
    string? path = Path.GetPathRoot(Environment.SystemDirectory);
    if (string.IsNullOrEmpty(path))
    {
        return "/" + FOLDER;
    }
    return Path.Combine(path, FOLDER);
}
11
  • 100% de acuerdo con vos.. pero tengamos algo en cuenta.. se esta validando lo mismo, pero no se esta comprobando lo mismo... o sea.. cuando llega a esa linea, no entiende que la funcion anterior esta validando lo que va a retornar la funcion siguiente, y es para una ultra discusion, si la ide deberia o no darse cuenta que la funcion anterior si valida que la funcion posterior no va a devolver null... es muy complejo, y se resuelve con el ! al final, porque de estos, hay mil ejemplos donde la ide no puede darse cuenta que el valor no es nulo...
    – gbianchi
    Commented el 4 may. a las 0:13
  • el problema no es solo ese... si no que tampoco quiero que en ningun momento retorne null prefiero que tire excepcion y se cierre ... pero lo que para mi es aceptable para otros es mala idea no se por que ... yo deberia poder controlar y decidir como se comporta la aplicacion ... segun mi criterio y logica y no dejar que el programa haga lo que le da la gana ... Commented el 4 may. a las 2:45
  • @FranciscoNuñezIALover Ok con el tema de la excepción, pero eso no está en la pregunta, la cual es relativa al warning. Y recuerda que las respuestas no son solo para el OP, sino que también para otros casos, razón por la cual yo he respondido aun cuando tú ya habías solucionado tu problema.
    – Yussef
    Commented el 4 may. a las 4:21
  • @Yussef respeto tu opinion, pero de que me sirve una respuesta que no soluciona mi problema o contexto ... solucionas el problema para la comunidad ... comunidad que no ha publicando nada asociado a tu respuesta ... como que la idea de contexto no se implementa correctamente ... Commented el 4 may. a las 4:31
  • es como que yo lleve el carro (un pick up) al mecanico (ustedes), por que quiero cambiarle las llantas a una mas anchas ... pero tu verificas el carro decides que es mejor cambiarle la suspencion por que esta debil y en la cabina del pickup montar a toda la comunidad ... de que me sirvio a mi (el OP) ... ultimamente las respuesta se salen de contexto... se van por la tangente ... desvian el enfoque ... no se que beneficio trae eso al usuario ... Commented el 4 may. a las 4:36
-1

El problema esta en la implementacion de Path.GetPathRoot ya que este requiere de 2 cosas, que le pasen un string y que su respuesta sea guardada en un nullable ... y esto no cumple con las necesidades de lo que se busca ... ya que la variable root no es nulable y no se quiere implementar asi ...

Tambien el otro detalle es que el IDE por alguna loca funcionalidad esta metiendo la variable path dentro del warning information provocando que visualmente se entienda que la variable path, forma parte del problema cuando no es asi ...

si se quiere estrictamente un string y evitar los warning entonce lo correcto seria implementar una logica personalizada que no de pie al retorno de un tipo null, ya que Path.GetPathRoot no tiene un modo estricto:

internal static string GetWorkDirectory()
{
    return Path.Combine(GetPathRoot(Environment.SystemDirectory), @"Test");
}


public static string GetPathRoot(string path)
{
    if (string.IsNullOrEmpty(path))
    {
        throw new ArgumentNullException(nameof(path), "The 'path' argument cannot be null or empty.");
    }
    if (!Path.IsPathRooted(path))
    {
        throw new ArgumentException("The path does not contain a valid directory separator.", nameof(path));
    }
    return path[..(path.IndexOfAny(['\\', '/']) + 1)];
}

Con esto ya le dices adios a los nullables y warning.

3
  • 4
    Esto.. es una mala idea.. ahora no solo tenes que controlar el null, tambien tenes que controlar dos excepciones que antes no existian....
    – gbianchi
    Commented el 3 may. a las 22:45
  • @gbianchi por que es mala idea si es lo que necesito que haga ... Commented el 4 may. a las 2:42
  • verificado despues de evaluar todas las respuesta esta es la unica que ha solucionado el su completo contexto el problema, el visual studio 2022 presenta aun problemas con algunos warning information. Commented el 7 may. a las 13:46

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.