Skip to main content
Commonmark migration
Origen Enlace

#Edición final

Edición final

##Problemas

Problemas

##Errores en el código de la pregunta

Errores en el código de la pregunta

###C++

C++

###C#

C#

##Solución

Solución

###C++

C++

###C#

C#

#Edición final

##Problemas

##Errores en el código de la pregunta

###C++

###C#

##Solución

###C++

###C#

Edición final

Problemas

Errores en el código de la pregunta

C++

C#

Solución

C++

C#

se añadieron 215 caracteres en el cuerpo
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Si la función devuelve != -1, el buffer se rellena con los nombres de los dispositivos detectados, insertando && para separar uno de otros.

###C#

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
            CharSet = CharSet.Unicode, CallingConvention = CallingConvention.Cdecl)]
        private static extern int escribirArchivo( StringBuilder buff );

private void seleccionDispositivos( ) {
  int result;

  StringBuilder sb = new StringBuilder( 20000 ); // 20000 caracteres.

  result = escribirArchivo( sb );

  if( result == -1 ) {
    Console.WriteLine( "Error: {0}", sb.ToString( ) );
  } else if( result == 0 ) {
    Console.WriteLine( "No se han detectado dispositivos" );
  } else {
    // OBTENEMOS LAS CADENAS; SE ENCUENTRAN SEPARADAS POR &&
    var lineas = sb.ToString( ).Split( new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries );

    Console.WriteLine( "Dispositivos detectados: {0}", result );

    foreach( string value in lineas) {
      Console.WriteLine( value );
    }
  }
}

###C#

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
            CharSet = CharSet.Unicode, CallingConvention = CallingConvention.Cdecl)]
        private static extern int escribirArchivo( StringBuilder buff );

private void seleccionDispositivos( ) {
  int result;

  StringBuilder sb = new StringBuilder( 20000 ); // 20000 caracteres.

  result = escribirArchivo( sb );

  if( result == -1 ) {
    Console.WriteLine( "Error: {0}", sb.ToString( ) );
  } else if( result == 0 ) {
    Console.WriteLine( "No se han detectado dispositivos" );
  } else {
    var lineas = sb.ToString( ).Split( new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries );

    Console.WriteLine( "Dispositivos detectados: {0}", result );

    foreach( string value in lineas) {
      Console.WriteLine( value );
    }
  }
}

Si la función devuelve != -1, el buffer se rellena con los nombres de los dispositivos detectados, insertando && para separar uno de otros.

###C#

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
            CharSet = CharSet.Unicode, CallingConvention = CallingConvention.Cdecl)]
        private static extern int escribirArchivo( StringBuilder buff );

private void seleccionDispositivos( ) {
  int result;

  StringBuilder sb = new StringBuilder( 20000 ); // 20000 caracteres.

  result = escribirArchivo( sb );

  if( result == -1 ) {
    Console.WriteLine( "Error: {0}", sb.ToString( ) );
  } else if( result == 0 ) {
    Console.WriteLine( "No se han detectado dispositivos" );
  } else {
    // OBTENEMOS LAS CADENAS; SE ENCUENTRAN SEPARADAS POR &&
    var lineas = sb.ToString( ).Split( new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries );

    Console.WriteLine( "Dispositivos detectados: {0}", result );

    foreach( string value in lineas) {
      Console.WriteLine( value );
    }
  }
}
se eliminaron 334 caracteres en el cuerpo
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68

Antes de nada: Ni idea de C#, pero basándome en esta respuesta de StackOverflow, vemos que tu problema está en que estás pasandohas utilizado una formación de string de C# ... que no tienen nada que ver ni con el típico char * ni con la clase std::string de C++.

Ese es el problema: Desde C++ estás interpretando los datos como algo que no son ... con el resultado que ya has comprobado: comportamiento indeterminado. Lo que viene a ser que peta, según la terminología actualnomenclatura moderna :-)

Una posible solución puede ser#Edición final

Edición global, con todas las modificaciones aplicadas. La respuesta es grande (C++):... y publicar todas las modificaciones, una a continuación de otra ... esto parecería un libro.

Muchisimas gracias al usuario @gbianchi; sin su ayuda, esto no hubiera sido posible.

