Mensajes

Lo primero que debes hacer es verificar el numero de llaves ya que tal como esta tu fragmento las ultimas dos lineas:

Código (cpp) [Seleccionar] Expandir

}
fi.close();
Están fuera de la función main.

Ademas:

* Si no utilizas los argumentos pasados mediante la linea de comandos eliminalos, deja la definición de la función como "int main() ...".

* Si la declaración de los arrays m1 y m2 funciona depende del compilador utilizado, no es parte del estándar de C++ (para el caso utilizar objetos de la clase vector).

* Ya que todo lo que lees del archivo son valores de tipo int no es necesario leer por linea con getline, en su lugar utiliza directamente el operador ">>".

* Para leer la primera matriz utilizas tres bucles cuando solo necesitas dos.

Para leer mediante un objeto de la clase vector la primera matriz el código fuente seria mas o menos así:

Código (cpp) [Seleccionar] Expandir

#include <iostream>
#include <fstream>
#include <vector>

using namespace::std;

int main()
{
   ifstream fi("bur26a.csv");
   
   if (fi.good()){
      int num_elem;
      fi >> num_elem;
      vector< vector<int> > mat(num_elem, vector<int>(num_elem));
     
      for (int i = 0; i != num_elem; ++i)
         for (int j = 0; j != num_elem; ++j)
            fi >> mat[i][j];
      fi.close();
     
      for (int i = 0; i != num_elem; ++i){
         for (int j = 0; j != num_elem; ++j)
            cout << ' ' << mat[i][j];
         
         cout << endl;
      }
   }
   
   return 0;
}

Para leer la segunda matriz el proceso es similar.

Un saludo

validación robusta? no tengo idea que significa.

Significa que se debe verificar que toda entrada este dentro de los parámetros esperados, por ejemplo ponte a pensar que pasara si a la función que crea un nodo se le pasa un valor que no es parte de la enumeración de tipos. También se debe verificar el resultado de toda función que pueda fallar (el mejor ejemplo es la lectura de datos de un archivo y las funciones de reserva de memoria).

Un saludo

[cout undeclared (first use in this function)]

estoy aprendiendo a programar y les agradecería si me pueden ayudar con este programa que me da error: cout undeclared (first use in this function)

Si estas aprendiendo C++ el encabezado que debes incluir es <iostream> y debes indicar que el mentado objeto se encuentra en el nombre de espacios estándar mediante el prefijo "std::", de esta forma:

Código (cpp) [Seleccionar] Expandir

#include <iostream>

int main()
{
   int cont1 = 0, cont2 = 0, i;
   float rut, nota;
   
   for (i = 0; i < 10; i++){
     
      // Proceso: lectura de notas y calculo del promedio
     
   }
   
   std::cout << "Cantidad de aprobados es: " << cont1 << std::endl;
   std::cout << "Cantidad de reprobados es " << cont2 << std::endl;
   
   return 0;
}
Una alternativa al uso de "std::" es agregar la sentencia:

Código (cpp) [Seleccionar] Expandir

using namespace::std;
Justo antes de la definición de main en tu programa.

Por supuesto debes agregar las correcciones que ya te indico leosansan.

Un saludo

[ivancea96]

No me deja ingresar el Nombre, sino que me sale seguido Sex cuando deberia salir en la siguiente linea luego de yo digitar el nombre

Ese problema se debe al uso intercalado del operador ">>" y la función getline, para solucionarlo se debe eliminar el resto de la linea después de pedir el numero de empleados y, como ya te comento ivancea96, después de pedir el pago. Un programa de ejemplo sobre el uso de la función ignore se encuentra en el tema getline().

Y la declaración del array "data" indica el numero de elementos de este mediante un valor calculado en tiempo de ejecución, el problema: su soporte depende del compilador (no es parte del estándar de C++), por ello mejor utiliza un objeto de la clase vector (hay muchos ejemplos sobre el uso de esa clase en la base de datos de los foros, solo es cuestión de utilizar el motor de búsqueda).

Un saludo

