Skip to content

Homework Class 6#4

Open
niskhakov wants to merge 1 commit intomasterfrom
Homework6
Open

Homework Class 6#4
niskhakov wants to merge 1 commit intomasterfrom
Homework6

Conversation

@niskhakov
Copy link
Owner

Домашняя работа к 6 уроку. Скрипт временно создает таблицы в базе данных shop_data, создает записи в таблицах, а затем берет эти данные и записывает в файл output.csv в формате csv, после удаляет все созданные таблицы(осторожно, таблицы после выполнения скрипта будут удалены, даже если эти таблицы сущестовали ранее).

@niskhakov niskhakov requested a review from Frodox August 6, 2018 04:13
Copy link
Collaborator

@Frodox Frodox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всё хорошо, кроме вот маленькой мелочи с безопасностью :)


def __init__(self):
try:
self.conn = pg.connect(host="localhost", database="shop_data", user="py_user", password="py_pass", port=6789)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

давай креды только из файла всё таки читать, а не хардкодить)

try:
self.conn = pg.connect(host="localhost", database="shop_data", user="py_user", password="py_pass", port=6789)
except pg.Error as e:
print("I'm unable to connect to the db")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ошибки в stderr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants