search

SeidChr / AutoTagger

Proof of Concept for a Instagram Best-Tag-Finder
1 stars 1 forks source link

Code Review #11

Open DariosKrimsKrams opened 6 years ago

DariosKrimsKrams commented 6 years ago

Der von mir gefactored'te CrawlerV1 ist feature-complete und crawlt die Userseiten. Crawling Logik von CrawlerV2 übernommen. Ich bitte um ein Code Review @tempel3 @Vittel :)

SeidChr commented 6 years ago

Dafür sind die Pull requests :)

SeidChr commented 6 years ago

Ah ja es meist du mit Logik übernommen? Erzähle mir bitte nicht du hast alles von parser 2 in 1 verschoben? -_-

DariosKrimsKrams commented 6 years ago

ich habe den CrawlerV1 komplett neu aufgebaut bzw V1 als Grundlage genommen und die Userpages ergänzt. U.a. habe ich mir paar regex etc. von V2 geklaut

DariosKrimsKrams commented 6 years ago

der alte V1 war sowohl was crawling anging, ausbaufähig, und vom Code her unsauber. Die alte V1 Logik oder deine V2 Logik kannst du nun super einfach mit dem neuen V1 nachbauen. Ist sehr dynamisch aufgebaut.. bzw ob es gut aufgebaut ist, das möchte ich ja von dir/euch wissen :)

SeidChr commented 6 years ago

Ohne es gesehen zu haben (habe leider derzeit super wenig zeit neben der arbeit) würde ich vermuten du wolltest einen v3 bauen.. Werde es mir bei Gelegenheit ansehen.

DariosKrimsKrams commented 6 years ago

Also keine Angst, habe an deinem V2 nichts verschlimmbessert ;) ja, sozusagen einen V3 - nur, dass der V1 nun überflüggig war, da V2 das besser hinbekommt :)