没错，我给androidx修了一个bug！

不容易啊，必须先截图留恋😁

这个bug是发生在xml中给AppcompatTextView设置textFontWeight，但是却无法生效。修复bug的代码也很简单，总共就几行代码，但是在找引起这个bug的原因和后面给androidx提pr却花了很久。

//AppcompatTextHelper  
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P&& mFontWeight != TEXT_FONT_WEIGHT_UNSPECIFIED && mFontTypeface != null) {mFontTypeface = Api28Impl.create(mFontTypeface, mFontWeight,(mStyle & Typeface.ITALIC) != 0);

事情的起因是这样的，在Android开发中，我们经常会遇到设计需要我们让TextView支持medium，bold等不同字重样式，原生TextView给了一个textStyle的属性，但是这个属性只支持bold，italic，normal三种样式，这肯定是满足不了设计小姐姐的需求的，而且很多时候bold的样式都比设计出的样式要粗一些。

以前的老办法就是导入不同字重样式的字体文件到app里面，但是很多字体包要支持的语言比较多的话，字体包文件又会比较大，会造成Apk包体积增大。裁剪字体包也可以，但是样式字体包文件多了，也是一件麻烦事，这些杂事很多又不会算到开发时间里面，我就想还有没有其他的解决办法。

网上搜索了下，大多数都是让使用textFontWeight属性的，这个属性倒是可以支持0-1000的字重设置，但是就是兼容性不好，只支持大于api28的手机使用。

Attribute textFontWeight is only used in API level 28 and higher (current min is 21)

但是从我实际使用来看，好像这个属性在TextView上也并没起作用。

<TextViewstyle="@style/TextStyle"android:layout_width="wrap_content"android:layout_height="wrap_content"android:text="Font100"android:textFontWeight="100" />
...
<TextViewstyle="@style/TextStyle"android:layout_width="wrap_content"android:layout_height="wrap_content"android:text="Font100"android:textFontWeight="900" />

我用一个LinearLayout装了9个TextView，textFontWeight设置从100-900，然后运行在api34的模拟器上，可以看到每个TextView都没应用上textFontWeight属性。

然后我就去Read The Fucking Source Code，看看TextView的textFontWeiget的相关源码。TextView设置textFontWeiget的相关源码就是从setTypefaceFromAttrs到resolveStyleAndSetTypeface，最后就是setTypeface。

private void setTypefaceFromAttrs(@Nullable Typeface typeface, @Nullable String familyName,@XMLTypefaceAttr int typefaceIndex, @Typeface.Style int style,@IntRange(from = -1, to = FontStyle.FONT_WEIGHT_MAX) int weight) {if (typeface == null && familyName != null) {// Lookup normal Typeface from system font map.final Typeface normalTypeface = Typeface.create(familyName, Typeface.NORMAL);resolveStyleAndSetTypeface(normalTypeface, style, weight);} else if (typeface != null) {resolveStyleAndSetTypeface(typeface, style, weight);} else {  // both typeface and familyName is null.switch (typefaceIndex) {case SANS:resolveStyleAndSetTypeface(Typeface.SANS_SERIF, style, weight);break;case SERIF:resolveStyleAndSetTypeface(Typeface.SERIF, style, weight);break;case MONOSPACE:resolveStyleAndSetTypeface(Typeface.MONOSPACE, style, weight);break;case DEFAULT_TYPEFACE:default:resolveStyleAndSetTypeface(null, style, weight);break;}}
}

private void resolveStyleAndSetTypeface(@NonNull Typeface typeface, @Typeface.Style int style,@IntRange(from = -1, to = FontStyle.FONT_WEIGHT_MAX) int weight) {if (weight >= 0) {weight = Math.min(FontStyle.FONT_WEIGHT_MAX, weight);final boolean italic = (style & Typeface.ITALIC) != 0;setTypeface(Typeface.create(typeface, weight, italic));} else {setTypeface(typeface, style);}
}

从代码可以看出来，设置textFontWeight就是靠Typeface.create(typeface, weight, italic)。
这样来说，我们的代码应该是没有问题的呀，可是为啥没有起效呢？我想了很久，期间我想用hook的办法，把每个TextView的textFontWeight都打印出来看看，然后我就写了一个WeightPrinterTextView工具类。

class WeightPrinterTextView @JvmOverloads constructor(context: Context,attrs: AttributeSet? = null
) : TextView(context, attrs) {
}

然后我把xml中的TextView都替换成这个WeightPrinterTextView，意想不到的是，textFontWeight这个属性竟然又起效果了😂。

<LinearLayoutstyle="@style/WeightLL"android:layout_height="match_parent"android:orientation="vertical"><demo.simple.fontweighttextview.WeightPrinterTextViewandroid:id="@+id/textView1"style="@style/TextStyle"android:layout_width="wrap_content"android:layout_height="wrap_content"android:text="Font100"android:textFontWeight="100" /><demo.simple.fontweighttextview.WeightPrinterTextViewstyle="@style/TextStyle"android:layout_width="wrap_content"android:layout_height="wrap_content"android:text="Font200"android:textFontWeight="200" />...<demo.simple.fontweighttextview.WeightPrinterTextViewstyle="@style/TextStyle"android:layout_width="wrap_content"android:layout_height="wrap_content"android:text="Font800"android:textFontWeight="800" /><demo.simple.fontweighttextview.WeightPrinterTextViewstyle="@style/TextStyle"android:layout_width="wrap_content"android:layout_height="wrap_content"android:text="Font900"android:textFontWeight="900" />
</LinearLayout>

啊，这？又把我秀到了，不愧是Android啊！不信邪的我，还重写clean了项目，然后xml又换回了TextView，再运行，结论就是TextView就是不行，但是自定义继承TextView的WeightPrinterTextView就是可以，这就奇怪了。

一次偶然的断点调试让我发现了端倪，在xml中声明的TextView竟然变成了AppcompatTextView，我写了几个方法验证了下，发现了如下规律：在xml中定义的TextView会转换成AppcompatTextView，但是自定义继承的TextView是不会转换类型的。

其实到这里，我大概就明白了应该是AppcompatTextView的问题，因为我之前也看过相关源码，知道如果Activity是继承自AppcompatActivity的，在createView的时候，会自动把相关的基础控件转成Appcompat的相关关联控件。

接下来又是喜闻乐见的Read The Fucking Source Code时间，首先肯定从View的构造函数看起，因为View属性一般都是在构造函数中读取的。

public AppCompatTextView(@NonNull Context context, @Nullable AttributeSet attrs, int defStyleAttr) {super(TintContextWrapper.wrap(context), attrs, defStyleAttr);ThemeUtils.checkAppCompatTheme(this, getContext());mBackgroundTintHelper = new AppCompatBackgroundHelper(this);mBackgroundTintHelper.loadFromAttributes(attrs, defStyleAttr);mTextHelper = new AppCompatTextHelper(this);mTextHelper.loadFromAttributes(attrs, defStyleAttr);mTextHelper.applyCompoundDrawablesTints();mTextClassifierHelper = new AppCompatTextClassifierHelper(this);AppCompatEmojiTextHelper emojiTextViewHelper = getEmojiTextViewHelper();emojiTextViewHelper.loadFromAttributes(attrs, defStyleAttr);
}

从代码来看好像也只有loadFromAttributes方法关联性比较大，然后再进入这个方法里面，但是这里嵌套的方法就更多了，就不详细阐述了。如果光是看代码去找问题就可大海捞针了，这里推荐我最喜欢的断点单步调试方法，在你觉得有关系的相关代码上打上断点，然后等着代码执行，一步一步的看调试结果就行了。

但是有一个注意的点就是：你要选择和运行代码匹配的库版本才行，比如你导入了两个appcompat的库，断点打在了1.0.0版本的源码上，但是运行的却是2.0.0的库，或者就是断点打在了Android14的源码上，但是运行的手机或模拟器却是Android13的系统。

反正经过我的各种调试，各种跳转，最终定位到了AppCompatTextHelper的updateTypefaceAndStyle方法。大致解释下造成这个bug的原因：就是updateTypefaceAndStyle被调用了两次，第一次的mFontWeight值是对的，但是mFontTypeface却是fontWeiget不对的，所以第二次就设置成了错误的FontTypeface，所以我们只需要像下面的代码，重新创建一个新的mFontTypeface就好了。

//AppcompatTextHelper  
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.P&& mFontWeight != TEXT_FONT_WEIGHT_UNSPECIFIED && mFontTypeface != null) {mFontTypeface = Api28Impl.create(mFontTypeface, mFontWeight,(mStyle & Typeface.ITALIC) != 0);

本来到这里就应该结束了，我只需要用hook或者反射等相关方法解决掉自己的需求就好了，但是突然想到我好像从来没给Google的仓库提交过代码，之前也就给AndroidUtilCode，ijkplayer这种国内开发者的仓库提过pr，不管了，试试就逝世😂！

我去仔细读了androidx的readme文档，大致流程就是先在Android Issue Tracker创建一个issue，这一点很坑，我从它这链接点进去，说我没有create issue的权限，我还以为不用完成这一步，这也导致了最开始这个pr搁置了很久也没人来code review。
其实是需要在下拉框，重新选择子模块的。

然后再是添加自己的Test方法，覆盖测试用例，完善文档说明，最后push代码，new一个新的pr，选择一个code reviewer。

 @Testpublic void testTextFontWeight() {final AppCompatTextView textView = mActivity.findViewById(R.id.text_view_text_font_weight);if (textView.getTypeface() == null) return;final int textFontWeight = textView.getTypeface().getWeight();assertEquals(textFontWeight, 900);}

还有就是在选择code reviewer可能需要注意一下，能多选几个就多选几个吧，我选择了一个对appcompat贡献最多的一个老哥，结果到现在都还在pending review状态，期间都打算发邮件去沟通一下了，我看其他的pr都是同时选择了好几个code reviewer，最后还是在issue tracker创建了一个issue，才有好心的Google老哥帮忙review了一下，thank you bro😁。

提交pr后，google-cla-bot就会让你同意一个Contributor License，同意之后就等着code review就行了。

code review期间liutikas (Aurimas)大佬帮我完善了测试用例，nona-google (Seigo Nonaka)大佬approved了我的code changes，谢谢他们，Thanks(･ω･)ﾉ。


总的来说，这次给androidx提交代码还是很开心，因为AOSP里面也有我写的代码啦，哈哈哈~ 从readme得知，github上这个androidx仓库只是AOSP里面的androidx一个镜像，在github上合并的pr都会同步到AOSP里面去。

好了这篇文章就到这里结束了~假的，我骗你的。因为我在看androidx里面源码的时候，发现了一个神奇的类，那就是TypefaceCompat。它里面有个create的方法，可以适配不同的系统版本，创建带有weight的Typeface。

public static Typeface create(@NonNull Context context, @Nullable Typeface family,@IntRange(from = 1, to = 1000) int weight, boolean italic) {if (context == null) {throw new IllegalArgumentException("Context cannot be null");}Preconditions.checkArgumentInRange(weight, 1, 1000, "weight");if (family == null) {family = Typeface.DEFAULT;}return sTypefaceCompatImpl.createWeightStyle(context, family, weight, italic);
}

在这个类初始化的时候，static代码块里面会创建自适应的sTypefaceCompatImpl。

static {if (Build.VERSION.SDK_INT >= 29) {sTypefaceCompatImpl = new TypefaceCompatApi29Impl();} else if (Build.VERSION.SDK_INT >= 28) {sTypefaceCompatImpl = new TypefaceCompatApi28Impl();} else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {sTypefaceCompatImpl = new TypefaceCompatApi26Impl();} else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N&& TypefaceCompatApi24Impl.isUsable()) {sTypefaceCompatImpl = new TypefaceCompatApi24Impl();} else if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {sTypefaceCompatImpl = new TypefaceCompatApi21Impl();} else {sTypefaceCompatImpl = new TypefaceCompatBaseImpl();}
}

本来我是打算再提交个pr，让AppcompatTextView能不区分版本的支持textFontWeight，但是在自己写代码的时候好像发现为啥Google的大佬不这样做的原因了，可能是因为Typeface.getWeight这个方法是在Api28添加的，这样就不好写单元测试了。其实要做感觉也能做，就是给AppcompatTextView增加getWeight的方法，但是这样又感觉有点抽象😂。

所以就此，我又搞了一个新的仓库😄，用来适配全版本的textFontWeight。

simplepeng/FontWeightTextView: 让textFontWeight属性支持Api29(Android9-p)以下

代码确实很简单，就读取了下textFontWeight的属性，然后调用了下TypefaceCompat.create，以至于群里老哥都吐槽：就这么几行代码还要搞一个库😂？

就此，本篇文章确实结束啦，完结撒花✿✿ヽ(°▽°)ノ✿。