Аапвпвапвапавпавпвыапвыапва?

Question

[cenox]

Привет, что делать в данной ситуации????

Answer 1

[Павел Трушин]

Дорогой друг, вопрос.. Что именно ты хочешь сделать?
Для определения людей from_id > 0 для определения группы from_id < 0
Я так понял ты хочешь получать картинку пользователя и группы.. Если да, то просто ставишь проверку на from_id, как я описал выше. Для пользователя используешь метод ("users.get"), для группы ("groups.getById").

if ans.reply_message.from_id > 0:
    user_photo = await bot.api.request("users.get", {"user_ids": ans.reply_message.from_id, "fields": "photo_max_orig"})
else:
    user_photo = await bot.api.request("groups.getById", {"user_ids": ans.reply_message.from_id, "fields": "photo_max_orig"})

Comments

[Павел Трушин]

Ты сравниваешь peer_id, что является id-чата у бота. Нужно сравнивать id-пользователя, который прислал сообщение. Также вижу, что ты просто продублировал один и тот же код, что не есть хорошо)
Здесь ты проверяешь, если from_id > 0, то это написал человек и ты используешь ("users.get"), иначе ("groups.getById"). Ты просто меняешь методы получения для человека и группы, весь остальной код точно такой же)

txt = ans.reply_message.text
if ans.reply_message.from_id > 0:
    user = await bot.api.request("users.get", {"user_ids": ans.reply_message.from_id, "fields": "photo_max_orig"})
else:
    user = await bot.api.request("groups.getById", {"group_ids": ans.reply_message.from_id, "fields": "photo_max_orig"})
p = requests.get(user[0]['photo_max_orig'])
out = open(r'rus.jpg', "wb")
out.write(p.content)
out.close()
if len(txt) >= 239:
  txt = txt[:239]
a = quote(txt, f"{user[0]['first_name']} {user[0]['last_name']} (с)")
a.get('rus.jpg')
photo = await photo_uploader.upload_message_photo('qresult.jpg')
await ans(attachment=photo)

[cenox]

PirraToZ, спасибо дружище!

[Павел Трушин]

cenox, Да, действительно, так как у группы есть только name и screen_name, где name - название группы, screen_name - короткое имя группы, когда укорачиваешь ссылку на группу, вместо club34378428. Замени кусочки кода на это:

if ans.reply_message.from_id > 0:
    user = await bot.api.request("users.get", {"user_ids": ans.reply_message.from_id, "fields": "photo_max_orig"})
    full_name = f"{user[0]['first_name']} {user[0]['last_name']}"
else:
    user = await bot.api.request("groups.getById", {"group_ids": ans.reply_message.from_id, "fields": 
"photo_max_orig"})
    full_name = user[0]["name"]

a = quote(txt, f"{full_name} (с)")

[Павел Трушин]

cenox, рад, что сам нашёл решение, но лучше используй моё т.к. у тебя опять "повтор" кода

[Павел Трушин]

cenox, выкладывай код с нормальными отступами, так будет куда проще ориентироваться в коде)
Как я понял - тебе нужно убрать ВСЕ отступы на 1 шаг начиная с строчки (включительно):
p = requests.get(user[0]['photo_max_orig'])

[cenox]

PirraToZ, спасибо, можешь еще помочь? Когда ввожу команду "цитата", просто, без ответа на сообщение, в консоль выдает это:

txt = ans.reply_message.text
AttributeError: 'NoneType' object has no attribute 'text'

Как это чинить?
Еще почему-то когда используешь команду на группу, текст вставляет, но аватарку использует самого бота

[Павел Трушин]

cenox,

if ans.reply_message.text is not None:
    #весь остальной код команды цитата

На счет аватарки группы.. Проверь метод для группы, скорее всего ты отсылаешь не ans.reply_message.from_id

[Павел Трушин]

cenox, Тогда попробуй в методе для группы изменить "group_ids" на "user_ids"

[cenox]

PirraToZ, изменил на user_ids, но ничего не изменилось

[Павел Трушин]

cenox, как оказалось vk-api принимает группы с числами как "club842984" или "game_foryou"
В каждой строчке после того, как вы поняли, что ans.reply_message.from_id < 0.

if '-' not in ans.reply_message.from_id:
    user = await bot.api.request("groups.getById", {"group_ids": ans.reply_message.from_id, "fields": "photo_max_orig"})
    full_name = user[0]["name"]
else:
    user = await bot.api.request("groups.getById", {"group_ids": str(ans.reply_message.from_id).replace("-", "club"), "fields": "photo_max_orig"})
    full_name = user[0]["name"]

ans.fwd_messages - содержит в себе None или лист с 1 или несколькими сообщениями. В любом случае, ты должен обратиться к индексу сообщений/я, которое переслал человек. Итог: в любом случае, где используется ans.fwd_messages[0] или ans.fwd_messages[0].from_id или ans.fwd_messages[0].text

[cenox]

PirraToZ, ошибка:

if '-' not in ans.reply_message.from_id:
TypeError: argument of type 'int' is not iterable

