Foro de elhacker.net

Hola a todos mi nombre es Laura y soy nueva por aquí, espero que todos estén bien. Mi duda es la siguiente yo quiero crear una matriz con ciertos elementos. Luego de llenarla quiero poder un elemento de mi matriz posición por posición según lo que ingrese el usuario, sea norte, sur, este, oeste, para así ir acumulando un puntaje. Yo cree el siguiente código pero no me funciona y no se que estoy haciendo mal. Disculpen la molestias y entiendan que soy principiante por lo que debo de tener los errores mas estúpidos, solo les muestro el norte porque mi código esta hecho un desastre y es muy largo, y como siempre se repite lo mismo quiero saber que tengo mal y así poder arreglarlo en los demás. Gracias de antemano, y sean un poco pacientes todavía soy principiante.

#include <iostream>
#include <string>
char matriz[20][20];
using namespace std;
//----------------------------------------------------------------------
float Puntaje(int x, int y){
	float Pts;
	if (matriz[x][y]=='$'){
		Pts=1.0;				
	}else if (matriz[x][y]=='%'){
		Pts=1.5;
	}else if (matriz[x][y]=='&'){
		Pts=2.0;
	}
	return Pts;
}
//----------------------------------------------------------------------
void Mover_posicion( int fil, int col, char matriz[][20]){
	bool estado=false;
	int i,j,c,pasos,x,y,l,m,Ac;
	string mov;
	do{
	for( i=0;i<fil;i++){
		for( j=0;j<col;j++){
			if(matriz[i][j]=='I'){
				x=i;
				y=j;
			}
			if(matriz[i][j]=='F'){
				l=i;
				m=j;
			}
		}
    }
    cin>>mov>>pasos;
 
    for(c=0;c<pasos;c++){
		if(mov=="N"){
 
    if((x-pasos) < 0){
	 Ac=Ac + Puntaje(x,y);
    if(matriz[x][y]==matriz[l][m]){
 	  for (i = 0; i < fil; i++){
        for ( j = 0; j < col; j++)
           cout << matriz[i][j] << " ";
        cout << endl;  
    }
	}else{  
         matriz[x][y] = matriz[x][y]; 
            matriz[x][y] = '#';
 
             if(matriz[i][j]=='I'  && matriz[x][y]!='|'  ){
 
	for (x = 0; x < fil; x++){
        for ( y = 0; y < col; y++)
           cout << matriz[x][y] << " ";
           cout << endl;
 
         } 
         }else if(matriz[x--][y]!='|'){
 
	for (i = 0; i < fil; i++)
    {
        for ( j = 0; j < col; j++)
           cout << matriz[i][j] << " ";
        cout << endl;  
    }
    }
} 
}
 x--;
}  
}
if(matriz[x][y]==matriz[l][m]){
	estado=true;
}
 
 
    }while(estado!=true);
    cout<<"Ha llegado"<<"su acumulado es:"<<Ac<<endl;
}
//----------------------------------------------------------------------
	int main(){
int fil,col,i,j; 
 
cout<<"Ingrese filas y columnas"<<endl;
cin>>fil>>col;
 
for( i=0;i<fil;i++){
for( j=0;j<col;j++){          
do{
        cin >> matriz[i][j];
 
 }while(matriz[i][j]!='I' && matriz[i][j]!='F' && matriz[i][j]!='$' && matriz[i][j]!='&' && matriz[i][j]!='|' && matriz[i][j]!='%');
}
}    
Mover_posicion( fil, col, matriz);
}

Podrias explicar de forma mas clara que es lo que quieres hacer exactamente; la función Mover_posicion carece totalmente de consistencia que intentas hacer con ella

Saludos 

lo siento si no se entiende, estoy comenzando. Lo que yo intente hacer fue que según la coordenada que ingrese es el usuario (norte, sur, este, oeste) se mueva el carácter
I tantas veces como lo indico el usuario. Ej si el usuario ingresa N 2, el carácter debería moverse nos espacios al norte y en su camino ir colocando el #. Yo trate de colocar el x-- para que fuera de posición en posición y así poder ir acumulando los puntos que hace la acción puntaje. Cambien le coloque un do while para que se saliera cuando la posición de I fuera igual  a F, se saliera y me escribiera el mensaje de ha llegado y de cuanto es mi acumulado. Lo siento si mi código es un desastre solo que parece que las matrices y yo no nos llevamos. No se, si estoy responde a tu pregunta

