SoloCodigo

bueno soy nuevo en el foro, tengo 15 años y recien estoy empezando a programar. Estoy intentando hacer un programa en C que entre otras cosas clasifica un compuesto quimico, pero cuando tengo que llamar a la funcion ecuacion(), le paso 4 argumentos:

2 strings (cada una viene de otra funcion llamada simb() )
2 enteros (los escribo manualmente)

El problema es que me toma bien el primer string y ambos enteros, pero el segundo string me lo toma igual que el primero, aca les dejo el codigo, perdonen si es alguna macana mia xD

Aca llamo a la funcion:

Aca la declaracion:

(no les copio toda la declaracion porque el error esta cuando paso los argumentos, ya que comprobe los argumentos antes de pasarlos y estaban bien, pero al comprobarlos una vez pasados pasaba lo que comente arriba)

Por ejemplo:
- paso la valencia del hidrogeno y del otro elemento (ambas 1) manualmente
- paso el simbolo del hidrogeno mediante una funcion que hice simb()
- paso el simbolo del elemento guardado en la variable tercera (por ejemplo aluminio) tambien mediante simb()

seria como poner ecuacion(1,1,H,Al)

pero cuando compruebo los valores dentro de la funcion ecuacion, es como si hubiera pasado (1,1,H,H)


Gracias y perdonen si lo puse medio confuso

Tienes razon, probe pasando manualmente los simbolos y los tomo bien, entonces el error debe ser de simb()... aca te dejo la declaracion:


de alguna forma debe de quedar guardado el simbolo anterior y devuelve el mismo :S

Gracias por tu ayuda

es que no se eso de depurar, como dije, soy nuevo, cuando vuelva del colegio busco por internet como hacerlo

si quieren les dejo el codigo fuente

No, asi es como debe salir, pero en el codigo sin modificar tambien funcionaba pasando los argumentos "manualmente", el tema esta cuando los paso con simb()

no entiendo bien lo que dices al principio, pero stdio.h esta incluida, ya que en quimica.c hago:
#include "elementos.h"
y en elementos.h hago:
#include <stdio.h>

y con respecto a las variables, hay alguna variable/funcion no definida al principio?
es que no me doy cuenta :S

Muchas gracias por gastar tu tiempo y hasta modificar el codigo (no conozco c++ todavia ya que primero quiero aprender bien c)

Editado: Mirando bien me di cuenta que meti algunos printf() antes de la declaracion de algunas variables locales (el codigo esta lleno de printf() que son para revisar el valor de algunas variables, y cuando ya funciones las borro o le agrego las // a esa linea y listo)

Igualmente arregle esto y sige sin pasarme bien el segundo argumento de ecuacion() con simb()

corrigeme si me equivoco, pero la estructura del programa quedaria algo asi como:

Jeje sorry si escribo confuso

EDITO: recien lei tu respuesta, entendi lo de static que me dijiste, asi que ahora lo arreglo, en el codigo que esta en esta respuesta ya esta arreglado lo de declarar las variables primero, lo que si no entendi es lo de que no pongo el tipo de retorno. Por ejemplo en elementos.h, metalono esta con el tipo de retorno int, no se que me corregiste (exepto que te hallas fijado en la "estructura" que puse en la respuesta anterior, que era solo representativa y no tenia ninguno de los tipos de retorno))
Igualmente ya funciona y ahora que me aclaraste lo de static ya voy a poder seguir con la funcion formula() para oxidos y haluros.
GRACIAS
--------------------------------------------------------------------------------------------

Logre hacerlo bien haciendo entre otras cosas que en vez de pasar simb("hidrogeno") pase directamente "H" (al pedo usaba simb ahi)

