Как генерировать случайные пуринаты для случайных фигур в python

Question

Это код программы, в которой я пытаюсь получить случайные координаты для рисования случайной формы, теперь случайный цвет работает, но когда я запускаю программу, я получаю эту ошибку :

(Traceback (most recent call last): 
    File "/home/04danielbrew/Desktop/Python/Random Coloured Square.py", line 9, in <module> 
    randomh=random.randint(1,1080) 
AttributeError: 'builtin_function_or_method' object has no attribute 'randint) 

И спасибо заранее, и мне не нужен самый продвинутый код, поскольку я занимаюсь только наукой о GCSE.

from tkinter import* 
from random import* 
####Set veriables 
colors=["mintcream","orangered","gold","coral","red", "orange", "yellow", "green", "blue","violet","midnightblue"] 
canvas_height=1080 
canvas_width=1920 
canvas_colour='black' 
line_width=1 
randomh=random.randint(1,1080) 
randomw=random.randint(1,1920) 
randomh2=random.randint(1,1080) 
randomw2=random.randint(1,1920) 
####Define function 
def square(self): 
    canvas.create_line(randomh,randomw,randomh2,randomw2, width=line_width,   fill=random.choice(colors)) 
    canvas.create_line(300,100,400,100, width=line_width, fill=random.choice(colors)) 
    canvas.create_line(400,100,400,200, width=line_width, fill=random.choice(colors)) 
    canvas.create_line(400,200,300,200, width=line_width, fill=random.choice(colors)) 
###Main program 
window=Tk() 
window.title("Daniel Random Amazballs lines") 
canvas=Canvas(bg=canvas_colour,height=canvas_height,width=canvas_width, highlightthickness=0) 
canvas.pack() 
###Click in window to start 
window.bind("<Button-1>",square) 
window.mainloop() 

Answer 1

В качестве лучшей практики, а также для вещего кода, делать не from <module> import *.

В этом случае вместо этого вы можете использовать from random import randint, choice.

Тогда ваши звонки будут выглядеть примерно так: randomh = randint(1,1080) и choice(colors)

Дело в том, что from <module> import * почти всегда неправильный путь - это загрязняет пространство имен верхнего уровня с всех от <module>.

Explicit is better than implicit

Это не о расширенном коде, это про научиться писать код, который четко выражает свое намерение.

Answer 2

Поскольку вы импортируете все из случайного, вам не нужно называть random.randint(), вам просто нужно вызвать randint() прямо.

так:

randomh = randint(1,1080) 
randomw = randint(1,1920) 
randomh2 = randint(1,1080) 
randomw2 = randint(1,1920) 

Как сказал GOMAD, это не очень хорошая практика импорта все из модуля, особенно если вы используете только один метод.

Кроме того, случайный предполагается использовать только с импортом. Чувствуете разницу в:

import random 
random.choice([1,2,3,4]) 

и

from random import choice 
choice([1,2,3,4]) 

Первый ясно, что он делает, он выбирает один из элементов в массиве. Хотя второе неясно, он выбрал массив как что-то? Представляет ли он выбор пользователю?