방금 정적 메소드 가져 오기가 좋지 않다는 리뷰 의견이 있습니다. 정적 가져 오기는 주로 정적 메소드가있는 DA 클래스의 메소드입니다. 따라서 비즈니스 논리의 중간에 현재 클래스에 속하는 것처럼 보이는 da 활동이있었습니다.
import static some.package.DA.*;
class BusinessObject {
void someMethod() {
....
save(this);
}
}
리뷰어는 코드를 바꾸는 데 열중하지 않았지만 그렇게하지는 않았지만 나는 그에게 동의합니다. 정적 가져 오기가 아닌 이유 중 하나는 메소드가 정의 된 위치가 혼란스럽고 현재 클래스가 아니고 수퍼 클래스가 아니기 때문에 정의를 식별하기에는 너무 시간이 걸리기 때문입니다 (웹 기반 검토 시스템에는 클릭 할 수 없음) IDE와 같은 링크 🙂 나는 이것이 중요하다고 생각하지 않습니다. 정적 가져 오기는 여전히 새롭고 곧 우리는 그것들을 찾는 데 익숙해 질 것입니다.
그러나 내가 동의하는 다른 이유는 규정되지 않은 메소드 호출이 현재 객체에 속하는 것으로 보이므로 컨텍스트를 뛰어 넘지 않아야하기 때문입니다. 그러나 그것이 실제로 속해 있다면, 그 슈퍼 클래스를 확장하는 것이 합리적입니다.
그래서, 때 않습니다 그것은 정적 수입 방법에 대한 이해가? 언제 했어요? 정규화되지 않은 전화 모양을 좋아했습니까?
편집 : 대중적인 의견은 아무도 정적 가져 오기 메소드를 현재 클래스의 메소드로 혼동하지 않을 것입니다. 예를 들어 java.lang.Math 및 java.awt.Color의 메소드입니다. 그러나 abs와 getAlpha가 모호하지 않으면 readEmployee가 왜 그런지 알 수 없습니다. 많은 프로그래밍 선택에서와 마찬가지로, 나는 이것 또한 개인적인 선호 사항이라고 생각합니다.
답변 해 주셔서 감사합니다. 질문을 닫습니다.
답변
이 기능을 릴리스했을 때의 Sun 안내서에서 가져온 것입니다 (원본에서 강조 표시).
정적 가져 오기를 언제 사용해야합니까? 매우 드물게! 상수의 로컬 사본을 선언하거나 상속을 남용하려는 유혹이있는 경우에만 사용하십시오 (Constant Interface Antipattern). … 정적 가져 오기 기능을 과도하게 사용하면 프로그램을 읽을 수없고 유지 관리 할 수 없어서 가져 오는 모든 정적 멤버로 네임 스페이스를 오염시킬 수 있습니다. 코드를 작성한 후 몇 개월이 지난 후에도 코드 멤버는 정적 멤버의 클래스를 알 수 없습니다. 클래스에서 모든 정적 멤버를 가져 오는 것은 특히 가독성에 해로울 수 있습니다. 하나 또는 두 개의 멤버 만 필요한 경우 개별적으로 가져 오십시오.
( https://docs.oracle.com/javase/8/docs/technotes/guides/language/static-import.html )
구체적으로 설명하고 싶은 두 부분이 있습니다.
- “상속을 남용하려는”유혹이있을 때만 정적 가져 오기를 사용하십시오 . 이 경우 BusinessObject를 갖고 싶은 유혹을
extend some.package.DA
받았습니까? 그렇다면 정적 가져 오기가이를보다 깔끔하게 처리 할 수 있습니다. 확장을 꿈꿔 본 적이 없다면some.package.DA
정적 가져 오기를 제대로 사용 하지 않는 것일 수 있습니다. 입력 할 때 문자를 몇 개만 저장하는 데 사용하지 마십시오. - 개별 회원을 가져옵니다.
import static some.package.DA.save
대신 말하십시오DA.*
. 이렇게하면이 가져온 방법의 출처를 훨씬 쉽게 찾을 수 있습니다.
개인적으로, 나는이 언어 기능을 사용하지 않은 매우 결코 방법으로 만 상수 또는 열거 거의 항상 거의합니다. 절충은 거의 가치가 없습니다.
답변
정적 가져 오기에 대한 또 다른 합리적인 용도는 JUnit 4를 사용하는 것입니다. 이전 버전의 JUnit 메소드 에서는 테스트 클래스 확장 이후 assertEquals
와 같고 fail
상속되었습니다 junit.framework.TestCase
.
// old way
import junit.framework.TestCase;
public class MyTestClass extends TestCase {
public void myMethodTest() {
assertEquals("foo", "bar");
}
}
JUnit 4에서 테스트 클래스는 더 이상 확장 할 필요가 없으며 TestCase
대신 주석을 사용할 수 있습니다. 그런 다음 다음에서 assert 메소드를 정적으로 가져올 수 있습니다 org.junit.Assert
.
// new way
import static org.junit.Assert.assertEquals;
public class MyTestClass {
@Test public void myMethodTest() {
assertEquals("foo", "bar");
// instead of
Assert.assertEquals("foo", "bar");
}
}
JUnit은 이것을 사용하여 문서화 합니다.
답변
항목 19 의 끝 부분 인 Effective Java, Second Edition 에서는 유틸리티 클래스의 상수를 많이 사용하는 경우 정적 가져 오기를 사용할 수 있습니다 . 이 원칙은 상수와 메소드의 정적 가져 오기에 적용된다고 생각합니다.
import static com.example.UtilityClassWithFrequentlyUsedMethods.myMethod;
public class MyClass {
public void doSomething() {
int foo= UtilityClassWithFrequentlyUsedMethods.myMethod();
// can be written less verbosely as
int bar = myMethod();
}
}
이것은 장단점이 있습니다. 메소드가 정의 된 위치에 대한 즉각적인 정보가 손실되는 대신 코드를 좀 더 읽기 쉽게 만듭니다. 그러나 좋은 IDE를 사용하면 정의로 이동할 수 있으므로 큰 문제는 아닙니다.
가져온 파일의 항목을 여러 번 사용하는 경우에만이 방법을 조금만 사용해야합니다.
편집 : 이 질문에서 언급 한 것처럼 메소드에 더 구체적으로 업데이트되었습니다. 가져 오는 내용 (상수 또는 방법)에 관계없이 원칙이 적용됩니다.
답변
나는 그것들이 가독성 관점에서 문제가 될 수 있으며 드물게 사용해야한다는 데 동의합니다. 그러나 일반적인 정적 방법을 사용하면 실제로 가독성을 높일 수 있습니다. 예를 들어, JUnit 테스트 클래스에서 메소드 assertEquals
는 어디에서 왔는지 분명합니다. 의 메소드와 유사합니다 java.lang.Math
.
답변
정적 가져 오기는 Arrays
and 같은 utils 클래스를 사용할 때 중복 클래스 이름을 제거하는 데 실제로 유용하다고 생각합니다 Assertions
.
왜 그런지 모르지만 로스 는 그가 언급 한 문서 에서 이것을 언급 한 마지막 문장을 생략했다 .
적절하게 사용하면 정적 가져 오기를 사용하면 클래스 이름이 반복되는 상용구를 제거하여 프로그램을 더 읽기 쉽게 만들 수 있습니다.
이 블로그에서 기본적으로 복사 : https://medium.com/alphadev-thoughts/static-imports-are-great-but-underused-e805ba9b279f
예를 들어 :
테스트의 주장
이것은 우리 모두가 동의하는 가장 명백한 경우입니다
Assertions.assertThat(1).isEqualTo(2);
// Use static import instead
assertThat(1).isEqualTo(2);
유틸리티 클래스와 열거 형
utils 클래스를 사용하면 코드를 쉽게 읽을 수 있도록 클래스 이름을 제거 할 수있는 경우가 많습니다
List<Integer> numbers = Arrays.asList(1, 2, 3);
// asList method name is enough information
List<Integer> numbers = asList(1, 2, 3);
java.time 패키지에는 사용해야 할 몇 가지 경우가 있습니다.
// Get next Friday from now, quite annoying to read
LocalDate.now().with(TemporalAdjusters.next(DayOfWeek.FRIDAY));
// More concise and easier to read
LocalDate.now().with(next(FRIDAY));
사용하지 않을 때의 예
// Ok this is an Optional
Optional.of("hello world");
// I have no idea what this is
of("hello world");
답변
칼라에 많이 사용하고 있습니다.
static import java.awt.Color.*;
색상이 다른 것과 혼동 될 가능성은 거의 없습니다.
답변
Java와 함께 OpenGL을 사용할 때 정적 가져 오기 를 사용 하는 것이 좋습니다. 이는 “유틸리티 클래스의 상수를 많이 사용”하는 경우입니다. 범주의
그것을 고려하십시오
import static android.opengl.GLES20.*;
원본 C 코드를 이식하고 다음과 같이 읽을 수있는 것을 작성할 수 있습니다.
glActiveTexture(GL_TEXTURE0);
glBindTexture(GL_TEXTURE_2D, texture);
glUniform1i(samplerUniform, 0);
glBindBuffer(GL_ARRAY_BUFFER, vtxBuffer);
glVertexAttribPointer(vtxAttrib, 3, GL_FLOAT, false, 0, 0);
그 보편적 인 못생긴 것 대신 :
GLES20.glActiveTexture(GLES20.GL_TEXTURE0);
GLES20.glBindTexture(GLES20.GL_TEXTURE_2D, texture);
GLES20.glUniform1i(samplerUniform, 0);
GLES20.glBindBuffer(GLES20.GL_ARRAY_BUFFER, vtxBuffer);
GLES20.glVertexAttribPointer(vtxAttrib, 3, GLES20.GL_FLOAT, false, 0, 0);