Decidme que tengo mal :,(

[carlosrebisch]

Llevo literalmente HORAS y siempre me sale algún error, o no me lee o no se que mas
si algun buen samaritano me ayudaría a indicarme que tengo mal o que debo cambiar, se lo agradeceré muchísimo.

FILE* fileHandle;
fileHandle= fopen("nationality.lst", "rb");
if (fileHandle== NULL)
{
fileHandle= fopen("nationality.lst", "wb");
String list[]= {"España", "Marruecos"};
//fwrite();
}
else
{
int checkError= 0;
checkError= fseek(fileHandle, 2, SEEK_SET);
//checkError= fseek(fileHandle, -8, SEEK_END);

int* identifier= 0;
//realloc(identifier, 8);
/*int lugar= 0;
lugar= fseek(fileHandle, 0, SEEK_CUR);*/
checkError= fread(identifier, 8, 1, fileHandle);
if (identifier== "_CVFast_")
{
fseek(fileHandle, 0, SEEK_SET);
char* item= "";
char* character= "";
unsigned int counter= 0;
do
{

fread(character, 1, 1, fileHandle);
if (character!= "\n")
{
item= strcat(item, character);
}
else
{
unsigned char i;
String itemStr= "";
for (i= 0; i< sizeof(item); i++)
{
itemStr[i]= item[i];
}
ComboBox3->Items->Add(itemStr);
}
} while (feof(fileHandle)&& item!= "_CVFast_");


}

}
fclose(fileHandle);

[aguml]

Por que mezclas C y C++? Por que no usas fstream para trabajar con archivos? Faltan partes del código que son fundamentales como la declaración de las cabeceras. No sabemos que error te está dando tampoco ni que lenguaje es exactamente aunque pienso que puede ser c++Builder o similar ya que usas un objeto que existe en el.

[carlosrebisch]

#include <vcl.h>
#include <string.h>
#include <stdio.h>
#pragma hdrstop

#include "Unit1.h"
#pragma package(smart_init)
#pragma resource "*.dfm"

void __fastcall TForm1::FormCreate(TObject *Sender)

FILE* fileHandle;
. fileHandle= fopen("nationality.lst", "rb");
. if (fileHandle== NULL)
. {
. fileHandle= fopen("nationality.lst", "wb");
. String list[]= {"España", "Marruecos"};
. }
. else
. {
. int checkError= 0;
. checkError= fseek(fileHandle, 2, SEEK_SET);
. checkError= fseek(fileHandle, -8, SEEK_END);
.
. int* identifier= 0;
. //realloc(identifier, 8);
. /*int lugar= 0;
. lugar= fseek(fileHandle, 0, SEEK_CUR);*/
. checkError= fread(identifier, 8, 1, fileHandle);
. if (identifier== "_CVFast_")
. {
. fseek(fileHandle, 0, SEEK_SET);
. char* item= "";
. char* character= "";
. unsigned int counter= 0;
. do
. {
. fread(character, 1, 1, fileHandle);
. if (character!= "\n")
. {
. item= strcat(item, character);
. }
. else
. {
. unsigned char i;
. String itemStr= "";
. for (i= 0; i< sizeof(item); i++)
. {
. itemStr[i]= item[i];
. }
. ComboBox3->Items->Add(itemStr);
. }
. } while (feof(fileHandle)&& item!= "_CVFast_");
.
.
. }
.
. }
.fclose(fileHandle);
estoy usando c++ con el programa de C++ builder XE6
y lo que quiero hacer es que en un combobox (comboBox3) aparezca la lista "nationality.lst"(da igual la extensión) que tengo dentro de la misma carpeta y que no tiene ningún error (lo he comprobado) pero por alguna razón no consigo que cuando visualizo el programa aparezca la lista. solo decidme con que función o que estoy tratando mal porque ya he intentado todo lo que se me ocurrió. Si podéis decirme con que función se haría rápido y limpio, se lo agradecería, porque ya me enrede tanto que yo no veo como podría hacerlo de una manera simple.

Simplemente que en el combobox salga la lista que tengo en un archivo aparte.

[aguml]

Ufff tiene muchos fallos. Pon como es el archivo para poder ayudarte mejor y tener una idea de lo que quieres hacer porque veo que usas fseek hasta en la sopa y no le veo sentido alguno.

Edito:
Te pongo un ejemplo usando lo que C++Builder pone a tu alcance para facilitarte las cosas:

Código C++:

//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
   : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::FormCreate(TObject *Sender)
{
   TStrings *lista;
   bool error=false;
 
   //Pido memoria para la lista
   try{
      lista=new TStringList;
   }catch(std::bad_alloc){
      error=true;
   }
 
   if(!error){
      if(FileExists("nationality.lst")){ //Compruebas si existe
         //Lleno la lista desde el archivo
         try{
            lista->LoadFromFile("nationality.lst");
         }catch(std::bad_alloc){
            error=true;
         }
      }else{
         try{
            //Lleno la lista con las cadenas deseadas
            lista->Add("¡Viva España!");
            lista->Add("Marruecos");
            lista->Add("Gibraltar");
            //Guardo la lista en el archivo
            lista->SaveToFile("nationality.lst");
            //Limpio la lista
            lista->Clear();
            //Lleno la lista desde el archivo
            lista->LoadFromFile("nationality.lst");
         }catch(std::bad_alloc){
            error=true;
         }
      }
 
      if(!error){
         //Le meto toda la lista de strings obtenida al combobox
         try{
            ComboBox1->Items->AddStrings(lista);
         }catch(std::bad_alloc){
            error=true;
         }
 
         //Coloco el primero como seleccionado si se añadio alguno
         if(ComboBox1->Items->Count > 0)
            ComboBox1->ItemIndex=0;
      }
      delete lista; //Libero la memoria ocupada por la lista
   }
 
   if(error)
      ShowMessage("No hay memoria suficiente");
}
//---------------------------------------------------------------------------
 

Es algo basico pero creo que hace lo que quieres y controlo solo los errores que se puedan producir por falta de memoria pero si hay otro tipo de error no lo estoy controlando.