##Problemas

  • Un string de C# no es compatible con C++. Y a la inversa.
  • Se utilizan, en C++, cadenas wchar_t, que son dependientes de la plataforma; en Windows, se usa un tipo de 16 bits para ello.
  • Se requiere enviar mas de una cadena entre ambos lenguajes. Desde C#, se entiende que después del 0 final de una cadena, no hay nada mas.

##Errores en el código de la pregunta

###C++

voidint obtenerNombreescribirArchivo( const wchar_t *in, wchar_tstring *out*data, int idxnum ) {
  int current = 0;

  while

Ese es completamente erróneo. Un string de C# no tiene nada que ver con un std::string de C++ ... ni un string[] de C# se parece en nada a un string * de C++.

###C#

[DllImport( idx < current )@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
 in += ( wcslen(CharSet in= )CharSet.Ansi, +CallingConvention 1= CallingConvention.Cdecl);
]
 

CharSet = CharSet.Ansi funcionaría ... si las cadenas en C++ usaran el tipo char. Pero no es así, son cadenas wchar_t.

string[] wcscpy(data out,= innew );
}
string[2];

Ya hemos visto que string[] de C# no es compatible con string * de C++.

##Solución

###C++

int escribirArchivo( wchar_t *buff, int length ) {
  wchar_t *ptr = buff;
  int nError = siNoError;
  SI_64 nDeviceCount = 0;

  std::ofstream ofs( "test.txt", std::ofstream::out );

  SI_CHK( SI_Load( LICENSE_PATH ) );

  SI_CHK( SI_GetInt( SI_SYSTEM, L"DeviceCount", &nDeviceCount ) );
  wprintf( L"Device count: %d\n", nDeviceCount );

  // SI NO HAY DISPOSITIVOS, SALIMOS.
  if( !nDeviceCount ) {
    ofs << "Sin dispositivos." << endl;
    return 0;
  } 

  *buff = 0;
  ofs << "Lista de Dispositivos: " << endl;

  // Iterate through each devices to print their name
  for( int n = 0; n < nDeviceCount; ++n ) {
    SI_WCif( szDeviceName[4096]n =) {wcscat( 0buff, }"&&" ); // SEPARADOR ENTRE NOMBRES -> &&

    wchar_t szDeviceName[4096];
    SI_CHK( SI_GetEnumStringByIndex( SI_SYSTEM, L"DeviceName", n, szDeviceName, 4096 ) );
    wcscpywcscat( ptrbuff, szDeviceName );
    ptr += ( wcslen( szDeviceName ) + 1 );

    // Show String
    ofs << "Dispositivo " << n << " " << szDeviceName << endl;
  }

  ofs.close( );

Error:
  if( SI_FAILED( nError ) ) {
    wprintf( L"An error occurred: %s\n", SI_GetErrorString( nError ) );
    // DEJAMOS EL ERROR EN EL BUFFER QUE RECIBIMOS COMO ARGUMENTO.
    wcscpy( buffer, SI_GetErrorString( nError ) );
  } else {
    wprintf(L"It all went well!\n");
  }
 
  // Cleanups the buffer, closes the camera and unloads SpecSensor
  SI_Close( g_hDevice );
  SI_Unload( );

  return SI_FAILED( nError ) ? -1 : reinterpret_cast< int >( nDeviceCount );
}

Usamos 2 funciones porque C# no se lleva bien con las secuencias de cadenas ... StringBuilder está pensado para contener solo 1, y StringBuilder.ToString( ) parará cuando llegue al primer caracter NUL (0).

Nuestra función escribirArchivo( ) devuelve:

La parte de¡ Cuidado !. No comprobamos el tamaño del bufer que recibimos como argumento; hay que tener cuidado con el tamaño del mismo al generarlo desde C# podría ser así:, asegurándonos que es lo bastante grande.

###C#

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
            CharSet = CharSet.AnsiUnicode, CallingConvention = CallingConvention.Cdecl)]
        private static extern int escribirArchivo( StringBuilder buff, int lengt );
        private static extern void obtenerNombre( StringBuilder in, StringBuilder out, int lengt );

