PanJiaChen / vue-element-admin

:tada: A magical vue admin https://panjiachen.github.io/vue-element-admin
MIT License
87.98k stars 30.47k forks source link

代码吐槽!觉得本项目有什么不足的地方欢迎来吐槽 #162

Closed aoeng closed 7 years ago

aoeng commented 7 years ago

作为一个后端人员,只是觉得代码有点丑陋而已 ,希望能找到一个更好更优质的vue admin 模板

PanJiaChen commented 7 years ago

@UpGod 能详细说一下,你觉得哪里不够优雅?

aoeng commented 7 years ago
import Vue from 'vue';
import App from './App';
import router from './router';
import store from './store';
import ElementUI from 'element-ui';
import 'element-ui/lib/theme-default/index.css';
import 'assets/custom-theme/index.css'; // 换肤版本element-ui css
import NProgress from 'nprogress'; // Progress 进度条
import 'nprogress/nprogress.css';// Progress 进度条 样式
import 'normalize.css/normalize.css';// normalize.css 样式格式化
import 'assets/iconfont/iconfont'; // iconfont 具体图标见https://github.com/PanJiaChen/vue-element-admin/wiki
import * as filters from './filters'; // 全局vue filter
import Multiselect from 'vue-multiselect';// 使用的一个多选框组件,element-ui的select不能满足所有需求
import 'vue-multiselect/dist/vue-multiselect.min.css';// 多选框组件css
import Sticky from 'components/Sticky'; // 粘性header组件
import IconSvg from 'components/Icon-svg';// svg 组件
import vueWaves from './directive/waves';// 水波纹指令
import errLog from 'store/errLog';// error log组件
import './mock/index.js';  // 该项目所有请求使用mockjs模拟
import { getToken } from 'utils/auth';

比如这里入口,不觉得很乱吗?

aoeng commented 7 years ago

整理一下,一种写法的放一起,

aoeng commented 7 years ago

可能是每个人的性格不同,看法不同 ,所以问题就此关闭

PanJiaChen commented 7 years ago

@UpGod main.js 是应该不这么凌乱 ,我这里优化了一下 你可以看一下,欢迎提出其它的不好的地方 https://github.com/PanJiaChen/vue-element-admin/blob/9768305166cd5a48d3f0f3db77745a562261055d/src/main.js#L1-L13

ttop5 commented 7 years ago

引入的东西太多的话,建议可以尝试将引入的依赖进行分类然后用一行空行隔开进行区分哈,比如(图中按照三方依赖和本地依赖分类): @PanJiaChen

screen shot 2017-08-21 at 4 11 44 pm
aoeng commented 7 years ago

有没有可能将以上的引入方式,改成一种

1. import Vue from 'vue' 
2. import App from './App' 
3. import 'normalize.css/normalize.css'
4. import '@/assets/iconfont/iconfont'
5. import { getToken } from 'utils/auth';
PanJiaChen commented 7 years ago

@UpGod 当然可以改啊,但这个是一种取舍或者习惯问题