Te mostré como se hacía esto en el tema que iniciaste anterior a este...

Temas a tratar:
- El código siempre entre etiquetas GeSHi. Puedes seleccionarlas encima del cuadro de texto cuando estás escribiendo un mensaje.
- Lo de no usar <conio.h> ya te lo han comentado. Sustituye <getche()> por <getchar()> y adiós <conio.h>.

- La función <registro()> devuelve un <float>. No tiene ningún sentido si siempre va a devolver 0. Si quieres modificar un <Usuario> o haces que <registro()> devuelva un Usuario y se lo asignas a un Usuario en el <main> o pasas un <Usuario> por referencia para poder modificarlo y que los cambios se guarden.
- No sé si programas en C usando <iostream> o programas en C++ usando <gets>. Si estás con C++ usa un <string> para el nombre y <cin> para las entradas de datos. Si es C y tienes que usar <gets>, usa <fgets>.

void registro(Usuario &user){
    cout << "Nombre: ";
    fgets(user.nombre, 100, stdin);
    cout << "Edad: ";
    scanf("%d", &user.edad);
    cout << "Registrado Usuario: " << user.nombre << " Edad: " << user.edad << endl;
} 

En este trozo de código te dejo la respuesta a como concatenar variables con cadenas de texto usando <cout>.

- El uso de <while> no es este. <while> se usa para un bucle que se repite. Lo que tú quieres usar aquí es un simple <if>.

while(res == 'N' || res == 'n'){
	cout<<"\n\nOk, que tenga un buen dia.";
	return 0;
 
// Forma correcta
if(res == 'N' || res == 'n')
    // todo lo demas
} 

- En el caso de que la respuesta sea <S> más de lo mismo, se utiliza un <if> no un <while>. En tu caso con el <if> cuando hayas elegido una opción te pedirá otra y otra y otra y otra. Aunque bueno sólo tienes definida la primera opción (imagino que el resto no lo has puesto por no hacer el código más largo )

- No uses <if> para comprobar las opciones. Usa un <switch>.

switch(opcion){
    case 1:
        // lo que sea si la opcion es 1
        break;
    case 2:
        // lo que sea si la opcion es 2
        break;
    // asi con todas las opciones que tengas y finalmente
    default:
        // lo que sea si la opcion no es ninguna de las anteriores
        break; // este es opcional
} 

- Además y si a la primera pregunta no se responde ni <N> ni <S>? . Se sale también pues entonces haz que si la respuesta es <S> se ejecute el resto del programa y sino, <else>, se salga y no uses tanto <if>.

- Siguiente trozo de código.

if(opcion == 1);{
	cout<<"\nUsted quiere ver la de Venom..."<<endl;
	if(regis.edad<18){
		cout<<"Lo siento pero no puedes verla, esta bien grotesca para alguien como usted"<<endl;
	}
	else{
		cout<<"Perfecto, son $40"<<endl;
	}
	return 0;
} 

- El punto y coma que va después del <if> de fuera sobra.
- El problema de la edad ya está arreglado con el cambio que he hecho a la función <registro()>.
- Concatenar el valor de la constante ya te lo he mostrado también.
- Hacer un descuento?? Pues vaya complicación.

else
    cobrar();
// el resto del programa
 
// funcion cobrar()
void cobrar(){
    const double DESCUENTO = 0.5;
    double precio = PRECIO; // PRECIO yo lo pondría <double> por si no es un entero.
    char respuesta;
    cout << "Eres estudiante (s/n)? ";
    respuesta = getchar();
    if(respuesta == 'S' || respuesta == 's')
        precio *= DESCUENTO;
    cout << "Perfecto, son $" << precio << endl;
} 

- Y una vez se ha elegido pagado, te echa del sistema con un <return> así que el <while> de antes para meter varios usuarios ha quedado inservible. Más vale que nadie quiera ver la de Venom porque sino el siguiente que venga se encuentra con el sistema caído y se queda sin peli.

- Hasta aquí todo bien, bueno contando todos estos fallos...
- Pero ahora, si la opción es 1 destruimos el sistema y si no es 1: creamos un usuario nuevo (y al anterior que le den por no saber elegir), creamos unas variables nuevas con el nombre de las viejas pero encima con los tipos cambiados. Una pena ya que iba a molar mucho que si el primero dice que quiere comprar un boleto y la máquina le echa por elegir mal el siguiente que venga compra sí o sí gracias al <while> (debe de ser una medida para recaudar más boletos ).

- Y ya después de confundir un poco al compilador y hacer que se enfade y nos escupa unos cuantos errores llegaríamos al final. Una pena que nunca lleguemos al final ya que el programa nunca llega a compilar. Aunque han usado medidas para vender más boletos al final si no funciona la máquina se quedan sin vender ninguno...

Ya tienes corregidos los errores pequeños y unas cuantas y repito, CUANTAS, recomendaciones para que ese código funcione mejor sólo te queda arreglar la función principal para que funcione. Vaya así dicho parece como si mi esfuerzo no hubiese servido de nada... 

PD: No había visto el <getch()> del final. Cambialo por <cin.get()> y manda a <conio.h> al quinto...

conio 

No. Tienes que modificar el mensaje y donde pone "Código GeSHi" despliegas el menú y eliges C y te saldrán dos etiquetas entre corchetes [ ].
Tienes que poner tu código entre los corchetes, es decir, [...] <tu código> [...].
Puedes darle a previsualizar para ver como ha quedado y el código tiene que quedar así:

// tu codigo 

Modifica el mensaje y mete el código entre etiquetas GeSHi para que podamos verlo bien.