private void seleccionDispositivos( ) {
  int result;

  StringBuilder sb = new StringBuilder( 20000 ); // 20000 caracteres.

  result = escribirArchivo( sb, sb.Capacity );

  if( result == -1 ) {
    Console.WriteLine( "Error: {0}", sb.ToString( ) );
  } else if( result == 0 ) {
    Console.WriteLine( "No se han detectado dispositivos" );
  } else {
    StringBuildervar devNamelineas = newsb.ToString( StringBuilder).Split( 1000new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries );

    Console.WriteLine( "Dispositivos detectados: {0}", result );

    forforeach( int idx = 0; idx <string result;value idx++in lineas) {
      obtenerNombre( sb, devName, idx );
      Console.WriteLine( devName.toString( )value );
    }
  }
}

Documentación adicional: Default Marshaling for Strings (Microsoft)

Actualizacion (por gbianchi, para que no culpen a Trauma si cometí un error)

La función tiene que estar definida en C# de la siguiente manera:

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
        CharSet = CharSet.Unicode, CallingConvention = CallingConvention.Cdecl)]

Notese la diferencia en el CharSet.

Además, la función en C++, tiene que mover el puntero a la longitud de la estructura, no a la longitud + 1:

ptr += wcslen( szDeviceName );

Y a la función en C++, habría que agregarle un lindo separador entre cada lectura de los dispositivos, ya que el StringBuilder se come la terminación del string, entonces necesitamos un separador, que podría ser algo así:

const wchar_t* separador = L"&&";

Y al final de cada línea leída del archivo, deberíamos agregar el mismo:

wcscpy( ptr, separador );
ptr += ( wcslen( separador ) );

De esta manera, ahora el StringBuilder tiene un solo string, pero separa cada línea con &&.

En C#, para leer esto, se podría hacer algo así:

var lineas = sb.ToString().Split(new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries);

Donde lineas es un arreglo de cada línea leída en el archivo.

Finalmente, en un repositorio de Github agrego un ejemplo minimo al respecto.

Antes de nada: Ni idea de C#, pero basándome en esta respuesta de StackOverflow, vemos que tu problema está en que estás pasando una formación de string de C# ... que no tienen nada que ver ni con el típico char * ni con la clase std::string de C++.

Ese es el problema: Desde C++ estás interpretando los datos como algo que no son ... con el resultado que ya has comprobado: comportamiento indeterminado. Lo que viene a ser que peta, según la terminología actual :-)

Una posible solución puede ser (C++):

void obtenerNombre( const wchar_t *in, wchar_t *out, int idx ) {
  int current = 0;

  while( idx < current ) in += ( wcslen( in ) + 1 );

  wcscpy( out, in );
}

int escribirArchivo( wchar_t *buff, int length ) {
  wchar_t *ptr = buff;
  int nError = siNoError;
  SI_64 nDeviceCount = 0;

  std::ofstream ofs( "test.txt", std::ofstream::out );

  SI_CHK( SI_Load( LICENSE_PATH ) );

  SI_CHK( SI_GetInt( SI_SYSTEM, L"DeviceCount", &nDeviceCount ) );
  wprintf( L"Device count: %d\n", nDeviceCount );

  // SI NO HAY DISPOSITIVOS, SALIMOS.
  if( !nDeviceCount ) {
    ofs << "Sin dispositivos." << endl;
    return 0;
  }

  ofs << "Lista de Dispositivos: " << endl;

  // Iterate through each devices to print their name
  for( int n = 0; n < nDeviceCount; ++n ) {
    SI_WC szDeviceName[4096] = { 0 };
    SI_CHK( SI_GetEnumStringByIndex( SI_SYSTEM, L"DeviceName", n, szDeviceName, 4096 ) );
    wcscpy( ptr, szDeviceName );
    ptr += ( wcslen( szDeviceName ) + 1 );

    // Show String
    ofs << "Dispositivo " << n << " " << szDeviceName << endl;
  }

  ofs.close( );

Error:
  if( SI_FAILED( nError ) ) {
    wprintf( L"An error occurred: %s\n", SI_GetErrorString( nError ) );
    // DEJAMOS EL ERROR EN EL BUFFER QUE RECIBIMOS COMO ARGUMENTO.
    wcscpy( buffer, SI_GetErrorString( nError ) );
  } else {
    wprintf(L"It all went well!\n");
  }
 
  // Cleanups the buffer, closes the camera and unloads SpecSensor
  SI_Close( g_hDevice );
  SI_Unload( );

  return SI_FAILED( nError ) ? -1 : reinterpret_cast< int >( nDeviceCount );
}

