code is not working , can anybody help?

[deanhystad]

tkinter is very forgiving of programmer stupidity. It is amazing how wrong your code can be and it will draw pretty windows. Believe me, I have been stupid plenty of times. Six times already while working on this post.

There are three reasons your button command does not work. Button expects command to be a function and you are passing a string. The second problem is that playermove(kare1,0,0) is not a function. It is a function call, and the button command will be bound to the return value of the function. The third problem is that 'kare1' is not yet defined when this code is executed.

The most common way to bind a button command to a function with parameters is to use a lambda expression.

command=lambda b=kare1:playermove(b, 0, 0)

You may say "But that is a function call! I can tell because of the parenthesis!" You are correct, there is a function call, but we are not binding command to the function call, we are binding it to the lambda expression. When the button is pressed the code calls the lambda expression, and the lambda expression calls the function. I know that may be a bit fuzzy right now, but you need to learn how these lambda expressions work because you will use them a lot if you write GUI applications.

If you want to pass your button as an argument to a function, you need to first define the button. This means you need to split creating the button and binding the button.

kare1 = tkinter.Button(text=' ', fg='yellow', bg='red')
kare1.configure('command=lambda b=kare1: playermove(b, 0, 0))

Your button should now call playermove() with the appropriate arguments. Now you need to fix playermove().

def playermove(buton,x1,y1):
    if play == 1:
        play = 2
        buton = tkinter.Label(text='            X\n            ',command='',fg='white',bg='black')
        buton.place(x=x1,y=y1)
    elif play == 2:
        play = 1
        button = tkinter.Label(text='            O\n            ',command='',fg='white',bg='black')
        buton.place(x=x1,y=y1)

You are passing a button to the function, but you don't use it. Instead you make a label and place it over the button. What you should do is change the buton text.

def playermove(buton):
    global play
    marker = 'X' if play == 1 else 'O'
    buton['text'] = marker
    play = 1 if play == 2 else 2

Since you are not making a label you don't need to pass the button coordinates. This simplifies the function and the function binding for the button.

Also notice that I declared "play" to be a global variable. Variables defined inside a function are by default saved in the function namespace. When the function returns, the variable ceases to exist. If you want to change "play" inside playermove and have that change be visible the next time playermove() is called, "play" needs to be a global variable. The way to tell a function that a variable is global is to use the "global" keyword followed by the function name.

Finally, there are better ways to make large buttons than what you are trying. If you want a big button, make the text large by using a large font. You can also specify the width and height of the button. While working on your code I made my own version. This is how I made my buttons:

    button = tk.Button(pencere, text=' ', font=('Times', 32, 'bold'),
                       fg='yellow', bg='red', width=4, height=2)
    button.configure(command=lambda b=button: playermove(b))

Setting the font size = 32 makes really large buttons. To make the buttons even larger I make them 4 characters wide and 2 characters high.