[por favor]

Los mensajes de error que mencionas se deben a que en C++ los encabezados parte de la biblioteca estándar de C no deben terminar en ".h", en su lugar debe prefijarse con "c". Ademas el nombre correcto del encabezado es <iostream>, estas directivas:

Código (cpp) [Seleccionar] Expandir

#include <conio.h>
#include <stdio.h>
#include <string.h>
#include <iostream.h>
#include <stdlib.h>
using namespace std;

Hay que cambiarlas a:

Código (cpp) [Seleccionar] Expandir

#include <cstdio>
#include <cstring>
#include <iostream>
#include <cstdlib>
using namespace std;

En cuanto a la biblioteca conio de Borland la recomendación es evitarla, las razones de ello se explican en el tema |Lo que no hay que hacer en C/C++. Nivel basico|.

Por ultimo cuando publiques código fuente por favor utiliza las etiquetas de código. Estas se agregan mediante el cuadro desplegable con el texto "GeSHi" en la pagina de composición de mensajes, solo tienes que seleccionar ahí el lenguaje (C, C++, etc.).

Un saludo

que les parece mi implementación?

Para empezar esta bien pero, como todo en programación, se puede mejorar.

Lo primero es eliminar el valor inicial de la enumeración ya que estas siempre empiezan con el valor cero, salvo una razón de peso se debe preferir el tipo double sobre el tipo float y evitar nombres en mayúsculas reservando estos para las macros y las constantes enumeradas.

Ademas:

* Los distintos tipos se pueden agrupar en una union:

Código (c) [Seleccionar] Expandir

#include <stdio.h>

enum tipo {INT, DOUBLE, STRING};
typedef enum tipo Tipo;

struct nodo {
   union {
      int Int;
      double Double;
      char   *str;
   } val;
   Tipo tipo;
   
   struct nodo *sig;
};
typedef struct nodo Nodo;

typedef Nodo *Lista;

* En lugar de una función por cada tipo puedes utilizar una función con un numero variable de argumentos donde el ultimo argumento con nombre indica el tipo del valor, de esa forma cuando agregues tipos solo debes modificar las sentencias de selección y no crear tantas funciones.

Mas o menos así:

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h>
#include <stdarg.h>
#include <string.h>

enum tipo {INT, DOUBLE, STRING};
typedef enum tipo Tipo;

struct nodo {
   union {
      int Int;
      double Double;
      char *String;
   } val;
   Tipo tipo;
   
   struct nodo *sig;
};
typedef struct nodo Nodo;

typedef Nodo *Lista;

/* ... */

Lista LG_Apilar(Lista lst, Tipo tipo, ...);
void LG_Imprimir(FILE *out, Lista lst);

int main(void)
{
   Lista lst = NULL;
   
   lst = LG_Apilar(lst, INT, 1);
   lst = LG_Apilar(lst, INT, 2);
   lst = LG_Apilar(lst, INT, 3);
   
   lst = LG_Apilar(lst, DOUBLE, 1.2);
   lst = LG_Apilar(lst, DOUBLE, 3.4);
   lst = LG_Apilar(lst, DOUBLE, 5.6);
   
   lst = LG_Apilar(lst, STRING, "Hugo");
   lst = LG_Apilar(lst, STRING, "Paco");
   lst = LG_Apilar(lst, STRING, "Luis");
   
   LG_Imprimir(stdout, lst);

   return EXIT_SUCCESS;
}

/* ... */

Lista LG_Apilar(Lista lst, Tipo tipo, ...)
{
   Lista nuevo;
   
   if ((nuevo = malloc(sizeof *nuevo)) != NULL){
      char *str;
      va_list ap;
     
      va_start(ap, tipo);
      switch (tipo){
      case INT:
         nuevo->val.Int = va_arg(ap, int);
         break;
      case DOUBLE:
         nuevo->val.Double = va_arg(ap, double);
         break;
      case STRING:
         str = va_arg(ap, char *);
         nuevo->val.String = malloc(strlen(str) + 1);
         strcpy(nuevo->val.String, str);
         break;
      }
      va_end(ap);
     
      nuevo->tipo = tipo;
      nuevo->sig = lst;
   }
   
   return nuevo;
}