Lo primero que no sé que idea tienes en la cabeza entonces no sé por qué haces algunas de las cosas que haces:
- El prototipo de la función <abrir()> no tiene parámetros, pero la definición de la función <abrir()> tiene un parámetro de tipo <int>.
- La función <menu()> está casi bien. Si la opción es 0 cambias el valor a una variable de tipo bool que no sé cual es su utilidad. 
- Además según tú, la función <abrir()> devuelve un valor de tipo <int> y tu función no devuelve nada.
- En la función <abrir()> compruebas que la variable <modo> sea 1 pero la variable <modo> no está inicializada.  Ahí lo único que tendría un poco de sentido sería que en vez de <modo == 1> quieras hacer <menu == 1>. Pero tienes que corregirlo.
- Y por último en todo el programa no llamas a la función <abrir()> 

Esto puede parecer una tontería ya que no tiene mucha complicación usar la librería de C++ <cstdlib> y las funciones <srand> y <rand> para generar números aleatorios. Pero qué pasa si no usas <cstdlib> o usas la función <time()> sin incluir <ctime>??
Estaba haciendo un programa en el que iba a generar unos números aleatorios pero se me ha olvidado incluir tanto la librería <cstdlib> como la función <srand> es decir que únicamente he usado la función <rand> sin más... Y el programa ha funcionado.
Me he puesto a hacer pruebas y os dejo los resultados (compilado desde Ubuntu 18.04 con g++ <programa.cpp> -o <programa>):

#include <iostream>
#include <list>
 
using namespace std;
 
int main(){
	const int MAX_ELEMENTS = 10;
	const int LIMIT_INF = 0, LIMIT_SUP = 5;
 
	list<int> myList;
 
	for(size_t i = 0; i < MAX_ELEMENTS; i++)
		myList.push_back(LIMIT_INF + rand() % (LIMIT_SUP - LIMIT_INF + 1));
 
	for(auto it = myList.cbegin(); it != myList.cend(); it++)
		cout << *it << "  ";
	cout << endl;
} 

Con este código compila el programa sin errores ni warnings pero sí es cierto que los números son siempre los mismos (lógico por no usar srand).

Entonces he probado a incluir la semilla sin nada más, es decir, como se ve aquí:

#include <iostream>
#include <list>
 
using namespace std;
 
int main(){
	const int MAX_ELEMENTS = 10;
	const int LIMIT_INF = 0, LIMIT_SUP = 5;
 
	list<int> myList;
 
	srand(time(NULL));
 
	for(size_t i = 0; i < MAX_ELEMENTS; i++)
		myList.push_back(LIMIT_INF + rand() % (LIMIT_SUP - LIMIT_INF + 1));
 
	for(auto it = myList.cbegin(); it != myList.cend(); it++)
		cout << *it << "  ";
	cout << endl;
} 

Y ahora tengo un programa que compila sin errores ni warnings y que genera los números aleatorios distintos cada vez que se ejecuta el programa (sin compilarlo antes de cada ejecución).
También si os dais cuenta tampoco he incluido <ctime> para usar la función <time()>.
He estado investigando <iostream> por si incluía <cstdlib> pero no he encontrado nada. Lo de usar <list> es simplemente porque era lo que estaba usando cuando me he dado cuenta de esto y no he querido quitarlo por si tenía algo que ver con la respuesta aunque tampoco he encontrado nada al respecto.

El prototipo de la función <abrir> no tiene parámetros pero luego en su implementación sí tiene un parámetro de tipo <int>
Si estás haciendo cambios en el programa vuelve a mandar el código para ver como lo tienes ahora ya que no es igual que el que mandaste al principio.

Y si por ejemplo tienes datos pendientes en el buffer pero no sabes que datos son, osea que no tiene porqué ser un salto de línea (0xa), has encontrado alguna forma de vaciar el buffer??
Es decir igual que:
Pero como representar el final de datos pendientes. No sé si me he explicado.

Primero: no entiendo muy bien el objetivo del programa, osea no entiendo que es lo que quieres calcular, por lo que haré uso de mi imaginación.
Segundo: por qué pasas como parámetros x e y si nada más empezar la función los modificas??
Tercero: estás seguro de que la suma y el resultado sólo deben ponerse a 0 al empezar la función y no habrá que ponerlos a 0 en cada iteración?? Y lo digo como pregunta porque repito que no sé cual es el propósito de la función pero estos suelen ser los errores más comunes por eso te los comento.

Y si no se arregla con lo que te he dicho explícame por favor el uso de esa función y cómo la usas en el <main> ya que no sé qué dos parámetros le pasas (aunque no haces uso de ellos).

Yo la verdad también tengo esa duda ya que por mucho que he leído y he buscado información no he encontrado respuesta a tu pregunta. Sin embargo aunque no suelo programar mucho en C no suelo tener el problema del buffer pero por ejemplo intentando ayudar en un post que dejo a continuación:
https://foro.elhacker.net/programacion_cc/necesito_ayuda_en_este_rpograma_quiero_que_pregunte_si_quiero_hacer_otro_correo-t489865.0.html
me pasaba lo mismo porque quedaban los saltos de línea en el buffer y entonces se me saltaban algunas de las entradas de datos. El tema de los saltos de línea pude solucionarlos como buenamente pude (puedes ver el código que dejé en ese post para ver las soluciones que encontré aunque son unas soluciones un poco caseras para salir del paso) ya que no quería usar <fflush> ya que también he leído que no debe usarse para el buffer de entrada, sino para el de salida.

Sin embargo me siguió quedando un problema en ese código que dejé comentado también para ver si alguien podía ayudarme a solucionarlo pero al final se ha quedado ahí y yo me he quedado sin una respuesta.
Espero que alguien nos pueda ayudar a resolver esta duda tan básica pero que todo el mundo pasa por alto...