Punteros en C

[markmb]

Hola a todos, estoy empezando en C (luego me meteré en C++) y estoy practicando con unos ejercicios que he encontrado por ahí. Ahora mismo tengo problemas con punteros:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
int main(void){
    char entrada[100];
    char *ult=entrada;
    char cantidadstr[10];
    int i,j,k = 0;
    int res;
 
    scanf ("%s",entrada);
    
    printf ("Hola %s",ult);
    while (*ult != "\0"){
        if(*ult == "\n"){
            if(i != 0){
                printf("\n");
            }
            i++;
            j = 0;
        }
        else if(i != 0){
            if(*ult == " "){
                j++;
                k=0;
                res = atoi(cantidadstr);
                printf("%d ",i);
                cantidadstr[10] = '\0';             
            }
            else if(j != 0){
                cantidadstr[k] = *ult;
            }
            
        }
        k++;
        *ult++;
    }
    return 0;
}
 

El puntero ult da problemas. Al compilar me salen warnings, sobre comparar punteros con int, pero el problema gordo, es que al ejecutar e insertar la cadena, me salta "Violación de segmento".

Por lo que he visto después de toda la mañana intentando arreglarlo, el error se sitúa en el printf primero, al llamar a ult.

Gracias de antemano!

[Gott]

Iniciado por markmb

Hola a todos, estoy empezando en C (luego me meteré en C++) y estoy practicando con unos ejercicios que he encontrado por ahí. Ahora mismo tengo problemas con punteros:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
int main(void){
    char entrada[100];
    char *ult=entrada;
    char cantidadstr[10];
    int i,j,k = 0;
    int res;
 
    scanf ("%s",entrada);
    
    printf ("Hola %s",ult);
    while (*ult != "\0"){
        if(*ult == "\n"){
            if(i != 0){
                printf("\n");
            }
            i++;
            j = 0;
        }
        else if(i != 0){
            if(*ult == " "){
                j++;
                k=0;
                res = atoi(cantidadstr);
                printf("%d ",i);
                cantidadstr[10] = '\0';             
            }
            else if(j != 0){
                cantidadstr[k] = *ult;
            }
            
        }
        k++;
        *ult++;
    }
    return 0;
}
 

El puntero ult da problemas. Al compilar me salen warnings, sobre comparar punteros con int, pero el problema gordo, es que al ejecutar e insertar la cadena, me salta "Violación de segmento".

Por lo que he visto después de toda la mañana intentando arreglarlo, el error se sitúa en el printf primero, al llamar a ult.

Gracias de antemano!

Es mejor si dices cual es el ejercicio que intentas hacer, porque para usar punteros asi como veo que intentas, que es cargar el texto por parte del usuario, tienes que hacerlo usando memoria dinamica, sino no podras!
Además en el while que hace por letra lo que tienes que preguntar es si *ult["algo aqui"] es != NULL..entiendes??
Trata de poner el ejercicio y lo vamos haciendo juntos ok??

[CalgaryCorpus]

Los arreglos definidos asi

tipo nombre_arreglo[cantidad];

pueden usar sin problemas indices desde 0 hasta (cantidad-1)
Usar un indice con valor "cantidad" o mayor para asignar es una receta para el desastre.

En tu caso k podria tener valor mayor que 10 y tu arreglo tiene 10 posiciones y veo una asignacion a la posicion 10, que he mencionado como caso conflictivo.

En resumen: Cuida los indices de los arreglos.

[markmb]

CalgaryCorpus: Con eso intentaba vaciar el arreglo....

Gott: El ejercicio te pide que dupliques los números que te dan con esta estructura:

Código:

4
5 0 1 2 3 4
0
3 -10 0 10
1 424248

La primera línea dice cuantas lineas habrá, y el primer número de cada línea dice cuántos números habrá en esa línea. El número de líneas llega hasta 1000, el número de números en la línea llega hasta 100, y no hay números mayores que 10⁹

Ahora al volver a repasar el enunciado, he visto un par de cosas:
- El tamaño del arreglo que almacena los datos, debería ser más grande, para cubrir todas las posibilidades
- Al final, cuando ya se muestran los resultados, me he equivocado de variable, y no la he multiplicado por dos (eso no es un error fatal, me hubiese dado cuenta )

A ver si me podéis ayudar, que ya me está entrando dolor de cabeza con el problemilla este

[markmb]

Después de realizar los cambios que mencionaba, llego a esto:

Código C:

#include <stdio.h>
#include <stdlib.h>
 
int main(void){
    char entrada[1001*11*101];
    /*Asi tenemos el tamano maximo:
    1001 por las 1000 posibles lineas, mas la primera
    11 por el tamano maximo del numero (1 + 9 ceros), mas el espacio o salto de linea siguiente
    101 por el numero de numeros por linea, mas el primero
    */
    char *ult=entrada;
    char cantidadstr[10];
    int i,j,k = 0;
    int res;
 
    scanf ("%s",entrada);
    printf ("\n%s",entrada);
    
    //poniendo ese print ahi arriba, ese me lo muestra, por tanto, el fallo esta en el puntero de debajo de esta linea
    printf ("Hola %s",ult);
    while (*ult != "\0"){
        if(*ult == "\n"){
            if(i != 0){
                printf("\n");
            }
            i++;
            j = 0;
        }
        else if(i != 0){
            if(*ult == " "){
                j++;
                k=0;
                res = atoi(cantidadstr);
                printf("%d ",res*2);
                //Este es el otro cambio que hablaba
                cantidadstr[10] = '\0';             
            }
            else if(j != 0){
                cantidadstr[k] = *ult;
            }
            
        }
        k++;
        *ult++;
    }
    return 0;
}
 

He marcado el lugar donde deja de funcionar. Me imagino que me hace falta entender un poco mejor los punteros, pero si no es con ejemplos y a golpes, no lo consigo...

[CalgaryCorpus]

Sugiero que cambies esta linea:

Código C++:

cantidadstr[k] = *ult;
 

por estas :

Código C++:

if( k >= 10 ) {
      printf( "Problemas en indice k, "
              "no puede ser mayor o igual que 10 para que este programa funcione bien\n" );
  }
  else {
     cantidadstr[k] = *ult;
}
 

Una linea previa cantidadstr[10] = '\0'; nunca estará bien, pues el arreglo cantidadstr no tiene 11 posiciones.


Ademas de modificar tu codigo para que la variable "k" nunca tome un valor mayor que (tamaño del arreglo - 1), la comparacion con caracteres se hace usando comillas simples no dobles.

por ejemplo, comparacion de igualdad con caracteres:
variable == 'a'

comparacion para diferente
variable != 'a'

la comilla doble es SIEMPRE para strings literales, nunca es para caracteres literales.

[estebanf93]

todo vector se cuenta desde el elemento 0, osea el ultimo elemento de tu vector es de indice 9

[markmb]

Si, tenéis razón, ese era un error, pero no era el error fatal. El error fatal, que me ha costado encontrar, es problema de comillas. Como vengo de PHP y Python, donde casi no hay diferencia entre comillas simples y dobles, pues ese es el problema.

Gracias igualmente!!