Mensajes

[¡¡¡¡ Saluditos! ..... !!!!]

Te falta reiniciar las variables fila1 y fila2 cada vez que invocas al while del contador.

Pero una forma, creo que más simple, y que produce la salida deseada:

* * * * * * * *
*  *  *  *  *  *  *  *
* * * * * * * *
*  *  *  *  *  *  *  *
* * * * * * * *
*  *  *  *  *  *  *  *
* * * * * * * *
*  *  *  *  *  *  *  *

podría ser:

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>

int main()
{
int fila1 = 1;
int fila2 = 1;
int contador = 1;

for(contador=1;contador<=4;contador++) {
fila1=fila2=1;
while (fila1 <= 8){
printf("* ");

fila1++;
}
printf("\n");
while (fila2 <= 8){
printf(" ");
printf("* ");

fila2++;
}
printf("\n");

}
return 0;
}


Claro que a lo mejor sólo puedes usar while. En este caso valdría:

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>

int main()
{
int fila1 = 1;
int fila2 = 1;
int contador = 1;

while (contador<=4) {
fila1=fila2=1;
while (fila1 <= 8){
printf("* ");

fila1++;
}
printf("\n");
while (fila2 <= 8){
printf(" ");
printf("* ");

fila2++;
}
printf("\n");
     contador++;
}
return 0;
}


¡¡¡¡ Saluditos! ..... !!!!

[¡¡¡¡ Saluditos! ..... !!!!]

Pregunta rara,rara.

Supongo que esto es lo que buscas:



El extremo de la línea donde está la frase "La asignación de su ...." se obtiene restando a 79 la longitud de la cadena. Se puede hacer contando a ojo o con strlen.

Código (cpp) [Seleccionar] Expandir



#include <iostream>
#include <stdlib.h>
#include <iomanip>
#include <cstring>

using namespace std;


int main()
{
   int i;
   system("cls");
   for (int i =0; i <= 79; i++) //Linea superior del marco
     cout << "\xCD";
   for (int i =1; i <= 20; i++){
     if (i==1)
     cout << "\xBA"<< setw(58) << "Esta en el menu de primera Clase"
             << setw(21)<<"\xBA";
     if (i==1)
        cout << "\xBA" <<"La asignacion de su asiento es: "<< setw(47)<< "\xBA";
     if (i>1)
     cout << "\xBA"<< setw(79)<< "\xBA";
     }
   for (int i =0; i <= 79; i++)//Linea inferior del marco
     if (i>59)
       cout << ' ';
   else
     cout << "\xCD";
    cout << ' '<<system("pause");
   return 0;
}


¡¡¡¡ Saluditos! ..... !!!!

[¡¡¡¡ Saluditos! ..... !!!!]

Pues a mi me corre sin problemas. ¿Qué problema tienes?.

¡¡¡¡ Saluditos! ..... !!!!

[¡¡¡¡ Saluditos! ..... !!!!]

Desactivando los include de Operaciones.h y Operaciones.cpp y poniendo la barra invertida "\" al final de las estrellitas el código funciona correctamente.

¿Podrías concretar menor la duda y/o problema que tienes?

¡¡¡¡ Saluditos! ..... !!!!

[¡¡¡¡ Saluditos! ..... !!!!]

Hay un mal uso de punteros ahí, estás escribiendo en una zona de la memoria que no deberías, ahora tanto como ser un virus... Por eso no uso antivirus, detecta un montón de amenazas falsas y deja pasar las de verdad. Lo mejor es usar el sentido común.

En fin, el problema es esto:

Código (cpp) [Seleccionar] Expandir

scanf("%d",&num);

Debería ser:

Código (cpp) [Seleccionar] Expandir

scanf("%d",&num[i]);

Puesto que tu idea será rellenar casilla a casilla no?

Pues no, no es eso.

En realidad lo tiene puesto. Mira su código con etiquetas Code:

Código (cpp) [Seleccionar] Expandir


# include <stdio.h>
# include <conio.h>
# include <math.h>
#include <stdlib.h>


main()