[Павел Трушин]

cenox, попробуй использовать этот код, я постарался сжать твой код, но при этом оставить функционал)

if ans.reply_message is not None or ans.fwd_messages is not None:
    if ans.reply_message is not None:
        txt = ans.reply_message
        from_id = ans.reply_message
    else:
        txt = ans.fwd_messages[0].text
        from_id = ans.fwd_messages[0].from_id
    if '-' not in f"{from_id}" and from_id > 0:
        user = await bot.api.request("users.get", {"user_ids": from_id, "fields": "photo_max_orig"})
        full_name = f"{user[0]['first_name']} {user[0]['last_name']}"
    else:
        if '-' not in f"{from_id}":
            user = await bot.api.request("groups.getById", {"group_ids": from_id, "fields": "photo_max_orig"})
            full_name = user[0]["name"]
        else:
            user = await bot.api.request("groups.getById", {"group_ids": str(from_id).replace("-", "club"), "fields": "photo_max_orig"})
            full_name = user[0]["name"]
    #Остальная часть команды цитата
    #from_id - это id человека на чьё сообщение ответили или переслали
else:
    await ans('Используй данную команду отвечая на сообщение')

[Павел Трушин]

cenox, обновил код выше, так как нашёл ошибку)

[cenox]

PirraToZ,

if '-' not in f"{from_id}" and from_id > 0:
TypeError: '>' not supported between instances of 'ForeignMessage' and 'int'

[Павел Трушин]

cenox, попробуй сделать так:
if '-' not in f"{from_id}" and int(from_id) >= 0:

[cenox]

PirraToZ,

if '-' not in f"{from_id}" and int(from_id) >= 0:
TypeError: int() argument must be a string, a bytes-like object or a number, not 'ForeignMessage'

[Павел Трушин]

cenox, Сделай вывод from_id и вызывай эту ошибку, мне интересно, что он держит внутри from_id в это время

[cenox]

PirraToZ, Я починил это, оказывается дело было в:

if ans.reply_message is not None:
          txt = ans.reply_message
          from_id = ans.reply_message
        else:
          txt = ans.fwd_messages[0].text
          from_id = ans.fwd_messages[0].from_id

Нужно было заменить

txt = ans.reply_message
          from_id = ans.reply_message

на

txt = ans.reply_message.text
					from_id = ans.reply_message.from_id

[Павел Трушин]

cenox, ну и хорошо, я значит просмотрел :) Всегда при ошибке делаю print() переменной, если дело в ней)

[cenox]

PirraToZ, кстати, это не работает:

else:
				await ans('Используй данную команду переслав сообщение или ответив на него')

Когда просто пишешь сообщение, в консоли выдает:

txt = ans.fwd_messages[0].text
IndexError: list index out of range

[Павел Трушин]

cenox, т.е. возникает ошибка, просто когда человек пишет любое сообщение?
Если да, то смотри проверку на какие команды отвечает бот.
Так как эта команда должна срабатывать только в том случае, когда человек написал:
Обращение к команде (Пересылая/отвечая на сообщение или просто название самой команды)
Если он дошёл до этого момента, то попробуй опять вывести переменную ans.fwd_messages

[cenox]

PirraToZ, ошибка возникает, когда вводишь команду "цитата", просто, без ответов и пересылок

[Павел Трушин]

cenox, сделай вывод двух переменных ans.fwd_messages и ans.reply_message в этот момент, когда у тебя просто сообщение без ответов и пересылок

[cenox]

PirraToZ, а по руски

[Павел Трушин]

cenox,
Сразу после if ans.reply_message is not None and ... :

print(ans.fwd_messages)
print(ans.reply_message)

[cenox]

PirraToZ, та же ошибка:

txt = ans.fwd_messages[0].text
IndexError: list index out of range

[Павел Трушин]

cenox, понимаю, но мне нужны значения, которые вывел твой бот через print

[cenox]

PirraToZ,
[]
None

[Павел Трушин]

cenox, тогда все просто

if ans.reply_message not in [None, []] or ans.fwd_messages is not None:
#или
if ans.reply_message is not None and ans.reply_message != []  or ans.fwd_messages is not None:

[cenox]

PirraToZ, нужно что-то из этого выбрать? и куда это вставлять вообще?

[Павел Трушин]

cenox, Да, нужно выбрать одну из проверок (не уверен, что первая работает).
Это нужно вставить вместо:

if ans.reply_message is not None or ans.fwd_messages is not None:

[cenox]

PirraToZ,

txt = ans.fwd_messages[0].text
IndexError: list index out of range

[Павел Трушин]

cenox,

if ans.reply_message not in [None, []] or ans.fwd_messages not in [None, []]:

[cenox]

PirraToZ, заработало, спасибо

[Павел Трушин]

cenox, не за что)

[fallus]

Понял.

[Павел Трушин]

fallus, что именно?

[fallus]

PirraToZ, во-первых суть вопроса. Вопрос очень хорошо развёрнут. По полочкам. А во-вторых, плавный переход от этого вопроса к развёрнутому ответу.