Skip to main content
Genial o SeuGundo problema! Acho que faltou formatar aquele bloco de código, a leitura tava bem complicada como texto.
Fonte Link
brasofilo
  • 6,7mil
  • 5
  • 30
  • 50

Têm dois problemas no seu código.

O primeiro problema é que quando quer comparar duas variáveis tem de usar o ==, ou ===. O que você usa é uma atribuição =, e isso não compara nada. Usando somente = vai fazer a variável à esquerda ficar com o valor que está na direita. Na verdade quando usa o = está a re-escrever a função .match(). Ou seja, o .match() passa a ter o valor que lhe atribuío e deixa de funcionar...

O seugundosegundo problema é a maneira como usar o .match(). Eu presumo que você quer usar o if no caso da str ter um certo conjunto de caracteres. Aí só precisa usar algo como str.match(/[1-2]/). Isto só por sí já retorna o que você precisa para usar no if. Por exemplo:

var str = '1233'; str.match(/[1-2]/); // dá verdadeiro no if, mais exatamente ["1"] str.match(/[1-2]/g); // dá verdadeiro, mais exatamente ["1", "2"] pois usa o "g" no match que retorna não só o primeiro encontrado

var str = '1233';
str.match(/[1-2]/);   // dá verdadeiro no if, mais exatamente ["1"]
str.match(/[1-2]/g);  // dá verdadeiro, mais exatamente ["1", "2"] pois usa o "g" no match que retorna não só o primeiro encontrado

Assim o que você deve usar é somente:

if (str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {

Uma vez que quer atribuír novo valor à variável old, e lembrando que uma atribuíção com = retorna um valor se não usar o var então o seu código poderia ser:

function verifica(str) {
    var old;
    if (old = str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {
        result = 1;
    } else if (old = str.match(/\d\d\d\d\d\.\d\d\d/)) {
        result = 2;
    }
    return result; // isto acrescentei eu, para a função retornar um valor
}

Têm dois problemas no seu código.

O primeiro problema é que quando quer comparar duas variáveis tem de usar o ==, ou ===. O que você usa é uma atribuição =, e isso não compara nada. Usando somente = vai fazer a variável à esquerda ficar com o valor que está na direita. Na verdade quando usa o = está a re-escrever a função .match(). Ou seja, o .match() passa a ter o valor que lhe atribuío e deixa de funcionar...

O seugundo problema é a maneira como usar o .match(). Eu presumo que você quer usar o if no caso da str ter um certo conjunto de caracteres. Aí só precisa usar algo como str.match(/[1-2]/). Isto só por sí já retorna o que você precisa para usar no if. Por exemplo:

var str = '1233'; str.match(/[1-2]/); // dá verdadeiro no if, mais exatamente ["1"] str.match(/[1-2]/g); // dá verdadeiro, mais exatamente ["1", "2"] pois usa o "g" no match que retorna não só o primeiro encontrado

Assim o que você deve usar é somente:

if (str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {

Uma vez que quer atribuír novo valor à variável old, e lembrando que uma atribuíção com = retorna um valor se não usar o var então o seu código poderia ser:

function verifica(str) {
    var old;
    if (old = str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {
        result = 1;
    } else if (old = str.match(/\d\d\d\d\d\.\d\d\d/)) {
        result = 2;
    }
    return result; // isto acrescentei eu, para a função retornar um valor
}

Têm dois problemas no seu código.

O primeiro problema é que quando quer comparar duas variáveis tem de usar o ==, ou ===. O que você usa é uma atribuição =, e isso não compara nada. Usando somente = vai fazer a variável à esquerda ficar com o valor que está na direita. Na verdade quando usa o = está a re-escrever a função .match(). Ou seja, o .match() passa a ter o valor que lhe atribuío e deixa de funcionar...

O segundo problema é a maneira como usar o .match(). Eu presumo que você quer usar o if no caso da str ter um certo conjunto de caracteres. Aí só precisa usar algo como str.match(/[1-2]/). Isto só por sí já retorna o que você precisa para usar no if. Por exemplo:

var str = '1233';
str.match(/[1-2]/);   // dá verdadeiro no if, mais exatamente ["1"]
str.match(/[1-2]/g);  // dá verdadeiro, mais exatamente ["1", "2"] pois usa o "g" no match que retorna não só o primeiro encontrado

Assim o que você deve usar é somente:

if (str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {

Uma vez que quer atribuír novo valor à variável old, e lembrando que uma atribuíção com = retorna um valor se não usar o var então o seu código poderia ser:

function verifica(str) {
    var old;
    if (old = str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {
        result = 1;
    } else if (old = str.match(/\d\d\d\d\d\.\d\d\d/)) {
        result = 2;
    }
    return result; // isto acrescentei eu, para a função retornar um valor
}
Fonte Link
Sergio
  • 135,1mil
  • 16
  • 201
  • 294

Têm dois problemas no seu código.

O primeiro problema é que quando quer comparar duas variáveis tem de usar o ==, ou ===. O que você usa é uma atribuição =, e isso não compara nada. Usando somente = vai fazer a variável à esquerda ficar com o valor que está na direita. Na verdade quando usa o = está a re-escrever a função .match(). Ou seja, o .match() passa a ter o valor que lhe atribuío e deixa de funcionar...

O seugundo problema é a maneira como usar o .match(). Eu presumo que você quer usar o if no caso da str ter um certo conjunto de caracteres. Aí só precisa usar algo como str.match(/[1-2]/). Isto só por sí já retorna o que você precisa para usar no if. Por exemplo:

var str = '1233'; str.match(/[1-2]/); // dá verdadeiro no if, mais exatamente ["1"] str.match(/[1-2]/g); // dá verdadeiro, mais exatamente ["1", "2"] pois usa o "g" no match que retorna não só o primeiro encontrado

Assim o que você deve usar é somente:

if (str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {

Uma vez que quer atribuír novo valor à variável old, e lembrando que uma atribuíção com = retorna um valor se não usar o var então o seu código poderia ser:

function verifica(str) {
    var old;
    if (old = str.match(/\d\d\d\d\d\.\d\d\d\d\d/)) {
        result = 1;
    } else if (old = str.match(/\d\d\d\d\d\.\d\d\d/)) {
        result = 2;
    }
    return result; // isto acrescentei eu, para a função retornar um valor
}