{
      int num [4];
      int con= 4;
      int i;
      long sum= 0L;
      float prom= 0.0f;
     
      printf("\n\tComo 4 numeros:\n")  ;
          for (i=0; i< con; i++)
          {
            printf("%2d>",i+1);
            scanf("%d",&num[i]);  /*<==AQUI esta puesto el [i] */
              }
         getch();
           
     }


Es el problema de no usar las benditas etiquetas Code ya que al encontrarse con [ i], sin el espacio en blanco que he dejado aposta, interpreta el [ i] como la etiqueta de las letras en cursiva. Por eso a partir de ahí las letras del post salen en cursiva.

Por cierto, el código anterior lo he copiado del original del post dándole a citar y copiándolo de ahí.

¡¡¡¡ Saluditos! ..... !!!!

[hay que indentar el código]

perdon por este reflote pero he estado volviendo a intentar sacar el ejercicio desde 0 y no hay manera  

alguien podria explicarme porque este codigo no funciona?, se que debo tener algun error de logica o algo pero no soy capaz de autocorregirme:

Esta vez no me voy a tomar la molestia de explicarte el código: analizalo, estudialo, compara con lo que tú habías puesto y haz lo mismo con lo códigos que anteriormente te habíamos posteado.

Se trata de aprender, ¿no?. Para ello lo mejor es lo que te indico.

Claro que cuando no has hecho ni *uto caso de las observaciones que te indiqué en el post anterior:

* main es int y requiere un return.

* hay que indentar el código, Si no lo quieres hacer por ti, hazlo por nosotros que es más sencillo de analizar.

* el uso de los printf. Sí, ya sé que en la UNI no te dejan, pero mientras aprendes ponlos y así observarás por ti mismo donde te equivocas. Ya cuando estés suelto podrás pasar, y sólo en principio, de ellos.

* vuelves a poner un if justo después de un for, cuando, como ya te indiqué anteriormente, puedes en este caso ahorrarte el if poniendo la condición en el for.

* por qué poner i<=10000000000 como condición cuando es más sencillo "numero/i!=0", que ya te lo había puesto. Ya que vas dividiendo el número entre potencias de 10, se trata de que cuando dé más pequeño que 10, o sea cero, se paren las divisiones. Además los int no llegan a tantas cifras como ceros tiene el número que has puesto. Es otro tema a estudiar por tu parte, los límites de los números que puedes usar.

*¿Por qué en la función cifras=2 como valor para inicializar la variable?. Hay que  calcularlas una a una y desde cifras=0.

*Esto está mal: for(cifra;cifra>1;cifra--). En todo caso sería for(i=*cifrass;i>=0;i--)
y la suma casi como la tenías: (*sumaa)+=(int)(num/(pow(10,i)));. Fíjate en lo de int, que es hacer un cast y no es otra cosa que forzar a una variable de un tipo, en este caso float por la presencia de pow, a entero, que es lo que corresponde a la suma de cifras. Es como obligar al programa a que redondee los decimales que pudiera originar pow y la división.

*el uso de pow implica que la variable sea float o double y tú la usas con int y claro salen cosas raras.

* y como pow implica el uso de float o double ya no vale % sino la función fmod

* ********

Vamos, que por lo que veo no has estudiado y/o exprimido los códigos anteriores. Así avanzarás a duras penas ..... y lo que es peor, nos aburres y dejaremos de hacerte caso.

* ¿Y ese cambio a punteros?, ¿te dio un flash o un mal aire?. Por si no es así te lo he respetado, modificando tu código justito lo necesario para que funcione y veas por comparación los errores que has cometido. Pero sería más lógico en tu caso, y con los conocimientos que se intuyen tienes, haber usado variables globales y dejarte de punteros. Claro que a lo mejor es lo que pretendías, usar punteros.

*Es que el final del código, por no hablar del comienzo, no tiene ni pies ni cabeza:

Código (cpp) [Seleccionar] Expandir


..............................

for(i;i<=10000000000;i=i*10){
   if(numero/i==0) break;
   cifra++;
}
cifrass=&cifra;
for(cifra;cifra>1;cifra--){
   s=numero/(pow(10,cifra));
   s=s+s;
}
cif=cifra;
sumaa=&s+(numero%10);
s=(numero/pow(10,cif));
primeracifraa=&s;
}
.........................