El problema es que solo sirve para hidruros y oxidos (donde el elemento H o O no varia. Cuando son dos elementos variables como en un haluro (no importa que no sepas lo q es, solamente q pueden ser dos elementos cualquiera, entonces si o si tendria q usar simb() en ambos)

Te subo el codigo esta vez arregle un par de cosas y ya funciona, pero sigo con la duda de por que ecuacion ecibia ambos argumentos simb() iguales

Hice una prueba haciendo que cada vez que simb() se ejecute me imprima en pantalla el valor q va a devolver.
Cuando le paso los argumentos a ecuacion() uso simb() dos veces, por lo tanto me imprimia el valor q devolvia en cada caso al argumento. Y para mi sorpresa me devolvio bien los dos, es decir, Al y H (suponiendo que escribo "Hidruro de aluminio" en la opcion 3 del programa).

Esto quiere decir que el error esta cuando ecuacion() me toma los dos argumentos?
si tienes tiempo y ganas de probarlo acordate que cuando llamo a ecuacion() en el codigo actual le paso un solo argumento con simb(), por eso no da error, pero haz la prueba poniendo en vez de "H" simb("hidrogeno")

GRACIAS

EDITO: recien lei tu respuesta, entendi lo de static que me dijiste, asi que ahora lo arreglo, en el codigo que esta en esta respuesta ya esta arreglado lo de declarar las variables primero, lo que si no entendi es lo de que no pongo el tipo de retorno. Por ejemplo en elementos.h, metalono esta con el tipo de retorno int, no se que me corregiste (exepto que te hallas fijado en la "estructura" que puse en la respuesta anterior, que era solo representativa y no tenia ninguno de los tipos de retorno))
Igualmente ya funciona y ahora que me aclaraste lo de static ya voy a poder seguir con la funcion formula() para oxidos y haluros.
GRACIAS
--------------------------------------------------------------------------------------------

Entonces mucho no sirve 

Ahora que me aclaraste eso de static ya puedo hacer que funcione con haluros
Gracias de nuevo

Es que las funciones mostrar, etc, solamente devuelven 0 (al no poner tipo de retorno es como poner int, cierto?) para que luego main termine (tambien podria hacerlas void), ya que en realidad esas funciones se utilizan una vez sola en el programa (para que son funciones entonces? jajaja, es solo para que me entienda mejor ya que no comento demasiado los programas como veras, tengo q acostumbrarme)

Ahora me surgio otra duda, en la funcion romanos tambien uso static
y la uso tambien 2 veces en este caso, y me devuelve los valores q quiero (distintos), por ejemplo "I" y la segunda vez "III"
porque?
no tendria que devolverme ambas veces "I"?
quizas no entendi tan bien lo que me explicaste

Gracias,

Edito: ya hice simb sin static y funciono, gracias

En C++ no hay tipos de retorno por defecto y tu programa en C no compila.

Mi programa lo compilo con el DevC++
esta compilando en c++? (es decir que no tiene nada que ver que el codigo este en un archivo.c?)
como cambio para que compile en C? (es que nunca me di cuenta si compilaba en c++, nunca me tiro ningun error con respecto a los tipos de retorno automaticos ni nada)

Si pones void es que no retorna nada (no podes usar return), 0 es un entero.

Con lo de void me referia a sacar el return tambiem.

Que una funcion se use una sola vez no significa que no deba retornar nada, no tiene nada que ver ...

Quise decir como que si la voy a usar solo una vez para que la hice funcion, si la podria haber escrito en main. Y por eso dije que solo era para entenderla mejor.

Gracias

Termine el programa!
Al fin lo deje tal cual queria, y todo gracias a tu ayuda

Como agradecimiento te dejo el programa, aunque no te va a servir de mucho exepto que sepas de quimica o (como yo) te sirva para la tarea de quimica del cole.
Jaja
Gracias

(por cierto, dentro del rar hay dos, uno compilado en ANSI C (o eso creo) y el otro en C comun (tampoco estoy seguro), aunque no tuve que cambiar demasiadas cosas)

EDIT:
Subo la version 3, que arregle un par de cosas y ademas agrege funciones
El programa ya acepta hidruros (metalicos y no metalicos), oxidos basicos y acidos, haluros, hidroxidos y oxoacidos
=)