Thread: text comparing

no not yours...in his...and not uninitialised....undefined

letters and digits are in specific places though quzah. not just spaces.

In mine there is a lot of unizialised variables because I forgot to translate a lot of things. My code is in spanish and I tried to translate it into English for you tu understand it better, but it was late and I forgot to translate a lot of things, for example, LETRA = LETTER and salida = out. I will rewrite it again :

Code:

#include <stdio.h>
#include <string.h>
#include <ctype.h>

#define LETTER "A" //patrón cuando encontramos una letra
#define NUMBER "1" //patrón cuando encontramos un número
#define SPACE " " //patrón cuando encontramos un espacio

#define PATTERN1 "1111 AAA"
#define PATTERN2 "AA 1111 AA" 
#define PATTERN3 "AA 11111"


int main(){
	char entry[81], out[10]="";
	strcpy(entrada, "SS 1235 J");

	int size = strnlen(entry, 81);
	if(size > 7 || size< 11){  //size of the spanish plates
		for(int i=0; i<size; i++){
			
			if(isdigit(entry[i])){
				strcat(out,NUMBER);
			
			}else if(isalpha(entry[i])){
				strcat(out,LETTER);
			
				//if the plate is AA 1111 A we make it become AA 1111 AA
				if(i==size-1 && strnlen(out, sizeof(out)) == 9){
					strcat(out,LETTER);
				}
			
			}else{
				// if the plate start with "A " we make it become "AA "
				if(i==1 && strcmp(&out[0],LETTER) == 0){
					strcat(out,LETTER);
				}
				strcat(out,SPACE);
			}
		}
		
		if(strcmp(out,PATTERN1) == 0 || strcmp(out,PATTERN2) == 0 || strcmp(out,PATTERN3) == 0){
			printf("spanish plate \n");
			return 0;
		}
	}
	printf("no spanish plate \n");
	return 1;
		
	
}

In the example you give me, there is a problem that make it complicated. Spanish plates can be :
1111 AAA
A 1111 A
A 1111 AA
AA 1111 A
AA 1111 AA
A 11111
AA 11111

In the example you give me, you will need 7 patterns. I tried to make it so you will only need 3

strcmp(&salida[0],LETRA) == 0
when i == 1 is equivalent to
salida[0] == 'A'

This is not exactly like that, I tried to do that if we find a space in the second position and in the first one there is a letter we add a letter to the second position and then we put a space. We need this if we had a plate that is A 11111, to make it become AA 11111, so we will use only one pattern for this case instead of 2.

Or maybe I didn't understand well, and what you mean is to use " salida[0] == 'A' " instead of "strcmp". I will try with this.


Maybe you think that it is better to use more patterns and make the code be easier.

I am sorry because I think I am causing much troubles.

Ok I think I'm trying to complicate it too much.

I will do test with your examples and I will choose the fastest one.

Maybe I am complicating it too much trying to use less patterns but maybe it is fastest to compare with 7 patterns than doing strange things to use only 3.

Thanks a lot for your help

Maybe I am confusing you more, but my boss just tell me that we are interested, not to see which plates are spanish, but which are not.

So realy what I need to know is which plate is not like the patterns.

Maybe with this information we can make a different programm or I have to use the same.

just pointing out that your previous versin didn't work with the 3 patterens. ie. A3AA 333 would return 1 when it should return 0. And hes trying to make it as fast as possible so I really think just changing its value to a binary number and testing it with if (pattern == 145) for example will far be the quickest option.

sl4nted, I was thinking about using your code, because finaly I have to discard spaces because of some problems whith the OCR.

But your system has a little problem. You define the pattern with bytes, so if the pattern is AA 11111 you enter 1F, but if it si A 11111 you also put 1F. That is not so bad. But if you enter AAA 11111 it will say OK is a good one, but it isn't. I think this will produce some confusions.

Finaly I think I will use a mix of yours and mine.

Code:

int parse2(char *p)
{
        int pos, spc, p_val;
        unsigned int i;

        i = 0, pos = 0, spc = 0;

        for (; *p != '\0'; p++, pos++)
        {
                if (*p == ' ')
                {
                        if ((pos == 2) && i == 0)
                        {
                                spc++;
                                continue;
                        }
                        else if (((pos == 4) || pos == 7) && i == 15)
                        {       spc++;
                                continue;
                        }
                        else return 0;
                }

                i <<= 1;

                if (((p_val = *p - '0') < 10) && p_val >= 0)
                        i += 1;
                else if (((p_val = *p - 'a') < 26) && p_val >= 0)
                        continue;
                else if (((p_val = *p - 'A') < 26) && p_val >= 0)
                        continue;
                else return 0;
        }

        switch (i)
        {
                case 31:
                case 120: if (spc == 1) return 1;
                case 60: if (spc == 2) return 1;
        }

        return 0;
}

works for only the 3 patterns you have described...if you want to add more, you'll have to add a case entry and a line to test for the space positions. if yu want it fast it has to be specific

Sorry my english is very bad.

I can't use the spaces because I tried with many examples and in one of them, the OCR put an example when there isn't. So it sais that it isn't a Spanish plate, but it is. So I discard spaces.

I read from the OCR, get the plate, erase the spaces and compare it with the patterns. That's wy I thought about using your the first code you wrote. But, making the 7 patterns I can have I realised that for differents plates I have the same pattern. So I realise that it won't work properly in some cases.

But I'm realy grateful for your help.

I coule care less what you use. milliseconds and microseconds are nothing. Unless your test data is huge, you'll see no differnce. If this is going to be speed tested against the rest of your class or somthing, you shoud use my method. But if you just want it not to lag when processing, you don't need to worry about that. you'll notice no differnce. quzahs is better for ease updating etc.