* s+s ...¿eso que es?

* uso de la variable cif cuando ya tienes cifras.

* primeracifraa=&s; Si ya habías hecho antes la asociación.

* ***************

Te dejo una salida del código que te pongo a posteriori sin y con con muchos printf,. Actívalos para que veas lo que el programa hace, que es ni  más ni menos que lo que tú quieres:

Introduzca un número:22 11 8 132 324 9 2132 6972 6515 0

22 SI cumple la condicion
11 SI cumple la condicion
8 es menor de 10
132 NO cumple la condicion
324 SI cumple la condicion
9 es menor de 10
2132 SI cumple la condicion
6972 SI cumple la condicion
6515 NO cumple la condicion

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>
#include <math.h>

void funcion(int numero, int *cifrass, int *primeracifraa, float *sumaa);

int main(){
 int i,numero, cifras=0, primeracifra=0;
 float suma=0;
 int *cifrass=&cifras;
 int *primeracifraa=&primeracifra;
 float *sumaa=&suma;
 printf ("Introduzca un n%cmero:",163);
 fflush (stdout);
 while((scanf("%d", &numero)==1 && numero!= 0 )!=0){
   cifras=0,suma=0;
   if (numero>9){
     funcion (numero, cifrass, primeracifraa, sumaa);
     if(*primeracifraa*((*cifrass)+1)==*sumaa) {
       printf("\n%d SI cumple la condicion",numero);
       fflush(stdout);
     }
     else
     printf("\n%d NO cumple la condicion",numero);
   }
   else
     printf("\n%d es menor de 10",numero);
 }
 return 0;
}

void funcion(int numero, int *cifrass, int *primeracifraa, float *sumaa){
 int i,num=numero,cifras=0;
 for(i=10;numero/i!=0;i=i*10)
     (*cifrass)++;
 (*primeracifraa)=numero/ pow(10,*cifrass);
 for(i=*cifrass;i>=0;i--){
   (*sumaa)+=(int)(num/(pow(10,i)));
   num=(int)fmod(num,pow(10,i)) ;
 }
}