http://foro.elhacker.net/programacion_cc/ayuda_con_programa_urgente-t404470.0.html
Voy a echar un ojo y después pongo algo que valga, aquí te dejo este enlace para que veas unos ejemplos muy buenos, porque así con prisa me ha salido to mal.  :-(

Te dejo un sencillo codigo que hace exactamente lo que pides, que es mover un elemento de la matriz hacia arriba la cantidad de posiciones que el usuario indique y rellenar el hueco dejado con '#'

if (mov == 'N' && x-pasos >= 0){
   matriz[x-pasos][y] = matriz[x][y];
  for (i = 0;  i < pasos; i++ )
      matriz[x-i][y] = '#';
   }

Que quieres hacer con estos caracteres I F  $  &  | % y con la condición de do while no entiendo a que quieres llegar

Saludos

Bueno con el do while del int main lo que quiero es que el usuario llene la matriz solo con los caracteres  I F  $  &  | % . Y el do while de la acción mover posición lo que hace es que cuanto la posición donde esta el carácter I es igual a F mi variable estado se vuelve verdad por lo que se sale y me imprime el mensaje ha llegado con el acumulado de los puntos de las posiciones por donde paso el carácter I. No se, si eso responde a tu pregunta, pero eso es lo que intento hacer y gracias por tratar de ayudarme a todos.

Posición del carácter I igual a F ????

¿¿¿???

Perdón por el comentario, pero te explicas como un libro cerrado

si, lo siento de verdad trata de explicar lo mejor que puedo pero las palabras no salen como debería. Lo que trato de decir es que a medida que mi carácter va subiendo y en el numero de pasos que indico llega al caracter F, es decir,  la posición  del carácter I pasa a ser la misma que el F. El se sale y no pide mas paso, para eso es el do while. Espero que me entienda, ya se que no debo ser profesora explico pésimo.

A ver si me entero.

* Tienes una matriz con valores determinados, entre ellos al menos una I y una F.

* Se elige una I y se le van pidiendo al usuario una serie de pasos. Cada paso desplaza I por la matriz.

* Si en uno de los pasos resulta que I va a estar ubicada en la posición donde se encuentra una F el programa deja de pedir más pasos

¿Es así?

Si estoy en lo cierto, tengo algunas dudas:

* Solo hay una I y una F o puede haber varias de cada una??

* Si se desplaza I por la matriz... que valor se pone en el espacio que queda vacío? Si I está en (1,0) y se mueve a (2,0), qué valor se pone en (1,0)??

Si no estoy en lo cierto, deberías replantearte un pequeño esfuerzo adicional para que se entienda el problema.

Un saludo.

 En cuanto a tus preguntas:
*El usuario solo va a ingresar una I y una F.
*Cuando se mueve se le coloca a la posición anterior el carácter #.
Y de verdad discúlpenme si no se entiende lo que trato de explicar.

Varias cosas:

* Tabula el código... es imposible leer un código que no está correctamente tabulado.

* Pon nombres con sentido a las variables. 'fil' y 'col' dan a entender que son índices para la fila y la columna... cuando realmente están indicando el total de filas y columnas... nuevamente lo que prima aquí es que el código sea lo más legible posible para evitar problemas.

* No uses variables globales, no sucede en este caso concreto, pero dan problemas. Es mejor pasar la matriz como parámetro.

* Cúrrate un poquito la interfaz de usuario. El sistema para rellenar la matriz es espantoso... si pones un carácter no válido no te avisa, no sabes qué está pasando... poner mensajes de error ante entradas incorrectas te ayuda a ver también que el programa hace lo que debe... si no pones mensajes no sabes si se está tragando información incorrecta.

Respecto a este punto... cómo sabes cuándo has dejado de rellenar la matriz y el programa te está pidiendo los movimientos?? sencillamente, con tu código, es imposible saberlo.

Queda más bonito algo del tipo:

const int FILAS = 20;
const int COLS = 20;
 
// ...
 
int main( )
{
  int num_filas, num_columnas;
 
  do
  {
    cout << "Numero de filas: ";
    cin >> num_filas;
    if ( num_filas < 1 || num_filas > FILAS )
      cout << "Valor incorrecto." << endl;
  } while ( num_filas < 1 || num_filas > FILAS );
 
  do
  {
    cout << "Numero de columnas: ";
    cin >> num_columnas;
    if ( num_columnas < 1 || num_columnas > COLS )
      cout << "Valor incorrecto." << endl;
  } while ( num_columnas < 1 || num_columnas > COLS );
 
  // ...
}

Es más código pero si algo funciona mal puedes ver qué es simplemente "jugando" con tu aplicación.

* Procura reducir el ámbito de las variables al mínimo posible:

int i,j,c,pasos,x,y,l,m,Ac;

Tienes esta línea al inicio de 'Mover_posicion'. No es necesario... si vas a usar un índice en un for puedes declarar la variable dentro del for:

for ( int i=0; i < 20; i++ )
{
}

Reutilizar variables dentro de una misma función es algo que hay que realizar con sumo cuidado para evitar meteduras de pata.

* Reutiliza código. Usas en varios puntos, por ejemplo, el código que imprime la matriz en pantalla. Saca ese código a una función y evita duplicarlo... por un lado, si ese código tiene errores, solo tendrás que corregirlos en un sitio, por otro, todas las invocaciones funcionarán igual.

void PintaMatriz( char matriz[][COLS], int filas, int columnas )
{ 
  for ( int i = 0; i < filas; i++)
  {
    for ( int j = 0; j < columnas ; j++)
      cout << matriz[i][j] << " ";
    cout << endl;
  }
}

* Cuidado con las asignaciones. La siguiente instrucción de tu programa no hace nada. ¿Metedura de pata o despiste?

matriz[x][y] = matriz[x][y];

* Controla el rango de valores de las variables:

  cout<<"Ingrese filas y columnas"<<endl;
  cin>>fil>>col;

¿Qué sucede si pongo 1000 y 244? Que el programa fallará al intentar escribir fuera de la matriz de 20x20 que has definido.

* Inicializa las variables:

  int c,pasos,x,y,l,m,Ac;
 
  // ...
  Ac=Ac + Puntaje(x,y); // Que valor tiene Ac aqui???

float Puntaje(int x, int y)
{
  float Pts;
  if (matriz[x][y]=='$')
    Pts=1.0;
  else if (matriz[x][y]=='%')
    Pts=1.5;
  else if (matriz[x][y]=='&')
    Pts=2.0;
 
  return Pts; // Cuantos puntos vale '|' ??
}

* No tienes ningún chequeo que impida introducir varios 'I' o 'F' así como tampoco compruebas que, obligatoriamente, se haya introducido una 'I' y una 'F' en la matriz.

* No compruebas que con 'pasos' te puedas salir de la matriz.

Tienes bastantes fallos en la aplicación. Te he indicado un poco lo que he ido viendo.

Dale un repaso al código, corrige las cosillas y seguimos revisando.