RSS torrent downloader
$begingroup$
Just looking for any tips on improving what I have. It does exactly what I want but i'm still learning Python and was wondering if there's something I can do to make it more efficient or simpler.
import feedparser
import os
import urllib
import re
dwnLoc = 'C:/Downloads/'
#shows currently watching
shwOne = 'RSS LINK'
shwTwo = 'RSS LINK2'
aniLst = [shwOne, shwTwo]
#the episodes currently on
oneCur = [72,73]
twoCur = [13]
allEpi = [oneCur, twoCur]
#parse rss feed to
#create lists for episodes to download, their download links, and file names
newEpi =
dwnLnk =
filNme =
for shows,epi in zip(aniLst,allEpi):
dump = feedparser.parse(shows)
epiTrk =
for post in dump.entries:
epiNum = [int(item) for item in post.title.split() if item.isdigit()]
if len(epiNum) == 1:
if epiNum[0] > epi[len(epi)-1]:
dwnLnk.append(post.link)
filNme.append(post.title + '.torrent')
epiTrk.append(epiNum[0])
newEpi.append(epiTrk)
#downloads and runs torrent
for link,name in zip(dwnLnk,filNme):
fileDownloaded = dwnLoc + name + '.torrent'
urllib.request.urlretrieve(link,fileDownloaded)
os.startfile(fileDownloaded)
#rewrites script to increment current episodes if one was downloaded
content =
with open(__file__,"r") as f:
for line in f:
content.append(line)
with open(__file__,"w") as f:
counter = 0
for i,item in enumerate(content):
if all(str1 in item for str1 in ['Cur',']']) and not any(str2 in item for str2 in ['all','if']):
endFnd = item.find(']')
epiLst = newEpi[counter]
itmAdd = item[:endFnd]
for epiAdd in reversed(epiLst):
itmAdd += ',' + str(epiAdd)
content[i] = itmAdd + ']n'
counter += 1
for j in range(len(content)):
f.write(content[j])
python beginner http rss
New contributor
$endgroup$
add a comment |
$begingroup$
Just looking for any tips on improving what I have. It does exactly what I want but i'm still learning Python and was wondering if there's something I can do to make it more efficient or simpler.
import feedparser
import os
import urllib
import re
dwnLoc = 'C:/Downloads/'
#shows currently watching
shwOne = 'RSS LINK'
shwTwo = 'RSS LINK2'
aniLst = [shwOne, shwTwo]
#the episodes currently on
oneCur = [72,73]
twoCur = [13]
allEpi = [oneCur, twoCur]
#parse rss feed to
#create lists for episodes to download, their download links, and file names
newEpi =
dwnLnk =
filNme =
for shows,epi in zip(aniLst,allEpi):
dump = feedparser.parse(shows)
epiTrk =
for post in dump.entries:
epiNum = [int(item) for item in post.title.split() if item.isdigit()]
if len(epiNum) == 1:
if epiNum[0] > epi[len(epi)-1]:
dwnLnk.append(post.link)
filNme.append(post.title + '.torrent')
epiTrk.append(epiNum[0])
newEpi.append(epiTrk)
#downloads and runs torrent
for link,name in zip(dwnLnk,filNme):
fileDownloaded = dwnLoc + name + '.torrent'
urllib.request.urlretrieve(link,fileDownloaded)
os.startfile(fileDownloaded)
#rewrites script to increment current episodes if one was downloaded
content =
with open(__file__,"r") as f:
for line in f:
content.append(line)
with open(__file__,"w") as f:
counter = 0
for i,item in enumerate(content):
if all(str1 in item for str1 in ['Cur',']']) and not any(str2 in item for str2 in ['all','if']):
endFnd = item.find(']')
epiLst = newEpi[counter]
itmAdd = item[:endFnd]
for epiAdd in reversed(epiLst):
itmAdd += ',' + str(epiAdd)
content[i] = itmAdd + ']n'
counter += 1
for j in range(len(content)):
f.write(content[j])
python beginner http rss
New contributor
$endgroup$
$begingroup$
This time it should be right.
$endgroup$
– qrs
50 mins ago
add a comment |
$begingroup$
Just looking for any tips on improving what I have. It does exactly what I want but i'm still learning Python and was wondering if there's something I can do to make it more efficient or simpler.
import feedparser
import os
import urllib
import re
dwnLoc = 'C:/Downloads/'
#shows currently watching
shwOne = 'RSS LINK'
shwTwo = 'RSS LINK2'
aniLst = [shwOne, shwTwo]
#the episodes currently on
oneCur = [72,73]
twoCur = [13]
allEpi = [oneCur, twoCur]
#parse rss feed to
#create lists for episodes to download, their download links, and file names
newEpi =
dwnLnk =
filNme =
for shows,epi in zip(aniLst,allEpi):
dump = feedparser.parse(shows)
epiTrk =
for post in dump.entries:
epiNum = [int(item) for item in post.title.split() if item.isdigit()]
if len(epiNum) == 1:
if epiNum[0] > epi[len(epi)-1]:
dwnLnk.append(post.link)
filNme.append(post.title + '.torrent')
epiTrk.append(epiNum[0])
newEpi.append(epiTrk)
#downloads and runs torrent
for link,name in zip(dwnLnk,filNme):
fileDownloaded = dwnLoc + name + '.torrent'
urllib.request.urlretrieve(link,fileDownloaded)
os.startfile(fileDownloaded)
#rewrites script to increment current episodes if one was downloaded
content =
with open(__file__,"r") as f:
for line in f:
content.append(line)
with open(__file__,"w") as f:
counter = 0
for i,item in enumerate(content):
if all(str1 in item for str1 in ['Cur',']']) and not any(str2 in item for str2 in ['all','if']):
endFnd = item.find(']')
epiLst = newEpi[counter]
itmAdd = item[:endFnd]
for epiAdd in reversed(epiLst):
itmAdd += ',' + str(epiAdd)
content[i] = itmAdd + ']n'
counter += 1
for j in range(len(content)):
f.write(content[j])
python beginner http rss
New contributor
$endgroup$
Just looking for any tips on improving what I have. It does exactly what I want but i'm still learning Python and was wondering if there's something I can do to make it more efficient or simpler.
import feedparser
import os
import urllib
import re
dwnLoc = 'C:/Downloads/'
#shows currently watching
shwOne = 'RSS LINK'
shwTwo = 'RSS LINK2'
aniLst = [shwOne, shwTwo]
#the episodes currently on
oneCur = [72,73]
twoCur = [13]
allEpi = [oneCur, twoCur]
#parse rss feed to
#create lists for episodes to download, their download links, and file names
newEpi =
dwnLnk =
filNme =
for shows,epi in zip(aniLst,allEpi):
dump = feedparser.parse(shows)
epiTrk =
for post in dump.entries:
epiNum = [int(item) for item in post.title.split() if item.isdigit()]
if len(epiNum) == 1:
if epiNum[0] > epi[len(epi)-1]:
dwnLnk.append(post.link)
filNme.append(post.title + '.torrent')
epiTrk.append(epiNum[0])
newEpi.append(epiTrk)
#downloads and runs torrent
for link,name in zip(dwnLnk,filNme):
fileDownloaded = dwnLoc + name + '.torrent'
urllib.request.urlretrieve(link,fileDownloaded)
os.startfile(fileDownloaded)
#rewrites script to increment current episodes if one was downloaded
content =
with open(__file__,"r") as f:
for line in f:
content.append(line)
with open(__file__,"w") as f:
counter = 0
for i,item in enumerate(content):
if all(str1 in item for str1 in ['Cur',']']) and not any(str2 in item for str2 in ['all','if']):
endFnd = item.find(']')
epiLst = newEpi[counter]
itmAdd = item[:endFnd]
for epiAdd in reversed(epiLst):
itmAdd += ',' + str(epiAdd)
content[i] = itmAdd + ']n'
counter += 1
for j in range(len(content)):
f.write(content[j])
python beginner http rss
python beginner http rss
New contributor
New contributor
edited 5 mins ago
200_success
130k16153419
130k16153419
New contributor
asked 6 hours ago
qrsqrs
62
62
New contributor
New contributor
$begingroup$
This time it should be right.
$endgroup$
– qrs
50 mins ago
add a comment |
$begingroup$
This time it should be right.
$endgroup$
– qrs
50 mins ago
$begingroup$
This time it should be right.
$endgroup$
– qrs
50 mins ago
$begingroup$
This time it should be right.
$endgroup$
– qrs
50 mins ago
add a comment |
0
active
oldest
votes
Your Answer
StackExchange.ifUsing("editor", function () {
return StackExchange.using("mathjaxEditing", function () {
StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
});
});
}, "mathjax-editing");
StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");
StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "196"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});
function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});
}
});
qrs is a new contributor. Be nice, and check out our Code of Conduct.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f214972%2frss-torrent-downloader%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
0
active
oldest
votes
0
active
oldest
votes
active
oldest
votes
active
oldest
votes
qrs is a new contributor. Be nice, and check out our Code of Conduct.
qrs is a new contributor. Be nice, and check out our Code of Conduct.
qrs is a new contributor. Be nice, and check out our Code of Conduct.
qrs is a new contributor. Be nice, and check out our Code of Conduct.
Thanks for contributing an answer to Code Review Stack Exchange!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
Use MathJax to format equations. MathJax reference.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f214972%2frss-torrent-downloader%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
$begingroup$
This time it should be right.
$endgroup$
– qrs
50 mins ago