Código (cpp) [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>  /*es solo por el system, en realidad esta de mas*/
#include <math.h>

void funcion(int numero, int *cifrass, int *primeracifraa, float *sumaa);

int main(){
 int i,numero, cifras=0, primeracifra=0;
 float suma=0;
 int *cifrass=&cifras;
 int *primeracifraa=&primeracifra;
 float *sumaa=&suma;
 //printf ("*cifrass= = %d   *primeracifraa=%d   *sumaa=%d\n",*cifrass,*primeracifraa,*sumaa);
 system("pause");*/
 printf ("Introduzca un n%cmero:",163);
 fflush (stdout);
 while((scanf("%d", &numero)==1 && numero!= 0 )!=0){
   //printf ("\nnum=%d",numero);
   cifras=0,suma=0;
   if (numero>9){
     funcion (numero, cifrass, primeracifraa, sumaa);
     if(*primeracifraa*((*cifrass)+1)==*sumaa) {
       printf("\n%d SI cumple la condicion",numero);
       fflush(stdout);
     }
     else
     printf("\n%d NO cumple la condicion",numero);
   }
   else
     printf("\n%d es menor de 10",numero);
 }
 return 0;
}

void funcion(int numero, int *cifrass, int *primeracifraa, float *sumaa){
 //puts("\nENTRO");
 //printf ("numero= %d   *cifrass= = %d   *primeracifraa=%d   *sumaa=%d\n",numero,*cifrass,*primeracifraa,*sumaa);
 int i,num=numero,cifras=0;
 //*cifrass=&cifras;
 for(i=10;numero/i!=0;i=i*10){
     (*cifrass)++;
     //printf ("\*cifra=%d   i=%d\n",*cifrass,i);
 }
 //printf ("numero= %d   *cifrass= = %d   *primeracifraa=%d  \n",numero,*cifrass,*primeracifraa);
 (*primeracifraa)=numero/ pow(10,*cifrass);
 //printf ("\*primeracifraa=%d \n",*primeracifraa);system("pause");

 for(i=*cifrass;i>=0;i--){
   (*sumaa)+=(int)(num/(pow(10,i)));
   num=fmod(num,pow(10,i)) ;
   //printf ("\nnum=%d   %d   sumaa=%g \n",num,(int) (num/pow(10,i)),*sumaa);system("pause");
 }
}


De todas formas, si querías usar punteros tal vez lo que sigue era lo adecuado:

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>
#include <math.h>


void funcion(int numero, int *cifras, int *primeracifra, int *suma);

int main(){
 int i,numero,cifras=0,suma=0,primeracifra=0;
 printf ("Introduzca un n%cmero:",163);
 fflush (stdout);
 while((scanf("%d", &numero)==1 && numero!= 0 )!=0){

   if (numero>9){
     funcion (numero, &cifras, &primeracifra, &suma);
     if(primeracifra*(cifras+1)==suma) {
       printf("\n%d SI cumple la condicion",numero);
       fflush(stdout);
     }
     else
     printf("\n%d NO cumple la condicion",numero);
   }
   else
     printf("\n%d es menor de 10",numero);
 }
 return 0;
}

void funcion(int numero, int *cifras, int *primeracifra, int *suma){
 int i,num=numero;
 *cifras=0;*suma=0;*primeracifra=0;
 for(i=10;numero/i!=0;i=i*10)
     (*cifras)++;
 *primeracifra=(int)(numero/ pow(10,*cifras));
 for(i=(*cifras);i>=0;i--){
   *suma+=(int)(numero/ pow(10,i));
   numero=(int)fmod(numero,pow(10,i)) ;
 }
}


O si lo quieres con variables globales, cosa que en principio se desaconseja:

Código (cpp) [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>
#include <math.h>

int  cifras, primeracifra, suma;

void funcion(int numero);

int main(){
 int i,numero;
 printf ("Introduzca un n%cmero:",163);
 fflush (stdout);
 while((scanf("%d", &numero)==1 && numero!= 0 )!=0){
   if (numero>9){
     funcion (numero);
     if(primeracifra*(cifras+1)==suma) {
       printf("\n%d SI cumple la condicion",numero);
       fflush(stdout);
     }
     else
     printf("\n%d NO cumple la condicion",numero);
   }
   else
     printf("\n%d es menor de 10",numero);
 }
 return 0;
}

void funcion(int numero){
 int i,num=numero;
 cifras=0;suma=0;primeracifra=0;
 for(i=10;numero/i!=0;i=i*10)
     cifras++;
 primeracifra=(int)(numero/ pow(10,cifras));
 for(i=cifras;i>=0;i--){
   (suma)+=(int)(num/ pow(10,i));
   num=(int)fmod(num,pow(10,i)) ;
 }
}


Espero que disfrutes entre tanto código.

¡¡¡¡ Saluditos! ..... !!!!

[leosansan]

leosansan si tienes razon el primer codigo me falto poner varias cosas, pero ahora el codigo que me da problemas es el ultimo que postee, el que tiene algunas cosas modificadas

Es que yo la prueba la hice con el segundo código.

¡¡¡¡ Saluditos! ..... !!!!

[xiruko]

Hola, bueno no me leí todo el hilo pero diría que se trata de problemas con el salto de línea y fgets(). Dejo aquí mi pequeño granito de arena:
........................................
Si no tiene nada que ver con el hilo pido disculpas, pero es que con tanto post la verdad que da un poco de pereza encontrar el problema.

Buena aportación xiruko, da una nueva perspectiva.

Ahora sólo falta que diga que se rechaza por menor o mayor número de cifras.

¡¡¡¡ Saluditos! ..... !!!!

[scanf]

.........................
   scanf ..............
..........................

Con toda humildad y respeto, creo que tomas la tangente usando el scanf cuando de lo que se trataba era de cómo actuar ante la función fgets.

Sinceramente espero una aportación tuya que nos saque de la duda, ya que este tema ha surgido en otro tema y nos tiene con soluciones, que si bien hacen correctamente la labor, hecho de menos una solución más elegante y/o simple por parte tuya. Para que veas la alta estima en que te tengo

¡¡¡¡ Saluditos! ..... !!!!

P.D: Otro hilo que trata el tema es error_fgets.