Skip to main content
8 de 11
se añadieron 73 caracteres en el cuerpo
Phi
  • 10.2k
  • 5
  • 28
  • 48

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:

  • == -1 -> Error. sb contiene el mensaje de error.

  • != -1 -> Ok. Número de dispositivos detectados (pueden ser 0).

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.

Trauma
  • 26.9k
  • 5
  • 42
  • 68