Hola, quería exponer una duda a cerca de una respuesta que hice a un usuario en el foro de C/C++.
En concreto este es el mensaje al que hago referencia: https://foro.elhacker.net/programacion_cc/crash_en_la_consola-t509690.0.html;msg2239804#msg2239804

Mi duda es a cerca de la norma de no resolver tareas a los usuarios del foro. E publicado una respuesta en la que incluyo un ejemplo completo del programa que necesita realizar a posteriori de que el usaurio ya ha conseguido dar con una solución y la ha publicado en el tema.

Mi motivación para aportar una respuesta completa a la tarea viene a coalición con lo que he expuesto en el mismo mensaje. En concreto trato de ejemplificar el uso de identación, funciones puras (lo que trata de usar el usuario que abre el tema) y proponer otra alternativa más organizada a la resolución de la tarea.

La razón de mi duda viene tras recibir un mensaje privado

No entendí cual era el problema asique respondí explicando mi motivación al publicar el código y preguntando cual era el motivo. No me convenció el argumento de la cantidad de lineas de código con respecto al usuario ya que el usuario omitió varios saltos de lineas que yo si añadí para que el código quedase mejor organizado así como el uso de más includes y funciones por mi parte en lugar de meter todo en una sola función, lo cual de por sí cada función son mínimo 4 o 5 lineas más. El prototipo, la declaracion, el return, el corchete final de la declaración y las llamadas que hago en cout, que decidí utilizar 3 llamadas independientes de cout en lugar de una sola para que el código quedase más claro

Tras lo cual:

Yo personalmente creo que mi código no es excesivo ni innecesario, asique en lugar de seguir respondiendo por privado y seguir argumentando (algo que obviamente no va a llevar a ningún lugar porque ya ambas partes hemos expuesto nuestra opinión a cerca del tema y no parece que vayamos a llegar a ningún lado) decido exponer mi duda aquí para zanjar el debate a cerca de si es correcta mi respuesta o no.
En caso de no serlo no tengo ningua objeción en que sea eliminado, modificado o lo que sea.

También me gustaría tener unas directrices claras de cuanto código es excesivo para poder seguir participando sin inclumplir las normas. O si la cantidad de código permitido en una respuesta mantiene algún tipo de relación con la cantidad de código sin identar que publica el usuario que abre el tema.

[MOD] Retirados los MP, estos no deben ser publicados en publico, al ser privados.

Te ha quedado un poco caótico, con muchísimas variables innecesarias emy algunas expresiones que puedes eliminar y no pasa nada.

Lineas 6, 7 y 8 deberían ir dentro de main. El motivo es que si declaras las variables globales, estas van a poder ser accesibles por librerías de otros programadores que incluyas en tu código, tus propias funciones que no deberían de poder acceder a ellas, se malgasta memoria al estar siempre accesibles hasta que se acabe el programa, etc. Usa solo globales cuando sea imprescindible.

Linea 11 y demás; deberías de identar (dar espacios antes de la palabra) dentro de cada cuerpo de función, bucle, condicional, etc. Es recomendable dar entre 2 y 4 (ambos incluidos) espacios. A tu gusto pero el número que eligas úsalo de forma consistente en todo el código del programa. Dar espacios facilita la interpretación del código a simple vista, mejora la lectura y es más sencillo detectar errores como la falta de un corchete.

En tu función int PedirEstudiantesEnRango(), retornas números, pero en ningún moemento haces nada de nada con el valor que se retorna. Si quieres una función que haga algo pero que no retorne nada, entonces declárala como void en lugar de int y te ahorras la linea de return. Lo lógico cuando usas funciones, es que hagan tareas más concretas y especializadas, y no funciones que hagan todo.

En lugar de usar globales dentro de la función PedirEstudianteEnRango, lo correcto es declarar las variables dentro de main y enviárselas como argumento de la llamada a la función. Y en la propia funcion las pones como parámetros. Ejemplo de lo que haces:

int numero1, numero2;
 
int sumar();
 
int main(){
numero1 = 30;
numero2 = 40;
sumar();
return 0;
}
 
int sumar(){
int resultado=numero1+numero2;
cout<<"El resultado es: "<<resultado;
return resultado;
}

Ejemplo de lo que deberías hacer:

int sumar(int num1, int num2);
 
int main() {
  int numero1 = 30;
  int numero2 = 40;
 
  cout << "El resultado es: " << sumar(30, 40);
}
 
int sumar(int num1, int num2) {
  return num1 + num2;
}

La linea 28 no hace nada. Tu compilador debería de lanzarte un warning avisándote de que eso ahí no se está usando para nada.

En las linea 29 y 31 los paréntesis en (i+1) no hacen nada, es como no ponerlos.

En la linea 51 tienes que convertir a doble ambos valores primero antes de dividirlos para que no trunque los decimales. Tal como lo tienes si divides 3 / 2 te da 1 en lugar de 1.5


Te dejo como lo haría yo por si te sirve de orientación para mejorar tu versión del programa conjuntamente con los consejos que te dejo y los que te ha dejado K-YreX. Repasa sus consejos y trata de entenderlos todos.

#include <iostream>
#include <string>
#include <vector>
 
using namespace std;
 
struct Estudiante {
  int edad;
  string sexo;
  string carrera;
};
 