Usamos 2 funciones porque C# no se lleva bien con las secuencias de cadenas ... StringBuilder está pensado para contener solo 1, y StringBuilder.ToString( ) parará cuando llegue al primer caracter NUL (0).

Nuestra función escribirArchivo( ) devuelve:

La parte de C# podría ser así:

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
            CharSet = CharSet.Ansi, CallingConvention = CallingConvention.Cdecl)]
        private static extern int escribirArchivo( StringBuilder buff, int lengt );
        private static extern void obtenerNombre( StringBuilder in, StringBuilder out, int lengt );

private void seleccionDispositivos( ) {
  int result;

  StringBuilder sb = new StringBuilder( 20000 ); // 20000 caracteres.

  result = escribirArchivo( sb, sb.Capacity );

  if( result == -1 ) {
    Console.WriteLine( "Error: {0}", sb.ToString( ) );
  } else if( result == 0 ) {
    Console.WriteLine( "No se han detectado dispositivos" );
  } else {
    StringBuilder devName = new StringBuilder( 1000 );

    Console.WriteLine( "Dispositivos detectados:" );

    for( int idx = 0; idx < result; idx++ ) {
      obtenerNombre( sb, devName, idx );
      Console.WriteLine( devName.toString( ) );
    }
  }
}

Documentación adicional: Default Marshaling for Strings (Microsoft)

Actualizacion (por gbianchi, para que no culpen a Trauma si cometí un error)

La función tiene que estar definida en C# de la siguiente manera:

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
        CharSet = CharSet.Unicode, CallingConvention = CallingConvention.Cdecl)]

Notese la diferencia en el CharSet.

Además, la función en C++, tiene que mover el puntero a la longitud de la estructura, no a la longitud + 1:

ptr += wcslen( szDeviceName );

Y a la función en C++, habría que agregarle un lindo separador entre cada lectura de los dispositivos, ya que el StringBuilder se come la terminación del string, entonces necesitamos un separador, que podría ser algo así:

const wchar_t* separador = L"&&";

Y al final de cada línea leída del archivo, deberíamos agregar el mismo:

wcscpy( ptr, separador );
ptr += ( wcslen( separador ) );

De esta manera, ahora el StringBuilder tiene un solo string, pero separa cada línea con &&.

En C#, para leer esto, se podría hacer algo así:

var lineas = sb.ToString().Split(new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries);

Donde lineas es un arreglo de cada línea leída en el archivo.

Finalmente, en un repositorio de Github agrego un ejemplo minimo al respecto.

Antes de nada: Ni idea de C#, pero basándome en esta respuesta de StackOverflow, vemos que tu problema está en que has utilizado una formación de string de C# ... que no tienen nada que ver ni con el típico char * ni con la clase std::string de C++.

Ese es el problema: Desde C++ estás interpretando los datos como algo que no son ... con el resultado que ya has comprobado: comportamiento indeterminado. Lo que viene a ser que peta, según la nomenclatura moderna :-)

#Edición final

Edición global, con todas las modificaciones aplicadas. La respuesta es grande ... y publicar todas las modificaciones, una a continuación de otra ... esto parecería un libro.

Muchisimas gracias al usuario @gbianchi; sin su ayuda, esto no hubiera sido posible.

##Problemas

  • Un string de C# no es compatible con C++. Y a la inversa.
  • Se utilizan, en C++, cadenas wchar_t, que son dependientes de la plataforma; en Windows, se usa un tipo de 16 bits para ello.
  • Se requiere enviar mas de una cadena entre ambos lenguajes. Desde C#, se entiende que después del 0 final de una cadena, no hay nada mas.

##Errores en el código de la pregunta

###C++

int escribirArchivo( string *data, int num ) {

Ese es completamente erróneo. Un string de C# no tiene nada que ver con un std::string de C++ ... ni un string[] de C# se parece en nada a un string * de C++.

###C#

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
    CharSet = CharSet.Ansi, CallingConvention = CallingConvention.Cdecl)]