void LG_Imprimir(FILE *out, Lista p)
{
   while (p != NULL){
      switch (p->tipo){
      case INT:
         fprintf(out, "%d", p->val.Int);
         break;
      case DOUBLE:
         fprintf(out, "%.2f", p->val.Double);
         break;
      case STRING:
         fprintf(out, "%s", p->val.String);
         break;
      }
      fputc('\n', out);
     
      p = p->sig;
   }
}

En ambos programas faltan muchas cosas como una validación robusta y la liberación de la memoria antes de finalizar el programa.

Un saludo

[vangodp]

Creo que hasta existe un conio2.h  pero te dice vangodp lo mejor seria no usarlo nunca [...]
Edito: Si no es estandar, no sería caer en lo mismo?

El problema no es el uso de extensiones (características especificas del compilador y/o plataforma) sino que una parte significativa de los cursos no documentan que es C/C++ estándar y cuales son las extensiones con el resultado de que al final el estudiante no sabe que puede utilizar sin importar el entorno y que no.

En resumen si conoces las diferencias y hay una razón de peso no hay problema en el uso de extensiones, el problema es si no existen.

Un saludo

[NULL]

Si logra encontrar un registro que tenga ese DNI deve devolver una copia de ese registro, entonces ¿que debería de poner en lugar de esos NULL para el caso de que no se encuentre el registro? ¿Que tengo que devolver?

Lo primero que debes hacer es publicar la declaración del alias (o macro) "Record" ya que la respuesta a tu problema depende de esa declaración (o definición, depende del caso). Lo mismo aplica a headerLength y recordLength.

fread devuelve un entero, pero no tengo idea que ponerle en lugar de NULL para evitar esos warnings.

El prototipo de fread es:

Código (c) [Seleccionar] Expandir

int fread(
   void *addr,        /* Direccion donde se almacenaran los caracteres */
   size_t size_elem,  /* Numero de bytes por elemento */
   size_t num_elem,   /* Numero de elementos a leer   */
   FILE *in           /* stream del cual se leeran los caracteres */
);
La función trata de leer de "in" un numero de bytes igual a size_elem * num_elem y los almacena a partir de la dirección addr. Si no se utiliza el valor de retorno de la función el orden del segundo y tercer argumento no importa. La función después del intento de lectura retorna un entero igual al numero de bytes leídos entre el numero de bytes por elemento.

En base a lo anterior el bucle que mencionas se debe cambiar a (para leer elemento por elemento del archivo):

Código (c) [Seleccionar] Expandir

/* Mientras se lea con exito un registro del archivo */
while (fread(&tmp, recordLength, 1, fd) == 1){
   /* Comparar el campo del registro con el valor buscado */
   if (strncmp(DNI,tmp.DNI,strlen(tmp.DNI)) == 0){
      struct Record data = tmp;
      return data;
   }
}

También hay que realizar otros cambios al programa, entre ellos:

* En caso de error puedes utilizar la función perror para que el mensaje de error se genere automáticamente.
* Si encuentras un registro con el valor buscado no cierras el archivo.
* La función strncmp solo compara los primeros N caracteres, si el valor del campo es "123" y el valor buscado "123456" la función retornara verdadero cuando las cadenas son distintas.

Un saludo

En C++ (según el estándar C++98) cuando utilizas una función parte de la biblioteca estándar de C debes incluir el encabezado correspondiente (en tu caso <cctype>), si no lo haces el programa es "ill formed", en buen cristiano no hay garantías sobre su comportamiento (cualquier cosa puede pasar desde obtener los resultados esperados hasta el legendario BSOD).

Un saludo

Habría que saber a que te refieres con "vectores dinámicos" : al uso de los operadores new y delete o a la clase vector, en cualquier caso puedes encontrar ejemplos de ambos utilizando el motor de búsqueda de los foros.

Un saludo