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.