The single `=` does variable assignment -- use `==` for comparisons. You also can't check two values at once in this way, you would need either `ccnum.length == 15 || ccnum.length == 16` or `[15, 16].include?(ccnum.length)`. The `if` is unneeded since boolean comparisons return `true` or `false` already.