int promedioEdadEstudiantesIngenieria( vector<Estudiante> &listado );
int porcentajeHombresUniversitarios( vector<Estudiante> &listado );
int porcentajeMujeresEstudianIngenieria( vector<Estudiante> &listado );
 
int main() {
  int numeroEstudiantes = 0;
  cout << "Cuantos estudiantes vas a introducir?" << endl;
  cin >> numeroEstudiantes;
 
  vector<Estudiante> listadoEstudiantes;
 
  for (int i = 0; i < numeroEstudiantes; ++i) {
    Estudiante estudiante;
    cout << "Edad del estudiante?" << endl;
    cin >> estudiante.edad;
 
    cout << "Sexo del estudiante?" << endl;
    cin >> estudiante.sexo;
 
    cout << "Carrera que cursa el estudiante?" << endl;
    cin >> estudiante.carrera;
 
    listadoEstudiantes.push_back(estudiante);
  }
 
  cout << "Promedio de edad de los estudiantes de ingenieria: " << promedioEdadEstudiantesIngenieria(listadoEstudiantes) << endl;
  cout << "Porcentaje de universitarios masculinos: " << porcentajeHombresUniversitarios(listadoEstudiantes) << "%" << endl;
  cout << "El " << porcentajeMujeresEstudianIngenieria(listadoEstudiantes) << "% de las mujeres universitarias estudian ingenieria" << endl;
 
  return 0;
}
 
 
int promedioEdadEstudiantesIngenieria( vector<Estudiante> &listado ) {
  int promedio = 0;
  int numeroDeEstudiantesIngenieria = 0;
  for (int i = 0; i < listado.size(); ++i) {
    if (listado[i].carrera == "Ingenieria" || listado[i].carrera == "ingenieria") {
      promedio += listado[i].edad;
      ++numeroDeEstudiantesIngenieria;
    }
  }
  return promedio / numeroDeEstudiantesIngenieria;
}
 
 
int porcentajeHombresUniversitarios( vector<Estudiante> &listado ) {
  int numeroHombres = 0;;
  for (int i = 0; i < listado.size(); ++i) {
    if (listado[i].sexo == "Hombre" || listado[i].sexo == "hombre") {
      ++numeroHombres;
    }
  }
  return 100 / listado.size() * numeroHombres;
}
 
 
int porcentajeMujeresEstudianIngenieria( vector<Estudiante> &listado ) {
  int numeroMujeresEnTodasCarreras = 0,
  numeroMujeresEstudianIngenieria = 0;
  for (int i = 0; i < listado.size(); ++i) {
    if (listado[i].sexo == "Mujer" || listado[i].sexo == "mujer") {
      ++numeroMujeresEnTodasCarreras;
      if (listado[i].carrera == "Ingenieria" || listado[i].carrera == "ingenieria") {
        ++numeroMujeresEstudianIngenieria;
      }
    }
  }
  return 100 / numeroMujeresEnTodasCarreras * numeroMujeresEstudianIngenieria;
}

Me refiería a esto.

for (int i = 0; i < recibidos.Length; i += 1) {
  richTextBox1.Text += recibidos[i];
  if (i % 8 == 0) {
    richTextBox1.Text += " ";
  } 
}

Uhh, no sabía que sucedía ese error.

Suma de 1 en 1 en lugar de 8 en 8 y cuando i % 8 == 0 añades un espacio.

Usa un bucle

string binario = "001000111100011110010111110000001011000000001101";
string resultado = "";
for (int i = 0; i < binario.Length; i += 8) {
  resultado += binario.Substring(i, 8) + " ";
}

Substring pilla desde el index hasta los siguientes 8 caracteres, asique si solo quedan por ejemplo 3, los pillas igual.

Te sobra un espacio al final. Puedes sumarle 8 al iterador para ver si en la siguiebte iteración ya no vas a entrar en el bucle A.K.A no añadas el espacio la última vez que iteras porque ya acabaste de añadir bytes. O reasignas con Substring usando 0, y resultado.Length -1

Puedes publicar aquí el script, o es muy largo?

La media se realiza sumando todos los elementos y dividiendo el resultado de la suma por el número de elementos.

La forma más rápida de recorrer un vector es con el

for(int element : myVector) {
  cout << element << endl;
}

Para sumar múltiples elementos, ve guardándolos en una variable.

int total = 0;
...
total += element;

Para calcular la media divides el total entre el número de elementos. Para calcular el número de elementos del vector puedes usar myVector.size() o myVector.length()

Pasa todo por el servidor. Haz un bot para jugar 24/7

No es lo mismo descargarte 2 teras cada vez que abres una aplicación que contrastar con el servidor enviando un hash si los archivos ya están descargados.

Te dejo una lista de cosas que puede que haga en un futuro con javascript:
- Gestor de paquetes tipo pip, npm, apt, pacman, dpkg, apk, pkg...
- Servidor VPN compatible con el cliente Android.
- Conversor Markdown, HTML, BBcode.
- Algoritmo de compresión.
- Emulador de sistemas de ficheros y comandos para el desarrollo de HoneyPots.
- IDS.
- Antivirus.
- Gestor de dependencias de programas.
- RPG.
- Editor de texto compartido (websockets)
- CMS.
- Scanner de directorios, dominios, subdominios...
- Navegador scriptable.
- Parser de lenguajes de programación.
- Sockets para Quickjs
- Virtual DOM con diff.
- Search Engine.