Как решить задачу на Python?

Question

[Влад Хоренко]

Есть задача:
Пример работы утилиты:
Сохранение значения value по ключу key_name:
$ storage.py --key key_name --val value
Получение значения по ключу key_name:
$ storage.py --key key_name
Задача: написать реализацию утилиты storage.py.
Утилита может вызваться со следующими параметрами:
--key <имя ключа> , где <имя ключа> - ключ по которому сохраняются/получаются значения
--val <значение>, где <значение> - сохраняемое значение.
Если при запуске утилиты переданы оба ключа, происходит добавление переданного значения по ключу и сохранение данных в файле. Если передано только имя ключа, происходит чтение файла хранилища и вывод на печать значений, которые были сохранены по данному ключу.  Обратите внимание, что значения по одному ключу не перезаписываются, а добавляются к уже сохраненным. Другими словами - по одному ключу могут храниться несколько значений. При выводе на печать, значения выводятся в порядке их добавления в хранилище. Формат вывода на печать для нескольких значений:
value_1, value_2
Обратите внимание на пробел после запятой. Если значений по ключу не было найдено, выведите пустую строку или None.
Для работы с аргументами командной строки используйте модуль argparse. Хранить данные в файле мы рекомендуем в формате JSON с помощью использования модуля стандартной библиотеки  json. Прежде чем отправлять ваше решение на проверку, протестируйте работу вашей утилиты на добавление нескольких ключей и разных значений.
Файл следует создавать с помощью модуля tempfile.
import os
import tempfile

storage_path = os.path.join(tempfile.gettempdir(), 'storage.data')
with open(storage_path, 'w') as f:
  ...
Пример работы:
$ python storage.py --key key_name --val value
$ python storage.py --key key_name
value

$ python storage.py --key multi_key --val value1
$ python storage.py --key multi_key --val value2
$ python storage.py --key multi_key
value1, value2

Я ее решил вот так:

import os
import sys
import tempfile
import json

storage_path = os.path.join(tempfile.gettempdir(), 'storage.data')
storage = {}
def key(dictionary, key):
    dictionary.update(key, None)
def key_and_value(dictionary, key, value):
    dictionary.update(key, value)
with open(storage_path, 'w') as f:
    if len(sys.argv) <= 3 and sys.argv[1] == '--key':
        if sys.argv[2] not in storage:
            key(storage, sys.argv[2])
            json.dump(storage, f)
        if sys.argv[2] in storage:
            with open(storage_path, 'w') as f:
                storage.get(key[sys.argv[2]])
    if len(sys.argv) > 3 and (sys.argv[1] == '--key' and sys.argv[3] == '--val'):
        key_and_value(storage, sys.argv[2], sys.argv[4])
        json.dump(storage, f)

Но у меня есть ошибки в ходе выполнения данной программы.
Пожалуйста, буду очень благодарен, если вы мне объясните как решить эту задачу, и покажите что у меня было не так.

Comments

[BorLaze]

Для начала, неплохо было бы отформатировать текст, заключить его в тег <code lang="python">...</code> и написать, какие же именно "есть ошибки в ходе выполнения данной программы"...

[dmshar]

Какие у вас ошибки?
И еще - у вас "не так" представление вашего кода. Оберните его в соответствующий тег.

[Влад Хоренко]

BorLaze, это программа не для браузера, это простая программа

[Влад Хоренко]

BorLaze,
1.
Traceback (most recent call last):
File "/home/vlad_horenko/PycharmProjects/Deep_in_Python/Week_2/Pract_1/main.py", line 13, in
if len(sys.argv) <= 3 and sys.argv[1] == '--key':
IndexError: list index out of range
2.
Traceback (most recent call last):
File "main.py", line 15, in
key(storage, sys.argv[2])
File "main.py", line 9, in key
dictionary.update(key, None)
TypeError: update expected at most 1 argument, got 2
3.

Traceback (most recent call last):
File "main.py", line 21, in
key_and_value(storage, sys.argv[2], sys.argv[4])
File "main.py", line 11, in key_and_value
dictionary.update(key, value)
TypeError: update expected at most 1 argument, got 2

[Влад Хоренко]

dmshar, 1.
Traceback (most recent call last):
File "/home/vlad_horenko/PycharmProjects/Deep_in_Python/Week_2/Pract_1/main.py", line 13, in
if len(sys.argv) <= 3 and sys.argv[1] == '--key':
IndexError: list index out of range
2.
Traceback (most recent call last):
File "main.py", line 15, in
key(storage, sys.argv[2])
File "main.py", line 9, in key
dictionary.update(key, None)
TypeError: update expected at most 1 argument, got 2
3.

Traceback (most recent call last):
File "main.py", line 21, in
key_and_value(storage, sys.argv[2], sys.argv[4])
File "main.py", line 11, in key_and_value
dictionary.update(key, value)
TypeError: update expected at most 1 argument, got 2

[Василий Дёмин]

Влад Хоренко, BorLaze предлагает вам поместить код программы в вопросе в тег code, чтобы она отображалась с подсветкой и отступами

[Василий Дёмин]

Влад Хоренко, ноль аргументов или один меньше трёх, нужно перед sys.argv[1] проверять, есть ли хотя бы два.

[Василий Дёмин]

Можно просто использовать модуль argparse, как указано в задаче, вместо самописного парсера аргументов

[Влад Хоренко]

Василий Дёмин, я не знаю как пользоваться этим модулем, а в ходе самой программы у меня задается или 2 или 4 аргумента, смотря какая команда будет

