Mensajes

Debería cambiar el resto de las funciones en las que pasé una lista como parámetro?

En aquellos casos en los que necesites modificar el valor del puntero (liberar memoria, modificar el tamaño de la memoria reservada, etc), SI. En estos casos es un imperativo para que los cambios se reflejen fuera de la función.

En el resto de casos no es necesario y puede ser incluso contraproducente. Lo más sano es facilitar a cada función la cantidad mínima de datos necesaria.

La materia no cubre objetos, por eso las listas las hacemos de esa manera.
Le discutiré eso pues mi profesor es graduado de la facultad más prestigiosa en ingeniería de mi país, donde estoy estudiando yo ahora, y sé que tiene sus razones para evitar que usemos los String.

Quizás, una de las pegas de la clase "string" es que no admite unicode... pero para eso está la clase "wstring". En cualquier caso, ambas clases forman parte del estándar... y aunque en la librería estándar existen bugs, no es motivo para no usarlos. Básicamente porque cualquier librería externa que tengas que usar (incluso las propias del sistema operativo) también están expuestas a fallos de programación y no por ello dejas de usarlas. Además, la librería estándar suele tener algoritmos altamente optimizados, por lo que suele ser la mejor opción para programas de uso general.

Un saludo.

Muy bueno el invertir cadena, con respecto a la PalabraMasLargar hay un bug.

La idea no era dar un código completo... si miras el algoritmo ves que separa palabras únicamente por espacios, pero se puede mejorar fácilmente para que sea capaz de reconocer otros tipos de separadores.

Código (cpp) [Seleccionar] Expandir

if ( *it2 == ' '  || *it2 == ',' || *it2 == '.' /* ... */ )

no logro ver como hacer para que cuando ya no hallan puestos disponibles el programa pregunte si desea un puesto en la otra clase y si la respuesta es SI que lo asigne

Código (cpp) [Seleccionar] Expandir

if(vacantes==false){cout<<"No hay vancantes en la clase #"<<clase<<endl; cin.get();cin.get();return;}

Ese if se ejecuta cuando no hay asientos en la clase seleccionada... solo tienes que poner ahí la lógica para que pregunte por un puesto en la nueva clase.

Un saludo.

Invertir un string... que tal así:

Código (cpp) [Seleccionar] Expandir


int main( )
{
  std::string cadena = "0123456789";
  std::string invertida( cadena.rbegin( ), cadena.rend( ) );

  std::cout << invertida << std::endl;
}

En cuanto a encontrar la palabra más larga... con iteradores se puede conseguir fácilmente:

Código (cpp) [Seleccionar] Expandir


void PalabraMasLarga( const std::string& cadena )
{
  std::string palabra;
  auto it = cadena.begin( );

  for( auto it2 = it; it2 != cadena.end( ); ++it2 )
  {
    if ( *it2 == ' ' )
    {
      size_t length = std::distance( it, it2 );
      if ( length > palabra.size( ) )
        palabra = std::string( it, it2 );
      it = it2+1;
    }
  }

  std::cout << palabra << std::endl;
}

El codigo usa dos iteradores, uno apunta al inicio de la palabra actual y el otro va avanzando hasta encontrar un espacio, si la longitud de la cadena es mayor que la palabra más larga encontrada hasta ahora, la sustituye.

Simple y rápido.

[ni la más mínima idea idea]

Las cadenas de tipo string no las tengo permitidas usar, el profesor nos dijo que eran sumamente inestables y que prefería que usemos char.

Me permito la licencia de decir públicamente que tu profesor no tiene ni la más mínima idea idea de programación. Las clases de la librería estándar están ahí por algo. Y su implementación es bastante eficiente y estable. Otra cosa es que, claro, al no tener mucha idea sobre programación orientada a objetos, prefiera que programéis lo más parecido a C posible... pero eso, como he dicho, no habla muy bien del nivel de tu profesor.

A ver, tal y como lo entiendo, el profesor debería motivar vuestra curiosidad y abriros los ojos, no encasillaros y obligaros a programar de una forma que no será la que os encontréis luego en "el mundo real".

El índice quería que empezara en 1, por eso el i++ previo.

Imagínate que el arreglo admite dos elementos y estás rellenando el segundo. Al hacer "i++", i pasa a valer "2" y en la siguiente línea intenas escribir en el elemento con índice "2", es decir, estarías escribiendo en una posición de memoria no válida.

pero tenía entendido que las listas y los array siempre pasan por referencia.

Cuando usas un puntero como argumento, todo lo que modifiques que se refiera a la memoria apuntada por el puntero se verá reflejado fuera de la función... pero el puntero, como variable, es algo local, luego modificar la posición de memoria apuntada por el puntero es un cambio local y no se refleja fuera de la función. No se si me explico.

Un saludo.

[Las funciones no deben terminar con ";"]

Las funciones no deben terminar con ";"

Esto sucede con la copia que has puesto de "LlenarVector", por ejemplo.

Estás programando en C++, no deberías usar los includes que acaben en ".h"

