search

morrolinux / subito-it-searcher

searcher for subito.it
100 stars 36 forks source link

Add include/exclude regex pattern to filter search result #24

Open elbowz opened 1 year ago

elbowz commented 1 year ago

The include/exclude filter are applied on item title, and saved in the json.

I'm not very happy of my solution. I'm spoking about print_sitrep() and print_queries() nested loop. Honestly, I'm lost inside the multiple loops for, and I don't well figure out the logic behind the json structure, expecially the position of minPrice and maxPrice.

{minPrice: {maxPrice: {...

For sure I have missed something in the program logic and structure...but my PR seems works.

About the minPrice and maxPrice, I see that subito.it have GET params (ps=30&pe=100), but can be used only inside a category, I guess.

I have also added a simple requirements.txt.

Thanks for your time and project!

note: you can do what ever you want with this PR...I have try to leave as much possible unchanged your code, but IMHO a different JSON structure and/or removing the [min,max]Price could be a way.

morrolinux commented 1 year ago

looking for testers

elbowz commented 1 year ago

You can trash my PR, it will not be a problem for me...

But I'm curious about the JSON structure. Why have you nested the minPrice and maxPrice?