Уведомления

Группа в Telegram: @pythonsu

#1 Окт. 4, 2016 18:54:25

marataziat
От: Алматы
Зарегистрирован: 2016-05-24
Сообщения: 104
Репутация: +  -5  -
Профиль   Отправить e-mail  

Оцените мой код :)

Я написал код:

 # -*- coding: utf-8 -*-
"""
Это библиотека для работы с
yandex speech kit cloud API
"""
from urllib import urlretrieve
#значения
http_api_server = "https://tts.voicetech.yandex.net/generate?" #speech сервер yandex
format = "mp3" #расширение выходного файла
filename = "testname" #это имя файла, нужно указывать только имя! Без расширения
key = "" #сюда нужно вставить ваш ключ
speaker = "zahar" #здесь должен быть ваш спикер к примеру zahar
lang = "ru" #язык для озвучивания по умолчанию ru
emotion = "good" #эмоция к примеру good
speed = "1" #скорость от 0.1 до ∞
text = "Это мой первый текст сгенерированный при помощи Yandex Spech Kit" #текст для генерации
#функции
def speachit(text,format,filename,lang,speaker,key,emotion,speed):
    text.replace (" ", "+")
    urlretrieve(http_api_server+"text="+text+"&"+"format="+format+"&"+"lang="+lang+"&"
        +"speaker="+speaker+"&"+"key="+key+"&"+"speed="+speed, filename+'.mp3')
Я показал его своему бате, он меня похвалил сказал все красиво. Но он бывший CPP разработчик и я хочу знать мнение бывалых python кодеров . Мне 13 лет так что сильно строго не судите



Я настоящий тюленялюб :)

Офлайн

#2 Окт. 4, 2016 19:02:56

marataziat
От: Алматы
Зарегистрирован: 2016-05-24
Сообщения: 104
Репутация: +  -5  -
Профиль   Отправить e-mail  

Оцените мой код :)

Дядя из Google не одобрил



Я настоящий тюленялюб :)

Офлайн

#3 Окт. 4, 2016 19:07:23

noob_saibot
Зарегистрирован: 2013-09-11
Сообщения: 495
Репутация: +  20  -
Профиль   Отправить e-mail  

Оцените мой код :)

marataziat
Я написал код:
Мне кажется format тут был бы кстати. Не пишите на русском.

НО. Я сам ни разу не средний питон программист, так что возможно профи натыкают носом.

Офлайн

#4 Окт. 4, 2016 19:08:04

noob_saibot
Зарегистрирован: 2013-09-11
Сообщения: 495
Репутация: +  20  -
Профиль   Отправить e-mail  

Оцените мой код :)

marataziat
Дядя из Google не одобрил
Оно и понятно, вы ж яндексом пользуетесь)))

Офлайн

#5 Окт. 4, 2016 20:03:46

JOHN_16
От: Россия, Петропавловск-Камчатск
Зарегистрирован: 2010-03-22
Сообщения: 3292
Репутация: +  221  -
Профиль   Отправить e-mail  

Оцените мой код :)

marataziat
http_api_server+“text=”+text+“&”+“format=”+format+“&”+“lang=”+lang+“&”
+“speaker=”+speaker+“&”+“key=”+key+“&”+“speed=”+speed, filename+'.mp3'
URL надо собирать не руками . В стандартной библиотеке есть для этого все, по памяти не помню названия модулей.
marataziat
text.replace (“ ”, “+”)
метод replace возвращает новую строку. Код ваш не работает так как вы ожидаете.
PEP8 не соблюден
Тут любой дядя специалист не одобрит




_________________________________________________________________________________
полезный блог о python john16blog.blogspot.com

Офлайн

#6 Окт. 4, 2016 20:41:02

ZerG
Зарегистрирован: 2012-04-05
Сообщения: 2627
Репутация: +  61  -
Профиль   Отправить e-mail  

Оцените мой код :)

Поддерживаю = .format() тут более в тему. Глаза сломать можно!
Кроме того на мой взгляд через чур много переменных для такой короткой логики!



Влодение рускай арфаграфией - это как владение кунг-фу: настаящие мастира не преминяют ево бес ниабхадимости

Офлайн

#7 Окт. 5, 2016 17:18:09

doza_and
От:
Зарегистрирован: 2010-08-15
Сообщения: 4138
Репутация: +  252  -
Профиль   Отправить e-mail  

Оцените мой код :)

На мой взгляд основной недостаток кода в том что он вообще ничего не делает, и оценка поэтому ему поэтому ноль :). Что там его содержимое оценивать. Если хотели потестить свой код, надо было тест и делать.



Офлайн

Board footer

Модераторировать

Powered by DjangoBB

Lo-Fi Version