[BorLaze]

Влад Хоренко, а доки кому пишутся? тем, кто знает? :-)

https://docs.python.org/3/library/argparse.html

читаешь, разбираешься, подключаешь, используешь...

[Алан Гибизов]

Рекомендую убрать лишние теги. Оставьте Python, остальные - «тег-спам», могут и наказать за это.

[Влад Хоренко]

BorLaze, у меня с английским плохо

[BorLaze]

Влад Хоренко, с гуглом у тебя тоже плохо, как я вижу?

Должен тебя расстроить - "Хочу стать Python разработчиком" при таком подходе так и останется "хочу".

[Влад Хоренко]

BorLaze, Я спросил изначально что у меня с кодом не так. Что бы сказали почему и что не так, потому что я еще не все знаю, и только учусь

[Vindicar]

Влад Хоренко, тебе сказали.
1. Читай, как пользоваться модулем argparse. Не умеешь по-английски - учи английский. Разработчик без знания английского - практически калека. тем более что беглая речь не требуется, достаточно умения читать со словарём. Это школьный уровень, на самом деле.
Но поскольку изучение языка - дело не очень быстрое, то либо используй машинный перевод, либо ищи документацию на русском. В любом случае, тебе направление поисков указали. Дальше сам.
2. По поводу ошибок. 1 - ты запускаешь скрипт без аргументов, а скрипт этого не ожидает. Решается либо проверкой длины sys.argv, либо использованием argparse, он это умеет.
2,3 - ты неправильно используешь метод update() у словаря. Он не предназначен для задания значения по ключу. Читай документацию.

[Влад Хоренко]

Vindicar, спасибо, я посмотрю

Answer 1

[Василий Дёмин]

Вы аргументы не правильно разбираете. Например,

if len(sys.argv) <= 3 and sys.argv[1] == '--key':
        if sys.argv[2] not in storage:

Если аргументов меньше или три, вы обращаетесь ко второму и третьему аргументу. Ноль аргументов под это условие подходит, поэтому у вас программа и крашится.

Ещё пример,

if sys.argv[2] in storage:
    with open(storage_path, 'w') as f:
        storage.get(key[sys.argv[2]])

Если в хранилище есть ключ, открываем файл и достаём из хранилища значение. Во-первых на данный момент, этот файл уже открыт, во-вторых вы не считываете значения из него в переменную storage, в-третьих вы достаёте значение из storage, но не выводите его в консоль.

Ещё неверный кусок:

def key_and_value(dictionary, key, value):
    dictionary.update(key, value)

Вам по условию нужно собирать список всех значений, а не только последнее.

Comments

[Влад Хоренко]

А как сделать так, чтобы когда скрипт закончился, и начался опять, чтобы не создавался заново новый словарь, а чтобы добавлялось к старому?

[Василий Дёмин]

Влад Хоренко, Считать из файла старый словарь и добавлять в него

[Влад Хоренко]

А как это сделать, подскажите пожалуйста
Вот код

import argparse
import tempfile
import os


parse = argparse.ArgumentParser()
parse.add_argument('-k', '--key', default=None, required=True)
parse.add_argument('-v', '--val', default=None, required=False)

key = parse.parse_args()
value = parse.parse_args()

dict_cop = {key.key : value.val}

dictionary = dict()
dictionary.update(dict_cop)

print(key.key)
print(value.val)
print(dictionary)

[Василий Дёмин]

1. Открыть файл на чтение
   
2. Декодировать его содержимое в словарь (json.load в помощь).
   

[Влад Хоренко]

У меня вышло так:

import argparse
import tempfile
import os
import json

storage_path = os.path.join(tempfile.gettempdir(), 'storage.data')

parse = argparse.ArgumentParser()
parse.add_argument('-k', '--key', default=None, required=True)
parse.add_argument('-v', '--val', default=None, required=False)

key = parse.parse_args()
value = parse.parse_args()

dict_cop = {key.key : value.val}

with open(storage_path, 'r') as f:
    dictionary = json.load(f)
with open(storage_path, 'w') as f:
    dictionary.update(dict_cop)
    json.dump(dictionary, f)
with open(storage_path, 'r') as f:
    print(json.load(f))

Ошибки:

Traceback (most recent call last):
File "storage.py", line 18, in
dictionary = json.load(f)
File "/usr/lib/python3.8/json/__init__.py", line 293, in load
return loads(fp.read(),
File "/usr/lib/python3.8/json/__init__.py", line 357, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.8/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.8/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Что там не так?

[Василий Дёмин]

Влад Хоренко, JSONDecodeError Expecting value: line 1 column 1 (char 0)

[Василий Дёмин]

Ошибка декодирования json

[Влад Хоренко]

Василий Дёмин, А как ее исправить?

[Василий Дёмин]

Влад Хоренко, Открывать файл на чтение если он существует. Плюс обрабатывать исключение, на случай если в файл записан невалидный json

[Влад Хоренко]

Василий Дёмин, у меня должен быть темпфайл, что с ним делать, чтобы ошибки не было, там вроде все правильно

[Василий Дёмин]

Влад Хоренко, Если он пустой, например, то парсинг не пройдёт. Если запускать скрипт в первый раз, файл не будет существовать, открытие его на чтение бросит исключение

[Влад Хоренко]

Василий Дёмин, Спасибо, я попробую.

[Алан Гибизов]

Влад Хоренко, в одном вопросе должен быть один вопрос. Следующий вопрос создавайте отдельно. См.п.3.7