fas.li.shink.in

fas.liとshink.inのクッキーを固定し、複数開いた時も同時進行できるようにする

< Feedback on fas.li.shink.in

Review: Good - script works

§
Posted: 12.03.2017
Edited: 12.03.2017

共通化について

可読性が落ちるのがあれですね

(function() {
var current_url = location.href;

var cookie_name = [];
cookie_name["fas.li"] = "csfi=";
cookie_name["shink.in"] = "csrf_sci=";

var domain = current_url.split("/")[2];

var FIXED_COOKIE = domain + "_fixed_cookie";

var cookie_value = document.cookie.split(cookie_name[domain])[1].split(";")[0];

if (current_url.split("/")[3] === "go") {
document.cookie = cookie_name[domain] + encodeURIComponent(GM_getValue(FIXED_COOKIE)) + ";domain=." + domain + ";path=/";
// console.info("GM_getValue(" + FIXED_COOKIE + ") = " + GM_getValue(FIXED_COOKIE));
} else {
GM_setValue(FIXED_COOKIE, cookie_value);
// console.info("GM_setValue(" + FIXED_COOKIE + ", " + cookie_value);
}
})();

plgdownAuthor
§
Posted: 12.03.2017

フラットになってすっきりしました でもやっぱりこうなると可読性が落ちてきますよね…しかし元のコードもネストのせいで大概か v1.1で何か起ったら何も考えずこれにしよう

§
Posted: 12.03.2017

処理自体はv1.1と全く同じなのでv1.1で問題が起こったら↑のコードも死ぬ可能性大なのがあれですね
(対応shortenerを追加するのが面倒な時なら、あるいは…?)

crokoSkipperの時みたいな小技も入れられなかったので、コードを読んでも面白くないのがちょっとなぁ…という気分です
強いて言えばsplitの使い方とdomainを使いまわしていることぐらいでしょうか

新しいコードも楽しみにしていますね

Post reply

Sign in to post a reply.