Tengo un problema al momento de programar para que sólo acepte 8 dígitos en un código de barras, tengo entendido que es con un strlen(.. pero me marca un error.
Lo que tiene que hacer este programa es que ingrese un código de barras y lo guarde pero sólo tiene que aceptar 8 dígitos, ni más ni menos, y tampoco letras.
Tengo este código pero me marca error:
if (strlen(Datos[c].CodigoBarras)=8) // Aquí necesito ayuda
{
cout<<"Correcto.";
}
else
{
cout<<"Incorrecto. Sólo se permiten 8 dígitos.";
}
Espero me puedan ayudar.
Cita de: FMR en 20 Octubre 2012, 22:31 PM
Tengo un problema al momento de programar para que sólo acepte 8 dígitos en un código de barras, tengo entendido que es con un strlen(.. pero me marca un error.
Lo que tiene que hacer este programa es que ingrese un código de barras y lo guarde pero sólo tiene que aceptar 8 dígitos, ni más ni menos, y tampoco letras.
Tengo este código pero me marca error:
Espero me puedan ayudar.
No sé C ni C++ , pero viendo la sintaxis asumo que esta es la solución.
if (strlen(Datos[c].CodigoBarras)==8) /* no es "=" es "==" ya que es comparacion y ahi lo estabas asignando*/
{
cout<<"Correcto.";
}
else
{
cout<<"Incorrecto. Sólo se permiten 8 dígitos.";
}Puedo estar equivocado, porque algunas sintaxis son diferentes en otros lenguajes, avisame si estoy mal, saludos.
Cita de: DonVidela en 20 Octubre 2012, 22:41 PM
No sé C ni C++ , pero viendo la sintaxis asumo que esta es la solución.
if (strlen(Datos[c].CodigoBarras)==8) /* no es "=" es "==" ya que es comparacion y ahi lo estabas asignando*/
{
cout<<"Correcto.";
}
else
{
cout<<"Incorrecto. Sólo se permiten 8 dígitos.";
}
Puedo estar equivocado, porque algunas sintaxis son diferentes en otros lenguajes, avisame si estoy mal, saludos.
Agregué eso que me dijiste y me sigue apareciendo equivocado, el error me lo marca en la palabra "Datos" y dice: "un argumento de tipo 'int' no es compatible con un parámetro de tipo 'const char'".
El operador de comparación es tal y como dice DonVidela , si no estarias asignando , por otro lado , el array Datos a que apunta ¿a una estructura? Pon el código entero porque si nó ...
Como dice avesudra, pon el codigo completo para que podamos ayudarte de manera más eficiente.
El código es una estructura pero lleva muchas más cosas como agregar registro, editar, borrar, etc, y el problema me aparece nadamás en esa parte, por lo que omití lo demás que no influye en el código.
include <iostream>
int c=0;
struct dato {
int CodigoBarras;
} Datos[10];
void main (){
cout<<"\t\tIngresa el Codigo de Barras\n";
flushall();
cin>>Datos[c-1].CodigoBarras;
if (strlen(Datos[c].CodigoBarras)==8) // Aquí necesito ayuda
{
cout<<"Correcto.";
}
else
{
cout<<"Incorrecto. Sólo se permiten 8 dígitos.";
}
}
Bueno aca no te puedo ayudar, puesto que no se que es Datos, para mi sería un arreglo que trabajo con C# , JAVA, PHP.
CitarAgregué eso que me dijiste y me sigue apareciendo equivocado, el error me lo marca en la palabra "Datos" y dice: "un argumento de tipo 'int' no es compatible con un parámetro de tipo 'const char'".
Si te fijas tienes Datos[10] y alega que el 10 que pusiste es un argumento de tipo entero, que no es compatible con una constante char, eso quiere decir que eso que tienes ahí (Porque no se si es un arreglo de objetos o no se en este lenguaje) espera algo como Datos['10'] , ahora como te dije antes, no conozco el lenguaje no se que será eso.
Cualquiera me puede corregir acá y te puede aportar más, pero espero darte algunas pistas.
Saludos
Citarif (strlen(Datos[c].CodigoBarras)==8) // Aquí necesito ayuda
Datos[c].CodigoBarras es un numero entero, y tu lo estas pasando como argumento a una funcion que espera un puntero a caracter. no se si lo sabras, pero
strlen() devuelve la longitud de la cadena pasada como parametro, por lo que pasarle un entero como parametro no tiene mucho sentido.
si lo que quieres hacer es tratar a un codigo de barras como un array de enteros, entonces tienes que hacerte alguna funcion que te diga el total de digitos que tiene el numero. ademas seria mas facil declarar el codigo de barras asi:
int CodigoBarras[10];si quieres trabajar el codigo de barras como una cadena para usar
strlen(), entonces debes declararlo como tal:
char CodigoBarras[10];espero que te ayude, un saludo!
Lo que hice fue cambiar la variable "CodigoBarras" de int a char y me funcionó ya que no contaba con que strlen() no funciona con enteros.
Ya con eso sí me acepta que sean ocho dígitos, ni más ni menos, pero acepta puros números y puras letras o mixto, y ese es otro problema que olvidé mencionar, el código de barras sólo tiene que aceptar números y no letras.
Espero que me haya explicado, y perdonen tantas preguntas mías, soy algo nuevo en esto del c++.
Cita de: FMR en 20 Octubre 2012, 22:31 PM
Tengo un problema al momento de programar para que sólo acepte 8 dígitos en un código de barras, tengo entendido que es con un strlen(.. pero me marca un error.
Lo que tiene que hacer este programa es que ingrese un código de barras y lo guarde pero sólo tiene que aceptar 8 dígitos, ni más ni menos, y tampoco letras.
Tengo este código pero me marca error:
Estas usando una función para char propiamente, y tu variable "Datos[c].CodigoBarras" es un numero entero. Lo que podrías hacer es calcular previamente los dígitos de dicho número y luego aplicar la condición a la variable contador si es 8 o no..
Para contar los dígitos de "VDatos[c].CodigoBarras" te valdría el siguiente código, cambiando "n" por "Datos[c].CodigoBarras":#include<iostream>
using namespace std;
int main() {
int n,contador=0;
cin>>n;
while(n>0){
contador++;
n=n/10;}
cout<<contador<<endl;
return 0;
}
Cita de: leosansan en 21 Octubre 2012, 13:42 PM
Estas usando una función para char propiamente, y tu variable "Datos[c].CodigoBarras" es un numero entero. Lo que podrías hacer es calcular previamente los dígitos de dicho número y luego aplicar la condición a la variable contador si es 8 o no..
Para contar los dígitos de "VDatos[c].CodigoBarras" te valdría el siguiente código, cambiando "n" por "Datos[c].CodigoBarras":
#include<iostream>
using namespace std;
int main() {
int n,contador=0;
cin>>n;
while(n>0){
contador++;
n=n/10;}
cout<<contador<<endl;
return 0;
}
Muchas gracias leosansan, me funcionó perfecto ya que si convertía el int a char me traería problemas con otras cosas en mi código y sin convertirlo de int a otro tipo me pareció perfecto, sólo una pregunta, ¿para qué se usa el n=n/10;? Todo lo entendí pero eso no muy bien, necesito comprenderlo bien ya que soy algo nuevo en el c++.
es muy simple... de hecho no hay que convertir a nada solo aplicar lógica PARA ESTE CASO QUE ya tiene establecido una longitud de 8...
if (abs(Datos[c].CodigoBarras) <= 99999999) {
puts("Existen 8 o menos dígitos");
} else {
puts("Hay mas de 8 dígitos");
}
Dulces Lunas!¡.
Cita de: FMR en 23 Octubre 2012, 03:38 AM
Muchas gracias leosansan, me funcionó perfecto ya que si convertía el int a char me traería problemas con otras cosas en mi código y sin convertirlo de int a otro tipo me pareció perfecto, sólo una pregunta, ¿para qué se usa el n=n/10;? Todo lo entendí pero eso no muy bien, necesito comprenderlo bien ya que soy algo nuevo en el c++.
Cada vez que haces n/10 divides entre 10 de forma entera, con lo que te "comes" un dígito del número inicial, al tiempo que "contador" se incrementa contando así los dígitos.
Saludos!.
Si gracias a todos por sus respuestas, ya solucioné ese problema, sólo que tengo otro, aparte de aceptar sólo 8 dígitos, necesito que sólo sean números y no letras, ¿saben como le puedo hacer con eso?.
Citarhttp://www.cplusplus.com/reference/clibrary/cctype/isdigit/
Cita de: leosansan en 2 Noviembre 2012, 13:15 PM
http://www.cplusplus.com/reference/clibrary/cctype/isdigit/
Eso o hacerte una funcioncica con un array (porque creo que no hay el tipo conjunto predefinido en c++). Así no tendrías que incluir otra librería :P
Cita de: FMR en 2 Noviembre 2012, 06:23 AM
ya solucioné ese problema, sólo que tengo otro, aparte de aceptar sólo 8 dígitos, necesito que sólo sean números y no letras, ¿saben como le puedo hacer con eso?.
CitarUna opción sería tomar N como cadena y convertirla a entero, para después contar el número de dígitos:
#include<iostream>
#include<cstdlib>
using namespace std;
int main() {
char N[20];
int n,contador=0;
while (contador!=8)
{
cout<<"Introduzca 8 digitos: ";
cin>> N;
n = atoi (N);
cout<<n<<endl;
contador=0;
while(n>0){
contador++;
n=n/10;
}
cout<<contador<<endl;
}
return 0;
}
char cadena[9];
int i;
fgets(cadena, 8, stdin);
for (i=0; i<8; i++) {
if (cadena[i] < '0' || cadena[i] > '9') {
printf("ERROR: Ingresa solo numeros!\n");
break;
}
}
con fgets() el usuario podria introducir mas digitos pero solo se aceptarian 8, y luego con el for irias comprobando que cada uno de ellos fuera un numero. si no lo es, te salta el error y luego sale del for. tendrias que implementar algo asi dentro de un while con su respectiva variable de control para repetir este proceso hasta que el usuario introdujese un numero valido.
un saludo!
Cita de: FMR en 2 Noviembre 2012, 06:23 AMya solucioné ese problema, sólo que tengo otro, aparte de aceptar sólo 8 dígitos, necesito que sólo sean números y no letras, ¿saben como le puedo hacer con eso?.
Solo tienes que seguir la recomendación de
BlackZeroX (Astaroth): leer el numero y verificar su valor (99,000,000 <= x <= 99,999,999).
Un saludo