RSS torrent downloader












1












$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])









share|improve this question









New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$












  • $begingroup$
    This time it should be right.
    $endgroup$
    – qrs
    55 mins ago
















1












$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])









share|improve this question









New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$












  • $begingroup$
    This time it should be right.
    $endgroup$
    – qrs
    55 mins ago














1












1








1





$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])









share|improve this question









New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$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 network-file-transfer rss






share|improve this question









New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.











share|improve this question









New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.









share|improve this question




share|improve this question








edited 11 secs ago









200_success

130k16153419




130k16153419






New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.









asked 6 hours ago









qrsqrs

62




62




New contributor




qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.





New contributor





qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.






qrs is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.












  • $begingroup$
    This time it should be right.
    $endgroup$
    – qrs
    55 mins ago


















  • $begingroup$
    This time it should be right.
    $endgroup$
    – qrs
    55 mins ago
















$begingroup$
This time it should be right.
$endgroup$
– qrs
55 mins ago




$begingroup$
This time it should be right.
$endgroup$
– qrs
55 mins ago










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.










draft saved

draft discarded


















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.










draft saved

draft discarded


















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.




draft saved


draft discarded














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





















































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







Popular posts from this blog

404 Error Contact Form 7 ajax form submitting

How to know if a Active Directory user can login interactively

TypeError: fit_transform() missing 1 required positional argument: 'X'