CharSet = CharSet.Ansi funcionaría ... si las cadenas en C++ usaran el tipo char. Pero no es así, son cadenas wchar_t.

string[] data = new string[2];

Ya hemos visto que string[] de C# no es compatible con string * de C++.

##Solución

###C++

int escribirArchivo( wchar_t *buff ) {
  int nError = siNoError;
  SI_64 nDeviceCount = 0;

  std::ofstream ofs( "test.txt", std::ofstream::out );

  SI_CHK( SI_Load( LICENSE_PATH ) );

  SI_CHK( SI_GetInt( SI_SYSTEM, L"DeviceCount", &nDeviceCount ) );
  wprintf( L"Device count: %d\n", nDeviceCount );

  // SI NO HAY DISPOSITIVOS, SALIMOS.
  if( !nDeviceCount ) {
    ofs << "Sin dispositivos." << endl;
    return 0;
  } 

  *buff = 0;
  ofs << "Lista de Dispositivos: " << endl;

  for( int n = 0; n < nDeviceCount; ++n ) {
    if( n ) wcscat( buff, "&&" ); // SEPARADOR ENTRE NOMBRES -> &&

    wchar_t szDeviceName[4096];
    SI_CHK( SI_GetEnumStringByIndex( SI_SYSTEM, L"DeviceName", n, szDeviceName, 4096 ) );
    wcscat( buff, szDeviceName );

    ofs << "Dispositivo " << n << " " << szDeviceName << endl;
  }

  ofs.close( );

Error:
  if( SI_FAILED( nError ) ) {
    wprintf( L"An error occurred: %s\n", SI_GetErrorString( nError ) );
    // DEJAMOS EL ERROR EN EL BUFFER QUE RECIBIMOS COMO ARGUMENTO.
    wcscpy( buffer, SI_GetErrorString( nError ) );
  } else {
    wprintf(L"It all went well!\n");
  }

  SI_Close( g_hDevice );
  SI_Unload( );

  return SI_FAILED( nError ) ? -1 : reinterpret_cast< int >( nDeviceCount );
}

Nuestra función escribirArchivo( ) devuelve:

¡ Cuidado !. No comprobamos el tamaño del bufer que recibimos como argumento; hay que tener cuidado con el tamaño del mismo al generarlo desde C#, asegurándonos que es lo bastante grande.

###C#

[DllImport(@"C:\Users\usuario\Dropbox\ANFACO\VISUAL_STUDIO\HIPERESPECTRAL\PRUEBA\Debug\Camara.dll",
            CharSet = CharSet.Unicode, CallingConvention = CallingConvention.Cdecl)]
        private static extern int escribirArchivo( StringBuilder buff );

private void seleccionDispositivos( ) {
  int result;

  StringBuilder sb = new StringBuilder( 20000 ); // 20000 caracteres.

  result = escribirArchivo( sb );

  if( result == -1 ) {
    Console.WriteLine( "Error: {0}", sb.ToString( ) );
  } else if( result == 0 ) {
    Console.WriteLine( "No se han detectado dispositivos" );
  } else {
    var lineas = sb.ToString( ).Split( new string[] { "&&" }, StringSplitOptions.RemoveEmptyEntries );

    Console.WriteLine( "Dispositivos detectados: {0}", result );

    foreach( string value in lineas) {
      Console.WriteLine( value );
    }
  }
}

Documentación adicional:

se añadieron 73 caracteres en el cuerpo
Origen Enlace
Phi
  • 10.2k
  • 5
  • 28
  • 48
Loading
se añadieron 106 caracteres en el cuerpo
Origen Enlace
gbianchi
  • 20.6k
  • 13
  • 40
  • 60
Loading
se añadieron 1165 caracteres en el cuerpo
Origen Enlace
gbianchi
  • 20.6k
  • 13
  • 40
  • 60
Loading
¡ Huy ! ¡ Menos mal que no lo ha visto nadie ! :-)
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68
Loading
A ver ahora ...
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68
Loading
se eliminaron 7 caracteres en el cuerpo
Origen Enlace
Phi
  • 10.2k
  • 5
  • 28
  • 48
Loading
se añadieron 437 caracteres en el cuerpo
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68
Loading
Origen Enlace
Trauma
  • 26.9k
  • 5
  • 42
  • 68
Loading