Is there a way to increase the cyclomatic capacity of an input variable

Question

I'm working on making a tic tac toe game and I have more complex code than my input statement can handle.

This is for a project in my class and I've done some research. nothing to me makes sense there so I came here. I tried to make it less complex but nothing can get around my head to do so. The bellow code is just a small portion of the results.

while while_statement == 3:
  X_variables = input('where do you place your X player1. {use (X , Y)}. ')
  if X_variables == '(1 , 1)':
    game = [[1 , 0 , 0],
      [0 , 0 , 0],
      [0 , 0 , 0]]
    print('|' , 'X' , '|' , ' ' , '|' , ' ' , '|')
    print('-------------')
    print('|' , ' ' , '|' , ' ' , '|' , ' ' , '|')
    print('-------------')
    print('|' , ' ' , '|' , ' ' , '|' , ' ' , '|')
  elif X_variables == '(1 , 2)':
    game = [[0 , 0 , 0],
      [1 , 0 , 0],
      [0 , 0 , 0]]
    print('|' , ' ' , '|' , ' ' , '|' , ' ' , '|')
    print('-------------')
    print('|' , 'X' , '|' , ' ' , '|' , ' ' , '|')
    print('-------------')
    print('|' , ' ' , '|' , ' ' , '|' , ' ' , '|')

I expect this code will continue with the game but the actual results will be an error in the X_variables input statement.

I get it from the input variable. that's what the problem is. it comes from there nothing else, and so if I could increase it, I could continue on with the rest of the results. — Kian Alford, Apr 09 '19 at 11:54
If you want to clean up the code . Create a class for representing the state of the game (the game variable here ) and define `__str__` on that class which returns a string representing printable game view — Sarath Sadasivan Pillai, Apr 09 '19 at 12:01
cyclomatic complexity : It is a quantitative measure of the number of linearly independent paths through a program's source code. Can you explain what exactly do you want to increase here? — Sarath Sadasivan Pillai, Apr 09 '19 at 12:20
as much as the complexity can hold. the error exactly is: "[mccabe] Cyclomatic complexity too high: 20 (threshold 15)". — Kian Alford, Apr 09 '19 at 13:35
that is not in this part mostly, try my answer below it would bring down the complexity. It would make sure complexity is not associated to input range — Sarath Sadasivan Pillai, Apr 09 '19 at 13:40

Sarath Sadasivan Pillai · Accepted Answer · 2019-04-09T12:33:54.617

0

Your code can be cleaned up to following for flexibility in terms of input. The code assumes input to be valid, you may want to handle exceptions later

class Board:
  def __init__(self,count):
    self.count = count
    self.game = [[0 for _ in range(count)] for _ in range(count)]

  def __repr__(self):
    line = "\n"+"- "*self.count+"\n"
    return line.join(["|".join([ 'X' if y == 1 else ' ' for y in x ]) for x in self.game])

  def place(self,x,y):
    self.game[x][y] = 1


.
.
.
board = Board(5)
.
.
.
while while_statement == 3:
  row,col = map(int,tuple(input("Input here:(format row,col)").split(',')))
  board.place(row,col)
  print(board)

edited Apr 09 '19 at 12:33

answered Apr 09 '19 at 12:25

Sarath Sadasivan Pillai

6,737
29
42

what exactly is happening. I don't see what is I tried it. II'm a bit confused on what class board is also. – Kian Alford Apr 09 '19 at 13:46
You can not copy paste it as such. Board represents a game , last 4 lines does what your while loop does Board is the tic-tac-toe board. The point is you don't need to have if statement for every index – Sarath Sadasivan Pillai Apr 09 '19 at 13:49
you can create an instance of Board , say `board = Board(5)` and try `board.place(1,2)` etc and try printing the board – Sarath Sadasivan Pillai Apr 09 '19 at 13:55
1

ok, thank you. How do I say this was answered?? I'm slightly new to this sight. – Kian Alford Apr 09 '19 at 18:01
you can accept my answer as the right answer by clicking the tick mark near the answer on left. Thanks – Sarath Sadasivan Pillai Apr 10 '19 at 07:52

Is there a way to increase the cyclomatic capacity of an input variable

1 Answers1