Esas cabeceras son de C. Puedes cargar la cabecera propia de C++ poniendo el prefijo "C" y eliminando el ".h" ( #include <cmath> en vez de #include <math.h> )

Dado que estás programando en C++, sería más cómodo y seguro para tí usar la clase "string" en vez de "char*" para almacenar cadenas.

Código (cpp) [Seleccionar] Expandir


int main( )
{
  // Tu estás usando este tipo de variables
  char* variable = "una prueba";

  // Cuando puedes usar esta otra. Fíjate que no hace falta usar strncpy para copiar los valores.
  std::string cadena = variable;
}

En C++ no hace falta usar "typedef" para declarar estructuras:

Código (cpp) [Seleccionar] Expandir


struct Dato
{
  int valor;
};

int main( )
{
  Dato dato; // Funciona sin typedef
  dato.valor = 10;
}

Los índices no los estás actualizando correctamente:

Código (cpp) [Seleccionar] Expandir


void LlenarVector(Equipo vectorequipos[], int &i) { //usando la función dada "leerequipo" lleno el vector y devuelvo el tamaño.
     FILE *archivo = fopen ("equipos.txt", "r");
   char nombre[31];
   int pA, pB;


   while (leerEquipo(archivo, nombre, pA, pB))
   {

     vectorequipos[i].paramA = pA; //estos dos son parámetros usados para simular partidos, son enteros.
     vectorequipos[i].paramB = pB;
     strncpy(vectorequipos[i].nombre, nombre, 31);
     i++;
         vectorequipos[i].indice = i; // <<<< AQUI!!!
   };
   fclose (archivo);
}


Fíjate que en la línea anterior estás actualizando "i", luego la línea marcada está modificando el índice del siguiente registro.

Cuidado al inicializar los arreglos:

Código (cpp) [Seleccionar] Expandir

void Aleatoriafecha(int l, int fechastotales, tipofixture *fixture){
    srand(time(NULL));//Para que no repita en caso de fixture nuevo
    int vec[fechastotales];

"fechastotales" es una variable. Esta forma de definir los arreglos no es estándar y puede darte problemas. Es más seguro crear un arreglo de tamaño fijo y más grande de lo necesario o recurrir a la memoria dinámica.

La función AleatoriaFecha no funciona

Si revisas el código, verás que "vec" únicamente va a contener un elemento SIEMPRE... en la función no hay ningún bucle.

La función "crearlista" no funciona

Código (cpp) [Seleccionar] Expandir


void crearlista(tipofixture *fixture){ //creo listas
fixture=NULL;

};

En primer lugar, el nombre de la función no se corresponde con el funcionamiento teórico de la función, ya que no crea nada, en tal caso inicializa un puntero. En segundo lugar, esta función no hace ABSOLUTAMENTE NADA, "tipofixture", si bien es un puntero, es una variable local. Esto quiere decir que cualquier cambio que realices sobre el puntero en sí (ojo, no sobre la memoria "apuntada" ), es un cambio que va a desaparecer al salir el código de la función. Si tu idea es que el cambio del puntero se refleje fuera de la función tienes que usar un puntero doble:

Código (cpp) [Seleccionar] Expandir


void crearlista(tipofixture **fixture){ //creo listas
*fixture=NULL;
};

Resumen

Tu código tiene bastantes inconsistencias. Deberías revisarlo con más detenimiento.

Ah si, y sería de agradecer que, para la siguiente ocasión, el código que facilitas pueda ser compilado... no es demasiado agradable tener que deducir el funcionamiento de tu programa únicamente viendo líneas de código.

Un saludo

C# es agua de otro costal, es código privativo Microsoft, pero es hijo de C++

Desde más o menos abril del año pasado, C# pasó a ser código abierto.

Te agradezco el código, no tenía constancia de esa función (funciona a la perfección, por cierto) pero a la vez soy del colectivo imbécil que le encanta reinventar la rueda, como bien dices pienso que para aprender está genial.

Una posible implementación de "next_permutation" copiada de la web cppreference:

Código (cpp) [Seleccionar] Expandir

template<class BidirIt>
bool next_permutation(BidirIt first, BidirIt last)
{
   if (first == last) return false;
   BidirIt i = last;
   if (first == --i) return false;

   while (1) {
       BidirIt i1, i2;

       i1 = i;
       if (*--i < *i1) {
           i2 = last;
           while (!(*i < *--i2)) ;

           std::iter_swap(i, i2);
           std::reverse(i1, last);
           return true;
       }
       if (i == first) {
           std::reverse(first, last);
           return false;
       }
   }
}

* Te he puesto el código directamente porque explicado queda un poco rebuscado

Si tu idea es pegarte con ello por aprender, perfecto.

En caso contrario, lo que estás buscando ya existe:

Código (cpp) [Seleccionar] Expandir

#include <algorithm>
#include <iostream>

int main( )
{
  std::string cadena = "ABCD";

  do
  {
    std::cout << cadena << std::endl;
  } while ( std::next_permutation( cadena.begin( ), cadena.end( ) ) );
}

Segundo yo no fui el desarrollador del código fue un colega mio como lo dije al principio de la publicación de dicho código, yo solamente compile el código y lo publique.

Si lo publicas, aunque no sea tuyo, pasas a adquirir cierta responsabilidad, ya que es tu nick el que figura junto al aporte.

Si alguien te dice "firma aquí", no vas a hacerlo sin el documento, entenderlo y estar conforme. Y es totalmente comprensible debido a las implicaciones legales que puede tener firmar algo.

Con lo que publicas en un foro pasa lo mismo. Si publicas algo, es tu nombre el que aparece junto al aporte y eso te hace responsable del contenido publicado. Por otro lado, dado que lo has publicado se entiende que estas conforme con el contenido, luego no es correcto excusarse diciendo que el código no es tuyo.

Si quieres aportar cosas a un foro, perfecto, pero no pierdas de vista que el aporte tiene que ser constructivo. Si no cumple ese requisito te puedes ahorrar el aporte. Una frase célebre de Hipócrates dice así: "Ante la enfermedad sólo pueden tomarse dos actitudes: o curar o, al menos, no perjudicar". En este caso, lo suyo es hacer aportaciones productivas o, al menos, no enseñar malos hábitos a los que están aprendiendo.

Un saludo.